Re: [PATCH net-next 02/12] net: dsa: vsc73xx: Add vlan filtering

From: Simon Horman
Date: Mon Jun 17 2024 - 08:10:53 EST


On Tue, Jun 11, 2024 at 09:49:54PM +0200, Pawel Dembicki wrote:
> This patch implements VLAN filtering for the vsc73xx driver.
>
> After starting VLAN filtering, the switch is reconfigured from QinQ to
> a simple VLAN aware mode. This is required because VSC73XX chips do not
> support inner VLAN tag filtering.
>
> Signed-off-by: Pawel Dembicki <paweldembicki@xxxxxxxxx>

...

Hi Pawel,

Two minor spelling nits from my side for your consideration.

> diff --git a/drivers/net/dsa/vitesse-vsc73xx.h b/drivers/net/dsa/vitesse-vsc73xx.h
> index 2997f7e108b1..fc8b7a73d652 100644
> --- a/drivers/net/dsa/vitesse-vsc73xx.h
> +++ b/drivers/net/dsa/vitesse-vsc73xx.h
> @@ -14,6 +14,27 @@
> */
> #define VSC73XX_MAX_NUM_PORTS 8
>
> +/**
> + * struct vsc73xx_portinfo - port data structure: contains storage data
> + * @pvid_vlan_filtering_configured: imforms if port have configured pvid in vlan
> + * fitering mode

fitering -> filtering

> + * @pvid_vlan_filtering: pvid vlan number used in vlan fitering mode

Likewise here.

Flagged by checkpatch.pl --codespell

> + * @pvid_tag_8021q_configured: imforms if port have configured pvid in tag_8021q
> + * mode
> + * @pvid_tag_8021q: pvid vlan number used in tag_8021q mode
> + * @untagged_tag_8021q_configured: imforms if port have configured untagged vlan
> + * in tag_8021q mode
> + * @untagged_tag_8021q: untagged vlan number used in tag_8021q mode
> + */

...