From: Liam Girdwood <lrg@slimlogic.co.uk>
To: Marek Szyprowski <m.szyprowski@samsung.com>
Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>,
Samuel Ortiz <sameo@linux.intel.com>,
linux-kernel@vger.kernel.org, kyungmin.park@samsung.com
Subject: Re: [PATCH v3] drivers: regulator: add Maxim 8998 driver
Date: Mon, 21 Jun 2010 18:50:58 +0100 [thread overview]
Message-ID: <1277142658.3288.46.camel@odin> (raw)
In-Reply-To: <1276671856-18902-1-git-send-email-m.szyprowski@samsung.com>
On Wed, 2010-06-16 at 09:04 +0200, Marek Szyprowski wrote:
> From: Kyungmin Park <kyungmin.park@samsung.com>
>
> This patch adds voltage regulator driver for Maxim 8998 chip. This chip
> is used on Samsung Aquila and GONI boards and provides following
> functionalities:
> - 4 BUCK voltage converters, 17 LDO power regulators and 5 other power
> controllers
> - battery charger
>
> This patch adds basic driver for voltage regulators and MAX 8998 MFD core.
>
> Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
>
> ---
>
> Changes sinve V2:
> - moved max8998_{read,write,update}_reg function to
> include/linux/mfd/max8998-private.h
> - removed max8998-rtc definition (rtc subfunction uses different i2c id
> (mfd functionality is still required for battery charger driver)
> - added iolock mutex to serialize register access correctly
>
> Changes since V1:
> - converted to MFD framework (for future extensions with RTC and battery
> charger drivers)
> - removed i2c register cache
> - removed dummy irq support (gpio keys will be used for setting irq_wake)
> - removed static voltage map tables
> - introduced atomic register update io function
> - other code cleanup suggested by Mark Brown
Applied.
Thanks
Liam
--
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk
prev parent reply other threads:[~2010-06-21 17:51 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-16 7:04 [PATCH v3] drivers: regulator: add Maxim 8998 driver Marek Szyprowski
2010-06-16 14:30 ` Mark Brown
2010-06-16 20:16 ` Liam Girdwood
2010-06-21 10:26 ` Samuel Ortiz
2010-06-17 13:35 ` Marek Szyprowski
2010-06-18 0:31 ` Mark Brown
2010-06-21 8:04 ` [PATCH] drivers: regulators: depend on MFD_MAX8998 Marek Szyprowski
2010-06-21 9:00 ` Mark Brown
2010-06-21 17:49 ` Liam Girdwood
2010-06-21 10:25 ` [PATCH v3] drivers: regulator: add Maxim 8998 driver Samuel Ortiz
2010-06-21 17:50 ` Liam Girdwood [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1277142658.3288.46.camel@odin \
--to=lrg@slimlogic.co.uk \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=kyungmin.park@samsung.com \
--cc=linux-kernel@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=sameo@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.