>-----Original Message----- >From: Shawn Guo [mailto:shawn.guo@xxxxxxxxxx] >Sent: Tuesday, June 18, 2013 4:46 PM >To: Lu Jingchang-B35083 >Cc: wsa@xxxxxxxxxxxxx; Estevam Fabio-R49496; s.hauer@xxxxxxxxxxxxxx; >linux-i2c@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; Jin >Zhengxiong-R64188; Li Xiaochun-B41219 >Subject: Re: [PATCH 2/2] i2c: imx: Add Vybrid VF610 I2C controller support > >On Tue, Jun 18, 2013 at 10:10:18AM +0800, Jingchang Lu wrote: >> Add Freescale Vybrid VF610 I2C controller support to imx I2C driver >> framework. >> Some operation is different from imx I2C controller. >> The register offset, the i2c clock divider value table, the module >> enabling(I2CR_IEN) which is just invert with imx, and the interrupt >> flag(I2SR) clearing opcode is w1c on VF610 but w0c on imx. >> >> Signed-off-by: Jason Jin <Jason.jin@xxxxxxxxxxxxx> >> Signed-off-by: Xiaochun Li <b41219@xxxxxxxxxxxxx> >> Signed-off-by: Jingchang Lu <b35083@xxxxxxxxxxxxx> >> --- >> drivers/i2c/busses/i2c-imx.c | 144 >> +++++++++++++++++++++++++++++++++++++------ >> 1 file changed, 125 insertions(+), 19 deletions(-) > >It seems to me that the patch introduces section mismatch warnings. > >WARNING: modpost: Found 2 section mismatch(es). >To see full details build your kernel with: >'make CONFIG_DEBUG_SECTION_MISMATCH=y' [Lu Jingchang-B35083] It seems that the "__initdata" causes these section mismatches, I will fix these in patch v2. Thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
