Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-21 Thread Sebastian Huber
On 19/06/2021 16:35, dufa...@hda.com wrote: I'm getting back to this as I have the HyperRAM working so I'm trying to set up appropriate linker settings. On Jun 10, 2021, at 01:43 , Sebastian Huber wrote: The initial stack needs to be in an accessible memory area. Currently it is placed

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-19 Thread dufault
I'm getting back to this as I have the HyperRAM working so I'm trying to set up appropriate linker settings. > On Jun 10, 2021, at 01:43 , Sebastian Huber > wrote: > > The initial stack needs to be in an accessible memory area. Currently it is > placed in this linker output section: > >

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-09 Thread Sebastian Huber
On 09/06/2021 20:47, Peter Dufault wrote: Similar comments apply to RAM. I'll be adding HyperRAM attached to FlexSPI port A chip select 2 and it won't be setup until we get past "bsp_start_hook_0()". So initial stack needs to be in on-chip RAM and we can't access anything outside of on-chip

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-09 Thread Chris Johns
On 9/6/21 5:05 pm, Christian Mauderer wrote: > On 09/06/2021 01:52, Chris Johns wrote: >> On 8/6/21 8:26 pm, Sebastian Huber wrote: >>> On 08/06/2021 05:07, Chris Johns wrote: On 7/6/21 6:40 pm, Christian Mauderer wrote:> I think the Options don't need documentation changes because

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-09 Thread Peter Dufault
Let's get back to the patch itself and the initialization of "FlexSPI" peripheral. The FlexSPI is one of the "imxrt" on-chip devices that can have flash attached to it, and that the on-chip boot ROM knows how to probe, configure, and then boot from. The "imxrt" chips don't have on-chip flash,

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-09 Thread Christian Mauderer
On 09/06/2021 01:52, Chris Johns wrote: On 8/6/21 8:26 pm, Sebastian Huber wrote: On 08/06/2021 05:07, Chris Johns wrote: On 7/6/21 6:40 pm, Christian Mauderer wrote:> I think the Options don't need documentation changes because the options in the waf based build system are now documented

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-08 Thread Chris Johns
On 8/6/21 8:26 pm, Sebastian Huber wrote: > On 08/06/2021 05:07, Chris Johns wrote: >> On 7/6/21 6:40 pm, Christian Mauderer wrote:> I think the Options don't need >> documentation changes because the options in the >>> waf based build system are now documented directly in the yaml files and >>>

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-08 Thread Gedare Bloom
On Tue, Jun 8, 2021 at 5:37 AM Christian Mauderer wrote: > > Hello Chris, > > On 08/06/2021 12:05, Chris Johns wrote: > > On 8/6/21 7:08 pm, Christian Mauderer wrote: > >> Hello Chris, > >> > >> On 08/06/2021 05:07, Chris Johns wrote: > >>> On 7/6/21 6:40 pm, Christian Mauderer wrote:> I think

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-08 Thread Christian Mauderer
Hello Chris, On 08/06/2021 12:05, Chris Johns wrote: On 8/6/21 7:08 pm, Christian Mauderer wrote: Hello Chris, On 08/06/2021 05:07, Chris Johns wrote: On 7/6/21 6:40 pm, Christian Mauderer wrote:> I think the Options don't need documentation changes because the options in the waf based

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-08 Thread Sebastian Huber
On 08/06/2021 05:07, Chris Johns wrote: On 7/6/21 6:40 pm, Christian Mauderer wrote:> I think the Options don't need documentation changes because the options in the waf based build system are now documented directly in the yaml files and printed if you generate the default config. Hmm I am

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-08 Thread Chris Johns
On 8/6/21 7:08 pm, Christian Mauderer wrote: > Hello Chris, > > On 08/06/2021 05:07, Chris Johns wrote: >> On 7/6/21 6:40 pm, Christian Mauderer wrote:> I think the Options don't need >> documentation changes because the options in the >>> waf based build system are now documented directly in the

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-08 Thread Christian Mauderer
Hello Chris, On 08/06/2021 05:07, Chris Johns wrote: On 7/6/21 6:40 pm, Christian Mauderer wrote:> I think the Options don't need documentation changes because the options in the waf based build system are now documented directly in the yaml files and printed if you generate the default

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-07 Thread Chris Johns
On 7/6/21 6:40 pm, Christian Mauderer wrote:> I think the Options don't need documentation changes because the options in the > waf based build system are now documented directly in the yaml files and > printed > if you generate the default config. Hmm I am not sure I agree with the premise the

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-07 Thread Christian Mauderer
Hello Gedare, I think the Options don't need documentation changes because the options in the waf based build system are now documented directly in the yaml files and printed if you generate the default config. But I think I should add a documentation for the ARM PLL. I'll send a patch as

Re: [PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-04 Thread Gedare Bloom
does this one need doco update for the option changes? On Fri, Jun 4, 2021 at 1:48 AM Christian Mauderer wrote: > > Calling the memory FLASH and EXTRAM instead of FLEXSPI and SDRAM makes > it simpler to support other types of external RAM. This patch also > removes some of the calculations and

[PATCH rtems 2/2] bsps/imxrt: Simplify linkcmds and make it flexible

2021-06-04 Thread Christian Mauderer
Calling the memory FLASH and EXTRAM instead of FLEXSPI and SDRAM makes it simpler to support other types of external RAM. This patch also removes some of the calculations and improves names and documentation to avoid pitfalls. It removes a unnecessary memory definition. Update #4180 ---