| Message ID | 20241114055926.38502-1-zhangjiao2@cmss.chinamobile.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | mailbox: imx: Modify the incorrect format specifier | expand |
On Thu, Nov 14, 2024 at 01:59:26PM +0800, zhangjiao2 wrote: > From: zhang jiao <zhangjiao2@cmss.chinamobile.com> > > The format specifier of "unsigned int" in snprintf() > should be "%u", not "%i". Replace %i with %u in snprintf() because it is "unsigned int". Reviewed-by: Frank Li <Frank.Li@nxp.com> > > Signed-off-by: zhang jiao <zhangjiao2@cmss.chinamobile.com> > --- > drivers/mailbox/imx-mailbox.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/mailbox/imx-mailbox.c b/drivers/mailbox/imx-mailbox.c > index f815dab3be50..3fe51b2de89e 100644 > --- a/drivers/mailbox/imx-mailbox.c > +++ b/drivers/mailbox/imx-mailbox.c > @@ -782,7 +782,7 @@ static int imx_mu_init_generic(struct imx_mu_priv *priv) > cp->chan = &priv->mbox_chans[i]; > priv->mbox_chans[i].con_priv = cp; > snprintf(cp->irq_desc, sizeof(cp->irq_desc), > - "%s[%i-%i]", dev_name(priv->dev), cp->type, cp->idx); > + "%s[%i-%u]", dev_name(priv->dev), cp->type, cp->idx); > } > > priv->mbox.num_chans = IMX_MU_CHANS; > @@ -819,7 +819,7 @@ static int imx_mu_init_specific(struct imx_mu_priv *priv) > cp->chan = &priv->mbox_chans[i]; > priv->mbox_chans[i].con_priv = cp; > snprintf(cp->irq_desc, sizeof(cp->irq_desc), > - "%s[%i-%i]", dev_name(priv->dev), cp->type, cp->idx); > + "%s[%i-%u]", dev_name(priv->dev), cp->type, cp->idx); > } > > priv->mbox.num_chans = num_chans; > -- > 2.33.0 > > >
diff --git a/drivers/mailbox/imx-mailbox.c b/drivers/mailbox/imx-mailbox.c index f815dab3be50..3fe51b2de89e 100644 --- a/drivers/mailbox/imx-mailbox.c +++ b/drivers/mailbox/imx-mailbox.c @@ -782,7 +782,7 @@ static int imx_mu_init_generic(struct imx_mu_priv *priv) cp->chan = &priv->mbox_chans[i]; priv->mbox_chans[i].con_priv = cp; snprintf(cp->irq_desc, sizeof(cp->irq_desc), - "%s[%i-%i]", dev_name(priv->dev), cp->type, cp->idx); + "%s[%i-%u]", dev_name(priv->dev), cp->type, cp->idx); } priv->mbox.num_chans = IMX_MU_CHANS; @@ -819,7 +819,7 @@ static int imx_mu_init_specific(struct imx_mu_priv *priv) cp->chan = &priv->mbox_chans[i]; priv->mbox_chans[i].con_priv = cp; snprintf(cp->irq_desc, sizeof(cp->irq_desc), - "%s[%i-%i]", dev_name(priv->dev), cp->type, cp->idx); + "%s[%i-%u]", dev_name(priv->dev), cp->type, cp->idx); } priv->mbox.num_chans = num_chans;