[U-Boot] [PATCHv4 0/3] drivers/ddr/altera: Add the DDR controller driver for SoCFPGA

Marek Vasut marex at denx.de
Fri Jul 17 22:22:11 CEST 2015


On Friday, July 17, 2015 at 09:58:20 PM, Dinh Nguyen wrote:
> On 07/12/2015 02:50 PM, Marek Vasut wrote:
> > On Friday, June 26, 2015 at 10:01:47 PM, Marek Vasut wrote:
> >> On Friday, June 26, 2015 at 06:43:13 PM, Marek Vasut wrote:
> >>> On Wednesday, June 03, 2015 at 05:52:47 AM,
> >>> dinguyen at opensource.altera.com
> >>> 
> >>> wrote:
> >>>> From: Dinh Nguyen <dinguyen at opensource.altera.com>
> >>>> 
> >>>> Hi,
> >>>> 
> >>>> This is v4 of the patch series that adds the DDR controller driver for
> >>>> Altera's SoCFPGA platform.
> >>>> 
> >>>> v4:
> >>>> - Further cleanup by removing comments that do not apply for Cyclone5.
> >>>> - Removed more unused functions
> >>>> 
> >>>> Thanks,
> >>> 
> >>> I applied this to u-boot/master and tried building for
> >>> socfpga_cyclone5, this is what I'm getting after I fixed these patches
> >>> to actually apply to u-boot/master (there was a minor conflict in
> >>> Makefile):
> >>> 
> >>> drivers/ddr/altera/sdram.c:11:35: fatal error: asm/arch/sdram_config.h:
> >>> No such file or directory
> >>> 
> >>>  #include <asm/arch/sdram_config.h>
> >> 
> >> OK, these files were misplaced by the patch. Now I got it to compile,
> >> only to see that when I try to init DRAM on sockit using those, PHY
> >> calibration fails (run_mem_calibrate() returns 1).
> >> 
> >> btw. I also think sdram_config.h should go into board/altera/ instead .
> > 
> > Just a quick heads-up, I'm cleaning this and the SPL up. I got SoCkit
> > booting from SD/MMC and I'm over 50 patches now. I'll submit them when
> > I'm done with this. I'd like to avoid merge conflicts now, so please
> > wait for my submission if possible.
> 
> So you want me to wait regarding the SDRAM patches?

Hi,

yeah, I have some insane amount of cleanup patches and fixes already. I
will post them once I'm done. What I am sorely missing is the UniPHY
register interface documentation, it looks like that is not public, so
I don't exactly know if what sequencer.c does is really correct. Sometimes
I have serious doubts about that too . Can you give me the documentation
please ?

Best regards,
Marek Vasut


More information about the U-Boot mailing list