On Fri, Aug 29, 2014 at 10:19:16PM +0400, Dmitry Eremin-Solenikov wrote: > From: Dirk Behme <dirk.be...@de.bosch.com> > > With GPIO #0, this if statement will always fail. Remove this, the > check for gpio_is_valid() is sufficent here.
No, read the archives - this will break boards using zero as default. Any current boards should be using DT and so shouldn't be using fixed GPIO numbers in the first place which will mean they'll not end up getting zero as a valid GPIO. If you are using zero as a GPIO for some reason provide a way to specify that the GPIO is a real GPIO and not just the default value for the struct. > Change-Id: Ia10e8e9c74e11deba852fcd44040cc275b583cfd > Signed-off-by: Gokulkrishnan Nagarajan <gokulkrishnan.nagara...@in.bosch.com> > Signed-off-by: Dirk Behme <dirk.be...@de.bosch.com> In any case I can't apply this since you didn't sign it off and please don't include noise from your internal review system in upstream postings.
signature.asc
Description: Digital signature
