Re: [net-next PATCH v5 5/9] dt-bindings: net: add QCA807x PHY defines

From: Krzysztof Kozlowski
Date: Fri Feb 02 2024 - 02:42:15 EST


On 01/02/2024 16:17, Christian Marangi wrote:
> From: Robert Marko <robert.marko@xxxxxxxxxx>
>
> Add DT bindings defined for Qualcomm QCA807x PHY series related to
> calibration and DAC settings.

Nothing from this file is used and your commit msg does not provide
rationale "why", thus it does not look like something for bindings.
Otherwise please point me which patch with *driver* uses these bindings.

>
> Signed-off-by: Robert Marko <robert.marko@xxxxxxxxxx>
> Signed-off-by: Christian Marangi <ansuelsmth@xxxxxxxxx>
> ---
> include/dt-bindings/net/qcom-qca807x.h | 30 ++++++++++++++++++++++++++

Use filename matching compatible, so vendor,device. No wildcards, unless
your compatible also has them.

> 1 file changed, 30 insertions(+)
> create mode 100644 include/dt-bindings/net/qcom-qca807x.h
>



Best regards,
Krzysztof