| Message ID | 20170313184750.429-4-krzk@kernel.org |
|---|---|
| State | New |
| Headers | show |
On 03/13/2017 03:47 PM, Krzysztof Kozlowski wrote: > Short declaration of 'i' was in the middle of declarations with > assignments. Make it a little bit more readable. Additionally switch > from "unsigned" to "unsigned int" as this pattern is more widely used. > No functional change. > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > --- > hw/misc/exynos4210_pmu.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/misc/exynos4210_pmu.c b/hw/misc/exynos4210_pmu.c > index e30dbc7d3d83..63a8ccd35559 100644 > --- a/hw/misc/exynos4210_pmu.c > +++ b/hw/misc/exynos4210_pmu.c > @@ -401,8 +401,8 @@ static uint64_t exynos4210_pmu_read(void *opaque, hwaddr offset, > unsigned size) > { > Exynos4210PmuState *s = (Exynos4210PmuState *)opaque; > - unsigned i; > const Exynos4210PmuReg *reg_p = exynos4210_pmu_regs; > + unsigned int i; > > for (i = 0; i < PMU_NUM_OF_REGISTERS; i++) { > if (reg_p->offset == offset) { > @@ -420,8 +420,8 @@ static void exynos4210_pmu_write(void *opaque, hwaddr offset, > uint64_t val, unsigned size) > { > Exynos4210PmuState *s = (Exynos4210PmuState *)opaque; > - unsigned i; > const Exynos4210PmuReg *reg_p = exynos4210_pmu_regs; > + unsigned int i; > > for (i = 0; i < PMU_NUM_OF_REGISTERS; i++) { > if (reg_p->offset == offset) { >
diff --git a/hw/misc/exynos4210_pmu.c b/hw/misc/exynos4210_pmu.c index e30dbc7d3d83..63a8ccd35559 100644 --- a/hw/misc/exynos4210_pmu.c +++ b/hw/misc/exynos4210_pmu.c @@ -401,8 +401,8 @@ static uint64_t exynos4210_pmu_read(void *opaque, hwaddr offset, unsigned size) { Exynos4210PmuState *s = (Exynos4210PmuState *)opaque; - unsigned i; const Exynos4210PmuReg *reg_p = exynos4210_pmu_regs; + unsigned int i; for (i = 0; i < PMU_NUM_OF_REGISTERS; i++) { if (reg_p->offset == offset) { @@ -420,8 +420,8 @@ static void exynos4210_pmu_write(void *opaque, hwaddr offset, uint64_t val, unsigned size) { Exynos4210PmuState *s = (Exynos4210PmuState *)opaque; - unsigned i; const Exynos4210PmuReg *reg_p = exynos4210_pmu_regs; + unsigned int i; for (i = 0; i < PMU_NUM_OF_REGISTERS; i++) { if (reg_p->offset == offset) {
Short declaration of 'i' was in the middle of declarations with assignments. Make it a little bit more readable. Additionally switch from "unsigned" to "unsigned int" as this pattern is more widely used. No functional change. Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> --- hw/misc/exynos4210_pmu.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)