-V3 TPS6507x MFD driver
in [Kernel]
|
From: Mark Brown on 6 Apr 2010 07:00 On Mon, Apr 05, 2010 at 08:23:55PM -0600, Todd Fischer wrote: > This is the third posting of the TPS6507x driver set. I have > resolved the concurrent access issue identified by Mark Brown. > Instead of using a mutex, I used the i2c_transfer() as done in > the similar routine in the wm8400-core.c file. All Acked-by: Mark Brown <broonie(a)opensource.wolfsonmicro.com> When reposting a patch series that has previously been reviewed it's useful to record any acks and similar that were given for patches that have been reposted unmodified - this makes life easier for reviewers since it saves having to re-review previously posted patches. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Liam Girdwood on 6 Apr 2010 07:10 Hi Samuel, Please feel free to upstream via MFD. On Tue, 2010-04-06 at 11:51 +0100, Mark Brown wrote: > On Mon, Apr 05, 2010 at 08:23:55PM -0600, Todd Fischer wrote: > > > This is the third posting of the TPS6507x driver set. I have > > resolved the concurrent access issue identified by Mark Brown. > > Instead of using a mutex, I used the i2c_transfer() as done in > > the similar routine in the wm8400-core.c file. > > All > > Acked-by: Mark Brown <broonie(a)opensource.wolfsonmicro.com> Signed-off-by: Liam Girdwood <lrg(a)slimlogic.co.uk> -- Freelance Developer, SlimLogic Ltd ASoC and Voltage Regulator Maintainer. http://www.slimlogic.co.uk -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Samuel Ortiz on 7 Apr 2010 04:50 Hi Todd, On Mon, Apr 05, 2010 at 08:23:55PM -0600, Todd Fischer wrote: > > This is the third posting of the TPS6507x driver set. I have > resolved the concurrent access issue identified by Mark Brown. > Instead of using a mutex, I used the i2c_transfer() as done in > the similar routine in the wm8400-core.c file. > > The TPS6507x family of Texas Instruments power management ICs (pmic) are > multi-function chips that include voltage regulation and touch screen > controller capabilities. This patch set adds a TPS6507x multi-function > device driver and change the TPS6507x regulator driver to use the > TPS6507x MFD driver instead of interacting with the chip directly. The > changes are needed before a touch screen driver can be added. > > The patch set applies cleanly to the MFD repository: > git://git.kernel.org/pub/scm/linux/kernel/git/sameo/mfd-2.6.git If the > MFD and voltage regulator maintainer would prefer I post patches for a > different repository, please let me know. All 4 patches applied to my for-next branch. Some nit picking advices for your future patches: - Include the Signed-off-by and Acked-by you collect along the review cycles. - Have shorter patch subject, they typically are less than 80 chars. - Describe the changes when moving from patchset vN to patchset vN+1. Also we prefer "[PATCH vN]" to "[PATCH]-vN", it makes our life easier when applying your patches. Thanks a lot for your work. Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|
Pages: 1 Prev: [PATCH 1/2] KVM MMU: remove unused field Next: proc: copy_to_user() returns unsigned |