Re: [PATCH v2 5/6] of_spi: Add fetching of of_gpio flags to of_spi_register_master

From: Andreas Larsson
Date: Tue Feb 12 2013 - 04:39:38 EST


On 2013-02-11 01:22, Grant Likely wrote:
On Tue, 29 Jan 2013 15:53:42 +0100, Andreas Larsson <andreas@xxxxxxxxxxx> wrote:
When using a gpio chip select with a OF_GPIO_ACTIVE_LOW flag, this needs to be
known to the controller driver.

Signed-off-by: Andreas Larsson <andreas@xxxxxxxxxxx>

Out of curiosity, what do you need the flags for? Polarity of the CS
signal? I think it is long past time to revisit baking polarity control
into core of gpiolib. Would you mind investigating doing that instead of
having to manage it manually in each and every driver?

I don't even need it on my hardware. It seemed like a good thing to have via of_spi_register_master. I am OK with just dropping this.

Cheers,
Andreas

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/