diff mbox series

[U-Boot,v4,03/18] rockchip: enable boot0-hook for all Rockchip SoCs

Message ID 1507318152-8524-4-git-send-email-philipp.tomsich@theobroma-systems.com
State Superseded
Delegated to: Philipp Tomsich
Headers show
Series rockchip: back-to-bootrom: replace assembly-implementation with C-code | expand

Commit Message

Philipp Tomsich Oct. 6, 2017, 7:28 p.m. UTC
Rockchip SoCs bootrom design is like this:
- First 2KB or 4KB internal memory is for bootrom stack and heap;
- Then the first 4-byte suppose to be a TAG like 'RK33';
- The the following memory address end with '0004' is the first
  instruction load and running by bootrom;

Let's use the boot0 hook to reserve the first 4-byte tag for all
the Rockchip SoCs.

Signed-off-by: Philipp Tomsich <philipp.tomsich@theobroma-systems>
[Commit message:]
Signed-off-by: Kever Yang <kever.yang@rock-chips.com>

Signed-off-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
---

Changes in v4: None
Changes in v3: None
Changes in v2: None

 arch/arm/Kconfig               | 1 +
 arch/arm/mach-rockchip/Kconfig | 2 --
 2 files changed, 1 insertion(+), 2 deletions(-)

Comments

Andy Yan Oct. 9, 2017, 9:37 a.m. UTC | #1
Hi Philipp:


On 2017年10月07日 03:28, Philipp Tomsich wrote:
> Rockchip SoCs bootrom design is like this:
> - First 2KB or 4KB internal memory is for bootrom stack and heap;
> - Then the first 4-byte suppose to be a TAG like 'RK33';
> - The the following memory address end with '0004' is the first
>    instruction load and running by bootrom;
>
> Let's use the boot0 hook to reserve the first 4-byte tag for all
> the Rockchip SoCs.
>
> Signed-off-by: Philipp Tomsich <philipp.tomsich@theobroma-systems>
> [Commit message:]

     What's this ?
> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
>
> Signed-off-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
> ---
>
> Changes in v4: None
> Changes in v3: None
> Changes in v2: None
>
>   arch/arm/Kconfig               | 1 +
>   arch/arm/mach-rockchip/Kconfig | 2 --
>   2 files changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 5d1ce3e..d5050d0 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1109,6 +1109,7 @@ config ARCH_ROCKCHIP
>   	select DM_USB if USB
>   	select DM_PWM
>   	select DM_REGULATOR
> +	select ENABLE_ARM_SOC_BOOT0_HOOK
>   	imply CMD_FASTBOOT
>   	imply FASTBOOT
>   	imply FAT_WRITE
> diff --git a/arch/arm/mach-rockchip/Kconfig b/arch/arm/mach-rockchip/Kconfig
> index d9b25d5..31e9864 100644
> --- a/arch/arm/mach-rockchip/Kconfig
> +++ b/arch/arm/mach-rockchip/Kconfig
> @@ -74,7 +74,6 @@ config ROCKCHIP_RK3368
>   	imply SPL_SEPARATE_BSS
>   	imply SPL_SERIAL_SUPPORT
>   	imply TPL_SERIAL_SUPPORT
> -	select ENABLE_ARM_SOC_BOOT0_HOOK
>   	select DEBUG_UART_BOARD_INIT
>   	select SYS_NS16550
>   	help
> @@ -112,7 +111,6 @@ config ROCKCHIP_RK3399
>   	select SPL_SEPARATE_BSS
>   	select SPL_SERIAL_SUPPORT
>   	select SPL_DRIVERS_MISC_SUPPORT
> -	select ENABLE_ARM_SOC_BOOT0_HOOK
>   	select DEBUG_UART_BOARD_INIT
>   	help
>   	  The Rockchip RK3399 is a ARM-based SoC with a dual-core Cortex-A72
Philipp Tomsich Oct. 9, 2017, 10:21 a.m. UTC | #2
> On 9 Oct 2017, at 11:37, Andy Yan <andy.yan@rock-chips.com> wrote:
> 
> Hi Philipp:
> 
> 
> On 2017年10月07日 03:28, Philipp Tomsich wrote:
>> Rockchip SoCs bootrom design is like this:
>> - First 2KB or 4KB internal memory is for bootrom stack and heap;
>> - Then the first 4-byte suppose to be a TAG like 'RK33';
>> - The the following memory address end with '0004' is the first
>>   instruction load and running by bootrom;
>> 
>> Let's use the boot0 hook to reserve the first 4-byte tag for all
>> the Rockchip SoCs.
>> 
>> Signed-off-by: Philipp Tomsich <philipp.tomsich@theobroma-systems>
>> [Commit message:]
> 
>     What's this ?

This means "I took the commit message from Kever’s original boot0-series,
but the content of the patch was rewritten”.

>> Signed-off-by: Kever Yang <kever.yang@rock-chips.com>
>> 
>> Signed-off-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
>> ---
>> 
>> Changes in v4: None
>> Changes in v3: None
>> Changes in v2: None
>> 
>>  arch/arm/Kconfig               | 1 +
>>  arch/arm/mach-rockchip/Kconfig | 2 --
>>  2 files changed, 1 insertion(+), 2 deletions(-)
>> 
>> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
>> index 5d1ce3e..d5050d0 100644
>> --- a/arch/arm/Kconfig
>> +++ b/arch/arm/Kconfig
>> @@ -1109,6 +1109,7 @@ config ARCH_ROCKCHIP
>>  	select DM_USB if USB
>>  	select DM_PWM
>>  	select DM_REGULATOR
>> +	select ENABLE_ARM_SOC_BOOT0_HOOK
>>  	imply CMD_FASTBOOT
>>  	imply FASTBOOT
>>  	imply FAT_WRITE
>> diff --git a/arch/arm/mach-rockchip/Kconfig b/arch/arm/mach-rockchip/Kconfig
>> index d9b25d5..31e9864 100644
>> --- a/arch/arm/mach-rockchip/Kconfig
>> +++ b/arch/arm/mach-rockchip/Kconfig
>> @@ -74,7 +74,6 @@ config ROCKCHIP_RK3368
>>  	imply SPL_SEPARATE_BSS
>>  	imply SPL_SERIAL_SUPPORT
>>  	imply TPL_SERIAL_SUPPORT
>> -	select ENABLE_ARM_SOC_BOOT0_HOOK
>>  	select DEBUG_UART_BOARD_INIT
>>  	select SYS_NS16550
>>  	help
>> @@ -112,7 +111,6 @@ config ROCKCHIP_RK3399
>>  	select SPL_SEPARATE_BSS
>>  	select SPL_SERIAL_SUPPORT
>>  	select SPL_DRIVERS_MISC_SUPPORT
>> -	select ENABLE_ARM_SOC_BOOT0_HOOK
>>  	select DEBUG_UART_BOARD_INIT
>>  	help
>>  	  The Rockchip RK3399 is a ARM-based SoC with a dual-core Cortex-A72
> 
>
diff mbox series

Patch

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 5d1ce3e..d5050d0 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -1109,6 +1109,7 @@  config ARCH_ROCKCHIP
 	select DM_USB if USB
 	select DM_PWM
 	select DM_REGULATOR
+	select ENABLE_ARM_SOC_BOOT0_HOOK
 	imply CMD_FASTBOOT
 	imply FASTBOOT
 	imply FAT_WRITE
diff --git a/arch/arm/mach-rockchip/Kconfig b/arch/arm/mach-rockchip/Kconfig
index d9b25d5..31e9864 100644
--- a/arch/arm/mach-rockchip/Kconfig
+++ b/arch/arm/mach-rockchip/Kconfig
@@ -74,7 +74,6 @@  config ROCKCHIP_RK3368
 	imply SPL_SEPARATE_BSS
 	imply SPL_SERIAL_SUPPORT
 	imply TPL_SERIAL_SUPPORT
-	select ENABLE_ARM_SOC_BOOT0_HOOK
 	select DEBUG_UART_BOARD_INIT
 	select SYS_NS16550
 	help
@@ -112,7 +111,6 @@  config ROCKCHIP_RK3399
 	select SPL_SEPARATE_BSS
 	select SPL_SERIAL_SUPPORT
 	select SPL_DRIVERS_MISC_SUPPORT
-	select ENABLE_ARM_SOC_BOOT0_HOOK
 	select DEBUG_UART_BOARD_INIT
 	help
 	  The Rockchip RK3399 is a ARM-based SoC with a dual-core Cortex-A72