[PATCH v3 10/22] usb: chipidea: Consolidate extcon notifiers
Stephen Boyd
stephen.boyd at linaro.org
Tue Sep 6 10:48:19 PDT 2016
On Sun, Sep 4, 2016 at 7:39 PM, Peter Chen <hzpeterchen at gmail.com> wrote:
> On Fri, Sep 02, 2016 at 06:03:06PM -0700, Stephen Boyd wrote:
>> On Thu, Sep 1, 2016 at 8:17 PM, Peter Chen <hzpeterchen at gmail.com> wrote:
>> > On Wed, Aug 31, 2016 at 05:40:24PM -0700, Stephen Boyd wrote:
>> >>
>> >>
>> >> if (cable->state)
>> >> - val |= OTGSC_ID;
>> >> + val &= ~OTGSC_ID; /* A device */
>> >> else
>> >> - val &= ~OTGSC_ID;
>> >> + val |= OTGSC_ID; /* B device */
>
> One more comment, would you change the comment to "host" and "device"?
> Unless we are supporting OTG-compliance device, otherwise, we should
> not mention "A" or "B" for USB device.
>
Done
More information about the linux-arm-kernel
mailing list