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
| ||
|
Message-ID: <20230328110247.GE15196@pengutronix.de> Date: Tue, 28 Mar 2023 13:02:47 +0200 From: Oleksij Rempel <o.rempel@...gutronix.de> To: Alexandre TORGUE <alexandre.torgue@...s.st.com> Cc: Maxime Coquelin <mcoquelin.stm32@...il.com>, Rob Herring <robh+dt@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Jérôme Pouiller <jerome.pouiller@...abs.com>, devicetree@...r.kernel.org, netdev@...r.kernel.org, linux-kernel@...r.kernel.org, kernel@...gutronix.de, linux-stm32@...md-mailman.stormreply.com, linux-arm-kernel@...ts.infradead.org Subject: Re: [PATCH v1] ARM: dts: stm32: prtt1c: Add PoDL PSE regulator nodes On Tue, Mar 28, 2023 at 11:58:34AM +0200, Alexandre TORGUE wrote: > Hi Oleksij > > On 3/23/23 13:32, Oleksij Rempel wrote: > > This commit introduces Power over Data Line (PoDL) Power Source > > Equipment (PSE) regulator nodes to the PRTT1C devicetree. The addition > > of these nodes enables support for PoDL in PRTT1C devices, allowing > > power delivery and data transmission over a single twisted pair. > > > > The new PoDL PSE regulator nodes provide voltage capability information > > of the current board design, which can be used as a hint for system > > administrators when configuring and managing power settings. This > > update enhances the versatility and simplifies the power management of > > PRTT1C devices while ensuring compatibility with connected Powered > > Devices (PDs). > > > > After applying this patch, the power delivery can be controlled from > > user space with a patched [1] ethtool version using the following commands: > > ethtool --set-pse t1l2 podl-pse-admin-control enable > > to enable power delivery, and > > ethtool --show-pse t1l2 > > to display the PoDL PSE settings. > > > > By integrating PoDL PSE support into the PRTT1C devicetree, users can > > benefit from streamlined power and data connections in their > > deployments, improving overall system efficiency and reducing cabling > > complexity. > > > > [1] https://lore.kernel.org/all/20230317093024.1051999-1-o.rempel@pengutronix.de/ > > > > Signed-off-by: Oleksij Rempel <o.rempel@...gutronix.de> > > --- > > Please, fix the introduction of those new yaml validation errors: > > arch/arm/boot/dts/stm32mp151a-prtt1c.dtb: ethernet-pse-1: $nodename:0: > 'ethernet-pse-1' does not match '^ethernet-pse(@.*)?$' > From schema: > /Documentation/devicetree/bindings/net/pse-pd/podl-pse-regulator.yaml > arch/arm/boot/dts/stm32mp151a-prtt1c.dtb: ethernet-pse-2: $nodename:0: > 'ethernet-pse-2' does not match '^ethernet-pse(@.*)?$' > From schema: /local/home/frq08678/STLINUX/kernel/my-kernel/stm32/Documentation/devicetree/bindings/net/pse-pd/podl-pse-regulator.yaml Using ethernet-pse@1 will require to use "reg" or "ranges" properties. Which makes no sense in this use case. I need to fix the schema instead by allowing this patter with following regex: "^ethernet-pse(@.*|-[0-9a-f])*$" Should I send schema fix together with this patch? Regards, Oleksij -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
Powered by blists - more mailing lists