linux-kernel - Re: [PATCH v3 2/2] arm64: dts: ti: Add k3-am67a-beagley-ai

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAOCHtYg8DTuGncoxKPvTOyKvGrF-zNMwVvKcNWzRHxT5Hvt7nw@mail.gmail.com>
Date: Wed, 28 Aug 2024 11:34:20 -0500
From: Robert Nelson <robertcnelson@...il.com>
To: Nishanth Menon <nm@...com>
Cc: linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org, 
	devicetree@...r.kernel.org, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Vignesh Raghavendra <vigneshr@...com>, Andrew Davis <afd@...com>, Jai Luthra <j-luthra@...com>, 
	Roger Quadros <rogerq@...nel.org>, Siddharth Vadapalli <s-vadapalli@...com>, 
	Jared McArthur <j-mcarthur@...com>, Jason Kridner <jkridner@...gleboard.org>, 
	Deepak Khatri <lorforlinux@...gleboard.org>, Drew Fustini <drew@...gleboard.org>
Subject: Re: [PATCH v3 2/2] arm64: dts: ti: Add k3-am67a-beagley-ai

On Wed, Aug 28, 2024 at 9:38 AM Robert Nelson <robertcnelson@...il.com> wrote:
>
> On Sat, Aug 24, 2024 at 1:44 PM Nishanth Menon <nm@...com> wrote:
> >
> > On 12:04-20240822, Robert Nelson wrote:
> > Minor nmits below:
> >
> > > BeagleBoard.org BeagleY-AI is an easy to use, affordable open source
> > > hardware single board computer based on the Texas Instruments AM67A,
> > > which features a quad-core 64-bit Arm CPU subsystem, 2 general-purpose
> > > digital-signal-processors (DSP) and matrix-multiply-accelerators (MMA),
> > > GPU, vision and deep learning accelerators, and multiple Arm Cortex-R5
> > > cores for low-power, low-latency GPIO control.
> > >
> > [...]
> > > +
> > > +     vdd_3v3: regulator-2 {
> > > +             compatible = "regulator-fixed";
> > > +             regulator-name = "vdd_3v3";
> > > +             regulator-min-microvolt = <3300000>;
> > > +             regulator-max-microvolt = <3300000>;
> > > +             vin-supply = <&vsys_5v0>;
> > > +             regulator-always-on;
> > > +             regulator-boot-on;
> > > +     };
> > > +
> > > +     vdd_mmc1: regulator-mmc1 {
> >
> >         Also responding:
> >
> > > Okay, i'll change these... I already see a problem, as I've got two
> > > 3v3... can we use 'regulator-[0-9]v[0-9]-X' ?
> >
> >
> > let us just call them regulator-3 and so on incrementally. You can
> > already name with regulator-name property. We really don't need the "v'
> > specification here.
>
> Okay, renamed all as `regulator-X`
>
>
> >
> > [...]
> > > +&wkup_i2c0 {
> > > +     pinctrl-names = "default";
> > > +     pinctrl-0 = <&wkup_i2c0_pins_default>;
> > > +     clock-frequency = <100000>;
> > > +     bootph-all;
> > > +     status = "okay";
> > > +
> > > +     tps65219: pmic@30 {
> > > +             compatible = "ti,tps65219";
> > > +             reg = <0x30>;
> > > +             buck1-supply = <&vsys_5v0>;
> > > +             buck2-supply = <&vsys_5v0>;
> > > +             buck3-supply = <&vsys_5v0>;
> > > +             ldo1-supply = <&vdd_3v3>;
> > > +             ldo3-supply = <&vdd_3v3>;
> > > +             ldo4-supply = <&vdd_3v3>;
> > > +
> > > +             pinctrl-names = "default";
> > > +             pinctrl-0 = <&pmic_irq_pins_default>;
> > > +             interrupt-parent = <&gic500>;
> > > +             interrupts = <GIC_SPI 224 IRQ_TYPE_LEVEL_HIGH>;
> > > +             interrupt-controller;
> > > +             #interrupt-cells = <1>;
> > > +
> > > +             system-power-controller;
> > > +             ti,power-button;
> > > +             bootph-all;
> >
> > Flip the bootph-all above the system-power-controller to stay within the
> > coding style boundaries?
>
> Fixed!
>
> >
> > > +
> > > +             regulators {
> >         [...]
> > > +};
> > > +
> > > +&sdhci1 {
> > > +     /* SD/MMC */
> > > +     vmmc-supply = <&vdd_mmc1>;
> > > +     vqmmc-supply = <&vdd_sd_dv>;
> > > +     pinctrl-names = "default";
> > > +     pinctrl-0 = <&main_mmc1_pins_default>;
> > > +     disable-wp;
> > > +     cd-gpios = <&main_gpio1 48 GPIO_ACTIVE_LOW>;
> > > +     cd-debounce-delay-ms = <100>;
> > > +     ti,fails-without-test-cd;
> > > +     bootph-all;
> >
> > Nit: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/dts-coding-style.rst#n117
> > keep the  bootph property above the vendor prefixed ones..
>
> Fixed!
>
> >
> > > +     status = "okay";
> > > +};
> > > --
> > > 2.39.2
> > >
> >
> >
> > Additionally, with dtbs_check on next-20240823, I get:
> >
> > /workdir/arch/arm64/boot/dts/ti/k3-am67a-beagley-ai.dtb: leds: led-0: Unevaluated properties are not allowed ('linux,default-trigger' was unexpected)
> > /workdir/arch/arm64/boot/dts/ti/k3-am67a-beagley-ai.dtb: leds: led-0:linux,default-trigger: 'oneOf' conditional failed, one must be fixed:
> >
> > Please fix appropriately.
>
> fixed with: linux,default-trigger = "off";

Nope, also not allowed..

/builds/RobertCNelson/arm64-multiplatform/KERNEL/arch/arm64/boot/dts/ti/k3-am67a-beagley-ai.dtb:
leds: led-0:linux,default-trigger: 'oneOf' conditional failed, one
must be fixed:
'off' is not one of ['backlight', 'default-on', 'heartbeat',
'disk-activity', 'disk-read', 'disk-write', 'timer', 'pattern',
'audio-micmute', 'audio-mute', 'bluetooth-power', 'flash',
'kbd-capslock', 'mtd', 'nand-disk', 'none', 'torch', 'usb-gadget',
'usb-host', 'usbport']

I guess, 'none' is close... off/etc..

Thanks

-- 
Robert Nelson
https://rcn-ee.com/