[PATCH net-next 09/30] net: dsa: mt7530: empty default case on mt7530_setup_port5()

Vladimir Oltean olteanv at gmail.com
Wed May 24 11:05:29 PDT 2023


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 */

> +		break;
>  	}



More information about the Linux-mediatek mailing list