| Message ID | 20230311131325.9750-2-nm@ti.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | dt-bindings: pinctrl: ti-k3: Move k3.h to arch specific | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
On 11/03/2023 14:13, Nishanth Menon wrote: > Drop the pinmux header reference. Examples should just show the node > definition. You could mention that it is not used. Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On 16:43-20230311, Krzysztof Kozlowski wrote: > On 11/03/2023 14:13, Nishanth Menon wrote: > > Drop the pinmux header reference. Examples should just show the node > > definition. > > You could mention that it is not used. > > Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Yup. will update the series. Thanks for reviewing.
diff --git a/Documentation/devicetree/bindings/net/ti,k3-am654-cpsw-nuss.yaml b/Documentation/devicetree/bindings/net/ti,k3-am654-cpsw-nuss.yaml index 900063411a20..837d6db299c4 100644 --- a/Documentation/devicetree/bindings/net/ti,k3-am654-cpsw-nuss.yaml +++ b/Documentation/devicetree/bindings/net/ti,k3-am654-cpsw-nuss.yaml @@ -222,7 +222,6 @@ additionalProperties: false examples: - | - #include <dt-bindings/pinctrl/k3.h> #include <dt-bindings/soc/ti,sci_pm_domain.h> #include <dt-bindings/net/ti-dp83867.h> #include <dt-bindings/interrupt-controller/irq.h>
Drop the pinmux header reference. Examples should just show the node definition. Signed-off-by: Nishanth Menon <nm@ti.com> --- Looks like there are regressions in this yaml file already when checked with dt_bindings_check, but that is a seperate activity of it's own. If net maintainers are OK, I'd like to pick this up as part of the pinctrl fixes series for next.. Documentation/devicetree/bindings/net/ti,k3-am654-cpsw-nuss.yaml | 1 - 1 file changed, 1 deletion(-)