| Message ID | 20110808101546.GA25572@maxin (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | ae65eb729dd3d3309fb3b7a1badc7b67cada2357 |
| Headers | show |
ping. On Mon, Aug 8, 2011 at 1:15 PM, Maxin B. John <maxin.john@gmail.com> wrote: > The pointer "va" returned from "phys_to_virt(pa)" is never used in > "sgtable_fill_kmalloc()".So,it is safe to remove this set-but-unused variable. > > Signed-off-by: Maxin B. John <maxin.john@gmail.com> > --- > diff --git a/arch/arm/plat-omap/iovmm.c b/arch/arm/plat-omap/iovmm.c > index c60737c..79e7fed 100644 > --- a/arch/arm/plat-omap/iovmm.c > +++ b/arch/arm/plat-omap/iovmm.c > @@ -423,9 +423,6 @@ static void sgtable_fill_kmalloc(struct sg_table *sgt, u32 pa, u32 da, > { > unsigned int i; > struct scatterlist *sg; > - void *va; > - > - va = phys_to_virt(pa); > > for_each_sg(sgt->sgl, sg, sgt->nents, i) { > unsigned bytes; > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
* Maxin B John <maxin.john@gmail.com> [110810 04:06]: > ping. Sorry I've been on vacation, applying to fixes. Regards, Tony > On Mon, Aug 8, 2011 at 1:15 PM, Maxin B. John <maxin.john@gmail.com> wrote: > > The pointer "va" returned from "phys_to_virt(pa)" is never used in > > "sgtable_fill_kmalloc()".So,it is safe to remove this set-but-unused variable. > > > > Signed-off-by: Maxin B. John <maxin.john@gmail.com> > > --- > > diff --git a/arch/arm/plat-omap/iovmm.c b/arch/arm/plat-omap/iovmm.c > > index c60737c..79e7fed 100644 > > --- a/arch/arm/plat-omap/iovmm.c > > +++ b/arch/arm/plat-omap/iovmm.c > > @@ -423,9 +423,6 @@ static void sgtable_fill_kmalloc(struct sg_table *sgt, u32 pa, u32 da, > > { > > unsigned int i; > > struct scatterlist *sg; > > - void *va; > > - > > - va = phys_to_virt(pa); > > > > for_each_sg(sgt->sgl, sg, sgt->nents, i) { > > unsigned bytes; > > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/arm/plat-omap/iovmm.c b/arch/arm/plat-omap/iovmm.c index c60737c..79e7fed 100644 --- a/arch/arm/plat-omap/iovmm.c +++ b/arch/arm/plat-omap/iovmm.c @@ -423,9 +423,6 @@ static void sgtable_fill_kmalloc(struct sg_table *sgt, u32 pa, u32 da, { unsigned int i; struct scatterlist *sg; - void *va; - - va = phys_to_virt(pa); for_each_sg(sgt->sgl, sg, sgt->nents, i) { unsigned bytes;
The pointer "va" returned from "phys_to_virt(pa)" is never used in "sgtable_fill_kmalloc()".So,it is safe to remove this set-but-unused variable. Signed-off-by: Maxin B. John <maxin.john@gmail.com> --- -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html