Re: [PATCH v1 00/13] intel-lpss: support non-ACPI platforms

[Andy Shevchenko]

On Tue, 2015-11-24 at 16:11 +0100, Rafael J. Wysocki wrote:
> On Tuesday, November 24, 2015 12:22:46 PM Andy Shevchenko wrote:
> > This series includes few logical sets that bring a support of non-
> > ACPI
> > platforms for Intel Skylake.
> > 
> > First part is a refactoring of built-in device properties support:
> > Â- keep single value inside the structure
> > Â- provide helper macros to define built-in properties
> > Â- fall back to secondary fwnode if primary has no asked property
> > 
> > Second one is modifications to MFD code and intel-lpss.c driver in
> > particular
> > to define and pass built-in properties to the individual drivers.
> > 
> > Last part is a fix for I2C bug found on Lenovo Yoga hardware and a
> > first
> > converted user.
> > 
> > Built-in device properties is an alternative to platform data. It
> > provides a
> > unified API that drivers can use to cover all cases at once: DT,
> > ACPI, and
> > built-in properties.
> > 
> > With this series applied platform data can be considered obsolete.
> > Moreover,
> > built-in device properties allows to adjust existing configuration,
> > for
> > example, in cases when ACPI values are wrong on some platforms.
> > 
> > The series has been tested on available hardware and doesn't break
> > current
> > behaviour. But we ask you, Kevin, to apply the series on your side
> > and check
> > with Lenovo hardware.
> > 
> > Andy Shevchenko (8):
> > Â device property: always check for fwnode type
> > Â device property: rename helper functions
> > Â device property: refactor built-in properties support
> > Â device property: keep single value inplace
> > Â device property: improve readability of macros
> > Â device property: return -EINVAL when property isn't found in ACPI
> > Â device property: Fallback to secondary fwnode if primary misses
> > the
> > ÂÂÂÂproperty
> > Â mfd: core: propagate device properties to sub devices drivers
> > 
> > Heikki Krogerus (1):
> > Â device property: helper macros for property entry creation
> > 
> > Mika Westerberg (4):
> > Â mfd: intel-lpss: Add support for passing device properties
> > Â mfd: intel-lpss: Pass HSUART configuration via properties
> > Â mfd: intel-lpss: Pass SDA hold time to I2C host controller driver
> > Â i2c: designware: Convert to use unified device property API
> > 
> > Âdrivers/acpi/property.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂ10 +-
> > Âdrivers/base/property.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ| 298
> > +++++++++++++++++++++-------
> > Âdrivers/i2c/busses/i2c-designware-platdrv.c |ÂÂ48 ++---
> > Âdrivers/mfd/intel-lpss-acpi.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂ18 +-
> > Âdrivers/mfd/intel-lpss-pci.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂ43 +++-
> > Âdrivers/mfd/intel-lpss.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂ16 +-
> > Âdrivers/mfd/intel-lpss.hÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂ2 +
> > Âdrivers/mfd/mfd-core.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂ3 +
> > Âinclude/linux/mfd/core.hÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂ5 +
> > Âinclude/linux/property.hÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂ87 ++++++--
> > Â10 files changed, 394 insertions(+), 136 deletions(-)
> 
> All patches in this series look good to me overall, but please fix
> the build
> problems reported by 0-day and resubmit.

Thanks for fast response. We will do that later this week while
gathering comments from others.



> Thanks,
> Rafael
> 

-- 
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/