Re: [PATCH 2/3] rtc: ds1307: add Intersil ISL12057 support

[arno@natisbad.org (Arnaud Ebalard)]

Alexandre Belloni <alexandre.belloni@free-electrons.com> writes:

> Intersil ISL12057 is a drop-in replacement for DS1337. It can be supported
> by the ds1307 driver.
>
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> ---
>  drivers/rtc/Kconfig      | 8 ++++----
>  drivers/rtc/rtc-ds1307.c | 6 ++++++
>  2 files changed, 10 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/rtc/Kconfig b/drivers/rtc/Kconfig
> index f47c2f5ff70d..ba0b3e7ce4c5 100644
> --- a/drivers/rtc/Kconfig
> +++ b/drivers/rtc/Kconfig
> @@ -198,14 +198,14 @@ config RTC_DRV_AS3722
>  	  will be called rtc-as3722.
>  
>  config RTC_DRV_DS1307
> -	tristate "Dallas/Maxim DS1307/37/38/39/40, ST M41T00, EPSON RX-8025"
> +	tristate "Dallas/Maxim DS1307/37/38/39/40, ST M41T00, EPSON RX-8025, ISL12057"
>  	help
>  	  If you say yes here you get support for various compatible RTC
>  	  chips (often with battery backup) connected with I2C. This driver
>  	  should handle DS1307, DS1337, DS1338, DS1339, DS1340, ST M41T00,
> -	  EPSON RX-8025 and probably other chips. In some cases the RTC
> -	  must already have been initialized (by manufacturing or a
> -	  bootloader).
> +	  EPSON RX-8025, Intersil ISL12057 and probably other chips. In some
> +	  cases the RTC must already have been initialized (by manufacturing or
> +	  a bootloader).
>  
>  	  The first seven registers on these chips hold an RTC, and other
>  	  registers may add features such as NVRAM, a trickle charger for
> diff --git a/drivers/rtc/rtc-ds1307.c b/drivers/rtc/rtc-ds1307.c
> index 4c5890864d9c..52be48f2e3c0 100644
> --- a/drivers/rtc/rtc-ds1307.c
> +++ b/drivers/rtc/rtc-ds1307.c
> @@ -186,6 +186,7 @@ static const struct i2c_device_id ds1307_id[] = {
>  	{ "mcp7941x", mcp794xx },
>  	{ "pt7c4338", ds_1307 },
>  	{ "rx8025", rx_8025 },
> +	{ "isl1207", ds_1337 },

          "isl12057"