Re: [net-next PATCH 0/8] configuration support for switch headers & phy

2021-03-31 Thread Hariprasad Kelam
; ; Geethasowjanya Akula ; > Jerin Jacob Kollanukkaran ; Subbaraya Sundeep Bhatta > > Subject: [EXT] Re: [net-next PATCH 0/8] configuration support for switch > headers & phy > > > The usecase is simple, unlike DSA tag, this 4byte FDSA tag doesn't > > have a et

Re: [net-next PATCH 0/8] configuration support for switch headers & phy

2021-03-28 Thread Andrew Lunn
> The usecase is simple, unlike DSA tag, this 4byte FDSA tag doesn't > have a ethertype, > so HW cannot recognize this header. If such packers arise, then HW parsing > will > fail and RSS will not work. > > Hypothetically if we introduce some communication between MAC driver > and DSA driver, > w

Re: [net-next PATCH 0/8] configuration support for switch headers & phy

2021-03-28 Thread Sunil Kovvuri
On Thu, Mar 25, 2021 at 6:51 PM Andrew Lunn wrote: > > On Thu, Mar 25, 2021 at 06:32:12PM +0530, Sunil Kovvuri wrote: > > On Thu, Mar 25, 2021 at 6:20 PM Andrew Lunn wrote: > > > > > > > > So you completely skipped how this works with mv88e6xxx or > > > > > prestera. If you need this private flag

Re: [net-next PATCH 0/8] configuration support for switch headers & phy

2021-03-25 Thread Andrew Lunn
On Thu, Mar 25, 2021 at 06:32:12PM +0530, Sunil Kovvuri wrote: > On Thu, Mar 25, 2021 at 6:20 PM Andrew Lunn wrote: > > > > > > So you completely skipped how this works with mv88e6xxx or > > > > prestera. If you need this private flag for some out of mainline > > > > Marvell SDK, it is very unlike

Re: [net-next PATCH 0/8] configuration support for switch headers & phy

2021-03-25 Thread Sunil Kovvuri
On Thu, Mar 25, 2021 at 6:20 PM Andrew Lunn wrote: > > > > So you completely skipped how this works with mv88e6xxx or > > > prestera. If you need this private flag for some out of mainline > > > Marvell SDK, it is very unlikely to be accepted. > > > > > > Andrew > > > > What we are trying

Re: [net-next PATCH 0/8] configuration support for switch headers & phy

2021-03-25 Thread Andrew Lunn
> > So you completely skipped how this works with mv88e6xxx or > > prestera. If you need this private flag for some out of mainline > > Marvell SDK, it is very unlikely to be accepted. > > > > Andrew > > What we are trying to do here has no dependency on DSA drivers and > neither impacts t

Re: [net-next PATCH 0/8] configuration support for switch headers & phy

2021-03-25 Thread Sunil Kovvuri
> > > Hi Hariprasad > > > > > > Private flags sound very wrong here. I would expect to see some > > > integration > > > between the switchdev/DSA driver and the MAC driver. > > > Please show how this works in combination with drivers/net/dsa/mv88e6xxx > > > or drivers/net/ethernet/marvell/prestera

Re: [net-next PATCH 0/8] configuration support for switch headers & phy

2021-03-23 Thread Andrew Lunn
On Tue, Mar 23, 2021 at 06:13:28PM +, Hariprasad Kelam wrote: > > Hi Andrew , > > Please see inline, No need to say that. That is the correct way to right emails. > > Hi Hariprasad > > > > Private flags sound very wrong here. I would expect to see some integration > > between the switchdev

Re: [net-next PATCH 0/8] configuration support for switch headers & phy

2021-03-23 Thread Hariprasad Kelam
m ; > Linu Cherian ; Geethasowjanya Akula > ; Jerin Jacob Kollanukkaran ; > Subbaraya Sundeep Bhatta > Subject: [EXT] Re: [net-next PATCH 0/8] configuration support for switch > headers & phy > > On Sun, Mar 21, 2021 at 05:39:50PM +0530, Hariprasad Kelam wrote: > > This series

Re: [net-next PATCH 0/8] configuration support for switch headers & phy

2021-03-21 Thread Andrew Lunn
On Sun, Mar 21, 2021 at 05:39:50PM +0530, Hariprasad Kelam wrote: > This series of patches add support for parsing switch headers and > configuration support for phy modulation type(NRZ or PAM4). > > PHYs that support changing modulation type ,user can configure it > through private flags pam4. >

[net-next PATCH 0/8] configuration support for switch headers & phy

2021-03-21 Thread Hariprasad Kelam
This series of patches add support for parsing switch headers and configuration support for phy modulation type(NRZ or PAM4). PHYs that support changing modulation type ,user can configure it through private flags pam4. Marvell switches support DSA(distributed switch architecture) with different