Re: [U-Boot] [PATCH] ARM SOCFPGA: add resetmgr command so reset can be deasserted in bootcmd (for example on peripheral dma interfaces after fpga has been programmed).

2017-12-16 Thread Frank Mori Hess
Marek Vasut wrote: > Please always CC the list. Do NOT top-post. You do realize I was replying to an email you sent to my personal address and you didn't even send to the list? > What is your goal here ? To put things in context, my larger goal is to update u-boot from the old version altera

[U-Boot] [PATCH] ARM SOCFPGA: add resetmgr command so reset can be deasserted in bootcmd (for example on peripheral dma interfaces after fpga has been programmed).

2017-12-14 Thread Frank Mori Hess
--- arch/arm/mach-socfpga/reset_manager_gen5.c | 31 + + 1 file changed, 31 insertions(+) diff --git a/arch/arm/mach-socfpga/reset_manager_gen5.c b/arch/arm/mach- socfpga/reset_manager_gen5.c index aa88adb414..6ad5d2a362 100644 ---

[U-Boot] [PATCH] ARM SOCFPGA: add resetmgr command so reset can be deasserted in bootcmd (for example on peripheral dma interfaces after fpga has been programmed).

2017-12-14 Thread Frank Mori Hess
--- arch/arm/mach-socfpga/reset_manager_gen5.c | 31 ++ 1 file changed, 31 insertions(+) diff --git a/arch/arm/mach-socfpga/reset_manager_gen5.c b/arch/arm/mach-socfpga/reset_manager_gen5.c index aa88adb414..6ad5d2a362 100644 ---

[U-Boot] tiny printf limits

2017-12-05 Thread Frank Mori Hess
Shouldn't tiny printf abort or something when it encounters a printf format string it doesn't support? Right now, it just silently skips it while leaving the associated argument in place, resulting in the arguments getting shifted onto the wrong format strings. -- Frank

Re: [U-Boot] arm socfpga: Revert "spi: cadence_qspi_apb: Support 32 bit AHB address"

2017-12-04 Thread Frank Mori Hess
Since your commit broke my platform to fix yours, shouldn't it be reverted and TI platforms use your pending patch queue? On Dec 3, 2017 23:14, "Vignesh R" <vigne...@ti.com> wrote: On Sunday 03 December 2017 09:29 PM, Frank Mori Hess wrote: >

[U-Boot] arm socfpga: Revert "spi: cadence_qspi_apb: Support 32 bit AHB address"

2017-12-03 Thread Frank Mori Hess
This reverts commit dac3bf20fb2c9b03476be0d73db620f62ab3cee1. Signed-off-by: Frank Mori Hess <fmh...@gmail.com> --- drivers/spi/cadence_qspi_apb.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/spi/cadence_qspi_apb.c b/drivers/spi/cadence_qspi_apb.c

Re: [U-Boot] arm socfpga: Revert "spi: cadence_qspi_apb: Support 32 bit AHB address"

2017-12-03 Thread Frank Mori Hess
Oops, ignore that patch I didn't merge it correctly On Sun, Dec 3, 2017 at 10:36 AM, Frank Mori Hess <fmh...@gmail.com> wrote: > This reverts commit dac3bf20fb2c9b03476be0d73db620f62ab3cee1. > > Signed-off-by: Frank Mori Hess <fmh...@gmail.com> > --- > drivers

Re: [U-Boot] arm socfpga: Revert "spi: cadence_qspi_apb: Support 32 bit AHB address"

2017-12-03 Thread Frank Mori Hess
On Sun, Dec 3, 2017 at 10:49 AM, Fabio Estevam <feste...@gmail.com> wrote: > On Sun, Dec 3, 2017 at 1:36 PM, Frank Mori Hess <fmh...@gmail.com> wrote: >> This reverts commit dac3bf20fb2c9b03476be0d73db620f62ab3cee1. > > Please explain the reasoning for the revert. It

[U-Boot] arm socfpga: Revert "spi: cadence_qspi_apb: Support 32 bit AHB address"

2017-12-03 Thread Frank Mori Hess
. I'm was using version 2016.11 of u-boot. Signed-off-by: Frank Mori Hess <fmh...@gmail.com> --- drivers/spi/cadence_qspi_apb.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/spi/cadence_qspi_apb.c b/drivers/spi/cadence_qspi_apb.c index e02f2217f4..b300f36607

Re: [U-Boot] arm socfpga: Revert "spi: cadence_qspi_apb: Support 32 bit AHB address"

2017-12-03 Thread Frank Mori Hess
>plat->ahbbase & CQSPI_INDIRECTTRIGGER_ADDR_MASK = 0x0 for SoCFPGA, but now >you changed it such that 0xffa0 is written into the register. Same does >apply for all your changes below. On Sun, Dec 3, 2017 at 10:54 AM, Fabio Estevam <feste...@gmail.com> wrote: > On Sun,

[U-Boot] commit dac3bf20fb2c9b03476be0d73db620f62ab3cee1 breaks cadence driver

2017-12-03 Thread Frank Mori Hess
Hi, I've been debugging why u-boot spl crashes in a loop when I boot off a cadence qspi flash. I narrowed it down to the changes from commit dac3bf20fb2c9b03476be0d73db620f62ab3cee1 which removes CQSPI_INDIRECTTRIGGER_ADDR_MASK. Restoring the mask allows the spl to successfully load the main

Re: [U-Boot] arm socfpga: Revert "spi: cadence_qspi_apb: Support 32 bit AHB address"

2017-12-04 Thread Frank Mori Hess
On Mon, Dec 4, 2017 at 11:41 AM, Vignesh R <vigne...@ti.com> wrote: > Hi, > > On 04-Dec-17 6:41 PM, Frank Mori Hess wrote: >> Since your commit broke my platform to fix yours, shouldn't it be >> reverted and TI platforms use your pending patch queue? >>