On Wed 20 Aug 01:06 PDT 2014, Srinivas Kandagatla wrote: > Hi Bjorn, > Hi Srinivas, Thanks for the testing. I'm reworking the driver to incorporate yours, Linus' and Ivans feedback. > Two things which I noticed while trying out this driver to drive a reset > line. > > 1> gpio numbering for pinconf vs gpio are not consistent, they differ by > an offset of 1. > After scratching my head regarding this I now see that there's a off by one in most of the gpio functions. I've rewroked this to make more sense now. The gpio numbers has to start at 1, as all documentation is based on that. [...] > > 2> Looking back at v3.4 kernel, for gpio modes, BIT(0) of bank 0 is set > to enable gpio mode. without this bit driver does not work for output pins. > Thanks, I missed that. Unfortunately, setting that bit results in input not working - the interrupt bits are never set for gpios that have that bit set. I'm trying to figure out why this is the case before sending out the new version... Regards, Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html