| Message ID | alpine.DEB.2.00.1210260411140.27963@utopia.booyaka.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
* Paul Walmsley <paul@pwsan.com> [121025 22:05]: > On Fri, 26 Oct 2012, Paul Walmsley wrote: > > > On Mon, 8 Oct 2012, Afzal Mohammed wrote: > > > > > platform data now contains a field to indicate whether > > > soc belongs to omap34xx family, use it instead of > > > cpu_is_* check. > > > > > > This helps in removing dependency of platform specific > > > header file - cpu.h > > > > > > Signed-off-by: Afzal Mohammed <afzal@ti.com> > > > > This one breaks an N800 multi-OMAP build here: > > It also breaks an OMAP3+4 config: > > drivers/built-in.o: In function `omap2_onenand_probe': > /home/paul/test_build/temp/test_cleanup_prcm_8634155e_with_fixes/20121025214236/linux/drivers/mtd/onenand/omap2.c:742: > undefined reference to `omap2_onenand_read_bufferram' > /home/paul/test_build/temp/test_cleanup_prcm_8634155e_with_fixes/20121025214236/linux/drivers/mtd/onenand/omap2.c:743: > undefined reference to `omap2_onenand_write_bufferram' > /home/paul/test_build/temp/test_cleanup_prcm_8634155e_with_fixes/20121025214236/linux/drivers/mtd/onenand/omap2.c:742: > undefined reference to `omap2_onenand_read_bufferram' > /home/paul/test_build/temp/test_cleanup_prcm_8634155e_with_fixes/20121025214236/linux/drivers/mtd/onenand/omap2.c:743: > undefined reference to `omap2_onenand_write_bufferram' > > Fix below. Thanks, I'd like to pull this fix in too along with the others. Regards, Tony > drivers/mtd/onenand/omap2.c | 18 ++++++++++++------ > 1 file changed, 12 insertions(+), 6 deletions(-) > > diff --git a/drivers/mtd/onenand/omap2.c b/drivers/mtd/onenand/omap2.c > index f87cf39..99f96e1 100644 > --- a/drivers/mtd/onenand/omap2.c > +++ b/drivers/mtd/onenand/omap2.c > @@ -555,13 +555,19 @@ static int omap2_onenand_write_bufferram(struct mtd_info *mtd, int area, > > #else > > -int omap2_onenand_read_bufferram(struct mtd_info *mtd, int area, > - unsigned char *buffer, int offset, > - size_t count); > +static int omap2_onenand_read_bufferram(struct mtd_info *mtd, int area, > + unsigned char *buffer, int offset, > + size_t count) > +{ > + return -ENOSYS; > +} > > -int omap2_onenand_write_bufferram(struct mtd_info *mtd, int area, > - const unsigned char *buffer, > - int offset, size_t count); > +static int omap2_onenand_write_bufferram(struct mtd_info *mtd, int area, > + const unsigned char *buffer, > + int offset, size_t count) > +{ > + return -ENOSYS; > +} > > #endif > > -- > 1.7.10.4
On Friday 26 October 2012 10:11 PM, Tony Lindgren wrote: > * Paul Walmsley<paul@pwsan.com> [121025 22:05]: >> It also breaks an OMAP3+4 config: >> Fix below. > Thanks, I'd like to pull this fix in too along with the > others. Thanks Paul and Tony for taking care of this issue. I am sorry about the issue and had been on vacation last week. Regards Afzal
diff --git a/drivers/mtd/onenand/omap2.c b/drivers/mtd/onenand/omap2.c index f87cf39..99f96e1 100644 --- a/drivers/mtd/onenand/omap2.c +++ b/drivers/mtd/onenand/omap2.c @@ -555,13 +555,19 @@ static int omap2_onenand_write_bufferram(struct mtd_info *mtd, int area, #else -int omap2_onenand_read_bufferram(struct mtd_info *mtd, int area, - unsigned char *buffer, int offset, - size_t count); +static int omap2_onenand_read_bufferram(struct mtd_info *mtd, int area, + unsigned char *buffer, int offset, + size_t count) +{ + return -ENOSYS; +} -int omap2_onenand_write_bufferram(struct mtd_info *mtd, int area, - const unsigned char *buffer, - int offset, size_t count); +static int omap2_onenand_write_bufferram(struct mtd_info *mtd, int area, + const unsigned char *buffer, + int offset, size_t count) +{ + return -ENOSYS; +} #endif