RE: [PATCH 0/2] arm: socfpga: arria10: allow to reprogram FPGA with warm reboot
> -Original Message- > From: Michał Barnaś > Sent: Thursday, February 29, 2024 9:49 PM > To: Dinh Nguyen > Cc: u-boot@lists.denx.de; Marcel Ziswiler ; > Marek Vasut ; Martyn Welch > ; Paweł Anikiel ; Simon > Glass ; Simon Goldschmidt > ; Svyatoslav Ryhel > ; Chee, Tien Fong ; Tom > Rini > Subject: Re: [PATCH 0/2] arm: socfpga: arria10: allow to reprogram FPGA with > warm reboot > > On Thu, Feb 29, 2024 at 2:03 PM Dinh Nguyen wrote: > > > > > > > > On 2/22/24 09:20, Michał Barnaś wrote: > > > > > > By default, the board requires power cycle (cold boot) to program > > > the FPGA with bitstream. This change adds Kconfig that allows to > > > enable reprogramming the FPGA with every boot. This makes the update > > > process of the bitstream on the filesystem to be applied with simple > > > system reboot. > > > > > > > > > > If we want to enable the reprogramming on every boot, would it make > > sense to just do it and not even bother with the Kconfig option? > > > > Dinh > > The FPGA programming part takes quite a long time, so it increases the > reboot time significantly. > I don't think that everyone needs the reprogramming to happen every time. > We need that because our boards are closed in the lab and the power is not > easily accessible, so in case of update to the bitstream, we should be able to > remotely update it with a pure warm reboot. > So I thought that this should be set by Kconfig to not bother other users with > longer reboot times. > > Michał Reprogram should be avoided when FPGA already in user mode, this is for FPGA boot first use case. Full reprogram is slow in SPL, because very small buffer in this small OCRAM can only be used to process the full RBF. Since this is an optional configuration, I don't see any issue with these changes. Tien Fong
Re: [PATCH 0/2] arm: socfpga: arria10: allow to reprogram FPGA with warm reboot
On Thu, Feb 29, 2024 at 2:03 PM Dinh Nguyen wrote: > > > > On 2/22/24 09:20, Michał Barnaś wrote: > > > > By default, the board requires power cycle (cold boot) to program the > > FPGA with bitstream. This change adds Kconfig that allows to enable > > reprogramming the FPGA with every boot. This makes the update process > > of the bitstream on the filesystem to be applied with simple system > > reboot. > > > > > > If we want to enable the reprogramming on every boot, would it make > sense to just do it and not even bother with the Kconfig option? > > Dinh The FPGA programming part takes quite a long time, so it increases the reboot time significantly. I don't think that everyone needs the reprogramming to happen every time. We need that because our boards are closed in the lab and the power is not easily accessible, so in case of update to the bitstream, we should be able to remotely update it with a pure warm reboot. So I thought that this should be set by Kconfig to not bother other users with longer reboot times. Michał
Re: [PATCH 0/2] arm: socfpga: arria10: allow to reprogram FPGA with warm reboot
On 2/22/24 09:20, Michał Barnaś wrote: By default, the board requires power cycle (cold boot) to program the FPGA with bitstream. This change adds Kconfig that allows to enable reprogramming the FPGA with every boot. This makes the update process of the bitstream on the filesystem to be applied with simple system reboot. If we want to enable the reprogramming on every boot, would it make sense to just do it and not even bother with the Kconfig option? Dinh
[PATCH 0/2] arm: socfpga: arria10: allow to reprogram FPGA with warm reboot
By default, the board requires power cycle (cold boot) to program the FPGA with bitstream. This change adds Kconfig that allows to enable reprogramming the FPGA with every boot. This makes the update process of the bitstream on the filesystem to be applied with simple system reboot. Michał Barnaś (2): arm: socfpga: arria10: add option to reprogram the FPGA every reboot chameleonv3: set in defconfig for FPGA to reprogram every reboot arch/arm/mach-socfpga/Kconfig | 8 arch/arm/mach-socfpga/spl_a10.c | 8 configs/socfpga_chameleonv3_defconfig | 1 + 3 files changed, 17 insertions(+) -- 2.44.0.rc1.240.g4c46232300-goog