Re: [PATCH 3/6] riscv: load addresses for Sipeed MAIX

2020-08-12 Thread Rick Chen
Hi Heinrich

> On 8/6/20 9:07 AM, Rick Chen wrote:
> >> From: U-Boot [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Heinrich 
> >> Schuchardt
> >> Sent: Wednesday, July 29, 2020 11:43 PM
> >> To: Sean Anderson
> >> Cc: Michal Simek; Tom Rini; Simon Glass; Bin Meng; Alexander Graf; 
> >> u-boot@lists.denx.de; Heinrich Schuchardt
> >> Subject: [PATCH 3/6] riscv: load addresses for Sipeed MAIX
> >>
> >> Define default load addresses and the device tree name for the Sipeed MAIX.
> >>
> >> Signed-off-by: Heinrich Schuchardt 
> >> ---
> >>  include/configs/sipeed-maix.h | 9 +
> >>  1 file changed, 9 insertions(+)
> >
> > Acked-by: Rick Chen 
> >
>
> Thanks for reviewing.
>
> I have reassigned the patch to you in patchwork. The rest of the series
> was already merged via the last EFI pull request.

It is better to be pulled with the whole patch series together via EFI tree.
Because the title of this patch sets is [PATCH 0/6] efi_selftest:
allow UEFI testing on MAIX.

And you said that you still try the address for OpenSBI in previous mail.

Thanks,
Rick

>
> Best regards
>
> Heinrich


Re: [PATCH 3/6] riscv: load addresses for Sipeed MAIX

2020-08-07 Thread Heinrich Schuchardt
On 8/6/20 9:07 AM, Rick Chen wrote:
>> From: U-Boot [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Heinrich 
>> Schuchardt
>> Sent: Wednesday, July 29, 2020 11:43 PM
>> To: Sean Anderson
>> Cc: Michal Simek; Tom Rini; Simon Glass; Bin Meng; Alexander Graf; 
>> u-boot@lists.denx.de; Heinrich Schuchardt
>> Subject: [PATCH 3/6] riscv: load addresses for Sipeed MAIX
>>
>> Define default load addresses and the device tree name for the Sipeed MAIX.
>>
>> Signed-off-by: Heinrich Schuchardt 
>> ---
>>  include/configs/sipeed-maix.h | 9 +
>>  1 file changed, 9 insertions(+)
>
> Acked-by: Rick Chen 
>

Thanks for reviewing.

I have reassigned the patch to you in patchwork. The rest of the series
was already merged via the last EFI pull request.

Best regards

Heinrich


Re: [PATCH 3/6] riscv: load addresses for Sipeed MAIX

2020-08-06 Thread Rick Chen
> From: U-Boot [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Heinrich 
> Schuchardt
> Sent: Wednesday, July 29, 2020 11:43 PM
> To: Sean Anderson
> Cc: Michal Simek; Tom Rini; Simon Glass; Bin Meng; Alexander Graf; 
> u-boot@lists.denx.de; Heinrich Schuchardt
> Subject: [PATCH 3/6] riscv: load addresses for Sipeed MAIX
>
> Define default load addresses and the device tree name for the Sipeed MAIX.
>
> Signed-off-by: Heinrich Schuchardt 
> ---
>  include/configs/sipeed-maix.h | 9 +
>  1 file changed, 9 insertions(+)

Acked-by: Rick Chen 


Re: [PATCH 3/6] riscv: load addresses for Sipeed MAIX

2020-08-01 Thread Heinrich Schuchardt
On 7/29/20 5:42 PM, Heinrich Schuchardt wrote:
> Define default load addresses and the device tree name for the Sipeed MAIX.
>
> Signed-off-by: Heinrich Schuchardt 
> ---
>  include/configs/sipeed-maix.h | 9 +
>  1 file changed, 9 insertions(+)
>
> diff --git a/include/configs/sipeed-maix.h b/include/configs/sipeed-maix.h
> index a46473fc78..978db6fb00 100644
> --- a/include/configs/sipeed-maix.h
> +++ b/include/configs/sipeed-maix.h
> @@ -21,4 +21,13 @@
>  /* For early init */
>  #define K210_SYSCTL_BASE 0x5044
>
> +#ifndef CONFIG_EXTRA_ENV_SETTINGS
> +#define CONFIG_EXTRA_ENV_SETTINGS \
> + "loadaddr=0x8004\0" \
> + "fdt_addr_r=0x8001\0" \
> + "scriptaddr=0x80008000\0" \
> + "kernel_addr_r=0x8004\0" \
> + "fdtfile=kendryte/" CONFIG_DEFAULT_DEVICE_TREE ".dtb\0"
> +#endif
> +
>  #endif /* CONFIGS_SIPEED_MAIX_H */
> --
> 2.27.0
>

Probably we should choose the addresses here higher to allow for
OpenSBI. I am still trying to make OpenSBI load U-Boot on the board.

Best regards

Heinrich


[PATCH 3/6] riscv: load addresses for Sipeed MAIX

2020-07-29 Thread Heinrich Schuchardt
Define default load addresses and the device tree name for the Sipeed MAIX.

Signed-off-by: Heinrich Schuchardt 
---
 include/configs/sipeed-maix.h | 9 +
 1 file changed, 9 insertions(+)

diff --git a/include/configs/sipeed-maix.h b/include/configs/sipeed-maix.h
index a46473fc78..978db6fb00 100644
--- a/include/configs/sipeed-maix.h
+++ b/include/configs/sipeed-maix.h
@@ -21,4 +21,13 @@
 /* For early init */
 #define K210_SYSCTL_BASE 0x5044

+#ifndef CONFIG_EXTRA_ENV_SETTINGS
+#define CONFIG_EXTRA_ENV_SETTINGS \
+   "loadaddr=0x8004\0" \
+   "fdt_addr_r=0x8001\0" \
+   "scriptaddr=0x80008000\0" \
+   "kernel_addr_r=0x8004\0" \
+   "fdtfile=kendryte/" CONFIG_DEFAULT_DEVICE_TREE ".dtb\0"
+#endif
+
 #endif /* CONFIGS_SIPEED_MAIX_H */
--
2.27.0