On 10/9/2018 10:31 AM, Igor Russkikh wrote:
> Signed-off-by: Igor Russkikh <igor.russk...@aquantia.com>
> Signed-off-by: Pavel Belous <pavel.bel...@aquantia.com>

<...>

> @@ -168,3 +171,16 @@ atl_dev_info_get(struct rte_eth_dev *dev, struct 
> rte_eth_dev_info *dev_info)
>  RTE_PMD_REGISTER_PCI(net_atlantic, rte_atl_pmd);
>  RTE_PMD_REGISTER_PCI_TABLE(net_atlantic, pci_id_atl_map);
>  RTE_PMD_REGISTER_KMOD_DEP(net_atlantic, "* igb_uio | uio_pci_generic");
> +
> +RTE_INIT(atl_init_log);
> +static void
> +atl_init_log(void)

You can merge these two:

RTE_INIT(atl_init_log)
{
....
}

> +{
> +     atl_logtype_init = rte_log_register("pmd.atlantic.init");

Log type string format is "pmd.net.atlantic.init"

> +     if (atl_logtype_init >= 0)
> +             rte_log_set_level(atl_logtype_init, RTE_LOG_DEBUG);
> +     atl_logtype_driver = rte_log_register("pmd.atlantic.driver");

same here related to format.

Reply via email to