Re: [PATCH v1 1/1] mfd: lpss: Add Intel Kaby Lake PCH-H PCI IDs

[Lee Jones <lee.jones@linaro.org>]

On Fri, 22 Jul 2016, Andy Shevchenko wrote:

> On Mon, Jul 4, 2016 at 6:24 PM, Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> > From: Mika Westerberg <mika.westerberg@linux.intel.com>
> >
> > Intel Kaby Lake PCH-H has the same LPSS than Intel Sunrisepoint. Add the new
> > IDs to the list of supported devices.
> 
> I'm sorry for an empty ping, but we would like to clarify what is the
> plan for this patch. Will it be material for v4.8-rc1?

I don't tend to accept !bug-fix patches post -rc6, and the
merge-window is due to open imminently.  So it's due for v4.9.

> > Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
> > ---
> >  drivers/mfd/intel-lpss-pci.c | 23 +++++++++++++++++++++++
> >  1 file changed, 23 insertions(+)
> >
> > diff --git a/drivers/mfd/intel-lpss-pci.c b/drivers/mfd/intel-lpss-pci.c
> > index 1d79a3c..d19569a 100644
> > --- a/drivers/mfd/intel-lpss-pci.c
> > +++ b/drivers/mfd/intel-lpss-pci.c
> > @@ -111,6 +111,19 @@ static const struct intel_lpss_platform_info bxt_i2c_info = {
> >         .properties = bxt_i2c_properties,
> >  };
> >
> > +static const struct intel_lpss_platform_info kbl_info = {
> > +       .clk_rate = 120000000,
> > +};
> > +
> > +static const struct intel_lpss_platform_info kbl_uart_info = {
> > +       .clk_rate = 120000000,
> > +       .clk_con_id = "baudclk",
> > +};
> > +
> > +static const struct intel_lpss_platform_info kbl_i2c_info = {
> > +       .clk_rate = 133000000,
> > +};
> > +
> >  static const struct pci_device_id intel_lpss_pci_ids[] = {
> >         /* BXT A-Step */
> >         { PCI_VDEVICE(INTEL, 0x0aac), (kernel_ulong_t)&bxt_i2c_info },
> > @@ -181,6 +194,16 @@ static const struct pci_device_id intel_lpss_pci_ids[] = {
> >         { PCI_VDEVICE(INTEL, 0xa160), (kernel_ulong_t)&spt_i2c_info },
> >         { PCI_VDEVICE(INTEL, 0xa161), (kernel_ulong_t)&spt_i2c_info },
> >         { PCI_VDEVICE(INTEL, 0xa166), (kernel_ulong_t)&spt_uart_info },
> > +       /* KBL-H */
> > +       { PCI_VDEVICE(INTEL, 0xa2a7), (kernel_ulong_t)&kbl_uart_info },
> > +       { PCI_VDEVICE(INTEL, 0xa2a8), (kernel_ulong_t)&kbl_uart_info },
> > +       { PCI_VDEVICE(INTEL, 0xa2a9), (kernel_ulong_t)&kbl_info },
> > +       { PCI_VDEVICE(INTEL, 0xa2aa), (kernel_ulong_t)&kbl_info },
> > +       { PCI_VDEVICE(INTEL, 0xa2e0), (kernel_ulong_t)&kbl_i2c_info },
> > +       { PCI_VDEVICE(INTEL, 0xa2e1), (kernel_ulong_t)&kbl_i2c_info },
> > +       { PCI_VDEVICE(INTEL, 0xa2e2), (kernel_ulong_t)&kbl_i2c_info },
> > +       { PCI_VDEVICE(INTEL, 0xa2e3), (kernel_ulong_t)&kbl_i2c_info },
> > +       { PCI_VDEVICE(INTEL, 0xa2e6), (kernel_ulong_t)&kbl_uart_info },
> >         { }
> >  };
> >  MODULE_DEVICE_TABLE(pci, intel_lpss_pci_ids);
> >
> 
> 
> 

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog