Hi, chris Could you please help review these patches again? Expect your comments. Thanks in advance. Regards Haijun. > -----Original Message----- > From: linux-mmc-owner@xxxxxxxxxxxxxxx [mailto:linux-mmc- > owner@xxxxxxxxxxxxxxx] On Behalf Of Zhang Haijun-B42677 > Sent: Friday, December 14, 2012 10:52 AM > To: Chris Ball; kbuild test robot > Cc: linux-mmc@xxxxxxxxxxxxxxx; Huang Changming-R66093; Anton Vorontsov > Subject: RE: [mmc:mmc-next 45/63] of_iommu.c:(.text+0x114dbc): undefined > reference to `__aeabi_uldivmod' > > Hi, chris > > Two extra patches had been sent to fix these err. > These patches had been Acked-by Anton. > Could you please help review these patches again? Expect your comments. > Thanks in advance. > > 1) Subject: [PATCH] Powerpc eSDHC: Use u64 to calculate the timeout > value to avoid overflow > 2) Subject: [PATCH 2/2 v3] mmc:core: Add limit to data and erase > timeout value calculation > 3) Subject: [PATCH 1/2 v5] Powerpc/eSDHC: Calculate the applicable mmc > erase timeout value > Extra: > 4) Subject: [PATCH 1/2 V2] mmc:block : Convert the cmd_timeou_ms to > u64 to avoid overflow > 5) Subject: [PATCH 2/2 V2] mmc:host: Use u64 to calculate the timeout > value to avoid overflow > > > Regards > Haijun. > > > -----Original Message----- > > From: linux-mmc-owner@xxxxxxxxxxxxxxx [mailto:linux-mmc- > > owner@xxxxxxxxxxxxxxx] On Behalf Of Zhang Haijun-B42677 > > Sent: Monday, November 26, 2012 6:03 PM > > To: Chris Ball; kbuild test robot > > Cc: linux-mmc@xxxxxxxxxxxxxxx; Huang Changming-R66093 > > Subject: RE: [mmc:mmc-next 45/63] of_iommu.c:(.text+0x114dbc): > > undefined reference to `__aeabi_uldivmod' > > > > Hi Chris > > > > I see patches modified the public C file and Powerpc Host C file. But > > it also used by other host C file. > > So need to replace all the u64 division by macro div_u64 to avoid this > > error. > > Two new patches had send and reviewed by Anton. > > I think they can fix the `__aeabi_uldivmod' error. > > All the patches work fine on my branch. > > > > Regards > > Haijun. > > > > > > > -----Original Message----- > > > From: linux-mmc-owner@xxxxxxxxxxxxxxx [mailto:linux-mmc- > > > owner@xxxxxxxxxxxxxxx] On Behalf Of Chris Ball > > > Sent: Monday, November 26, 2012 7:51 AM > > > To: kbuild test robot > > > Cc: Zhang Haijun-B42677; linux-mmc@xxxxxxxxxxxxxxx; Huang Changming- > > > R66093 > > > Subject: Re: [mmc:mmc-next 45/63] of_iommu.c:(.text+0x114dbc): > > > undefined reference to `__aeabi_uldivmod' > > > > > > Hi, > > > > > > On Sun, Nov 25 2012, kbuild test robot wrote: > > > > tree: git://git.kernel.org/pub/scm/linux/kernel/git/cjb/mmc.git > > mmc- > > > next > > > > head: b1dc46e6cd5f314f52a9864cc7ba1b7624b54bd7 > > > > commit: d29254e21ddddee37b3c8525817398d96d6403f3 [45/63] mmc: core: > > > > Use u64 to calculate the timeout value to avoid overflow > > > > config: make ARCH=arm omap2plus_defconfig > > > > > > > > All error/warnings: > > > > > > > > drivers/built-in.o: In function `mmc_omap_start_request': > > > > of_iommu.c:(.text+0x114dbc): undefined reference to > > `__aeabi_uldivmod' > > > > > > I've removed Haijun's three patches from mmc-next while they're > > > reworked to fix this. Thanks! > > > > > > - Chris. > > > -- > > > Chris Ball <cjb@xxxxxxxxxx> <http://printf.net/> > > > One Laptop Per Child > > > -- > > > To unsubscribe from this list: send the line "unsubscribe linux-mmc" > > > in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo > > > info at http://vger.kernel.org/majordomo-info.html > > > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-mmc" > > in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo > > info at http://vger.kernel.org/majordomo-info.html > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-mmc" in > the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at > http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
