Re: [PATCH] pinctrl: Fix Kconfig indentation

From: Krzysztof Kozlowski
Date: Wed Nov 20 2019 - 20:27:40 EST


On Wed, 20 Nov 2019 at 22:15, Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> wrote:
>
> Hello Krzysztof,
>
> On 11/20/19 2:40 PM, Krzysztof Kozlowski wrote:
> > Adjust indentation from spaces to tab (+optional two spaces) as in
> > coding style with command like:
> > $ sed -e 's/^ /\t/' -i */Kconfig
>
> > config PINCTRL_ARTPEC6
> > - bool "Axis ARTPEC-6 pin controller driver"
> > + bool "Axis ARTPEC-6 pin controller driver"
>
> Here you replace spaces for tabs before the prompt type.
>
> > config PINCTRL_SM8150
> > tristate "Qualcomm Technologies Inc SM8150 pin controller driver"
>
> Here you leave the spaces before intact.
>
> Intentional?

No, not intentional. Thanks for pointing it out. I made the sed
patterns precise - 8 spaces -> tab, and there you have 7 spaces
(ugh!). I'll send a follow up.

Best regards,
Krzysztof