Re: [PATCH 4/5] irqchip/irq-imx-gpcv2: Make error messages more consistent

From: Lucas Stach
Date: Thu Dec 06 2018 - 06:32:25 EST


Am Mittwoch, den 05.12.2018, 23:31 -0800 schrieb Andrey Smirnov:
> Make error messages more consistent by making sure each starts with
> "%pOF:".
>
> > Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> > Cc: Jason Cooper <jason@xxxxxxxxxxxxxx>
> > Cc: Marc Zyngier <marc.zyngier@xxxxxxx>
> Cc: cphealy@xxxxxxxxx
> Cc: l.stach@xxxxxxxxxxxxxx
> > Cc: Leonard Crestez <leonard.crestez@xxxxxxx>
> > Cc: "A.s. Dong" <aisheng.dong@xxxxxxx>
> > Cc: Richard Zhu <hongxing.zhu@xxxxxxx>
> Cc: linux-imx@xxxxxxx
> Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx
> Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>

Reviewed-by: Lucas Stach <l.stach@xxxxxxxxxxxxxx>

> ---
> Âdrivers/irqchip/irq-imx-gpcv2.c | 4 ++--
> Â1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/irqchip/irq-imx-gpcv2.c b/drivers/irqchip/irq-imx-gpcv2.c
> index 077d56b3183a..c2b2b3128ddd 100644
> --- a/drivers/irqchip/irq-imx-gpcv2.c
> +++ b/drivers/irqchip/irq-imx-gpcv2.c
> @@ -212,7 +212,7 @@ static int __init imx_gpcv2_irqchip_init(struct device_node *node,
> Â
> > Â cd = kzalloc(sizeof(struct gpcv2_irqchip_data), GFP_KERNEL);
> > Â if (!cd) {
> > - pr_err("kzalloc failed!\n");
> > + pr_err("%pOF: kzalloc failed!\n", node);
> > Â return -ENOMEM;
> > Â }
> Â
> @@ -220,7 +220,7 @@ static int __init imx_gpcv2_irqchip_init(struct device_node *node,
> Â
> > Â cd->gpc_base = of_iomap(node, 0);
> > Â if (!cd->gpc_base) {
> > - pr_err("fsl-gpcv2: unable to map gpc registers\n");
> > + pr_err("%pOF: unable to map gpc registers\n", node);
> > Â kfree(cd);
> > Â return -ENOMEM;
> > Â }