Re: [PATCH 0/8] Introduce initial TI's am62a support

2022-11-08 Thread Bryan Brattlof
On November  4, 2022 thus sayeth Andrew Davis:
> On 11/4/22 8:08 AM, Tom Rini wrote:
> > On Fri, Nov 04, 2022 at 11:49:39AM +, Peter Robinson wrote:
> > > Hi Bryan,
> > > 
> > > > This series will introduce basic support (SD and UART) support for Texas
> > > > Instruments AM62Ax SK EVM.
> > > > 
> > > > The am62ax shares many of the same features as the am62x however it uses
> > > > a new 32bit controller and therefore depends on the patch I sent last
> > > > week updating the macros used by the k3-ddrss ram driver[0].
> > > > 
> > > > Here is some proof of life & more documentation if you're interested :)
> > > > 
> > > > Bootlog:https://paste.sr.ht/~bryanb/e0a418ba7dd452749d2dd1efb5e91b2875a01708
> > > > Technical Reference Manual:https://www.ti.com/lit/zip/spruj16
> > > > Schematics:https://www.ti.com/lit/zip/sprr459
> > > 
> > > Does this board need a readme for how to build the firmware, these
> > > days there generally needs to be ATF and probably a slew of other
> > > firmwares linked into a FIT image or similar to build the entire
> > > firmware bundle, a readme would likely be a useful addition for people
> > > getting started if there's not a generic TI 64 bit build doc, and if
> > > there is that likely needs an update to include this SoC/board. I
> > > didn't see anything that looked like that in the file list below.
> > 
> > Agreed, something under doc/board/ti/ is needed as well for the series,
> > thanks!
> > 
> 
> I see we do have some files at board/ti/{j721e,j721s2}/README with some
> good info on all this firmware source/building. Much of that info is
> common and could be factored out into a "generic TI 64 bit build doc".
> 

Cool! I can add this to the series :)

Thanks for reviewing everyone
~Bryan


Re: [PATCH 0/8] Introduce initial TI's am62a support

2022-11-04 Thread Andrew Davis

On 11/4/22 8:08 AM, Tom Rini wrote:

On Fri, Nov 04, 2022 at 11:49:39AM +, Peter Robinson wrote:

Hi Bryan,


This series will introduce basic support (SD and UART) support for Texas
Instruments AM62Ax SK EVM.

The am62ax shares many of the same features as the am62x however it uses
a new 32bit controller and therefore depends on the patch I sent last
week updating the macros used by the k3-ddrss ram driver[0].

Here is some proof of life & more documentation if you're interested :)

Bootlog:https://paste.sr.ht/~bryanb/e0a418ba7dd452749d2dd1efb5e91b2875a01708
Technical Reference Manual:https://www.ti.com/lit/zip/spruj16
Schematics:https://www.ti.com/lit/zip/sprr459


Does this board need a readme for how to build the firmware, these
days there generally needs to be ATF and probably a slew of other
firmwares linked into a FIT image or similar to build the entire
firmware bundle, a readme would likely be a useful addition for people
getting started if there's not a generic TI 64 bit build doc, and if
there is that likely needs an update to include this SoC/board. I
didn't see anything that looked like that in the file list below.


Agreed, something under doc/board/ti/ is needed as well for the series,
thanks!



I see we do have some files at board/ti/{j721e,j721s2}/README with some
good info on all this firmware source/building. Much of that info is
common and could be factored out into a "generic TI 64 bit build doc".

Andrew


Re: [PATCH 0/8] Introduce initial TI's am62a support

2022-11-04 Thread Tom Rini
On Fri, Nov 04, 2022 at 11:49:39AM +, Peter Robinson wrote:
> Hi Bryan,
> 
> > This series will introduce basic support (SD and UART) support for Texas
> > Instruments AM62Ax SK EVM.
> >
> > The am62ax shares many of the same features as the am62x however it uses
> > a new 32bit controller and therefore depends on the patch I sent last
> > week updating the macros used by the k3-ddrss ram driver[0].
> >
> > Here is some proof of life & more documentation if you're interested :)
> >
> > Bootlog:https://paste.sr.ht/~bryanb/e0a418ba7dd452749d2dd1efb5e91b2875a01708
> > Technical Reference Manual:https://www.ti.com/lit/zip/spruj16
> > Schematics:https://www.ti.com/lit/zip/sprr459
> 
> Does this board need a readme for how to build the firmware, these
> days there generally needs to be ATF and probably a slew of other
> firmwares linked into a FIT image or similar to build the entire
> firmware bundle, a readme would likely be a useful addition for people
> getting started if there's not a generic TI 64 bit build doc, and if
> there is that likely needs an update to include this SoC/board. I
> didn't see anything that looked like that in the file list below.

Agreed, something under doc/board/ti/ is needed as well for the series,
thanks!

-- 
Tom


signature.asc
Description: PGP signature


Re: [PATCH 0/8] Introduce initial TI's am62a support

2022-11-04 Thread Peter Robinson
Hi Bryan,

> This series will introduce basic support (SD and UART) support for Texas
> Instruments AM62Ax SK EVM.
>
> The am62ax shares many of the same features as the am62x however it uses
> a new 32bit controller and therefore depends on the patch I sent last
> week updating the macros used by the k3-ddrss ram driver[0].
>
> Here is some proof of life & more documentation if you're interested :)
>
> Bootlog:https://paste.sr.ht/~bryanb/e0a418ba7dd452749d2dd1efb5e91b2875a01708
> Technical Reference Manual:https://www.ti.com/lit/zip/spruj16
> Schematics:https://www.ti.com/lit/zip/sprr459

