Re: [PATCH 1/3] omap4: Add i2c board support on omap4430 sdp platform

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* Santosh Shilimkar <santosh.shilimkar@xxxxxx> [100512 01:22]:
> This patch adds the i2c board support for OMAP4430 SDP platform. The
> necessary drivers support patch is posted earlier.
> 
> https://patchwork.kernel.org/patch/80659/
>
> --- a/arch/arm/plat-omap/i2c.c
> +++ b/arch/arm/plat-omap/i2c.c
> @@ -35,6 +35,7 @@
>  #define OMAP2_I2C_BASE1		0x48070000
>  #define OMAP2_I2C_BASE2		0x48072000
>  #define OMAP2_I2C_BASE3		0x48060000
> +#define OMAP2_I2C_BASE4		0x48350000
>  
>  static const char name[] = "i2c_omap";
>  
> @@ -54,9 +55,16 @@ static struct resource i2c_resources[][2] = {
>  #if	defined(CONFIG_ARCH_OMAP2) || defined(CONFIG_ARCH_OMAP3)
>  	{ I2C_RESOURCE_BUILDER(OMAP2_I2C_BASE2, INT_24XX_I2C2_IRQ) },
>  #endif
> +#if	defined(CONFIG_ARCH_OMAP4)
> +	{ I2C_RESOURCE_BUILDER(OMAP2_I2C_BASE2, OMAP44XX_IRQ_I2C2) },
> +#endif
>  #if	defined(CONFIG_ARCH_OMAP3)
>  	{ I2C_RESOURCE_BUILDER(OMAP2_I2C_BASE3, INT_34XX_I2C3_IRQ) },
>  #endif
> +#if	defined(CONFIG_ARCH_OMAP4)
> +	{ I2C_RESOURCE_BUILDER(OMAP2_I2C_BASE3, OMAP44XX_IRQ_I2C3) },
> +	{ I2C_RESOURCE_BUILDER(OMAP2_I2C_BASE4, OMAP44XX_IRQ_I2C4) },
> +#endif
>  };
>  
>  #define I2C_DEV_BUILDER(bus_id, res, data)		\

The above breaks omap3_defconfig. You'll end up with all of the above
in the array, and as the code is using the array index as the key,
you end up with wrong resources.

I've posted a fix for that in the MMC series and updated your
patch accordingly.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux