[PATCH net-next 09/30] net: dsa: mt7530: empty default case on mt7530_setup_port5()
Arınç ÜNAL
arinc.unal at arinc9.com
Wed May 24 23:51:12 PDT 2023
On 24.05.2023 21:05, Vladimir Oltean wrote:
> On Mon, May 22, 2023 at 03:15:11PM +0300, arinc9.unal at gmail.com wrote:
>> default:
>> - dev_err(ds->dev, "Unsupported p5_intf_sel %d\n",
>> - priv->p5_intf_sel);
>> - goto unlock_exit;
>
> You could have probably left a comment though (that doesn't cost in
> terms of compiled code):
>
> /* We never call mt7530_setup_port5() with P5_DISABLED */
I remove P5_DISABLED with later patches so I don't see a reason to add
this then remove it on the same patch series.
Arınç
More information about the Linux-mediatek
mailing list