Re: [PATCH net-next v4 1/2] dt-bindings: net: add DAPU Telecom DAP8211R(I) PHY binding

Artem Shimko <[email protected]>
Newsgroups dev.linux.lists.sashiko-reviews,org.kernel.vger.linux-devicetree
Message-ID <CAOPX747K9QrHdNy0Cn7bh3grWfi9mv2maEyo9RfVpVdA5B_4eQ@mail.gmail.com>
On Tue, Jul 28, 2026 at 10:53 AM <[email protected]> wrote:

> [Severity: Low]
> Does this patch series also need to add the 'dapu' vendor prefix to
> Documentation/devicetree/bindings/vendor-prefixes.yaml?
>
> It appears 'dapu' is used in the filename and compatible string, which will
> cause make dt_binding_check to emit warnings if the prefix is missing from
> the global registry.
I don't know.

But it seems it's finally clear how to use yamllint =) found the issue
$ make dt_binding_check DT_SCHEMA_FILES=dapu,dap8211r.yaml
  SCHEMA  Documentation/devicetree/bindings/processed-schema.json
  CHKDT   ./Documentation/devicetree/bindings
  LINT    ./Documentation/devicetree/bindings
  STYLE   ./Documentation/devicetree/bindings
  DTEX    Documentation/devicetree/bindings/net/dapu,dap8211r.example.dts
  DTC [C] Documentation/devicetree/bindings/net/dapu,dap8211r.example.dtb
$ yamllint Documentation/devicetree/bindings/net/dapu,dap8211r.yaml
Documentation/devicetree/bindings/net/dapu,dap8211r.yaml
  63:1      error    too many blank lines (1 > 0)  (empty-lines)

--
Best regards,
Artem
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.