I fullquote here because of the lists and people added to Cc.
On Thu, Nov 20, 2008 at 08:02:15PM -0800, David Brownell wrote:
> I just noticed:
> > commit 2e373952cc893207a8b47a5e68c2f5155f912449
> > Author: Phil Sutter <email@example.com>
> > Date: Sat Nov 1 15:13:21 2008 +0100
> > MIPS: RB532: Provide functions for gpio configuration
> > As gpiolib doesn't support pin multiplexing, it provides no way to
> > access the GPIOFUNC register. Also there is no support for setting
> > interrupt status and level. These functions provide access to them and
> > are needed by the CompactFlash driver.
> > Signed-off-by: Phil Sutter <firstname.lastname@example.org>
> > Signed-off-by: Ralf Baechle <email@example.com>
> The conventional way to do most of that is through the irq_chip
> associated with that block of IRQ-capable GPIOs.
> So ...
> > - /* Set the interrupt status and level for the CF pin */
> > - rb532_gpio_set_int_level(&rb532_gpio_chip->chip, CF_GPIO_NUM, 1);
> > - rb532_gpio_set_int_status(&rb532_gpio_chip->chip, CF_GPIO_NUM, 0);
> > + /* configure CF_GPIO_NUM as CFRDY IRQ source */
> > + rb532_gpio_set_func(0, CF_GPIO_NUM);
> ... the pinmux would indeed be a SOC-specific mechanism, kicked in
> only for boards that use that GPIO in that way, but ...
> > + rb532_gpio_direction_input(&rb532_gpio_chip->chip, CF_GPIO_NUM);
> ... normal gpio_request() + gpio_direction_input() would set the pin up, and
> > + rb532_gpio_set_ilevel(1, CF_GPIO_NUM);
> > + rb532_gpio_set_istat(0, CF_GPIO_NUM);
> status = request_irq(gpio_to_irq(CF_GPIO_NUM), ... )
> with appropriate IRQF_TRIGGER_* flags should solve that problem.
> Or even just set_irq_type(). (At least for one of the two
> registers updated there ... I can't guess what "istat" would be.)
> Just FYI at this point. Maybe you have a reason not to fit into
> the genirq framework.
> - Dave
Thanks for your hints, Dave. I already had a patch flying around adding
a set_type() function to the irq_chip (merely a dummy to shut up
warnings in the boot log). After extending it to cover the needs of the
mapped GPIO pins and some other work, I could drop that CompactFlash
initialisation code in gpio.c completely.
As a side effect, many of my changes to pata-rb532-cf got unnecessary
which I consider a very good sign.
I'll reply to this mail with the relevant patches for each list and