Re: Concerns with em.yaml YNL spec

[Andrew Lunn]

On Thu, Dec 11, 2025 at 03:54:53PM +0000, Donald Hunter wrote:
> Hi,
> 
> I just spotted the new em.yaml YNL spec that got merged in
> bd26631ccdfd ("PM: EM: Add em.yaml and autogen files") as part of [1]
> because it introduced new yamllint reports:
> 
> make -C tools/net/ynl/ lint
> make: Entering directory '/home/donaldh/net-next/tools/net/ynl'
> yamllint ../../../Documentation/netlink/specs
> ../../../Documentation/netlink/specs/em.yaml
>   3:1       warning  missing document start "---"  (document-start)
>   107:13    error    wrong indentation: expected 10 but found 12  (indentation)
> 
> I guess the patch series was never cced to netdev or the YNL
> maintainers so this is my first opportunity to review it.

Maybe submit a patch to this:

YAML NETLINK (YNL)
M:      Donald Hunter <donald.hunter@xxxxxxxxx>
M:      Jakub Kicinski <kuba@xxxxxxxxxx>
F:      Documentation/netlink/
F:      Documentation/userspace-api/netlink/intro-specs.rst
F:      Documentation/userspace-api/netlink/specs.rst
F:      tools/net/ynl/

adding

F:	Documentation/netlink/specs

I'm also surprised there is no L: line in the entry.

	Andrew