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

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

[Andy Shevchenko]

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@linux.intel.com>
---
 .../asm/arch-tangier/acpi/southcluster.asl    | 20 +++++++++++++++++++
 1 file changed, 20 insertions(+)

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)
-- 
2.25.1

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

[Simon Glass]

On Fri, 20 Mar 2020 at 11:46, Andy Shevchenko
<andriy.shevchenko@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@linux.intel.com>
> ---
>  .../asm/arch-tangier/acpi/southcluster.asl    | 20 +++++++++++++++++++
>  1 file changed, 20 insertions(+)

Reviewed-by: Simon Glass <sjg@chromium.org>

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

[Bin Meng]

On Sat, Mar 21, 2020 at 1:46 AM Andy Shevchenko
<andriy.shevchenko@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@linux.intel.com>
> ---
>  .../asm/arch-tangier/acpi/southcluster.asl    | 20 +++++++++++++++++++
>  1 file changed, 20 insertions(+)
>

applied to u-boot-x86, thanks!

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

[Andy Shevchenko]

On Wed, Mar 25, 2020 at 9:15 AM Bin Meng <bmeng.cn@gmail.com> wrote:
>
> On Sat, Mar 21, 2020 at 1:46 AM Andy Shevchenko
> <andriy.shevchenko@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@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?

-- 
With Best Regards,
Andy Shevchenko

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

[Bin Meng]

Hi Andy,

On Wed, Mar 25, 2020 at 7:41 PM Andy Shevchenko
<andy.shevchenko@gmail.com> wrote:
>
> On Wed, Mar 25, 2020 at 9:15 AM Bin Meng <bmeng.cn@gmail.com> wrote:
> >
> > On Sat, Mar 21, 2020 at 1:46 AM Andy Shevchenko
> > <andriy.shevchenko@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@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