[PATCH] arm pxa: remove "direction input" forcing for IRQ GPIOs

Rodolfo Giometti giometti at enneenne.com
Tue Nov 17 08:16:46 EST 2009


On Tue, Nov 17, 2009 at 02:06:53PM +0100, Daniel Mack wrote:
> On Tue, Nov 17, 2009 at 01:59:53PM +0100, Rodolfo Giometti wrote:
> > On Tue, Nov 17, 2009 at 01:43:34PM +0100, Daniel Mack wrote:
> > > On Tue, Nov 17, 2009 at 01:37:21PM +0100, Rodolfo Giometti wrote:
> > > > On Tue, Nov 17, 2009 at 01:34:22PM +0100, Daniel Mack wrote:
> > > > > For such situations, the pullup/pulldown functions are normally used.
> > > > > Have you tried that?
> > > > 
> > > > Yes, the code has been tested on my board and it works perfectly.
> > > 
> > > What I mean is a mfp_cfg_t configuration as input pin with pull flags
> > > MFP_PULL_{HIGH,LOW} set.
> > 
> > I see... no I didn't noticed such possibility.
> > 
> > however I suppose that even using such flags my patch is needed in
> > order to avoid pxa_gpio_irq_type() forcing the input direction...
> 
> No, that's exactly my point. You don't need to set the pin to output
> direction to make use of those pull features. More than that, when
> confiugured as output, the pull flags actually don't make sense.
> 
> As an IRQ is always inbound, I agree with Eric that the code should
> stay in there for safety and sanity.

I see what you mean... I'll test those flags and report results ASAP.

Thanks,

Rodolfo

-- 

GNU/Linux Solutions                  e-mail: giometti at enneenne.com
Linux Device Driver                          giometti at linux.it
Embedded Systems                     phone:  +39 349 2432127
UNIX programming                     skype:  rodolfo.giometti
Freelance ICT Italia - Consulente ICT Italia - www.consulenti-ict.it



More information about the linux-arm-kernel mailing list