[U-Boot] [PATCH] dm: avoid dropping pin control DT properties in case of SPL_PINCTRL

[U-Boot] [PATCH] dm: avoid dropping pin control DT properties in case of SPL_PINCTRL

[Vikas Manocha]

This patch replaces SPL_PINCTRL_FULL with SPL_PINCNTRL. It is to avoid removal
of pin control properties in case of SPL_PINCTRL. No impact in case of
SPL_PINCTRL_FULL as it depends on SPL_PINCTRL.

Signed-off-by: Vikas Manocha <vikas.manocha@st.com>
---
 dts/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dts/Kconfig b/dts/Kconfig
index 4b7d8b1..2374e62 100644
--- a/dts/Kconfig
+++ b/dts/Kconfig
@@ -73,8 +73,8 @@ config OF_LIST
 config OF_SPL_REMOVE_PROPS
 	string "List of device tree properties to drop for SPL"
 	depends on SPL_OF_CONTROL
-	default "interrupt-parent" if SPL_PINCTRL_FULL && SPL_CLK
-	default "clocks clock-names interrupt-parent" if SPL_PINCTRL_FULL
+	default "interrupt-parent" if SPL_PINCTRL && SPL_CLK
+	default "clocks clock-names interrupt-parent" if SPL_PINCTRL
 	default "pinctrl-0 pinctrl-names interrupt-parent" if SPL_CLK
 	default "pinctrl-0 pinctrl-names clocks clock-names interrupt-parent"
 	help
-- 
1.9.1

[U-Boot] [PATCH] dm: avoid dropping pin control DT properties in case of SPL_PINCTRL

[Vikas MANOCHA]

Hi Michal/Simon,

> -----Original Message-----
> From: Vikas MANOCHA
> Sent: Friday, March 24, 2017 2:48 PM
> To: u-boot at lists.denx.de
> Cc: Vikas MANOCHA <vikas.manocha@st.com>; Michal Simek <michal.simek@xilinx.com>; Simon Glass <sjg@chromium.org>
> Subject: [PATCH] dm: avoid dropping pin control DT properties in case of SPL_PINCTRL
> 
> This patch replaces SPL_PINCTRL_FULL with SPL_PINCNTRL. It is to avoid removal of pin control properties in case of SPL_PINCTRL. No
> impact in case of SPL_PINCTRL_FULL as it depends on SPL_PINCTRL.

Just a reminder to review this patch.

Cheers,
Vikas

> 
> Signed-off-by: Vikas Manocha <vikas.manocha@st.com>
> ---
>  dts/Kconfig | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/dts/Kconfig b/dts/Kconfig
> index 4b7d8b1..2374e62 100644
> --- a/dts/Kconfig
> +++ b/dts/Kconfig
> @@ -73,8 +73,8 @@ config OF_LIST
>  config OF_SPL_REMOVE_PROPS
>  	string "List of device tree properties to drop for SPL"
>  	depends on SPL_OF_CONTROL
> -	default "interrupt-parent" if SPL_PINCTRL_FULL && SPL_CLK
> -	default "clocks clock-names interrupt-parent" if SPL_PINCTRL_FULL
> +	default "interrupt-parent" if SPL_PINCTRL && SPL_CLK
> +	default "clocks clock-names interrupt-parent" if SPL_PINCTRL
>  	default "pinctrl-0 pinctrl-names interrupt-parent" if SPL_CLK
>  	default "pinctrl-0 pinctrl-names clocks clock-names interrupt-parent"
>  	help
> --
> 1.9.1

[U-Boot] [PATCH] dm: avoid dropping pin control DT properties in case of SPL_PINCTRL

[Simon Glass]

Hi Vikas,

On 29 March 2017 at 13:33, Vikas MANOCHA <vikas.manocha@st.com> wrote:
> Hi Michal/Simon,
>
>> -----Original Message-----
>> From: Vikas MANOCHA
>> Sent: Friday, March 24, 2017 2:48 PM
>> To: u-boot at lists.denx.de
>> Cc: Vikas MANOCHA <vikas.manocha@st.com>; Michal Simek <michal.simek@xilinx.com>; Simon Glass <sjg@chromium.org>
>> Subject: [PATCH] dm: avoid dropping pin control DT properties in case of SPL_PINCTRL
>>
>> This patch replaces SPL_PINCTRL_FULL with SPL_PINCNTRL. It is to avoid removal of pin control properties in case of SPL_PINCTRL. No
>> impact in case of SPL_PINCTRL_FULL as it depends on SPL_PINCTRL.
>
> Just a reminder to review this patch.

I do normally see patches that are copied to me, but I don't review
all of them and it can take me a week or two to get to them sometimes.
E.g at present I have about 250 patches in my queue so I only make
progress on the backlog when people send fewer patches :-)

Regards,
Simon

[U-Boot] [PATCH] dm: avoid dropping pin control DT properties in case of SPL_PINCTRL

[Simon Glass]

On 24 March 2017 at 15:47, Vikas Manocha <vikas.manocha@st.com> wrote:
> This patch replaces SPL_PINCTRL_FULL with SPL_PINCNTRL. It is to avoid removal
> of pin control properties in case of SPL_PINCTRL. No impact in case of
> SPL_PINCTRL_FULL as it depends on SPL_PINCTRL.
>
> Signed-off-by: Vikas Manocha <vikas.manocha@st.com>
> ---
>  dts/Kconfig | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

[U-Boot] [PATCH] dm: avoid dropping pin control DT properties in case of SPL_PINCTRL

[Simon Glass]

On 31 March 2017 at 22:22, Simon Glass <sjg@chromium.org> wrote:
> On 24 March 2017 at 15:47, Vikas Manocha <vikas.manocha@st.com> wrote:
>> This patch replaces SPL_PINCTRL_FULL with SPL_PINCNTRL. It is to avoid removal
>> of pin control properties in case of SPL_PINCTRL. No impact in case of
>> SPL_PINCTRL_FULL as it depends on SPL_PINCTRL.
>>
>> Signed-off-by: Vikas Manocha <vikas.manocha@st.com>
>> ---
>>  dts/Kconfig | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> Reviewed-by: Simon Glass <sjg@chromium.org>

Applied to u-boot-dm, thanks!