[PATCH 12/12] docs: andes-ae350.md: Update ae350 documentation for fdt driver support

[Yu-Chien Peter Lin]

On Thu, Sep 15, 2022 at 03:31:38AM +0100, Jessica Clarke wrote:
> On 15 Sept 2022, at 02:51, Yu Chien Peter Lin <peterlin at andestech.com> wrote:
> > 
> > We update ae350 documentation to add details about platform device tree.
> > The nodes and their properties must be provided to properly initialize 
> > data of underlying hardware and access their mmio registers.
> > 
> > Signed-off-by: Yu Chien Peter Lin <peterlin at andestech.com>
> > ---
> > docs/platform/andes-ae350.md | 191 ++++++++++++++++++++++++++++++++++-
> > 1 file changed, 187 insertions(+), 4 deletions(-)
> > 
> > diff --git a/docs/platform/andes-ae350.md b/docs/platform/andes-ae350.md
> > index 1cf83cf..2a0e242 100644
> > --- a/docs/platform/andes-ae350.md
> > +++ b/docs/platform/andes-ae350.md
> > @@ -18,13 +18,196 @@ The Andes AE350 platform does not have any platform-specific options.
> > Building Andes AE350 Platform
> > -----------------------------
> > 
> > -To use Linux v5.2 should be used to build Andes AE350 OpenSBI binaries by using
> > -the compile time option FW_FDT_PATH.
> > -
> > -AE350's dts is included in https://github.com/andestech/linux/tree/ast-v3_2_0-release-public
> > +AE350's dts is included in https://github.com/andestech/linux/tree/RISCV-Linux-5.4-ast-v5_1_0-branch
> > 
> > **Linux Kernel Payload**
> > 
> > ```
> > make PLATFORM=andes/ae350 FW_PAYLOAD_PATH=<linux_build_directory>/arch/riscv/boot/Image FW_FDT_PATH=<ae350.dtb path>
> > ```
> > +
> > +DTS Example: (Quad-core AX45MP)
> > +-------------------------------
> > +
> > +```
> > +	cpus {
> > +		#address-cells = <1>;
> > +		#size-cells = <0>;
> > +		timebase-frequency = <60000000>;
> > +
> > +		CPU0: cpu at 0 {
> > +			device_type = "cpu";
> > +			reg = <0>;
> > +			status = "okay";
> > +			compatible = "riscv";
> > +			riscv,isa = "rv64i2p0m2p0a2p0f2p0d2p0c2p0xv5-1p1xdsp0p0";
> 
> The versions mean it won’t validate even with the updated bindings in [1].

Hi Jessica,

Thanks, we'll update it to comply with bindings.

> > ...
> > +		wdt: wdt at f0500000 {
> > +			compatible = "andestech,atcwdt200";
> 
> Assuming atc stands for AndesTeCh that seems a bit redundant?
> 
> Jess

This is not an abbreviation but a term used in our manual and kernel driver.

Best regards,
Peter Lin

> 
> [1] https://lore.kernel.org/all/20220823183319.3314940-4-mail@conchuod.ie/
> 
> > +			reg = <0x00000000 0xf0500000 0x00000000 0x00001000>;
> > +			interrupts = <3 4>;
> > +			interrupt-parent = <&plic0>;
> > +			clock-frequency = <15000000>;
> > +		};
> > +
> > +		serial0: serial at f0300000 {
> > +			compatible = "andestech,uart16550", "ns16550a";
> > +			reg = <0x00000000 0xf0300000 0x00000000 0x00001000>;
> > +			interrupts = <9 4>;
> > +			interrupt-parent = <&plic0>;
> > +			clock-frequency = <19660800>;
> > +			current-speed = <38400>;
> > +			reg-shift = <2>;
> > +			reg-offset = <32>;
> > +			reg-io-width = <4>;
> > +			no-loopback-test = <1>;
> > +		};
> > +
> > +		smu: smu at f0100000 {
> > +			compatible = "andestech,atcsmu";
> > +			reg = <0x00000000 0xf0100000 0x00000000 0x00001000>;
> > +		};
> > +	};
> > +```
> > -- 
> > 2.34.1
> > 
> > 
> > -- 
> > opensbi mailing list
> > opensbi at lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/opensbi
>