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
> + */
...