On November 18, 2024 thus sayeth Andrew Davis: > On 11/17/24 11:34 PM, Bryan Brattlof wrote: > > From: Vignesh Raghavendra <vigneshr@xxxxxx> > > > > Add the initial board file for the AM62L3's Evaluation Module. > > > > Signed-off-by: Vignesh Raghavendra <vigneshr@xxxxxx> > > Signed-off-by: Bryan Brattlof <bb@xxxxxx> > > --- > > arch/arm64/boot/dts/ti/k3-am62l3-evm.dts | 54 ++++++++++++++++++++++++++++++++ > > 1 file changed, 54 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts > > new file mode 100644 > > index 0000000000000..2d59389765cab > > --- /dev/null > > +++ b/arch/arm64/boot/dts/ti/k3-am62l3-evm.dts > > @@ -0,0 +1,54 @@ > > +// SPDX-License-Identifier: GPL-2.0-only or MIT > > +/* > > + * Device Tree file for the AM62L3 Evaluation Module > > + * Copyright (C) 2024 Texas Instruments Incorporated - https://www.ti.com/ > > + * > > + * Technical Reference Manual: https://www.ti.com/lit/ug/sprujb4/sprujb4.pdf > > + */ > > + > > +/dts-v1/; > > + > > +#include "k3-am62l3.dtsi" > > + > > +/ { > > + compatible = "ti,am62l3-evm", "ti,am62l3"; > > + model = "Texas Instruments AM62L3 Evaluation Module"; > > + > > + aliases { > > + serial2 = &main_uart0; > > We usually have WKUP and MCU UARTs as serial0 and serial1, we don't > have that many, so we would never have serial1, only 0 and 2. Might > be time we drop this odd numbering convention, at least for AM62L > class parts.. > Good point. I'll see what I can come up with > > + }; > > + > > + chosen { > > + stdout-path = &main_uart0; > > + }; > > + > > + memory@80000000 { > > + reg = <0x00000000 0x80000000 0x00000000 0x80000000>; > > + device_type = "memory"; > > + bootph-all; > > + }; > > + > > + reserved-memory { > > Not needed until you have something to reserve. > Ah I agree. I'll drop this next time. > > + ranges; > > + #address-cells = <2>; > > + #size-cells = <2>; > > + }; > > +}; > > + > > +&pmx0 { > > + main_uart0_pins_default: main_uart0-default-pins { > > + pinctrl-single,pins = < > > + AM62LX_IOPAD(0x01d4, PIN_INPUT, 1) /* (D7) UART0_RXD */ > > + AM62LX_IOPAD(0x01d8, PIN_OUTPUT, 1) /* (A6) UART0_TXD */ > > + >; > > + bootph-all; > > + }; > > +}; > > + > > +&main_uart0 { > > + current-speed = <115200>; > > current-speed is only for UARTs that can't detect their current speed, > our UART and driver can, so this line isn't needed. > Nice! I'll drop this property as well ~Bryan
