Re: [PATCH v2 0/9] Introduce HPE GXP Architecture

2022-06-22 Thread Hawkins, Nick
Greetings All, I just wanted to make sure I did not make a mistake with how I submitted the second version of this patch as I am still learning the u-boot patch submission process. Thanks ahead of time for any feedback. Thanks for your time, -Nick Hawkins

RE: [PATCH v1 6/9] ARM: dts: Add device tree files for hpe gxp soc

2022-06-03 Thread Hawkins, Nick
> You will need to post a sync periodically. Every Linux release (full, not > -rc) would be great, every couple is fine. If there are changes I need in the device tree for u-boot that are not currently in Linux do they need to be in Linux first? Or is that what the usage of appending "-u-boo

RE: [PATCH v1 6/9] ARM: dts: Add device tree files for hpe gxp soc

2022-06-02 Thread Hawkins, Nick
-Original Message- From: Hawkins, Nick Sent: Thursday, June 2, 2022 3:56 PM To: Tom Rini Cc: Verdun, Jean-Marie ; u-boot@lists.denx.de Subject: RE: [PATCH v1 6/9] ARM: dts: Add device tree files for hpe gxp soc > > > The dts files need to be in linux-next (or similar) upstre

RE: [PATCH v1 6/9] ARM: dts: Add device tree files for hpe gxp soc

2022-06-02 Thread Hawkins, Nick
> > > The dts files need to be in linux-next (or similar) upstream before we > > > start taking them in to Linux, and then re-synced periodically. > > > > Hi Tom, > > > > When you say taking them into Linux do you mean these files for U-Boot > > would go into Torvalds Linux? Ideally we would li

RE: [PATCH v1 7/9] configs: gxp: add core support

2022-06-02 Thread Hawkins, Nick
> Can we figure any of that out dynamically instead? Since > CONFIG_SYS_SDRAM_SIZE is only used (for ARM) in board code, I'd rather see > this handled in there, with SZ_xxx and not use CONFIG_SYS_SDRAM_SIZE at all. Hi Tom, Would something like this be acceptable? int dram_init(void) { #ifd

RE: [PATCH v1 6/9] ARM: dts: Add device tree files for hpe gxp soc

2022-06-01 Thread Hawkins, Nick
> > > > The HPE SoC is new to linux. A basic device tree layout with minimum > > required for linux to boot including a timer and watchdog support has > > been created. > > > > The dts file is empty at this point but will be updated in subsequent > > updates as board specific features are en