Patchwork [U-Boot,01/10] mxs: Rename CONFIG_SPL_MX28_PSWITCH_WAIT to CONFIG_SPL_MXS_PSWITCH_WAIT

login
register
mail settings
Submitter Otavio Salvador
Date Jan. 30, 2013, 12:13 p.m.
Message ID <1359548001-14278-2-git-send-email-otavio@ossystems.com.br>
Download mbox | patch
Permalink /patch/216873/
State Changes Requested
Delegated to: Stefano Babic
Headers show

Comments

Otavio Salvador - Jan. 30, 2013, 12:13 p.m.
The power switch option is compatible with i.MX23 and i.MX28 so the
configration option needs to reflect it. We choose
'CONFIG_SPL_MXS_PSWITCH_WAIT' for the option name.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 arch/arm/cpu/arm926ejs/mxs/mxs_init.h       | 2 +-
 arch/arm/cpu/arm926ejs/mxs/spl_power_init.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
Marek Vasut - Jan. 30, 2013, 2:11 p.m.
Dear Otavio Salvador,

> The power switch option is compatible with i.MX23 and i.MX28 so the
> configration option needs to reflect it. We choose
> 'CONFIG_SPL_MXS_PSWITCH_WAIT' for the option name.

Acked-by: Marek Vasut <marex@denx.de>

I'm missing cover letter for this large series.

Best regards,
Marek Vasut
Otavio Salvador - Jan. 30, 2013, 3:29 p.m.
On Wed, Jan 30, 2013 at 12:11 PM, Marek Vasut <marex@denx.de> wrote:
> Dear Otavio Salvador,
>
>> The power switch option is compatible with i.MX23 and i.MX28 so the
>> configration option needs to reflect it. We choose
>> 'CONFIG_SPL_MXS_PSWITCH_WAIT' for the option name.
>
> Acked-by: Marek Vasut <marex@denx.de>
>
> I'm missing cover letter for this large series.

Hum, patman seems to be doing something wrong here. It should copy the
cover to all Cced people I think. Simon?

--
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br
Simon Glass - Feb. 1, 2013, 11:42 p.m.
Him

On Wed, Jan 30, 2013 at 7:29 AM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> On Wed, Jan 30, 2013 at 12:11 PM, Marek Vasut <marex@denx.de> wrote:
>> Dear Otavio Salvador,
>>
>>> The power switch option is compatible with i.MX23 and i.MX28 so the
>>> configration option needs to reflect it. We choose
>>> 'CONFIG_SPL_MXS_PSWITCH_WAIT' for the option name.
>>
>> Acked-by: Marek Vasut <marex@denx.de>
>>
>> I'm missing cover letter for this large series.
>
> Hum, patman seems to be doing something wrong here. It should copy the
> cover to all Cced people I think. Simon?

Yes - there is a pull request up to bring those patches into mainline,
after which it will do this.

http://patchwork.ozlabs.org/patch/217308/

Regards,
Simon

>
> --
> Otavio Salvador                             O.S. Systems
> E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
> Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Patch

diff --git a/arch/arm/cpu/arm926ejs/mxs/mxs_init.h b/arch/arm/cpu/arm926ejs/mxs/mxs_init.h
index 2ddc5bc..084def5 100644
--- a/arch/arm/cpu/arm926ejs/mxs/mxs_init.h
+++ b/arch/arm/cpu/arm926ejs/mxs/mxs_init.h
@@ -30,7 +30,7 @@  void early_delay(int delay);
 
 void mxs_power_init(void);
 
-#ifdef	CONFIG_SPL_MX28_PSWITCH_WAIT
+#ifdef	CONFIG_SPL_MXS_PSWITCH_WAIT
 void mxs_power_wait_pswitch(void);
 #else
 static inline void mxs_power_wait_pswitch(void) { }
diff --git a/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c b/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c
index e9d6302..287c698 100644
--- a/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c
+++ b/arch/arm/cpu/arm926ejs/mxs/spl_power_init.c
@@ -921,7 +921,7 @@  void mxs_power_init(void)
 	early_delay(1000);
 }
 
-#ifdef	CONFIG_SPL_MX28_PSWITCH_WAIT
+#ifdef	CONFIG_SPL_MXS_PSWITCH_WAIT
 void mxs_power_wait_pswitch(void)
 {
 	struct mxs_power_regs *power_regs =