Subject: Re: [PATCH net-next] Documentation: Add documentation for eswitch attribute

From: Jakub Kicinski <[email-masked]>
Date: Fri, 8 Mar 2024 20:14:35 -0800

On Fri, 8 Mar 2024 02:01:06 +0200 William Tu wrote:
> +   * - ``mode``
> +     - enum
> +     - The mode of the device. The mode can be one of the following:
> +
> +       * ``legacy`` Legacy SRIOV.
> +       * ``switchdev`` SRIOV switchdev offloads.

In my head `mode` is special because it controls on / off for
switchdev, and none of the other attrs do anything outside of
switchdev mode. But thinking about it now, I'm not sure if that's
actually the case. Let's clarify. And if other attrs indeed are
only meaningful in switchdev mode we should feature `mode` more
prominently than others. Separate section perhaps?

Please link to representors.rst and perhaps switchdev.rst ?
-- 
pw-bot: cr