Messages in this thread | | | From | Hariprasad Kelam <> | Subject | Re: [net-next PATCH 2/8] octeontx2-pf: Add ethtool priv flag to control PAM4 on/off | Date | Tue, 23 Mar 2021 18:59:39 +0000 |
| |
Hi Andrew,
Please see inline,
> -----Original Message----- > From: Andrew Lunn <andrew@lunn.ch> > Sent: Sunday, March 21, 2021 7:58 PM > To: Hariprasad Kelam <hkelam@marvell.com> > Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org; kuba@kernel.org; > davem@davemloft.net; Sunil Kovvuri Goutham <sgoutham@marvell.com>; > Linu Cherian <lcherian@marvell.com>; Geethasowjanya Akula > <gakula@marvell.com>; Jerin Jacob Kollanukkaran <jerinj@marvell.com>; > Subbaraya Sundeep Bhatta <sbhatta@marvell.com> > Subject: Re: [net-next PATCH 2/8] octeontx2-pf: Add ethtool priv flag to > control PAM4 on/off > > On Sun, Mar 21, 2021 at 05:39:52PM +0530, Hariprasad Kelam wrote: > > From: Felix Manlunas <fmanlunas@marvell.com> > > > > For PHYs that support changing modulation type (NRZ or PAM4), enable > > these > > commands: > > > > ethtool --set-priv-flags ethX pam4 on > > ethtool --set-priv-flags ethX pam4 off # means NRZ modulation > > ethtool --show-priv-flags ethX > > Why is this not derived from the link mode? How do other Vendors support > this in their high speed MAC/PHY combinations. > > Please stop using priv flags like this. This is not a Marvell specific problem. > Any high speed MAC/PHY combination is going to need some way to > configure this. So please think about the best generic solution. > > This combined with your DSA changes give me a bad feeling. It seems like you > are just trying to dump your SDK features into the kernel, without properly > integrating the features in a vendor neutral way. >
Thanks for your suggestion . Will try to evaluate this can be achieved through link mode or any generic solution. Will try to submit his feature as a separate patch.
Thanks, Hariprasad k > Andrew
| |