[PATCH 2/2] i2c: imx: Add Vybrid VF610 I2C controller support

Lu Jingchang-B35083 B35083 at freescale.com
Tue Jun 18 05:22:06 EDT 2013



>-----Original Message-----
>From: Shawn Guo [mailto:shawn.guo at linaro.org]
>Sent: Tuesday, June 18, 2013 4:46 PM
>To: Lu Jingchang-B35083
>Cc: wsa at the-dreams.de; Estevam Fabio-R49496; s.hauer at pengutronix.de;
>linux-i2c at vger.kernel.org; linux-arm-kernel at lists.infradead.org; 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 at freescale.com>
>> Signed-off-by: Xiaochun Li <b41219 at freescale.com>
>> Signed-off-by: Jingchang Lu <b35083 at freescale.com>
>> ---
>>  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!





More information about the linux-arm-kernel mailing list