Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1448562 > unrolled thread

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

Started byAndy Shevchenko <andy.shevchenko@gmail.com>
First post2016-07-22 14:50 +0200
Last post2016-07-22 16:00 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v1 1/1] mfd: lpss: Add Intel Kaby Lake PCH-H PCI IDs Andy Shevchenko <andy.shevchenko@gmail.com> - 2016-07-22 14:50 +0200
    Re: [PATCH v1 1/1] mfd: lpss: Add Intel Kaby Lake PCH-H PCI IDs Lee Jones <lee.jones@linaro.org> - 2016-07-22 16:00 +0200

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

FromAndy Shevchenko <andy.shevchenko@gmail.com>
Date2016-07-22 14:50 +0200
SubjectRe: [PATCH v1 1/1] mfd: lpss: Add Intel Kaby Lake PCH-H PCI IDs
Message-ID<rXI3U-5Un-5@gated-at.bofh.it>
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?

>
> 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);
> --
> 2.8.1
>



-- 
With Best Regards,
Andy Shevchenko

[toc] | [next] | [standalone]


#1448602

FromLee Jones <lee.jones@linaro.org>
Date2016-07-22 16:00 +0200
Message-ID<rXJ9E-6An-19@gated-at.bofh.it>
In reply to#1448562
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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web