On Thu, 2014-08-21 at 08:43 +0200, Thierry Reding wrote: > On Wed, Aug 20, 2014 at 06:57:59PM +0100, Alan Cox wrote: > > On Wed, 2014-08-20 at 12:12 +0200, Thierry Reding wrote: > > > On Tue, Aug 19, 2014 at 07:17:34PM +0300, Andy Shevchenko wrote: > > > > This small series makes the architecture of pwm-lpss driver cleaner. > > > > > > > > Since v3: > > > > - rebased on top of recent linux-next > > > > - rebased on top of Alan's patch for Braswell > > > > > > > > Since v2: > > > > - fix one more typo in Kconfig (tested with one line removed in my > > > > config now) > > > > > > > > Since v1: > > > > - hide struct definition in the core part > > > > - rename pwm-lpss-plat to pwm-lpss-platform > > > > - fix Kconfig dependencies and other typos > > > > - add Mika's Reviewed-by tag > > > > - fix spelling in the commit messages > > > > > > > > Andy Shevchenko (2): > > > > pwm: lpss: properly split driver to parts > > > > pwm: lpss: pci: move to use pcim_enable_device() > > > > > > > > drivers/pwm/Kconfig | 21 ++++++- > > > > drivers/pwm/Makefile | 2 + > > > > drivers/pwm/pwm-lpss-pci.c | 65 +++++++++++++++++++ > > > > drivers/pwm/pwm-lpss-platform.c | 69 ++++++++++++++++++++ > > > > drivers/pwm/pwm-lpss.c | 136 > > > > +++------------------------------------- > > > > drivers/pwm/pwm-lpss.h | 32 ++++++++++ > > > > 6 files changed, 198 insertions(+), 127 deletions(-) > > > > create mode 100644 drivers/pwm/pwm-lpss-pci.c > > > > create mode 100644 drivers/pwm/pwm-lpss-platform.c > > > > create mode 100644 drivers/pwm/pwm-lpss.h > > > > > > Yea or nay, Alan? > > > > Context ? > > Back when the LPSS PWM driver was first submitted I suggested that it > could be split up in this way (core + PCI and ACPI/platform drivers). > You had objections to doing that. Now Andy's doing exactly that, so I > would like your Acked-by (or NAK) on this change.
I think Andy proved you were right 8) Ack from me Alan -- To unsubscribe from this list: send the line "unsubscribe linux-pwm" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
