Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753877Ab3HVFO4 (ORCPT ); Thu, 22 Aug 2013 01:14:56 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:33667 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753853Ab3HVFOy (ORCPT ); Thu, 22 Aug 2013 01:14:54 -0400 Message-ID: <52159E37.90501@ti.com> Date: Thu, 22 Aug 2013 10:44:31 +0530 From: George Cherian User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130620 Thunderbird/17.0.7 MIME-Version: 1.0 To: Stephen Warren CC: , , , , , , , , , , , , , , Subject: Re: [PATCH 1/2] extcon: extcon-dra7xx: Add extcon driver for USB ID detection References: <1376648029-30659-1-git-send-email-george.cherian@ti.com> <1376648029-30659-2-git-send-email-george.cherian@ti.com> <52127284.6030506@wwwdotorg.org> <521312CC.8060003@ti.com> <52139F0F.6090507@wwwdotorg.org> <5214BB6C.3060903@ti.com> <5214FA64.3040902@wwwdotorg.org> In-Reply-To: <5214FA64.3040902@wwwdotorg.org> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1626 Lines: 46 On 8/21/2013 11:05 PM, Stephen Warren wrote: > On 08/21/2013 07:06 AM, George Cherian wrote: >> Hi Stephen, >> >> On 8/20/2013 10:23 PM, Stephen Warren wrote: >>>>> ID pins are connected to pcf8575, and the pcf8575's interrupt line is >>>>> inturn connected to >>>>> gpio bank6 pin 11, we use this gpio interrupt to detect the ID pin >>>> change. >>> In that case, the PCF8575 node needs to be a GPIO controller and an IRQ >>> controller, as does the driver for the PCF8575. This binding should have >>> a single entry in the gpios property, and the driver can call >>> gpio_to_irq() on that so it knows which IRQ to request. >> You meant some thing like this? >> >> pcf_usb: pcf8575@21 { >> compatible = "ti,pcf8575"; >> reg = <0x21>; >> gpio-controller; >> #gpio-cells = <2>; >> interrupt-parent = <&gpio6>; >> interrupts = <11 2>; >> interrupt-controller; >> #interrupt-cells = <2>; >> }; >> >> usb_vid_gpio { >> compatible = "ti,dra7xx-usb"; >> gpios = <&pcf_usb 1 0>; >> }; > Yes. > > Except that the compatible value for the usb_vid_gpio node still looks > wrong, since I think that node isn't anything to do with any particular SoC. Yes will fix that too in v2. -- -George -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/