Skip to content

Commit

Permalink
i.MX6: Fix bad printf format specifiers
Browse files Browse the repository at this point in the history
We should use printf format specifier "%u" instead of "%d" for
argument of type "unsigned int".

Reported-by: Euler Robot <euler.robot@huawei.com>
Signed-off-by: Alex Chen <alex.chen@huawei.com>
Message-id: 20201126111109.112238-4-alex.chen@huawei.com
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
  • Loading branch information
Alex Chen authored and pm215 committed Dec 10, 2020
1 parent a88ae03 commit 9197c7b
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 11 deletions.
20 changes: 10 additions & 10 deletions hw/misc/imx6_ccm.c
Expand Up @@ -96,7 +96,7 @@ static const char *imx6_ccm_reg_name(uint32_t reg)
case CCM_CMEOR:
return "CMEOR";
default:
sprintf(unknown, "%d ?", reg);
sprintf(unknown, "%u ?", reg);
return unknown;
}
}
Expand Down Expand Up @@ -235,7 +235,7 @@ static const char *imx6_analog_reg_name(uint32_t reg)
case USB_ANALOG_DIGPROG:
return "USB_ANALOG_DIGPROG";
default:
sprintf(unknown, "%d ?", reg);
sprintf(unknown, "%u ?", reg);
return unknown;
}
}
Expand Down Expand Up @@ -263,7 +263,7 @@ static uint64_t imx6_analog_get_pll2_clk(IMX6CCMState *dev)
freq *= 20;
}

DPRINTF("freq = %d\n", (uint32_t)freq);
DPRINTF("freq = %u\n", (uint32_t)freq);

return freq;
}
Expand All @@ -275,7 +275,7 @@ static uint64_t imx6_analog_get_pll2_pfd0_clk(IMX6CCMState *dev)
freq = imx6_analog_get_pll2_clk(dev) * 18
/ EXTRACT(dev->analog[CCM_ANALOG_PFD_528], PFD0_FRAC);

DPRINTF("freq = %d\n", (uint32_t)freq);
DPRINTF("freq = %u\n", (uint32_t)freq);

return freq;
}
Expand All @@ -287,7 +287,7 @@ static uint64_t imx6_analog_get_pll2_pfd2_clk(IMX6CCMState *dev)
freq = imx6_analog_get_pll2_clk(dev) * 18
/ EXTRACT(dev->analog[CCM_ANALOG_PFD_528], PFD2_FRAC);

DPRINTF("freq = %d\n", (uint32_t)freq);
DPRINTF("freq = %u\n", (uint32_t)freq);

return freq;
}
Expand Down Expand Up @@ -315,7 +315,7 @@ static uint64_t imx6_analog_get_periph_clk(IMX6CCMState *dev)
break;
}

DPRINTF("freq = %d\n", (uint32_t)freq);
DPRINTF("freq = %u\n", (uint32_t)freq);

return freq;
}
Expand All @@ -327,7 +327,7 @@ static uint64_t imx6_ccm_get_ahb_clk(IMX6CCMState *dev)
freq = imx6_analog_get_periph_clk(dev)
/ (1 + EXTRACT(dev->ccm[CCM_CBCDR], AHB_PODF));

DPRINTF("freq = %d\n", (uint32_t)freq);
DPRINTF("freq = %u\n", (uint32_t)freq);

return freq;
}
Expand All @@ -339,7 +339,7 @@ static uint64_t imx6_ccm_get_ipg_clk(IMX6CCMState *dev)
freq = imx6_ccm_get_ahb_clk(dev)
/ (1 + EXTRACT(dev->ccm[CCM_CBCDR], IPG_PODF));

DPRINTF("freq = %d\n", (uint32_t)freq);
DPRINTF("freq = %u\n", (uint32_t)freq);

return freq;
}
Expand All @@ -351,7 +351,7 @@ static uint64_t imx6_ccm_get_per_clk(IMX6CCMState *dev)
freq = imx6_ccm_get_ipg_clk(dev)
/ (1 + EXTRACT(dev->ccm[CCM_CSCMR1], PERCLK_PODF));

DPRINTF("freq = %d\n", (uint32_t)freq);
DPRINTF("freq = %u\n", (uint32_t)freq);

return freq;
}
Expand Down Expand Up @@ -385,7 +385,7 @@ static uint32_t imx6_ccm_get_clock_frequency(IMXCCMState *dev, IMXClk clock)
break;
}

DPRINTF("Clock = %d) = %d\n", clock, freq);
DPRINTF("Clock = %d) = %u\n", clock, freq);

return freq;
}
Expand Down
2 changes: 1 addition & 1 deletion hw/misc/imx6_src.c
Expand Up @@ -68,7 +68,7 @@ static const char *imx6_src_reg_name(uint32_t reg)
case SRC_GPR10:
return "SRC_GPR10";
default:
sprintf(unknown, "%d ?", reg);
sprintf(unknown, "%u ?", reg);
return unknown;
}
}
Expand Down

0 comments on commit 9197c7b

Please sign in to comment.