* [U-Boot] [PATCH] arm: socfpga: revert "set the fpga global bit to disable HPS to FPGA signals"
@ 2016-01-19 15:16 Dinh Nguyen
2016-01-19 15:50 ` Dinh Nguyen
2016-01-19 23:06 ` Marek Vasut
0 siblings, 2 replies; 4+ messages in thread
From: Dinh Nguyen @ 2016-01-19 15:16 UTC (permalink / raw)
To: u-boot
Revert "arm: socfpga: set the fpga global bit to disable HPS to FPGA signals"
Apparently, the logic for the FPGA global bit is not universal between Gen5
and Gen10 devices is not the same. Disabling this bit, while applicable to
Gen10 devices, will break FPGA programming on Gen5 devices.
Signed-off-by: Dinh Nguyen <dinguyen@opensource.altera.com>
---
drivers/fpga/socfpga.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/fpga/socfpga.c b/drivers/fpga/socfpga.c
index 431e159..4448250 100644
--- a/drivers/fpga/socfpga.c
+++ b/drivers/fpga/socfpga.c
@@ -269,7 +269,7 @@ int socfpga_load(Altera_desc *desc, const void *rbf_data, size_t rbf_size)
/* Prior programming the FPGA, all bridges need to be shut off */
/* Disable all signals from hps peripheral controller to fpga */
- writel(0, &sysmgr_regs->fpgaintfgrp_gbl);
+ writel(0, &sysmgr_regs->fpgaintfgrp_module);
/* Disable all signals from FPGA to HPS SDRAM */
#define SDR_CTRLGRP_FPGAPORTRST_ADDRESS 0x5080
--
1.9.1.315.g3f09db0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] arm: socfpga: revert "set the fpga global bit to disable HPS to FPGA signals"
2016-01-19 15:16 [U-Boot] [PATCH] arm: socfpga: revert "set the fpga global bit to disable HPS to FPGA signals" Dinh Nguyen
@ 2016-01-19 15:50 ` Dinh Nguyen
2016-01-19 23:03 ` Marek Vasut
2016-01-19 23:06 ` Marek Vasut
1 sibling, 1 reply; 4+ messages in thread
From: Dinh Nguyen @ 2016-01-19 15:50 UTC (permalink / raw)
To: u-boot
Hi Marek,
On 01/19/2016 09:16 AM, Dinh Nguyen wrote:
> Revert "arm: socfpga: set the fpga global bit to disable HPS to FPGA signals"
>
I apologize for the original patch "arm: socfpga: set the fpga global
bit to disable HPS to FPGA signals". I did not test the patch when it
was sent out. I did go back to testing it after I was able to
consolidate a bit the FPGA driver for Gen5 and Gen10 devices, and found
out that behavior for the global FPGA bit is different.
Dinh
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] arm: socfpga: revert "set the fpga global bit to disable HPS to FPGA signals"
2016-01-19 15:50 ` Dinh Nguyen
@ 2016-01-19 23:03 ` Marek Vasut
0 siblings, 0 replies; 4+ messages in thread
From: Marek Vasut @ 2016-01-19 23:03 UTC (permalink / raw)
To: u-boot
On Tuesday, January 19, 2016 at 04:50:06 PM, Dinh Nguyen wrote:
> Hi Marek,
Hi Dinh,
> On 01/19/2016 09:16 AM, Dinh Nguyen wrote:
> > Revert "arm: socfpga: set the fpga global bit to disable HPS to FPGA
> > signals"
>
> I apologize for the original patch "arm: socfpga: set the fpga global
> bit to disable HPS to FPGA signals". I did not test the patch when it
> was sent out. I did go back to testing it after I was able to
> consolidate a bit the FPGA driver for Gen5 and Gen10 devices, and found
> out that behavior for the global FPGA bit is different.
Good thing it went in after 2016.01 was out, so nothing that bad really
happened. Thanks for finding this.
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] arm: socfpga: revert "set the fpga global bit to disable HPS to FPGA signals"
2016-01-19 15:16 [U-Boot] [PATCH] arm: socfpga: revert "set the fpga global bit to disable HPS to FPGA signals" Dinh Nguyen
2016-01-19 15:50 ` Dinh Nguyen
@ 2016-01-19 23:06 ` Marek Vasut
1 sibling, 0 replies; 4+ messages in thread
From: Marek Vasut @ 2016-01-19 23:06 UTC (permalink / raw)
To: u-boot
On Tuesday, January 19, 2016 at 04:16:21 PM, Dinh Nguyen wrote:
> Revert "arm: socfpga: set the fpga global bit to disable HPS to FPGA
> signals"
>
> Apparently, the logic for the FPGA global bit is not universal between Gen5
> and Gen10 devices is not the same. Disabling this bit, while applicable to
> Gen10 devices, will break FPGA programming on Gen5 devices.
>
> Signed-off-by: Dinh Nguyen <dinguyen@opensource.altera.com>
Applied, thanks.
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-01-19 23:06 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-01-19 15:16 [U-Boot] [PATCH] arm: socfpga: revert "set the fpga global bit to disable HPS to FPGA signals" Dinh Nguyen
2016-01-19 15:50 ` Dinh Nguyen
2016-01-19 23:03 ` Marek Vasut
2016-01-19 23:06 ` Marek Vasut
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox