Re: [PATCH] extcon-usb-gpio: add enable pin support

[Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>]

Hello.

On 12/11/2015 07:05 AM, Chanwoo Choi wrote:

>> Sometimes  there's a real  OTG chip behind the USB ID signal mapped to a GPIO
>> pin: in my case it's Maxim Integrated MAX3355E which  integrates Vbus charge
>> pump and comparators and passes  thru the ID  signal  from an OTG connector.
>
> s/thru/through ?

    "Thru" is valid English.

>> This chip also has the SHDN# pin which  should be  driven high for the normal
>> operation  and low to  save power;  it  is connected to a GPIO pin as well on,
>> hence  we'll have  to  teach the driver to parse the new optional device tree
>> property, "enable-gpio"...

> This patch description includes the double space between words. Also, I think

    So what?

> you need to write the patch description again for formal style.

    Not sure I understand you.

> This patch adds the specific 'enable-gpio' pin to express the SHDN#pin for MAX3355E.
> I think it is not regular and standard case because maybe USB specification
> don't include the SHDN#pin information.

    Certainly, it's not a USB pin.

> I think it not appropriate way.
> Instead, you better to make the MAX3355 extcon driver to support this case.

    OK, just didn't want to duplicate most of this driver there...

> Thanks,
> Chanwoo

MBR, Sergei