Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753851Ab3ISMgk (ORCPT ); Thu, 19 Sep 2013 08:36:40 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:39863 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751914Ab3ISMgi (ORCPT ); Thu, 19 Sep 2013 08:36:38 -0400 Message-ID: <523AEFD1.8000109@ti.com> Date: Thu, 19 Sep 2013 07:36:33 -0500 From: Nishanth Menon User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.0 MIME-Version: 1.0 To: George Cherian CC: , Laurent Pinchart , Linus Walleij , Laurent Pinchart , Kuninori Morimoto , "linux-gpio@vger.kernel.org" , Linux Kernel Mailing List , Linux OMAP Mailing List Subject: Re: PCF857x and 16-bit GPIO expanders References: <20130917190732.GD15645@radagast> <2356541.62QktiIBr5@avalon> <20130918173636.GT21559@radagast> <523AB235.1000808@ti.com> <523AE8FF.9020209@ti.com> <523AEC0C.6010005@ti.com> In-Reply-To: <523AEC0C.6010005@ti.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3580 Lines: 90 On 09/19/2013 07:20 AM, George Cherian wrote: > On 9/19/2013 5:37 PM, Nishanth Menon wrote: >> On 09/19/2013 03:13 AM, George Cherian wrote: >>> On 9/18/2013 11:06 PM, Felipe Balbi wrote: >>>> Hi, >>>> >>>> On Wed, Sep 18, 2013 at 07:18:04PM +0200, Laurent Pinchart wrote: >>>>> On Wednesday 18 September 2013 13:16:27 Linus Walleij wrote: >>>>>> On Tue, Sep 17, 2013 at 9:07 PM, Felipe Balbi wrote: >>>>>>> has anyone ever successfully using gpio-pcf857x.c driver with 16-bit >>>>>>> gpio expanders ? We're having some issues here where toggling the last >>>>>>> gpio pin (gpio 15) on a PCF8575 device causes platform to hang and I >>>>>>> can't come up with any explanation of why would it hang... >>>>>> Bouncing the question to George, Laurent and Kuninori... >>>>> I've got a board with a PCF8575 chip, but it uses I/Os 8 to 14 only as far as >>>>> I know. >>>>> >>>>> I can try toggling I/O 15, but that will need to wait until next week as I'm >>>>> currently travelling without access to the hardware. >>>> alright, that'd help me a lot :-) Just want to make sure if we're having >>>> a board issue, or PCF8575 is a bit screwy ;-) >>> Is it on dra7x-evm if so which pcf device (i2c address)? >>> The pins i were interested were only 1 and 2 I never tried pin 15. >>> >>> Just tried toggling through sysfs and it works for me. >> When I look at the data sheet for PCF8575[1] Page 7, Figure 4 Write >> mode (output) >> I see the data writes are of the order: >> I2c 1's byte: address >> I2c 2'nd byte:P[7-0] >> I2c 3rd byte:P[17-10] > > I read it as an octal numbering. >> Note: bits 8,9 are missing not supported. > > In octal there is no 8 and 9 Where is octal coming into play here? P8 and 9 does not exist as per the data sheet -> look at the pinout on page 1[1] ->P00-P17 this is exactly what is described on page 2[1]: "The number of data bytes that can be sent successively is not limited After every two bytes, the previous data is overwritten. When the PCF8575 receives the pairs of data bytes, the first byte is referred to as P07 ? P00 and the second byte as P17 ? P10. The third byte is referred to as P07 ? P00, the fourth byte as P17 ? P10, and so on" For someone reading schematics and setting up the P15, if the person uses gpios = <&PCF8575 15 OF_GPIO_HIGH>; this will result in offset = 15, and as a result 0x80 will be send in byte 3, which from h/w point of view is P13 which could be controlling something weird! >> >> Now [2] claims that it does support PCF8575, however when I look at >> line 143[3] >> unsigned bit = 1 << offset; >> [snip] >> if (value) >> gpio->out |= bit; >> else >> gpio->out &= ~bit; >> >> There is no handling for the skip needed for bits 8 and 9.. Seems to >> me like a driver bug. > > In which case there is no driver bug It probably needs clarification: Depends on how the definition is done - to hit exact P15 in definition and to set P15, should I provide 17 as gpio number? That does not match what the h/w description is in the data sheet! >> [1] http://www.ti.com/lit/ds/symlink/pcf8575.pdf >> [2] >> http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/gpio/gpio-pcf857x.c >> [3] >> http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/gpio/gpio-pcf857x.c#n143 > > -- Regards, Nishanth Menon -- 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/