[PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-04-12 Thread sbabic
> The FlexSPI NOR boot offset does not require any special handling, > the image_offset is correct in either case (0x1000 for FlexSPI NOR > and 0x8000 for SD/eMMC) and the offset of u-boot.itb from the start > of flash.bin is always 0x58000 on MX8MN/MX8MP, which matches the > CONFIG_SYS_MMCSD_RAW_M

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-31 Thread Marek Vasut
On 3/31/22 20:02, Tim Harvey wrote: On Thu, Mar 31, 2022 at 9:41 AM Marek Vasut wrote: On 3/31/22 18:03, Tim Harvey wrote: On Thu, Mar 31, 2022 at 8:26 AM Marek Vasut wrote: On 3/31/22 17:09, Tim Harvey wrote: On Mon, Mar 28, 2022 at 7:55 AM Marek Vasut wrote: On 3/28/22 08:54, Ye Li w

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-31 Thread Tim Harvey
On Thu, Mar 31, 2022 at 9:41 AM Marek Vasut wrote: > > On 3/31/22 18:03, Tim Harvey wrote: > > On Thu, Mar 31, 2022 at 8:26 AM Marek Vasut wrote: > >> > >> On 3/31/22 17:09, Tim Harvey wrote: > >>> On Mon, Mar 28, 2022 at 7:55 AM Marek Vasut wrote: > > On 3/28/22 08:54, Ye Li wrote: >

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-31 Thread Marek Vasut
On 3/31/22 18:03, Tim Harvey wrote: On Thu, Mar 31, 2022 at 8:26 AM Marek Vasut wrote: On 3/31/22 17:09, Tim Harvey wrote: On Mon, Mar 28, 2022 at 7:55 AM Marek Vasut wrote: On 3/28/22 08:54, Ye Li wrote: Hi Marek, Hi, [...] 2. Update the u-boot.itb offset in imx8mp-u-boot.dtsi, set

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-31 Thread Tim Harvey
On Thu, Mar 31, 2022 at 8:26 AM Marek Vasut wrote: > > On 3/31/22 17:09, Tim Harvey wrote: > > On Mon, Mar 28, 2022 at 7:55 AM Marek Vasut wrote: > >> > >> On 3/28/22 08:54, Ye Li wrote: > >>> Hi Marek, > >> > >> Hi, > >> > >> [...] > >> > > 2. Update the u-boot.itb offset in imx8mp-u-boot.dt

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-31 Thread Marek Vasut
On 3/31/22 17:09, Tim Harvey wrote: On Mon, Mar 28, 2022 at 7:55 AM Marek Vasut wrote: On 3/28/22 08:54, Ye Li wrote: Hi Marek, Hi, [...] 2. Update the u-boot.itb offset in imx8mp-u-boot.dtsi, set the offset to 0x5f000. The previous offset 0x58000 is for SD, calculated by 0x6 - 0x80

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-31 Thread Tim Harvey
On Mon, Mar 28, 2022 at 7:55 AM Marek Vasut wrote: > > On 3/28/22 08:54, Ye Li wrote: > > Hi Marek, > > Hi, > > [...] > > >>> 2. Update the u-boot.itb offset in imx8mp-u-boot.dtsi, set the > >>> offset > >>> to 0x5f000. The previous offset 0x58000 is for SD, calculated by > >>> 0x6 - 0x8000 (

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-30 Thread Ye Li
On Thu, 2022-03-31 at 00:27 +0200, Marek Vasut wrote: > Caution: EXT Email > > On 3/29/22 11:56, Ye Li wrote: > > Hi, > > > > > > > > > > > > > > But once you want to upgrade the > > > > flash.bin, flexspi configurations will also be erased due to > > > > the > > > > block > > > > size. Then

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-30 Thread Fabio Estevam
Hi Marek, On Wed, Mar 30, 2022 at 7:27 PM Marek Vasut wrote: > I still believe it is better (=less confusing for users) to have one > unified flash.bin for all boot media. I agree with your approach. Thanks

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-30 Thread Marek Vasut
On 3/29/22 11:56, Ye Li wrote: Hi, But once you want to upgrade the flash.bin, flexspi configurations will also be erased due to the block size. Then you have to reprogram the configurations with flash.bin. So most of our customers add the flexspi configurations to flash.bin head. They don't us

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-29 Thread Ye Li
On Tue, 2022-03-29 at 11:01 +0200, Marek Vasut wrote: > Caution: EXT Email > > On 3/29/22 04:49, Ye Li wrote: > > Hi, > > > > > > > > > > > > > > If you change the ROM API driver, that will break our design. > > > > You > > > > can > > > > try to overwrite spl_romapi_get_uboot_base for your b

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-29 Thread Marek Vasut
On 3/29/22 04:49, Ye Li wrote: Hi, If you change the ROM API driver, that will break our design. You can try to overwrite spl_romapi_get_uboot_base for your board only. Since there are no users which boot from flexspi upstream, this design can still be fixed such that it does not require diffe

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-28 Thread Ye Li
On Mon, 2022-03-28 at 16:54 +0200, Marek Vasut wrote: > Caution: EXT Email > > On 3/28/22 08:54, Ye Li wrote: > > > > Hi Marek, > Hi, > > [...] > > > > > > > > > > > > > > 2. Update the u-boot.itb offset in imx8mp-u-boot.dtsi, set the > > > > offset > > > > to 0x5f000.  The previous offset

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-28 Thread Marek Vasut
On 3/28/22 08:54, Ye Li wrote: Hi Marek, Hi, [...] 2. Update the u-boot.itb offset in imx8mp-u-boot.dtsi, set the offset to 0x5f000.  The previous offset 0x58000 is for SD, calculated by 0x6 - 0x8000 (32KB image offset).   uboot: blob-ext@2 {   filename

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-27 Thread Ye Li
Hi Marek, On Wed, 2022-03-23 at 22:16 +0100, Marek Vasut wrote: > Caution: EXT Email > > On 3/23/22 03:42, Ye Li wrote: > > > > Hi Marek, > Hi, > > > > > > > > > > > > > > > > > > > > diff --git a/arch/arm/mach-imx/spl_imx_romapi.c > > > > > b/arch/arm/mach- > > > > > imx/spl_imx_romapi.c >

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-23 Thread Marek Vasut
On 3/23/22 03:42, Ye Li wrote: Hi Marek, Hi, diff --git a/arch/arm/mach-imx/spl_imx_romapi.c b/arch/arm/mach- imx/spl_imx_romapi.c index d827de375a6..c47f5a6bdb4 100644 --- a/arch/arm/mach-imx/spl_imx_romapi.c +++ b/arch/arm/mach-imx/spl_imx_romapi.c @@ -38,14 +38,8 @@ ulong spl_romapi_raw_se

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-22 Thread Ye Li
Hi Marek, On Mon, 2022-03-21 at 15:59 +0100, Marek Vasut wrote: > Caution: EXT Email > > On 3/21/22 04:35, Ye Li wrote: > > > > Hi Marek, > Hi, > > > > > > > > > diff --git a/arch/arm/mach-imx/spl_imx_romapi.c b/arch/arm/mach- > > > imx/spl_imx_romapi.c > > > index d827de375a6..c47f5a6bdb4 10

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-21 Thread Marek Vasut
On 3/21/22 04:35, Ye Li wrote: Hi Marek, Hi, diff --git a/arch/arm/mach-imx/spl_imx_romapi.c b/arch/arm/mach- imx/spl_imx_romapi.c index d827de375a6..c47f5a6bdb4 100644 --- a/arch/arm/mach-imx/spl_imx_romapi.c +++ b/arch/arm/mach-imx/spl_imx_romapi.c @@ -38,14 +38,8 @@ ulong spl_romapi_raw_se

Re: [EXT] [PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-20 Thread Ye Li
Hi Marek, On Wed, 2022-03-09 at 17:09 +0100, Marek Vasut wrote: > Caution: EXT Email > > The FlexSPI NOR boot offset does not require any special handling, > the image_offset is correct in either case (0x1000 for FlexSPI NOR > and 0x8000 for SD/eMMC) and the offset of u-boot.itb from the start >

[PATCH] ARM: imx: romapi: Repair FlexSPI NOR boot offset

2022-03-09 Thread Marek Vasut
The FlexSPI NOR boot offset does not require any special handling, the image_offset is correct in either case (0x1000 for FlexSPI NOR and 0x8000 for SD/eMMC) and the offset of u-boot.itb from the start of flash.bin is always 0x58000 on MX8MN/MX8MP, which matches the CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT