\

> -----Original Message-----
> From: Stanley.Miao [mailto:stanley.m...@windriver.com]
> Sent: Tuesday, April 20, 2010 12:03 PM
> To: linux-omap@vger.kernel.org
> Cc: t...@atomide.com; Hiremath, Vaibhav; Govindarajan, Sriramakrishnan;
> Aggarwal, Anuj
> Subject: [PATCH 3/7] AM3517: rename the i2c boardinfo to make it more
> readable
> 
> There are three i2c buses on am3517, now rename these three boardinfo
> structure name to am3517evm_i2c1_boardinfo, am3517evm_i2c2_boardinfo,
> and am3517evm_i2c3_boardinfo, to make it more readable.
> 
[Hiremath, Vaibhav] Since we haven't had any objection/suggestion to follow 
naming convention based on i2C indexes, I think we can merge this patch.

Acked-By: Vaibhav Hiremath <hvaib...@ti.com>

Thanks,
Vaibhav

> Signed-off-by: Stanley.Miao <stanley.m...@windriver.com>
> ---
>  arch/arm/mach-omap2/board-am3517evm.c |   20 ++++++++++----------
>  1 files changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/board-am3517evm.c b/arch/arm/mach-
> omap2/board-am3517evm.c
> index 6ae8805..3acfcae 100644
> --- a/arch/arm/mach-omap2/board-am3517evm.c
> +++ b/arch/arm/mach-omap2/board-am3517evm.c
> @@ -39,7 +39,7 @@
>  #define LCD_PANEL_BKLIGHT_PWR        182
>  #define LCD_PANEL_PWM                181
> 
> -static struct i2c_board_info __initdata am3517evm_i2c_boardinfo[] = {
> +static struct i2c_board_info __initdata am3517evm_i2c1_boardinfo[] = {
>       {
>               I2C_BOARD_INFO("s35390a", 0x30),
>               .type           = "s35390a",
> @@ -69,7 +69,7 @@ static void __init am3517_evm_rtc_init(void)
>               gpio_free(GPIO_RTCS35390A_IRQ);
>               return;
>       }
> -     am3517evm_i2c_boardinfo[0].irq = gpio_to_irq(GPIO_RTCS35390A_IRQ);
> +     am3517evm_i2c1_boardinfo[0].irq = gpio_to_irq(GPIO_RTCS35390A_IRQ);
>  }
> 
>  /*
> @@ -80,7 +80,7 @@ static void __init am3517_evm_rtc_init(void)
>  static struct pca953x_platform_data am3517evm_gpio_expander_info_0 = {
>       .gpio_base      = OMAP_MAX_GPIO_LINES,
>  };
> -static struct i2c_board_info __initdata am3517evm_tca6416_info_0[] = {
> +static struct i2c_board_info __initdata am3517evm_i2c2_boardinfo[] = {
>       {
>               I2C_BOARD_INFO("tca6416", 0x21),
>               .platform_data = &am3517evm_gpio_expander_info_0,
> @@ -94,7 +94,7 @@ static struct pca953x_platform_data
> am3517evm_ui_gpio_expander_info_1 = {
>  static struct pca953x_platform_data am3517evm_ui_gpio_expander_info_2 = {
>       .gpio_base      = OMAP_MAX_GPIO_LINES + 32,
>  };
> -static struct i2c_board_info __initdata am3517evm_ui_tca6416_info[] = {
> +static struct i2c_board_info __initdata am3517evm_i2c3_boardinfo[] = {
>       {
>               I2C_BOARD_INFO("tca6416", 0x20),
>               .platform_data = &am3517evm_ui_gpio_expander_info_1,
> @@ -108,10 +108,10 @@ static struct i2c_board_info __initdata
> am3517evm_ui_tca6416_info[] = {
>  static int __init am3517_evm_i2c_init(void)
>  {
>       omap_register_i2c_bus(1, 400, NULL, 0);
> -     omap_register_i2c_bus(2, 400, am3517evm_tca6416_info_0,
> -                     ARRAY_SIZE(am3517evm_tca6416_info_0));
> -     omap_register_i2c_bus(3, 400, am3517evm_ui_tca6416_info,
> -                     ARRAY_SIZE(am3517evm_ui_tca6416_info));
> +     omap_register_i2c_bus(2, 400, am3517evm_i2c2_boardinfo,
> +                     ARRAY_SIZE(am3517evm_i2c2_boardinfo));
> +     omap_register_i2c_bus(3, 400, am3517evm_i2c3_boardinfo,
> +                     ARRAY_SIZE(am3517evm_i2c3_boardinfo));
> 
>       return 0;
>  }
> @@ -311,8 +311,8 @@ static void __init am3517_evm_init(void)
>       /* RTC - S35390A */
>       am3517_evm_rtc_init();
> 
> -     i2c_register_board_info(1, am3517evm_i2c_boardinfo,
> -                             ARRAY_SIZE(am3517evm_i2c_boardinfo));
> +     i2c_register_board_info(1, am3517evm_i2c1_boardinfo,
> +                             ARRAY_SIZE(am3517evm_i2c1_boardinfo));
>  }
> 
>  static void __init am3517_evm_map_io(void)
> --
> 1.5.4.3

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

Reply via email to