> > > > + if (cryptodev->data->dev_private == NULL) { > > + RTE_LOG(ERR, PMD, "%s() Cannot allocate memzone > for private device data", > > + __func__); > > dpaa2_sec is already doing private log type, via DPAA2_SEC_LOG macro. > > You should go through your patch and make sure there are as few direct > calls > to RTE_LOG as possible. >
Thanks Stephen, missed that one which seems to be declared but not yet used.Will replace I may be missing something here. While seeing the point in a short common form such as PMD_DRV_LOG, why won't dpaa use the same syntax rather than adding another syntax variant which does pretty much the same?