On Fri, Feb 5, 2016 at 5:37 PM, Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> wrote:
> Am Freitag, den 05.02.2016, 14:11 +0800 schrieb Peter Chen:
> [...]
>> > The reference recommendation states that for single-configuration USB
>> > devices the compatible should contain all of the applicable strings from
>> > the list starting with 2) "usbVID,PID.REV" and ending with 11)
>> > "usb,device". Are we going to ignore this?
>>
>> I have not seen benefits if we write several compatibles in dts,
>> the information of compatibles listed in doc can be got during
>> the enumeration.
>>
>> I suggest we use the simple pattern for this compatible, in that
>> case, every one can be easy to follow it, and will not be confused
>> which compatibles should be used, and the style can be unify.
>
> Just pointing it out, a comment why this differs from the recommendation
> would be nice to avoid confusion.
>
Thanks, I will.
> [...]
>> > > + compatible = "usb05e3,0608";
>> > > + reg = <0x1>;
>> > > + };
>> >
>> > I'd have written this node as:
>> >
>> > hub: hub@1 {
>> > compatible = "usb5e3,608", "usb5e3,class9",
>> > "usb,class9", "usb,device";
>> > reg = <1>;
>> > };
>>
>> The reg should be hexadecimal, do we need to add "0x" before the value?
>
> The unit-address name part should be hexadecimal, the reg property value
> doesn't have to be. As long as the value is < 10 I don't see a problem.
>
I see.
>> > As another example, I'd like to introduce the USB WLAN Adapter soldered
>> > onto the imx6q-gk802 board to its power enable GPIO via the device tree:
>> >
>> > /* Internal USB port (USBH1) */
>> > &usbh1 {
>> > #address-cells = <1>;
>> > #size-cells = <0>;
>> > status = "okay";
>> >
>> > /* RTL8192CU 802.11n WLAN Adapter */
>> > device@1 {
>> > compatible = "usbbda,8176.200", "usbbda,8176",
>> > "usb,device";
>> > reg = <1>;
>> >
>> > enable-gpios = <&gpio2 0 GPIO_ACTIVE_LOW>;
>> > };
>> > };
>> >
>>
>> It is okay to use your example, but I still insist like below:
>
> Sorry for being imprecise. The hub example is fine for the
> documentation. I just wanted to illustrate how and why I am interested
> in this discussion.
>
Thanks for commenting it.
I will be in Chinese New Year holiday, will send next version after Feb 15th.
--
BR,
Peter Chen
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
