Remove duplicate newline `\n` from the debugging macros to avoid double `\n\n`.
Signed-off-by: Hemant Agrawal <hemant.agra...@nxp.com> --- drivers/bus/dpaa/base/fman/fman.c | 58 +++++++++++++++---------------- drivers/bus/dpaa/dpaa_bus.c | 12 +++---- 2 files changed, 34 insertions(+), 36 deletions(-) diff --git a/drivers/bus/dpaa/base/fman/fman.c b/drivers/bus/dpaa/base/fman/fman.c index 953f7d7061..41195eb0a7 100644 --- a/drivers/bus/dpaa/base/fman/fman.c +++ b/drivers/bus/dpaa/base/fman/fman.c @@ -244,13 +244,13 @@ fman_if_init(const struct device_node *dpa_node) /* Obtain the MAC node used by this interface except macless */ mac_phandle = of_get_property(dpa_node, mprop, &lenp); if (!mac_phandle) { - FMAN_ERR(-EINVAL, "%s: no %s\n", dname, mprop); + FMAN_ERR(-EINVAL, "%s: no %s", dname, mprop); return -EINVAL; } assert(lenp == sizeof(phandle)); mac_node = of_find_node_by_phandle(*mac_phandle); if (!mac_node) { - FMAN_ERR(-ENXIO, "%s: bad 'fsl,fman-mac\n", dname); + FMAN_ERR(-ENXIO, "%s: bad 'fsl,fman-mac", dname); return -ENXIO; } mname = mac_node->full_name; @@ -262,19 +262,19 @@ fman_if_init(const struct device_node *dpa_node) ports_phandle = of_get_property(mac_node, "fsl,fman-ports", &lenp); if (!ports_phandle) { - FMAN_ERR(-EINVAL, "%s: no fsl,port-handles\n", + FMAN_ERR(-EINVAL, "%s: no fsl,port-handles", mname); return -EINVAL; } assert(lenp == (2 * sizeof(phandle))); rx_node = of_find_node_by_phandle(ports_phandle[0]); if (!rx_node) { - FMAN_ERR(-ENXIO, "%s: bad fsl,port-handle[0]\n", mname); + FMAN_ERR(-ENXIO, "%s: bad fsl,port-handle[0]", mname); return -ENXIO; } tx_node = of_find_node_by_phandle(ports_phandle[1]); if (!tx_node) { - FMAN_ERR(-ENXIO, "%s: bad fsl,port-handle[1]\n", mname); + FMAN_ERR(-ENXIO, "%s: bad fsl,port-handle[1]", mname); return -ENXIO; } @@ -282,8 +282,7 @@ fman_if_init(const struct device_node *dpa_node) if (of_device_is_compatible(dpa_node, "fsl,dpa-ethernet")) { port_cell_idx = of_get_property(rx_node, "cell-index", &lenp); if (!port_cell_idx) { - FMAN_ERR(-ENXIO, - "%s: no cell-index for port\n", mname); + FMAN_ERR(-ENXIO, "%s: no cell-index for port", mname); return -ENXIO; } assert(lenp == sizeof(*port_cell_idx)); @@ -305,8 +304,7 @@ fman_if_init(const struct device_node *dpa_node) ext_args_cell_idx = of_get_property(ext_args_node, "cell-index", &lenp); if (!ext_args_cell_idx) { - FMAN_ERR(-ENXIO, - "%s: no cell-index for ext args\n", + FMAN_ERR(-ENXIO, "%s: no cell-index for ext args", mname); return -ENXIO; } @@ -343,7 +341,7 @@ fman_if_init(const struct device_node *dpa_node) /* Allocate an object for this network interface */ __if = rte_malloc(NULL, sizeof(*__if), RTE_CACHE_LINE_SIZE); if (!__if) { - FMAN_ERR(-ENOMEM, "malloc(%zu)\n", sizeof(*__if)); + FMAN_ERR(-ENOMEM, "malloc(%zu)", sizeof(*__if)); goto err; } memset(__if, 0, sizeof(*__if)); @@ -356,12 +354,12 @@ fman_if_init(const struct device_node *dpa_node) /* Map the CCSR regs for the MAC node */ regs_addr = of_get_address(mac_node, 0, &__if->regs_size, NULL); if (!regs_addr) { - FMAN_ERR(-EINVAL, "of_get_address(%s)\n", mname); + FMAN_ERR(-EINVAL, "of_get_address(%s)", mname); goto err; } phys_addr = of_translate_address(mac_node, regs_addr); if (!phys_addr) { - FMAN_ERR(-EINVAL, "of_translate_address(%s, %p)\n", + FMAN_ERR(-EINVAL, "of_translate_address(%s, %p)", mname, regs_addr); goto err; } @@ -369,7 +367,7 @@ fman_if_init(const struct device_node *dpa_node) PROT_READ | PROT_WRITE, MAP_SHARED, fman_ccsr_map_fd, phys_addr); if (__if->ccsr_map == MAP_FAILED) { - FMAN_ERR(-errno, "mmap(0x%"PRIx64")\n", phys_addr); + FMAN_ERR(-errno, "mmap(0x%"PRIx64")", phys_addr); goto err; } na = of_n_addr_cells(mac_node); @@ -380,13 +378,13 @@ fman_if_init(const struct device_node *dpa_node) fman_node = of_get_parent(mac_node); na = of_n_addr_cells(mac_node); if (!fman_node) { - FMAN_ERR(-ENXIO, "of_get_parent(%s)\n", mname); + FMAN_ERR(-ENXIO, "of_get_parent(%s)", mname); goto err; } fname = fman_node->full_name; cell_idx = of_get_property(fman_node, "cell-index", &lenp); if (!cell_idx) { - FMAN_ERR(-ENXIO, "%s: no cell-index)\n", fname); + FMAN_ERR(-ENXIO, "%s: no cell-index)", fname); goto err; } assert(lenp == sizeof(*cell_idx)); @@ -395,7 +393,7 @@ fman_if_init(const struct device_node *dpa_node) if (!get_once) { _errno = fman_get_ip_rev(fman_node); if (_errno) { - FMAN_ERR(-ENXIO, "%s: ip_rev is not available\n", + FMAN_ERR(-ENXIO, "%s: ip_rev is not available", fname); goto err; } @@ -425,7 +423,7 @@ fman_if_init(const struct device_node *dpa_node) char_prop = of_get_property(mac_node, "phy-connection-type", NULL); if (!char_prop) { - printf("memac: unknown MII type assuming 1G\n"); + FMAN_ERR(-EINVAL, "memac: unknown MII type assuming 1G"); /* Right now forcing memac to 1g in case of error*/ __if->__if.mac_type = fman_mac_1g; } else { @@ -440,7 +438,7 @@ fman_if_init(const struct device_node *dpa_node) __if->__if.mac_type = fman_mac_10g; } } else { - FMAN_ERR(-EINVAL, "%s: unknown MAC type\n", mname); + FMAN_ERR(-EINVAL, "%s: unknown MAC type", mname); goto err; } @@ -463,7 +461,7 @@ fman_if_init(const struct device_node *dpa_node) mac_addr = of_get_property(mac_node, "local-mac-address", &lenp); if (!mac_addr) { - FMAN_ERR(-EINVAL, "%s: no local-mac-address\n", + FMAN_ERR(-EINVAL, "%s: no local-mac-address", mname); goto err; } @@ -473,19 +471,19 @@ fman_if_init(const struct device_node *dpa_node) tx_channel_id = of_get_property(tx_node, "fsl,qman-channel-id", &lenp); if (!tx_channel_id) { - FMAN_ERR(-EINVAL, "%s: no fsl-qman-channel-id\n", + FMAN_ERR(-EINVAL, "%s: no fsl-qman-channel-id", tx_node->full_name); goto err; } regs_addr = of_get_address(rx_node, 0, &__if->regs_size, NULL); if (!regs_addr) { - FMAN_ERR(-EINVAL, "of_get_address(%s)\n", mname); + FMAN_ERR(-EINVAL, "of_get_address(%s)", mname); goto err; } phys_addr = of_translate_address(rx_node, regs_addr); if (!phys_addr) { - FMAN_ERR(-EINVAL, "of_translate_address(%s, %p)\n", + FMAN_ERR(-EINVAL, "of_translate_address(%s, %p)", mname, regs_addr); goto err; } @@ -493,7 +491,7 @@ fman_if_init(const struct device_node *dpa_node) PROT_READ | PROT_WRITE, MAP_SHARED, fman_ccsr_map_fd, phys_addr); if (__if->bmi_map == MAP_FAILED) { - FMAN_ERR(-errno, "mmap(0x%"PRIx64")\n", phys_addr); + FMAN_ERR(-errno, "mmap(0x%"PRIx64")", phys_addr); goto err; } @@ -507,7 +505,7 @@ fman_if_init(const struct device_node *dpa_node) */ rx_phandle = of_get_property(dpa_node, rprop, &lenp); if (!rx_phandle) { - FMAN_ERR(-EINVAL, "%s: no fsl,qman-frame-queues-rx\n", dname); + FMAN_ERR(-EINVAL, "%s: no fsl,qman-frame-queues-rx", dname); goto err; } @@ -542,7 +540,7 @@ fman_if_init(const struct device_node *dpa_node) tx_phandle = of_get_property(dpa_node, "fsl,qman-frame-queues-tx", &lenp); if (!tx_phandle) { - FMAN_ERR(-EINVAL, "%s: no fsl,qman-frame-queues-tx\n", dname); + FMAN_ERR(-EINVAL, "%s: no fsl,qman-frame-queues-tx", dname); goto err; } @@ -562,7 +560,7 @@ fman_if_init(const struct device_node *dpa_node) pools_phandle = of_get_property(dpa_node, "fsl,bman-buffer-pools", &lenp); if (!pools_phandle) { - FMAN_ERR(-EINVAL, "%s: no fsl,bman-buffer-pools\n", dname); + FMAN_ERR(-EINVAL, "%s: no fsl,bman-buffer-pools", dname); goto err; } /* For each pool, parse the corresponding node and add a pool object @@ -578,13 +576,13 @@ fman_if_init(const struct device_node *dpa_node) /* Allocate an object for the pool */ bpool = rte_malloc(NULL, sizeof(*bpool), RTE_CACHE_LINE_SIZE); if (!bpool) { - FMAN_ERR(-ENOMEM, "malloc(%zu)\n", sizeof(*bpool)); + FMAN_ERR(-ENOMEM, "malloc(%zu)", sizeof(*bpool)); goto err; } /* Find the pool node */ pool_node = of_find_node_by_phandle(*pools_phandle); if (!pool_node) { - FMAN_ERR(-ENXIO, "%s: bad fsl,bman-buffer-pools\n", + FMAN_ERR(-ENXIO, "%s: bad fsl,bman-buffer-pools", dname); rte_free(bpool); goto err; @@ -593,7 +591,7 @@ fman_if_init(const struct device_node *dpa_node) /* Extract the BPID property */ prop = of_get_property(pool_node, "fsl,bpid", &proplen); if (!prop) { - FMAN_ERR(-EINVAL, "%s: no fsl,bpid\n", pname); + FMAN_ERR(-EINVAL, "%s: no fsl,bpid", pname); rte_free(bpool); goto err; } @@ -691,7 +689,7 @@ fman_init(void) for_each_child_node(parent_node, dpa_node) { _errno = fman_if_init(dpa_node); if (_errno) { - FMAN_ERR(_errno, "if_init(%s)\n", dpa_node->full_name); + FMAN_ERR(_errno, "if_init(%s)", dpa_node->full_name); goto err; } } diff --git a/drivers/bus/dpaa/dpaa_bus.c b/drivers/bus/dpaa/dpaa_bus.c index 5d4352bb3c..e483428ddc 100644 --- a/drivers/bus/dpaa/dpaa_bus.c +++ b/drivers/bus/dpaa/dpaa_bus.c @@ -326,7 +326,7 @@ int rte_dpaa_portal_init(void *arg) dpaa_seqn_dynfield_offset = rte_mbuf_dynfield_register(&dpaa_seqn_dynfield_desc); if (dpaa_seqn_dynfield_offset < 0) { - DPAA_BUS_LOG(ERR, "Failed to register mbuf field for dpaa sequence number\n"); + DPAA_BUS_LOG(ERR, "Failed to register mbuf field for dpaa sequence number"); return -rte_errno; } @@ -673,7 +673,7 @@ rte_dpaa_bus_probe(void) if (dev->device_type == FSL_DPAA_ETH) { ret = rte_dpaa_setup_intr(dev->intr_handle); if (ret) - DPAA_BUS_ERR("Error setting up interrupt.\n"); + DPAA_BUS_ERR("Error setting up interrupt."); } } @@ -734,13 +734,13 @@ rte_dpaa_find_device(const struct rte_device *start, rte_dev_cmp_t cmp, while (dev != NULL) { if (cmp(&dev->device, data) == 0) { - DPAA_BUS_DEBUG("Found dev=(%s)\n", dev->device.name); + DPAA_BUS_DEBUG("Found dev=(%s)", dev->device.name); return &dev->device; } dev = TAILQ_NEXT(dev, next); } - DPAA_BUS_DEBUG("Unable to find any device\n"); + DPAA_BUS_DEBUG("Unable to find any device"); return NULL; } @@ -786,14 +786,14 @@ dpaa_bus_dev_iterate(const void *start, const char *str, /* Expectation is that device would be name=device_name */ if (strncmp(str, "name=", 5) != 0) { - DPAA_BUS_DEBUG("Invalid device string (%s)\n", str); + DPAA_BUS_DEBUG("Invalid device string (%s)", str); return NULL; } /* Now that name=device_name format is available, split */ dup = strdup(str); if (dup == NULL) { - DPAA_BUS_DEBUG("Dup string (%s) failed!\n", str); + DPAA_BUS_DEBUG("Dup string (%s) failed!", str); return NULL; } dev_name = dup + strlen("name="); -- 2.25.1