On Tue, 16 Apr 2024 19:23:02 -0700
Jakub Kicinski <kuba@xxxxxxxxxx> wrote:
> On Sun, 14 Apr 2024 16:21:55 +0200 Kory Maincent wrote:
> > + !(pse_has_podl(phydev->psec))) {
> > + NL_SET_ERR_MSG_ATTR(info->extack,
> > + tb[ETHTOOL_A_PODL_PSE_ADMIN_CONTROL],
> > + "setting PoDL PSE admin control not
> > supported");
> > + return -EOPNOTSUPP;
> > + }
> > + if (tb[ETHTOOL_A_C33_PSE_ADMIN_CONTROL] &&
> > + !(pse_has_c33(phydev->psec))) {
>
> nit: unnecessary parenthesis around the function call
Right, thanks for spotting it!
Regards,
--
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com
