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

[Matt Porter]

On Thu, Apr 10, 2014 at 07:41:20PM +0200, Linus Walleij wrote:
> 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?

Sounds fine to me, done.

-Matt
--
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/