* Tony Lindgren <tony@xxxxxxxxxxx> [091109 13:05]: > * Tobias Klauser <tklauser@xxxxxxxxxx> [091109 09:44]: > > Use the resource_size function instead of manually calculating the > > resource size. This reduces the chance of introducing off-by-one errors > > and actually fixes one in mailbox.c. > > > > Signed-off-by: Tobias Klauser <tklauser@xxxxxxxxxx> > > Acked-by: Tony Lindgren <tony@xxxxxxxxxxx> Hmm, sorry though this was part of some larger series.. Added to my omap-fixes queue. Regards, Tony > > --- > > arch/arm/mach-omap2/gpmc.c | 2 +- > > arch/arm/mach-omap2/mailbox.c | 2 +- > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/gpmc.c b/arch/arm/mach-omap2/gpmc.c > > index 004da69..67af370 100644 > > --- a/arch/arm/mach-omap2/gpmc.c > > +++ b/arch/arm/mach-omap2/gpmc.c > > @@ -366,7 +366,7 @@ int gpmc_cs_request(int cs, unsigned long size, unsigned long *base) > > if (r < 0) > > goto out; > > > > - gpmc_cs_enable_mem(cs, res->start, res->end - res->start + 1); > > + gpmc_cs_enable_mem(cs, res->start, resource_size(res)); > > *base = res->start; > > gpmc_cs_set_reserved(cs, 1); > > out: > > diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c > > index 46e1f2e..5ba3aa6 100644 > > --- a/arch/arm/mach-omap2/mailbox.c > > +++ b/arch/arm/mach-omap2/mailbox.c > > @@ -300,7 +300,7 @@ static int __devinit omap2_mbox_probe(struct platform_device *pdev) > > dev_err(&pdev->dev, "invalid mem resource\n"); > > return -ENODEV; > > } > > - mbox_base = ioremap(res->start, res->end - res->start); > > + mbox_base = ioremap(res->start, resource_size(res)); > > if (!mbox_base) > > return -ENOMEM; > > > > -- > > 1.6.3.3 > > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
