lkml.org 
[lkml]   [2016]   [Apr]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Date
    Subject[PATCH 3.16 028/217] ARM: davinci: make I2C support optional
    3.16.35-rc1 review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Arnd Bergmann <arnd@arndb.de>

    commit 8e58041114319ad290e85a7a82077dc9e2f93613 upstream.

    The davinci platform has tried to get support for the EEPROM right,
    but failed to get a clean build so far. At the moment, we get
    a warning whenever CONFIG_SYSFS is disabled, as that is needed by
    EEPROM_AT24:

    warning: (MACH_DAVINCI_EVM && MACH_SFFSDR && MACH_DAVINCI_DM6467_EVM && MACH_DAVINCI_DM365_EVM && MACH_DAVINCI_DA830_EVM && MACH_MITYOMAPL138 && MACH_MINI2440) selects EEPROM_AT24 which has unmet direct dependencies (I2C && SYSFS)

    Kevin Hilman initially added the 'select' to ensure that EEPROM_AT24
    is always enabled in machines that really want it for normal operation
    (i.e. for reading the MAC address). This broke when I2C was disabled,
    and Russell King followed up with another patch to select that as
    well.

    I now see that the SYSFS dependency is still missing, which leaves
    us with three options:

    a) add 'select SYSFS' in addition to the others
    b) change AT24_EEPPROM to work without sysfs (should be possible)
    c) remove all those selects again and get the files to build when
    I2C is disabled.

    I would really hate to do a) because adding select statements that
    hardwire user-selectable symbols is generally a bad idea. I first
    tried b) but then ended up redoing the patch from scratch to approach
    c), so we can also remove the other selects.

    I checked that CONFIG_I2C is still enabled with davinci_all_defconfig,
    so that does not have to change.

    Signed-off-by: Arnd Bergmann <arnd@arndb.de>
    Fixes: 45b146d746ea ("ARM: Davinci: Fix I2C build errors")
    Fixes: 22ca466847ad ("davinci: kconfig: select at24 eeprom for selected boards")
    Signed-off-by: Sekhar Nori <nsekhar@ti.com>
    Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
    ---
    arch/arm/mach-davinci/Kconfig | 14 +-------------
    arch/arm/mach-davinci/board-dm644x-evm.c | 8 ++++++--
    arch/arm/mach-davinci/board-dm646x-evm.c | 7 +++++++
    3 files changed, 14 insertions(+), 15 deletions(-)

    --- a/arch/arm/mach-davinci/Kconfig
    +++ b/arch/arm/mach-davinci/Kconfig
    @@ -65,8 +65,6 @@ config MACH_DAVINCI_EVM
    bool "TI DM644x EVM"
    default ARCH_DAVINCI_DM644x
    depends on ARCH_DAVINCI_DM644x
    - select EEPROM_AT24
    - select I2C
    help
    Configure this option to specify the whether the board used
    for development is a DM644x EVM
    @@ -74,8 +72,6 @@ config MACH_DAVINCI_EVM
    config MACH_SFFSDR
    bool "Lyrtech SFFSDR"
    depends on ARCH_DAVINCI_DM644x
    - select EEPROM_AT24
    - select I2C
    help
    Say Y here to select the Lyrtech Small Form Factor
    Software Defined Radio (SFFSDR) board.
    @@ -106,8 +102,6 @@ config MACH_DAVINCI_DM6467_EVM
    bool "TI DM6467 EVM"
    default ARCH_DAVINCI_DM646x
    depends on ARCH_DAVINCI_DM646x
    - select EEPROM_AT24
    - select I2C
    select MACH_DAVINCI_DM6467TEVM
    help
    Configure this option to specify the whether the board used
    @@ -120,8 +114,6 @@ config MACH_DAVINCI_DM365_EVM
    bool "TI DM365 EVM"
    default ARCH_DAVINCI_DM365
    depends on ARCH_DAVINCI_DM365
    - select EEPROM_AT24
    - select I2C
    help
    Configure this option to specify whether the board used
    for development is a DM365 EVM
    @@ -130,9 +122,7 @@ config MACH_DAVINCI_DA830_EVM
    bool "TI DA830/OMAP-L137/AM17x Reference Platform"
    default ARCH_DAVINCI_DA830
    depends on ARCH_DAVINCI_DA830
    - select EEPROM_AT24
    - select GPIO_PCF857X
    - select I2C
    + select GPIO_PCF857X if I2C
    help
    Say Y here to select the TI DA830/OMAP-L137/AM17x Evaluation Module.

    @@ -212,8 +202,6 @@ config DA850_WL12XX
    config MACH_MITYOMAPL138
    bool "Critical Link MityDSP-L138/MityARM-1808 SoM"
    depends on ARCH_DAVINCI_DA850
    - select EEPROM_AT24
    - select I2C
    help
    Say Y here to select the Critical Link MityDSP-L138/MityARM-1808
    System on Module. Information on this SoM may be found at
    --- a/arch/arm/mach-davinci/board-dm644x-evm.c
    +++ b/arch/arm/mach-davinci/board-dm644x-evm.c
    @@ -267,7 +267,7 @@ static struct platform_device rtc_dev =
    static struct snd_platform_data dm644x_evm_snd_data;

    /*----------------------------------------------------------------------*/
    -
    +#ifdef CONFIG_I2C
    /*
    * I2C GPIO expanders
    */
    @@ -614,6 +614,7 @@ static void __init evm_init_i2c(void)
    i2c_add_driver(&dm6446evm_msp_driver);
    i2c_register_board_info(1, i2c_info, ARRAY_SIZE(i2c_info));
    }
    +#endif

    #define VENC_STD_ALL (V4L2_STD_NTSC | V4L2_STD_PAL)

    @@ -783,7 +784,9 @@ static __init void davinci_evm_init(void
    pr_warn("%s: Cannot configure AEMIF.\n",
    __func__);

    +#ifdef CONFIG_I2C
    evm_leds[7].default_trigger = "nand-disk";
    +#endif
    if (HAS_NOR)
    pr_warning("WARNING: both NAND and NOR flash "
    "are enabled; disable one of them.\n");
    @@ -793,9 +796,10 @@ static __init void davinci_evm_init(void

    platform_add_devices(davinci_evm_devices,
    ARRAY_SIZE(davinci_evm_devices));
    +#ifdef CONFIG_I2C
    evm_init_i2c();
    -
    davinci_setup_mmc(0, &dm6446evm_mmc_config);
    +#endif
    dm644x_init_video(&dm644xevm_capture_cfg, &dm644xevm_display_cfg);

    davinci_serial_init(dm644x_serial_device);
    --- a/arch/arm/mach-davinci/board-dm646x-evm.c
    +++ b/arch/arm/mach-davinci/board-dm646x-evm.c
    @@ -122,6 +122,7 @@ static struct platform_device davinci_na

    #define HAS_ATA IS_ENABLED(CONFIG_BLK_DEV_PALMCHIP_BK3710)

    +#ifdef CONFIG_I2C
    /* CPLD Register 0 bits to control ATA */
    #define DM646X_EVM_ATA_RST BIT(0)
    #define DM646X_EVM_ATA_PWD BIT(1)
    @@ -317,6 +318,7 @@ static struct at24_platform_data eeprom_
    .setup = davinci_get_mac_addr,
    .context = (void *)0x7f00,
    };
    +#endif

    static u8 dm646x_iis_serializer_direction[] = {
    TX_MODE, RX_MODE, INACTIVE_MODE, INACTIVE_MODE,
    @@ -347,6 +349,7 @@ static struct snd_platform_data dm646x_e
    },
    };

    +#ifdef CONFIG_I2C
    static struct i2c_client *cpld_client;

    static int cpld_video_probe(struct i2c_client *client,
    @@ -714,6 +717,7 @@ static void __init evm_init_i2c(void)
    evm_init_cpld();
    evm_init_video();
    }
    +#endif

    #define CDCE949_XIN_RATE 27000000

    @@ -795,7 +799,10 @@ static __init void evm_init(void)
    if (ret)
    pr_warn("%s: GPIO init failed: %d\n", __func__, ret);

    +#ifdef CONFIG_I2C
    evm_init_i2c();
    +#endif
    +
    davinci_serial_init(dm646x_serial_device);
    dm646x_init_mcasp0(&dm646x_evm_snd_data[0]);
    dm646x_init_mcasp1(&dm646x_evm_snd_data[1]);
    \
     
     \ /
      Last update: 2016-04-27 03:01    [W:4.086 / U:0.132 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site