Re: [PATCH v3 0/3] MAX8998 changes for RTC

Previous thread: [PATCH 1/2] cifs: Include linux/err.h for IS_ERR and PTR_ERR by David Howells on Thursday, August 5, 2010 - 7:13 pm. (3 messages)

Next thread: [PATCH v3 1/3] mfd: MAX8998: Use struct i2c_client to argument on i2c operation functions by Joonyoung Shim on Thursday, August 5, 2010 - 7:28 pm. (3 messages)
From: Joonyoung Shim
Date: Thursday, August 5, 2010 - 7:27 pm

Hi.

This patchset is to support the RTC provided by the Maxim 8998 chip. For
this first, needs i2c interface changes and interrupts support of
max8998 mfd driver.

Changes since v1:
 - Fix missing braces of 3/3 patch from v1 review.
 - The rest is same with v1.

Changes since v2:
 - add missing free_irq and rtc_valid_tm on rtc driver of 3/3 patch
 - The rest is same with v2.

Joonyoung Shim (3):
      mfd: MAX8998: Use struct i2c_client to argument on i2c operation functions
      mfd: MAX8998: Add interrupts support
      rtc: Add MAX8998 rtc driver

 drivers/mfd/Kconfig                 |    2 +-
 drivers/mfd/Makefile                |    2 +-
 drivers/mfd/max8998-irq.c           |  255 +++++++++++++++++++++++++++++
 drivers/mfd/max8998.c               |   78 +++++++--
 drivers/regulator/max8998.c         |   17 ++-
 drivers/rtc/Kconfig                 |   10 ++
 drivers/rtc/Makefile                |    1 +
 drivers/rtc/rtc-max8998.c           |  300 +++++++++++++++++++++++++++++++++++
 include/linux/mfd/max8998-private.h |  106 +++++++++---
 include/linux/mfd/max8998.h         |   11 +-
 10 files changed, 726 insertions(+), 56 deletions(-)
 create mode 100644 drivers/mfd/max8998-irq.c
 create mode 100644 drivers/rtc/rtc-max8998.c
--

From: Joonyoung Shim
Date: Wednesday, August 25, 2010 - 1:55 am

Ping. Is there any feedback?

Thanks.

-- 
- Joonyoung Shim
--

From: Andrew Morton
Date: Wednesday, August 25, 2010 - 12:21 pm

On Wed, 25 Aug 2010 17:55:00 +0900

They look good to me.  Things are pretty delayed at present as I (and
presumably others) recover from the -rc merge window..

I merged them.  Sam's acks on the MFD bits made life easier.
--

From: Samuel Ortiz
Date: Friday, September 10, 2010 - 9:39 am

For some reasons I thought Liam was pushing this patchset through the last
merge window.
But now that the max8998 driver is upstreamed, I can carry those patches.

Cheers,
Samuel.


-- 
Intel Open Source Technology Centre
http://oss.intel.com/
--

From: Liam Girdwood
Date: Friday, September 10, 2010 - 9:45 am

Thanks

Liam
-- 
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk

--

Previous thread: [PATCH 1/2] cifs: Include linux/err.h for IS_ERR and PTR_ERR by David Howells on Thursday, August 5, 2010 - 7:13 pm. (3 messages)

Next thread: [PATCH v3 1/3] mfd: MAX8998: Use struct i2c_client to argument on i2c operation functions by Joonyoung Shim on Thursday, August 5, 2010 - 7:28 pm. (3 messages)