Re: [PATCH v6] dt-bindings: net: can: holt,hi311x: Convert to DT schema

["Rob Herring (Arm)" <robh@kernel.org>]

On Wed, 16 Sep 2026 09:58:01 +0800, Quchaosheng wrote:
> Convert the Holt HI-311X binding from free-form text to DT schema.
> 
> Keep the example and the properties as the original text describes them,
> including clocks as a required property.
> 
> Signed-off-by: Quchaosheng <quchaosheng000406@163.com>
> ---
> v2:
>  - Keep clocks required and drop the clock-frequency property.
>  - Drop the variant-specific handling that v2 introduced. It documented
>    properties the binding never had.
> 
> v3:
>  - Rewrite the commit message in imperative mood.
> 
> v6:
>  - No content change. Fix the patch format: the changelog was placed in the
>    commit message instead of after the '---' separator, which confused
>    Patchwork. Move it back where it belongs.
> 
>  - Drop an incorrect statement from the v2 changelog: clock-frequency was
>    described there as an ACPI-only property of the driver. That is wrong,
>    the driver has no ACPI support at all (no acpi_device_id, no ACPI
>    headers); device_property_read_u32() reads it from device tree too.
>    The property is simply not part of this DT binding, which is why it is
>    not documented. Keeping clocks required matches the sibling
>    microchip,mcp2510.yaml, whose driver also uses devm_clk_get_optional().
>  .../bindings/net/can/holt,hi311x.yaml         | 67 +++++++++++++++++++
>  .../bindings/net/can/holt_hi311x.txt          | 23 -------
>  2 files changed, 67 insertions(+), 23 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/net/can/holt,hi311x.yaml
>  delete mode 100644 Documentation/devicetree/bindings/net/can/holt_hi311x.txt
> 

Reviewed-by: Rob Herring (Arm) <robh@kernel.org>