Does this board need a readme for how to build the firmware, these
days there generally needs to be ATF and probably a slew of other
firmwares linked into a FIT image or similar to build the entire
firmware bundle, a readme would likely be a useful addition for people
getting started if there's not a generic TI 64 bit build doc, and if
there is that likely needs an update to include this SoC/board. I
didn't see anything that looked like that in the file list below.

Peter

> Thanks for reviewing!
> ~Bryan
>
> [0] https://lore.kernel.org/u-boot/20221024215328.22373-1...@ti.com/
>
> Bryan Brattlof (8):
>   arm: dts: introduce am62a7 dtbs from linux kernel
>   arm: dts: introduce am62a7 u-boot dtbs
>   ram: k3-ddrss: add am62a controller support
>   soc: ti: k3-socinfo: add am62a SoC entry
>   arm: mach-k3: introduce basic files to support the am62a
>   arm: mach-k3: am62a: introduce auto-generated SoC data
>   board: ti: introduce the basic files needed to support the am62a
>   configs: introduce configs for the am62a
>
>  arch/arm/dts/Makefile |3 +
>  arch/arm/dts/k3-am62a-ddr-1866mhz-32bit.dtsi  | 2798 
>  arch/arm/dts/k3-am62a-ddr.dtsi| 2814 +
>  arch/arm/dts/k3-am62a-main.dtsi   |  298 ++
>  arch/arm/dts/k3-am62a-mcu.dtsi|   39 +
>  arch/arm/dts/k3-am62a-wakeup.dtsi |   54 +
>  arch/arm/dts/k3-am62a.dtsi|  122 +
>  arch/arm/dts/k3-am62a7-r5-sk.dts  |  143 +
>  arch/arm/dts/k3-am62a7-sk-u-boot.dtsi |  140 +
>  arch/arm/dts/k3-am62a7-sk.dts |  223 ++
>  arch/arm/dts/k3-am62a7.dtsi   |  103 +
>  arch/arm/mach-k3/Kconfig  |   14 +-
>  arch/arm/mach-k3/Makefile |2 +
>  arch/arm/mach-k3/am62a7_init.c|  250 ++
>  arch/arm/mach-k3/am62ax/Makefile  |6 +
>  arch/arm/mach-k3/am62ax/clk-data.c|  317 ++
>  arch/arm/mach-k3/am62ax/dev-data.c|   73 +
>  arch/arm/mach-k3/arm64-mmu.c  |6 +-
>  .../arm/mach-k3/include/mach/am62a_hardware.h |   74 +
>  arch/arm/mach-k3/include/mach/am62a_spl.h |   49 +
>  arch/arm/mach-k3/include/mach/hardware.h  |4 +
>  arch/arm/mach-k3/include/mach/spl.h   |4 +
>  board/ti/am62ax/Kconfig   |   52 +
>  board/ti/am62ax/MAINTAINERS   |9 +
>  board/ti/am62ax/Makefile  |7 +
>  board/ti/am62ax/evm.c |   31 +
>  configs/am62ax_evm_a53_defconfig  |   79 +
>  configs/am62ax_evm_r5_defconfig   |  106 +
>  drivers/clk/ti/clk-k3.c   |6 +
>  drivers/firmware/ti_sci_static_data.h |4 +-
>  drivers/power/domain/ti-power-domain.c|6 +
>  drivers/ram/Kconfig   |1 +
>  drivers/ram/k3-ddrss/k3-ddrss.c   |1 +
>  drivers/soc/soc_ti_k3.c   |4 +
>  include/configs/am62ax_evm.h  |   68 +
>  include/dt-bindings/pinctrl/k3.h  |3 +
>  include/k3-clk.h  |1 +
>  include/k3-dev.h  |1 +
>  38 files changed, 7906 insertions(+), 9 deletions(-)
>  create mode 100644 arch/arm/dts/k3-am62a-ddr-1866mhz-32bit.dtsi
>  create mode 100644 arch/arm/dts/k3-am62a-ddr.dtsi
>  create mode 100644 arch/arm/dts/k3-am62a-main.dtsi
>  create mode 100644 arch/arm/dts/k3-am62a-mcu.dtsi
>  create mode 100644 arch/arm/dts/k3-am62a-wakeup.dtsi
>  create mode 100644 arch/arm/dts/k3-am62a.dtsi
>  create mode 100644 arch/arm/dts/k3-am62a7-r5-sk.dts
>  create mode 100644 arch/arm/dts/k3-am62a7-sk-u-boot.dtsi
>  create mode 100644 arch/arm/dts/k3-am62a7-sk.dts
>  create mode 100644 arch/arm/dts/k3-am62a7.dtsi
>  create mode 100644 arch/arm/mach-k3/am62a7_init.c
>  create mode 100644 arch/arm/mach-k3/am62ax/Makefile
>  create mode 100644 arch/arm/mach-k3/am62ax/clk-data.c
>  create mode 100644 arch/arm/mach-k3/am62ax/dev-data.c
>  create mode 100644 arch/arm/mach-k3/include/mach/am62a_hardware.h
>  create mode 100644 arch/arm/mach-k3/include/mach/am62a_spl.h
>  create mode 100644 board/ti/am62ax/Kconfig
>  create mode 100644 board/ti/am62ax/MAINTAINERS
>  create mode 100644 board/ti/am62ax/Makefile
>  create mode 100644