Re: [PATCH v3 1/3] pinctrl: Rename Broadcom Capri pinctrl driver

From: Linus Walleij
Date: Thu Apr 10 2014 - 13:41:34 EST


On Thu, Apr 10, 2014 at 6:56 PM, Matt Porter <matt.porter@xxxxxxxxxx> wrote:
> On Thu, Apr 10, 2014 at 06:51:12PM +0200, Linus Walleij wrote:
>> On Thu, Apr 10, 2014 at 6:49 PM, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:
>> > On Thu, Apr 3, 2014 at 12:40 AM, Sherman Yin <syin@xxxxxxxxxxxx> wrote:
>> >
>> >> To be consistent with other Broadcom drivers, the Broadcom Capri pinctrl
>> >> driver and its related CONFIG option are renamed to bcm281xx.
>> >>
>> >> Devicetree compatible string and binding documentation use
>> >> "brcm,bcm11351-pinctrl" to match the machine binding here:
>> >> Documentation/devicetree/bindings/arm/bcm/bcm11351.txt
>> >>
>> >> This driver supports pinctrl on BCM11130, BCM11140, BCM11351, BCM28145
>> >> and BCM28155 SoCs.
>> >>
>> >> Signed-off-by: Sherman Yin <syin@xxxxxxxxxxxx>
>> >> Reviewed-by: Matt Porter <mporter@xxxxxxxxxx>
>> >> Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
>> >> ---
>> >> drivers/pinctrl/Kconfig | 8 +-
>> >> drivers/pinctrl/Makefile | 2 +-
>> >> .../{pinctrl-capri.c => pinctrl-bcm281xx.c} | 1519 ++++++++++----------
>> >
>> > This last version from apr 7 does not change
>> > arch/arm/configs/bcm_defconfig:CONFIG_PINCTRL_CAPRI=y
>> >
>> > Doesn't this mean that you render this platform without a pin
>> > control driver and thus broken?
>>
>> Bah I see that it's a separate patch now, sorry...
>>
>> Looking closer.
>
> I can take that one through mach-bcm for 3.16 to keep these changes
> in one place.

Nah I might as well take all of it through the pin control tree.
Can I have your ACK on the defconfig change?

Yours,
Linus Walleij
--
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/