Message ID | cover.1597729692.git.landen.chao@mediatek.com (mailing list archive) |
---|---|
Headers | show |
Series | net-next: dsa: mt7530: add support for MT7531 | expand |
Tested full series on Bananapi-R2+R64 with5.9-rc1 (iperf3 no retransmitts, Throughput >900Mbit/s)
Tested-By: Frank Wunderlich <frank-w@public-files.de>
maybe you can include the port_change_mtu callback you've send me? or do you want to send it separately
regards Frank
Hi, Is port mirroring working? Port mirroring registers on MT7531 have moved, according to bpi's MT7531 reference manual. Please fix that as well.
Hi Is vlan also different? Am 19. August 2020 05:49:20 MESZ schrieb DENG Qingfang <dqfext@gmail.com>: >Hi, > >Is port mirroring working? Port mirroring registers on MT7531 have >moved, according to bpi's MT7531 reference manual. >Please fix that as well. > >_______________________________________________ >Linux-mediatek mailing list >Linux-mediatek@lists.infradead.org >http://lists.infradead.org/mailman/listinfo/linux-mediatek regards Frank
Hi DENG, MT7531 mirror port has been fixed by new definition of register base in header file. The logic of mirror port setting in 7530.c is reused. @@ -41,6 +42,33 @@ enum mt753x_id { #define MIRROR_PORT(x) ((x) & 0x7) #define MIRROR_MASK 0x7 +/* Registers for CPU forward control */ +#define MT7531_CFC 0x4 +#define MT7531_MIRROR_EN BIT(19) +#define MT7531_MIRROR_MASK (MIRROR_MASK << 16) +#define MT7531_MIRROR_PORT_GET(x) (((x) >> 16) & MIRROR_MASK) +#define MT7531_MIRROR_PORT_SET(x) (((x) & MIRROR_MASK) << 16) +#define MT7531_CPU_PMAP_MASK GENMASK(7, 0) + +#define MT753X_MIRROR_REG(id) (((id) == ID_MT7531) ? \ + MT7531_CFC : MT7530_MFC) +#define MT753X_MIRROR_EN(id) (((id) == ID_MT7531) ? \ + MT7531_MIRROR_EN : MIRROR_EN) +#define MT753X_MIRROR_MASK(id) (((id) == ID_MT7531) ? \ + MT7531_MIRROR_MASK : MIRROR_MASK) On Wed, 2020-08-19 at 11:49 +0800, DENG Qingfang wrote: > Hi, > > Is port mirroring working? Port mirroring registers on MT7531 have > moved, according to bpi's MT7531 reference manual. > Please fix that as well.