| Message ID | 20230412151332.3037845-1-fabrice.gasnier@foss.st.com |
|---|---|
| State | New |
| Headers | show |
| Series | [RFC] ARM: dts: stm32: remove typec port dtbs_check warning on stm32mp15xx-dkx | expand |
On 4/12/23 17:13, Fabrice Gasnier wrote: > According to usb-connector.yaml, typec connector should always use: > - ports property > - with port@0 (required). > This is also true for a single HS port. This removes dtbs_check warning: > stusb1600@28: connector: Unevaluated properties are not allowed ('port' > was unexpected) > > Downside is it introduces another warning when building with W=1: > > Warning (graph_child_address): /soc/i2c@5c002000/stusb1600@28/connector/ > ports: graph node has single child node 'port@0', #address-cells > /#size-cells are not necessary > > By removing #address-cells/#size-cells as suggested, another couple of > W=1 warning shows up: > Warning (avoid_default_addr_size): /soc/i2c@5c002000/stusb1600@28/ > connector/ports/port@0: Relying on default #address-cells value > Warning (avoid_default_addr_size): /soc/i2c@5c002000/stusb1600@28/ > connector/ports/port@0: Relying on default #size-cells value > > Is there a clean way or necessary change to reach a clean build and clean > dtbs_check ? > Maybe this kind of concern has already been addressed ? > As far as I understand, the OF graph allows a single port {} node (e.g. > code being reworked here). Could this be added in relevant dtschema, to > address the single HS port use case (usb-connector.yaml) ? > Rob, Krzysztof, do you an idea to figure out this issue ? Thanks in advance Alex > Signed-off-by: Fabrice Gasnier <fabrice.gasnier@foss.st.com> > --- > arch/arm/boot/dts/stm32mp15xx-dkx.dtsi | 11 ++++++++--- > 1 file changed, 8 insertions(+), 3 deletions(-) > > diff --git a/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi b/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi > index cefeeb00fc22..41e79c8cc752 100644 > --- a/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi > +++ b/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi > @@ -259,9 +259,14 @@ connector { > power-role = "dual"; > typec-power-opmode = "default"; > > - port { > - con_usbotg_hs_ep: endpoint { > - remote-endpoint = <&usbotg_hs_ep>; > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + port@0 { > + reg = <0>; > + con_usbotg_hs_ep: endpoint { > + remote-endpoint = <&usbotg_hs_ep>; > + }; > }; > }; > };
diff --git a/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi b/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi index cefeeb00fc22..41e79c8cc752 100644 --- a/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi +++ b/arch/arm/boot/dts/stm32mp15xx-dkx.dtsi @@ -259,9 +259,14 @@ connector { power-role = "dual"; typec-power-opmode = "default"; - port { - con_usbotg_hs_ep: endpoint { - remote-endpoint = <&usbotg_hs_ep>; + ports { + #address-cells = <1>; + #size-cells = <0>; + port@0 { + reg = <0>; + con_usbotg_hs_ep: endpoint { + remote-endpoint = <&usbotg_hs_ep>; + }; }; }; };
According to usb-connector.yaml, typec connector should always use: - ports property - with port@0 (required). This is also true for a single HS port. This removes dtbs_check warning: stusb1600@28: connector: Unevaluated properties are not allowed ('port' was unexpected) Downside is it introduces another warning when building with W=1: Warning (graph_child_address): /soc/i2c@5c002000/stusb1600@28/connector/ ports: graph node has single child node 'port@0', #address-cells /#size-cells are not necessary By removing #address-cells/#size-cells as suggested, another couple of W=1 warning shows up: Warning (avoid_default_addr_size): /soc/i2c@5c002000/stusb1600@28/ connector/ports/port@0: Relying on default #address-cells value Warning (avoid_default_addr_size): /soc/i2c@5c002000/stusb1600@28/ connector/ports/port@0: Relying on default #size-cells value Is there a clean way or necessary change to reach a clean build and clean dtbs_check ? Maybe this kind of concern has already been addressed ? As far as I understand, the OF graph allows a single port {} node (e.g. code being reworked here). Could this be added in relevant dtschema, to address the single HS port use case (usb-connector.yaml) ? Signed-off-by: Fabrice Gasnier <fabrice.gasnier@foss.st.com> --- arch/arm/boot/dts/stm32mp15xx-dkx.dtsi | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-)