diff options
| author | Linus Walleij <linus.walleij@linaro.org> | 2020-09-02 00:09:35 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-09-02 14:13:35 -0700 |
| commit | 7e1301ed1881447d2a25f9c6423738c33cbca133 (patch) | |
| tree | aa0ae8fe12420b66378109668c5209aed3432210 /drivers/net/dsa/realtek-smi-core.h | |
| parent | 6641a2c42b0a307b7638d10e5d4b90debc61389d (diff) | |
| download | linux-7e1301ed1881447d2a25f9c6423738c33cbca133.tar.gz linux-7e1301ed1881447d2a25f9c6423738c33cbca133.tar.bz2 linux-7e1301ed1881447d2a25f9c6423738c33cbca133.zip | |
net: dsa: rtl8366: Refactor VLAN/PVID init
The VLANs and PVIDs on the RTL8366 utilizes a "member
configuration" (MC) which is largely unexplained in the
code.
This set-up requires a special ordering: rtl8366_set_pvid()
must be called first, followed by rtl8366_set_vlan(),
else the MC will not be properly allocated. Relax this
by factoring out the code obtaining an MC and reuse
the helper in both rtl8366_set_pvid() and
rtl8366_set_vlan() so we remove this strict ordering
requirement.
In the process, add some better comments and debug prints
so people who read the code understand what is going on.
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/dsa/realtek-smi-core.h')
| -rw-r--r-- | drivers/net/dsa/realtek-smi-core.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/net/dsa/realtek-smi-core.h b/drivers/net/dsa/realtek-smi-core.h index 9a63b51e1d82..6f2dab7e33d6 100644 --- a/drivers/net/dsa/realtek-smi-core.h +++ b/drivers/net/dsa/realtek-smi-core.h @@ -25,6 +25,9 @@ struct rtl8366_mib_counter { const char *name; }; +/** + * struct rtl8366_vlan_mc - Virtual LAN member configuration + */ struct rtl8366_vlan_mc { u16 vid; u16 untag; @@ -119,7 +122,6 @@ int realtek_smi_setup_mdio(struct realtek_smi *smi); int rtl8366_mc_is_used(struct realtek_smi *smi, int mc_index, int *used); int rtl8366_set_vlan(struct realtek_smi *smi, int vid, u32 member, u32 untag, u32 fid); -int rtl8366_get_pvid(struct realtek_smi *smi, int port, int *val); int rtl8366_set_pvid(struct realtek_smi *smi, unsigned int port, unsigned int vid); int rtl8366_enable_vlan4k(struct realtek_smi *smi, bool enable); |