Re: [PATCH] extcon: extcon-dra7xx: Add Extcon driver for DRA7xx

[Guenter Roeck]

On 06/15/2014 10:41 PM, George Cherian wrote:
> On 6/16/2014 10:29 AM, Guenter Roeck wrote:
> > On 06/15/2014 07:42 PM, George Cherian wrote:
> > > This is the driver for the USB ID pin detection. This driver
> > > handles only the USB ID pin changes generated by cable
> > > insertion/removal.
> > >
> > > Signed-off-by: George Cherian <george.cherian@xxxxxx>
> >
> > Hi George,
> >
> > Curious: Why can't you use extcon-gpio ?
> Main reason being missing dt support.

Then why not add dt support to it ?

> > Also, I thought that Linux specific bindings would be unacceptable.
> > "ti,dra7xx-extcon" looks very linux specific to me. Did the rules change ?
> Then how about "ti,extcon-usbid" ?

"extcon" seems very linux specific to me.

Anyway, not arguing, just asking, since I wondered. If the extcon maintainer
and the dt folks are happy with your driver, so am I.

Guenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/