Messages in this thread | | | Date | Tue, 17 Jun 2014 17:58:33 +0530 | From | George Cherian <> | Subject | Re: [PATCH] extcon: extcon-dra7xx: Add Extcon driver for DRA7xx |
| |
On 6/16/2014 7:45 PM, Guenter Roeck wrote: > 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@ti.com> >>> >>> Hi George, >>> >>> Curious: Why can't you use extcon-gpio ? >> Main reason being missing dt support. > > Then why not add dt support to it ? Yes, I did sent a patch for the same. http://marc.info/?l=linux-kernel&m=140297766828100&w=2 http://marc.info/?l=linux-kernel&m=140297771028124&w=2 http://marc.info/?l=linux-kernel&m=140297768828109&w=2 > >>> >>> 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 >
-- -George
| |