[Buildroot] [PATCH v3 2/2] configs/microchip_mpfs_icicle: add support for Microchip's Icicle Kit

[Jamie.Gibbons at microchip.com]

Hi Thomas,

On Thu, 2023-08-10 at 15:53 +0200, Thomas Petazzoni wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know
> the content is safe
> 
> Hello Jamie,
> 
> On Thu, 10 Aug 2023 10:46:30 +0000
> <Jamie.Gibbons at microchip.com> wrote:
> 
> > THe HSS code is located in eNVM of the Icicle kit for PolarFire SoC
> > and
> > built outside of Buildroot.
> 
> ACK.
> 
> > The HSS paylaod generator which is a host tool to create HSS
> > formatted
> > payloads is built by buildroot. This is covered in the microchip-
> > hss-
> > payload-generator package. This tool is released under dual
> > MIT/GPLv2+
> > licenses.
> > https://github.com/polarfire-soc/hart-software-services/blob/master/tools/hss-payload-generator/LICENSE.md
> > 
> > The HSS payload generator is the only software compnent here that is
> > built inside Buildroot. The HSS code and OpenSBI are both built
> > outside
> > Buildroot, so their licences are not a concern.
> > 
> > I hope this clears things up further.
> 
> This is all clear.
> 
> Can this HSS code + OpenSBI stored in the eNVM be updated by the user?

Yes, it can be updated by the user.
> 

> If it's the case, would it make sense for Buildroot to be able to
> build
> it?

However, it does not make sense for Buildroot to be able to built it.
The HSS is tightly coupled with our FPGA flow and because of this we
believe it is best to keep building/changing the HSS code separate from
the Linux system development.

> 
> > > Does that make sense in the RISC-V world?
> > 
> > After a discussion with colllegues we believe that the above two
> > configs
> > should be available when BR2_riscv_g is selected as suggested. This
> > does
> > make sense.
> 
> Cool, we agree!
> 
> > Would you like me to include this change in a patchset to fix the
> > configs for the microchip_mpfs_icicle_defconfig or do you want to
> > make
> > this update yourself?
> 
> Feel free to send a patch set :-)

Great. Will send over a patch set for all the previously mentioned
changes over the coming days.

Thanks for everything,

Jamie.