[PATCH 2/2] configs: socfpga: Add QSPI boot for Arria 10 SoCDK

[Ley Foon Tan]

On Tue, Mar 3, 2020 at 8:16 PM Marek Vasut <marex at denx.de> wrote:
>
> On 3/3/20 10:21 AM, Ley Foon Tan wrote:
> > On Mon, Mar 2, 2020 at 6:40 PM Marek Vasut <marex at denx.de> wrote:
> >>
> >> On 3/2/20 10:33 AM, Ley Foon Tan wrote:
> >>> On Fri, Feb 21, 2020 at 9:25 AM Ley Foon Tan <ley.foon.tan at intel.com> wrote:
> >>>>
> >>>> Add QSPI boot settings for Arria 10 SoCDK.
> >>>>
> >>>> Signed-off-by: Ley Foon Tan <ley.foon.tan at intel.com>
> >>>> ---
> >>>>  include/configs/socfpga_arria10_socdk.h | 9 +++++++++
> >>>>  1 file changed, 9 insertions(+)
> >>>>
> >>>> diff --git a/include/configs/socfpga_arria10_socdk.h b/include/configs/socfpga_arria10_socdk.h
> >>>> index 645e66e6b0..e1d01c095f 100644
> >>>> --- a/include/configs/socfpga_arria10_socdk.h
> >>>> +++ b/include/configs/socfpga_arria10_socdk.h
> >>>> @@ -39,6 +39,15 @@
> >>>>  /* SPL memory allocation configuration, this is for FAT implementation */
> >>>>  #define CONFIG_SYS_SPL_MALLOC_SIZE     0x00015000
> >>>>
> >>>> +#define KERNEL_FIT_ADDR                __stringify(0x1200000)
> >>>> +
> >>>> +#define SOCFPGA_BOOT_SETTINGS \
> >>>> +       "kernelfit_addr=" KERNEL_FIT_ADDR "\0" \
> >>>> +       "qspiboot=setenv bootargs " CONFIG_BOOTARGS \
> >>>> +                       "root=/dev/mtdblock1 rw rootfstype=jffs2;" \
> >>>> +                       "bootm ${scriptaddr}\0" \
> >>>> +       "qspiload=sf probe; sf read ${scriptaddr} ${kernelfit_addr}\0" \
> >>>> +
> >>>>  /* The rest of the configuration is shared */
> >>>>  #include <configs/socfpga_common.h>
> >>>>
> >>>
> >>> Any comment on this patch?
> >>
> >> Can we get rid of rootfstype=jffs2 ? It's archaic and you can use UBI on
> >> top of SPI NOR too. Also, isn't there already some kernel_addr_r for the
> >> kernel address ?
> > Yes, I know SPI flash can use UBI FS too, but we only enable jffs2 now.
>
> JFFS2 is dead for a very long time. I only ever met it on some archaic
> altera hardware, everywhere else it's UBI, which makes me wonder -- what
> is the reason Altera is sticking to this antique ?

We haven't enable UBI for Gen5. Tien Fong did enable UBI for A10 and
submit the patches before.
But, he is busy on other tasks now and haven't continue rework the patches.
But, Gen5 haven't enable UBI yet.

>
> > kernelfit_addr is for kernel fit image offset in SPI flash, it is
> > different from kernel_addr_r.
>
> Shouldn't the MTD layout of the SPI NOR be described in mtdparts ?
It doesn't use mtdparts now. I will try enable it.


Regards
Ley Foon