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] [day] [month] [year] [list]
Message-ID: <aGFiCsXWaubBBk7I@lizhi-Precision-Tower-5810>
Date: Sun, 29 Jun 2025 11:55:54 -0400
From: Frank Li <Frank.li@....com>
To: Vladimir Zapolskiy <vz@...ia.com>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Piotr Wojtaszczyk <piotr.wojtaszczyk@...esys.com>,
	"open list:USB SUBSYSTEM" <linux-usb@...r.kernel.org>,
	"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@...r.kernel.org>,
	"moderated list:ARM/LPC32XX SOC SUPPORT" <linux-arm-kernel@...ts.infradead.org>,
	open list <linux-kernel@...r.kernel.org>, imx@...ts.linux.dev
Subject: Re: [PATCH v2 1/1] dt-bindings: usb: convert lpc32xx-udc.txt to yaml
 format

On Sat, Jun 28, 2025 at 03:57:44AM +0300, Vladimir Zapolskiy wrote:
> Hi Frank.
>
> On 6/26/25 00:43, Frank Li wrote:
> > Convert lpc32xx-udc.txt to yaml format.
> >
> > Additional changes:
> > - add clocks and put it into required list to match existed lp32xx.dtsi.
> > - remove usb-transceiver@2c at examples.
> >
> > Signed-off-by: Frank Li <Frank.Li@....com>
>
> Thank you a lot for doing this massive conversion work of NXP LPC
> device tree bindings, I would appreciate, if you put me to CC for
> all such changes.
>
> While you do one-to-one conversion, you do bring a lot of errors,
> because the .txt descriptions are broken.
>
> Please always reference to arch/arm/boot/dts/nxp/lpc/lpc32xx.dtsi
> and arch/arm/boot/dts/nxp/lpc/lpc18xx.dtsi, that's the correct
> version of NXP LPC DT bindings, otherwise something important
> can be missed by occasion.

Okay, I dependent on CHECK_DTBS but omit the factor the node with status
="disabled" will skip check.

Frank

>
> <snip>
>
> > +
> > +examples:
> > +  - |
> > +    #include <dt-bindings/clock/lpc32xx-clock.h>
> > +
> > +    usbd@...20000 {
> > +        compatible = "nxp,lpc3220-udc";
> > +        reg = <0x31020000 0x300>;
> > +        interrupt-parent = <&mic>;
>
> interrupt-parent = <sic1>;
>
> > +        interrupts = <0x3d 0>, <0x3e 0>, <0x3c 0>, <0x3a 0>;
>
> Should subscribe 32 and correct the type:
>
> interrupts = <29 4>, <30 4>, <28 4>, <26 8>;
>
> > +        clocks = <&usbclk LPC32XX_USB_CLK_DEVICE>;
> > +        transceiver = <&isp1301>;
> > +    };
>
> --
> Best wishes,
> Vladimir

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