2020-11-13 15:51:04

by Sekhar Nori

[permalink] [raw]
Subject: [PATCH v3] arm64: defconfig: Enable GPIO and I2C configs for TI's J721e platform

From: Faiz Abbas <[email protected]>

Add configs to help enable regulators that supply power to the SD card
on TI's J721e platform. These regulators are controlled by either
SoC gpios or gpios over i2c expander.

vmlinux size before and after patch follow:

Before:
text data bss dec hex filename
19755448 10376346 535572 30667366 1d3f266 vmlinux

After:
text data bss dec hex filename
19769232 10381390 536212 30686834 1d43e72 vmlinux

difference is 19,468 (dec)

Acked-by: Tero Kristo <[email protected]>
Signed-off-by: Faiz Abbas <[email protected]>
Signed-off-by: Sekhar Nori <[email protected]>
---
changes in v3:
- add size delta as requested by Nishanth Menon

arch/arm64/configs/defconfig | 2 ++
1 file changed, 2 insertions(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index c5f404ce2eec..e44d3f438530 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -438,6 +438,7 @@ CONFIG_I2C_IMX=y
CONFIG_I2C_IMX_LPI2C=y
CONFIG_I2C_MESON=y
CONFIG_I2C_MV64XXX=y
+CONFIG_I2C_OMAP=y
CONFIG_I2C_OWL=y
CONFIG_I2C_PXA=y
CONFIG_I2C_QCOM_CCI=m
@@ -498,6 +499,7 @@ CONFIG_PINCTRL_SDM845=y
CONFIG_PINCTRL_SM8150=y
CONFIG_PINCTRL_SM8250=y
CONFIG_GPIO_ALTERA=m
+CONFIG_GPIO_DAVINCI=y
CONFIG_GPIO_DWAPB=y
CONFIG_GPIO_MB86S7X=y
CONFIG_GPIO_MPC8XXX=y
--
2.17.1


2020-11-13 17:09:59

by Nishanth Menon

[permalink] [raw]
Subject: Re: [PATCH v3] arm64: defconfig: Enable GPIO and I2C configs for TI's J721e platform

On 21:19-20201113, Sekhar Nori wrote:
> From: Faiz Abbas <[email protected]>
>
> Add configs to help enable regulators that supply power to the SD card
> on TI's J721e platform. These regulators are controlled by either
> SoC gpios or gpios over i2c expander.
>
> vmlinux size before and after patch follow:
>
Sekhar,

> Before:
> text data bss dec hex filename
> 19755448 10376346 535572 30667366 1d3f266 vmlinux
>
> After:
> text data bss dec hex filename
> 19769232 10381390 536212 30686834 1d43e72 vmlinux
>
> difference is 19,468 (dec)
>
> Acked-by: Tero Kristo <[email protected]>
> Signed-off-by: Faiz Abbas <[email protected]>
> Signed-off-by: Sekhar Nori <[email protected]>
> ---
> changes in v3:
> - add size delta as requested by Nishanth Menon

https://git.kernel.org/pub/scm/linux/kernel/git/nmenon/linux.git/commit/?h=ti-k3-config-next&id=6b133f475a97a0839f02e3c0b937886b9adc2933

https://lore.kernel.org/linux-arm-kernel/[email protected]/


Is this a duplicate patch?
>
> arch/arm64/configs/defconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index c5f404ce2eec..e44d3f438530 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -438,6 +438,7 @@ CONFIG_I2C_IMX=y
> CONFIG_I2C_IMX_LPI2C=y
> CONFIG_I2C_MESON=y
> CONFIG_I2C_MV64XXX=y
> +CONFIG_I2C_OMAP=y
> CONFIG_I2C_OWL=y
> CONFIG_I2C_PXA=y
> CONFIG_I2C_QCOM_CCI=m
> @@ -498,6 +499,7 @@ CONFIG_PINCTRL_SDM845=y
> CONFIG_PINCTRL_SM8150=y
> CONFIG_PINCTRL_SM8250=y
> CONFIG_GPIO_ALTERA=m
> +CONFIG_GPIO_DAVINCI=y
> CONFIG_GPIO_DWAPB=y
> CONFIG_GPIO_MB86S7X=y
> CONFIG_GPIO_MPC8XXX=y
> --
> 2.17.1
>

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D

2020-11-13 17:19:47

by Sekhar Nori

[permalink] [raw]
Subject: Re: [PATCH v3] arm64: defconfig: Enable GPIO and I2C configs for TI's J721e platform

On 13/11/20 10:34 PM, Nishanth Menon wrote:
> On 21:19-20201113, Sekhar Nori wrote:
>> From: Faiz Abbas <[email protected]>
>>
>> Add configs to help enable regulators that supply power to the SD card
>> on TI's J721e platform. These regulators are controlled by either
>> SoC gpios or gpios over i2c expander.
>>
>> vmlinux size before and after patch follow:
>>
> Sekhar,
>
>> Before:
>> text data bss dec hex filename
>> 19755448 10376346 535572 30667366 1d3f266 vmlinux
>>
>> After:
>> text data bss dec hex filename
>> 19769232 10381390 536212 30686834 1d43e72 vmlinux
>>
>> difference is 19,468 (dec)
>>
>> Acked-by: Tero Kristo <[email protected]>
>> Signed-off-by: Faiz Abbas <[email protected]>
>> Signed-off-by: Sekhar Nori <[email protected]>
>> ---
>> changes in v3:
>> - add size delta as requested by Nishanth Menon
>
> https://git.kernel.org/pub/scm/linux/kernel/git/nmenon/linux.git/commit/?h=ti-k3-config-next&id=6b133f475a97a0839f02e3c0b937886b9adc2933
>
> https://lore.kernel.org/linux-arm-kernel/[email protected]/
>
>
> Is this a duplicate patch?

Looks like, please discard. For some reason, Faiz's v3 did not show-up
in my inbox and v2 applied cleanly to linux-next/master. So I assume it
was not refreshed.

Thanks,
Sekhar

2020-11-13 18:37:19

by Nishanth Menon

[permalink] [raw]
Subject: Re: [PATCH v3] arm64: defconfig: Enable GPIO and I2C configs for TI's J721e platform

On 22:45-20201113, Sekhar Nori wrote:
> On 13/11/20 10:34 PM, Nishanth Menon wrote:
> > On 21:19-20201113, Sekhar Nori wrote:
> >> From: Faiz Abbas <[email protected]>
> >>
> >> Add configs to help enable regulators that supply power to the SD card
> >> on TI's J721e platform. These regulators are controlled by either
> >> SoC gpios or gpios over i2c expander.
> >>
> >> vmlinux size before and after patch follow:
> >>
> > Sekhar,
> >
> >> Before:
> >> text data bss dec hex filename
> >> 19755448 10376346 535572 30667366 1d3f266 vmlinux
> >>
> >> After:
> >> text data bss dec hex filename
> >> 19769232 10381390 536212 30686834 1d43e72 vmlinux
> >>
> >> difference is 19,468 (dec)
> >>
> >> Acked-by: Tero Kristo <[email protected]>
> >> Signed-off-by: Faiz Abbas <[email protected]>
> >> Signed-off-by: Sekhar Nori <[email protected]>
> >> ---
> >> changes in v3:
> >> - add size delta as requested by Nishanth Menon
> >
> > https://git.kernel.org/pub/scm/linux/kernel/git/nmenon/linux.git/commit/?h=ti-k3-config-next&id=6b133f475a97a0839f02e3c0b937886b9adc2933
> >
> > https://lore.kernel.org/linux-arm-kernel/[email protected]/
> >
> >
> > Is this a duplicate patch?
>
> Looks like, please discard. For some reason, Faiz's v3 did not show-up
> in my inbox and v2 applied cleanly to linux-next/master. So I assume it
> was not refreshed.


OK. Thanks for confirming - keeping Faiz's version in the tree. :)
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D