diff mbox series

[v1] x86: acpi: Add I²C timings to Intel Merrifield platform

Message ID 20200320174631.33857-1-andriy.shevchenko@linux.intel.com
State New
Headers show
Series [v1] x86: acpi: Add I²C timings to Intel Merrifield platform | expand

Commit Message

Andy Shevchenko March 20, 2020, 5:46 p.m. UTC
There is established way to provide I?C timings, or actually counters,
to the OS via ACPI. Fill them for Intel Merrifield platform.

Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
---
 .../asm/arch-tangier/acpi/southcluster.asl    | 20 +++++++++++++++++++
 1 file changed, 20 insertions(+)

Comments

Simon Glass March 23, 2020, 3:36 p.m. UTC | #1
On Fri, 20 Mar 2020 at 11:46, Andy Shevchenko
<andriy.shevchenko at linux.intel.com> wrote:
>
> There is established way to provide I?C timings, or actually counters,
> to the OS via ACPI. Fill them for Intel Merrifield platform.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> ---
>  .../asm/arch-tangier/acpi/southcluster.asl    | 20 +++++++++++++++++++
>  1 file changed, 20 insertions(+)

Reviewed-by: Simon Glass <sjg at chromium.org>
Bin Meng March 25, 2020, 7:14 a.m. UTC | #2
On Sat, Mar 21, 2020 at 1:46 AM Andy Shevchenko
<andriy.shevchenko at linux.intel.com> wrote:
>
> There is established way to provide I?C timings, or actually counters,
> to the OS via ACPI. Fill them for Intel Merrifield platform.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> ---
>  .../asm/arch-tangier/acpi/southcluster.asl    | 20 +++++++++++++++++++
>  1 file changed, 20 insertions(+)
>

applied to u-boot-x86, thanks!
Andy Shevchenko March 25, 2020, 11:41 a.m. UTC | #3
On Wed, Mar 25, 2020 at 9:15 AM Bin Meng <bmeng.cn at gmail.com> wrote:
>
> On Sat, Mar 21, 2020 at 1:46 AM Andy Shevchenko
> <andriy.shevchenko at linux.intel.com> wrote:
> >
> > There is established way to provide I?C timings, or actually counters,
> > to the OS via ACPI. Fill them for Intel Merrifield platform.
> >
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> > ---
> >  .../asm/arch-tangier/acpi/southcluster.asl    | 20 +++++++++++++++++++
> >  1 file changed, 20 insertions(+)
> >
>
> applied to u-boot-x86, thanks!

There is a v2 of it. Should I send a fix up or you can replace?
Bin Meng March 25, 2020, 1 p.m. UTC | #4
Hi Andy,

On Wed, Mar 25, 2020 at 7:41 PM Andy Shevchenko
<andy.shevchenko at gmail.com> wrote:
>
> On Wed, Mar 25, 2020 at 9:15 AM Bin Meng <bmeng.cn at gmail.com> wrote:
> >
> > On Sat, Mar 21, 2020 at 1:46 AM Andy Shevchenko
> > <andriy.shevchenko at linux.intel.com> wrote:
> > >
> > > There is established way to provide I?C timings, or actually counters,
> > > to the OS via ACPI. Fill them for Intel Merrifield platform.
> > >
> > > Signed-off-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> > > ---
> > >  .../asm/arch-tangier/acpi/southcluster.asl    | 20 +++++++++++++++++++
> > >  1 file changed, 20 insertions(+)
> > >
> >
> > applied to u-boot-x86, thanks!
>
> There is a v2 of it. Should I send a fix up or you can replace?

Sorry I replied the wrong email. I did apply the v2 patch, and
manually added Simon's RB tag.

Regards,
Bin
diff mbox series

Patch

diff --git a/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl b/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl
index 95413c1112..aa1793b383 100644
--- a/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl
+++ b/arch/x86/include/asm/arch-tangier/acpi/southcluster.asl
@@ -250,6 +250,16 @@  Device (PCI0)
             Return (STA_VISIBLE)
         }
 
+        Name (SSCN, Package ()
+        {
+            0x02F8, 0x037B, Zero,
+        })
+
+        Name (FMCN, Package ()
+        {
+            0x0087, 0x010A, Zero,
+        })
+
         Name (RBUF, ResourceTemplate()
         {
             FixedDMA(0x0009, 0x0000, Width32bit, )
@@ -270,6 +280,16 @@  Device (PCI0)
         {
             Return (STA_VISIBLE)
         }
+
+        Name (SSCN, Package ()
+        {
+            0x02F8, 0x037B, Zero,
+        })
+
+        Name (FMCN, Package ()
+        {
+            0x0087, 0x010A, Zero,
+        })
     }
 
     Device (GPIO)