On Sat, 13 Nov 2021 10:01:39 +0530 Apeksha Gupta <apeksha.gu...@nxp.com> wrote:
> + /* Tx deferred start is not supported */ > + if (tx_conf->tx_deferred_start) { > + ENETFEC_PMD_ERR("%p:Tx deferred start not supported", > + (void *)dev); Small observations: 1. Cast to void * is unnecessary in a print/log statement 2. Why not print something human friendly like device name?