Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1271474 > unrolled thread
| Started by | "Opensource [Adam Thomson]" <Adam.Thomson.Opensource@diasemi.com> |
|---|---|
| First post | 2015-11-17 18:30 +0100 |
| Last post | 2015-11-18 12:40 +0100 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
RE: [PATCH v2 1/2] ASoC: da7218: Add bindings documentation for DA7218 audio codec "Opensource [Adam Thomson]" <Adam.Thomson.Opensource@diasemi.com> - 2015-11-17 18:30 +0100
Re: [PATCH v2 1/2] ASoC: da7218: Add bindings documentation for DA7218 audio codec Rob Herring <robh@kernel.org> - 2015-11-17 19:00 +0100
RE: [PATCH v2 1/2] ASoC: da7218: Add bindings documentation for DA7218 audio codec "Opensource [Adam Thomson]" <Adam.Thomson.Opensource@diasemi.com> - 2015-11-18 12:40 +0100
| From | "Opensource [Adam Thomson]" <Adam.Thomson.Opensource@diasemi.com> |
|---|---|
| Date | 2015-11-17 18:30 +0100 |
| Subject | RE: [PATCH v2 1/2] ASoC: da7218: Add bindings documentation for DA7218 audio codec |
| Message-ID | <qvSeS-81H-15@gated-at.bofh.it> |
T24gTm92ZW1iZXIgMTEsIDIwMTUgMjA6MjAsIFJvYiBIZXJyaW5nIHdyb3RlOg0KDQo+ID4gKy0g ZGxnLG1pY2JpYXMxLWx2bCA6IFZvbHRhZ2UgKG1WKSBmb3IgTWljIEJpYXMgMQ0KPiA+ICsJWzwx MjAwPiwgPDE2MDA+LCA8MTgwMD4sIDwyMDAwPiwgPDIyMDA+LCA8MjQwMD4sIDwyNjAwPiwgPDI4 MDA+LA0KPiA8MzAwMD5dDQo+ID4gKy0gZGxnLG1pY2JpYXMyLWx2bCA6IFZvbHRhZ2UgKG1WKSBm b3IgTWljIEJpYXMgMg0KPiA+ICsJWzwxMjAwPiwgPDE2MDA+LCA8MTgwMD4sIDwyMDAwPiwgPDIy MDA+LCA8MjQwMD4sIDwyNjAwPiwgPDI4MDA+LA0KPiA8MzAwMD5dDQo+IA0KPiBVbml0cyBwbGVh c2UgKC1taWNyb3ZvbHQpLg0KDQpJIHJlZmVyIGJhY2sgdG8gb3VyIHByZXZpb3VzIGRpc2N1c3Np b24gKGh0dHBzOi8vbGttbC5vcmcvbGttbC8yMDE1LzEwLzgvNjYxKS4NClRoaXMgZG9lc24ndCBh ZGQgYW55dGhpbmcgYW5kIG1ha2VzIHRoZSBiaW5kaW5nIG5hbWUgdW5uZWNlc3NhcmlseSBsb25n LiBXaHkgaXMNCnRoaXMgYmVpbmcgZW5mb3JjZWQ/IFdob2V2ZXIgdXNlcyB0aGUgYmluZGluZyB3 aWxsIGhhdmUgdG8gbG9vayBhdCB0aGUNCmRvY3VtZW50YXRpb24gdG8gdW5kZXJzdGFuZCB3aGlj aCB2YWx1ZXMgYXJlIHZhbGlkIGFueXdheSwgc28gdGhpcyBzZWVtcyBsaWtlDQpjcnVmdC4NCg0K PiA+ICtPcHRpb25hbCBwcm9wZXJ0aWVzOg0KPiA+ICstIGRsZyxqYWNrLXJhdGUgOiBUaW1lIGJl dHdlZW4gamFjayBkZXRlY3QgbWVhc3VyZW1lbnRzICh1cykNCj4gPiArCVs8NT4sIDwxMD4sIDwy MD4sIDw0MD4sIDw4MD4sIDwxNjA+LCA8MzIwPiwgPDY0MD5dDQo+IA0KPiBVbml0cw0KDQpkaXR0 by4NCg== -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Rob Herring <robh@kernel.org> |
|---|---|
| Date | 2015-11-17 19:00 +0100 |
| Message-ID | <qvSHU-8dG-5@gated-at.bofh.it> |
| In reply to | #1271474 |
On Tue, Nov 17, 2015 at 11:27 AM, Opensource [Adam Thomson] <Adam.Thomson.Opensource@diasemi.com> wrote: > On November 11, 2015 20:20, Rob Herring wrote: > >> > +- dlg,micbias1-lvl : Voltage (mV) for Mic Bias 1 >> > + [<1200>, <1600>, <1800>, <2000>, <2200>, <2400>, <2600>, <2800>, >> <3000>] >> > +- dlg,micbias2-lvl : Voltage (mV) for Mic Bias 2 >> > + [<1200>, <1600>, <1800>, <2000>, <2200>, <2400>, <2600>, <2800>, >> <3000>] >> >> Units please (-microvolt). > > I refer back to our previous discussion (https://lkml.org/lkml/2015/10/8/661). > This doesn't add anything and makes the binding name unnecessarily long. Why is > this being enforced? Whoever uses the binding will have to look at the > documentation to understand which values are valid anyway, so this seems like > cruft. It is simply standard, best practice for new bindings. Certainly there are examples that don't follow this, but they are either old or escaped review. Drop the 'lvl' part if you are so concerned about length. Rob -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | "Opensource [Adam Thomson]" <Adam.Thomson.Opensource@diasemi.com> |
|---|---|
| Date | 2015-11-18 12:40 +0100 |
| Message-ID | <qw9fH-2gd-11@gated-at.bofh.it> |
| In reply to | #1271497 |
T24gTm92ZW1iZXIgMTcsIDIwMTUgMTc6NTQsIFJvYiBIZXJyaW5nIHdyb3RlOg0KDQo+ID4+ID4g Ky0gZGxnLG1pY2JpYXMxLWx2bCA6IFZvbHRhZ2UgKG1WKSBmb3IgTWljIEJpYXMgMQ0KPiA+PiA+ ICsgICBbPDEyMDA+LCA8MTYwMD4sIDwxODAwPiwgPDIwMDA+LCA8MjIwMD4sIDwyNDAwPiwgPDI2 MDA+LCA8MjgwMD4sDQo+ID4+IDwzMDAwPl0NCj4gPj4gPiArLSBkbGcsbWljYmlhczItbHZsIDog Vm9sdGFnZSAobVYpIGZvciBNaWMgQmlhcyAyDQo+ID4+ID4gKyAgIFs8MTIwMD4sIDwxNjAwPiwg PDE4MDA+LCA8MjAwMD4sIDwyMjAwPiwgPDI0MDA+LCA8MjYwMD4sIDwyODAwPiwNCj4gPj4gPDMw MDA+XQ0KPiA+Pg0KPiA+PiBVbml0cyBwbGVhc2UgKC1taWNyb3ZvbHQpLg0KPiA+DQo+ID4gSSBy ZWZlciBiYWNrIHRvIG91ciBwcmV2aW91cyBkaXNjdXNzaW9uIChodHRwczovL2xrbWwub3JnL2xr bWwvMjAxNS8xMC84LzY2MSkuDQo+ID4gVGhpcyBkb2Vzbid0IGFkZCBhbnl0aGluZyBhbmQgbWFr ZXMgdGhlIGJpbmRpbmcgbmFtZSB1bm5lY2Vzc2FyaWx5IGxvbmcuIFdoeSBpcw0KPiA+IHRoaXMg YmVpbmcgZW5mb3JjZWQ/IFdob2V2ZXIgdXNlcyB0aGUgYmluZGluZyB3aWxsIGhhdmUgdG8gbG9v ayBhdCB0aGUNCj4gPiBkb2N1bWVudGF0aW9uIHRvIHVuZGVyc3RhbmQgd2hpY2ggdmFsdWVzIGFy ZSB2YWxpZCBhbnl3YXksIHNvIHRoaXMgc2VlbXMgbGlrZQ0KPiA+IGNydWZ0Lg0KPiANCj4gSXQg aXMgc2ltcGx5IHN0YW5kYXJkLCBiZXN0IHByYWN0aWNlIGZvciBuZXcgYmluZGluZ3MuIENlcnRh aW5seSB0aGVyZQ0KPiBhcmUgZXhhbXBsZXMgdGhhdCBkb24ndCBmb2xsb3cgdGhpcywgYnV0IHRo ZXkgYXJlIGVpdGhlciBvbGQgb3INCj4gZXNjYXBlZCByZXZpZXcuDQo+IA0KPiBEcm9wIHRoZSAn bHZsJyBwYXJ0IGlmIHlvdSBhcmUgc28gY29uY2VybmVkIGFib3V0IGxlbmd0aC4NCg0KSWYgdGhp cyBpcyBhIHN0YW5kYXJkLCB0aGVuIG1heWJlIHRoaXMgc2hvdWxkIGJlIGV4cGxpY2l0bHkgZG9j dW1lbnRlZCBhbmQNCnB1c2hlZCB0byBhbGwgb3RoZXIgbWFpbnRhaW5lcnMgc28gaXQgaXNuJ3Qg ZXNjYXBpbmcgcmV2aWV3PyBPdGhlcndpc2UgeW91J3JlDQpsZWZ0IHdpdGggYSBkaXNwYXJpdHkg d2hpY2ggc3VnZ2VzdHMgaXQgaXNuJ3Qgc3RhbmRhcmQgYXQgYWxsLiBQZXJzb25hbGx5IEkNCnN0 YW5kIGJ5IHRoYXQgZm9yIGRldmljZSBzcGVjaWZpYyBiaW5kaW5ncyB0aGlzIHNob3VsZG4ndCBh cHBseSBhcyBpdCBnYWlucw0Kbm90aGluZyBleGNlcHQgYWNoaWV2aW5nIGFuIG92ZXJseSBsb25n IGJpbmRpbmcsIG9yIHlvdSBlbmQgdXAgY3V0dGluZyBkb3duIHRoZQ0KZGVzY3JpcHRpdmUgcGFy dCBvZiB0aGUgbmFtZSB0byBhY2NvbW1vZGF0ZSB1bml0cyBhdCB0aGUgZW5kLg0K -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web