[PATCH 01/14] pinctrl: SPEAr: add spi chipselect control driver

Viresh Kumar viresh.kumar at linaro.org
Fri Nov 16 01:19:30 EST 2012


On 15 November 2012 19:47, Linus Walleij <linus.walleij at linaro.org> wrote:
> On Sun, Nov 11, 2012 at 5:39 AM, Viresh Kumar <viresh.kumar at linaro.org> wrote:
>
>> From: Shiraz Hashim <shiraz.hashim at st.com>
>>
>> SPEAr platform provides a provision to control chipselects of ARM PL022
>> Prime Cell spi controller through its system registers, which otherwise
>> remains under PL022 control which some protocols do not want.
>>
>> This commit intends to provide the spi chipselect control in software
>> over gpiolib interface. Since it is tied to pinctrl, we place it under
>> 'drivers/pinctrl/spear/' directory.
>>
>> spi chip drivers can use the exported gpiolib interface to define their
>> chipselect through DT or platform data.
>>
>> Cc: Linus Walleij <linus.walleij at linaro.org>
>> Signed-off-by: Shiraz Hashim <shiraz.hashim at st.com>
>> Reviewed-by: Vipin Kumar <vipin.kumar at st.com>
>> Signed-off-by: Viresh Kumar <viresh.kumar at linaro.org>
>
> This sure looks good, sorry for the delay.
>
> Just one question: since this driver is not using any
> pinctrl interfaces, why is it under drivers/pinctrl/*?
>
> Shouldn't it be under drivers/gpio from a technical
> point of view?
>
> I think I'd accept it under drivers/pinctrl/spear/*
> if you just want this to keep everything SPEAr-related
> in one place though, so enlighten me.

That's because they are controlling few pads. They are not
actually gpio's but just pads that we are controlling.

That's why we thought they are better in this place.

--
viresh


More information about the devicetree-discuss mailing list