lkml.org 
[lkml]   [2018]   [Sep]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH] pinctrl: icelake: Fix the resource number for community-4/5
    On Mon, Sep 24, 2018 at 8:04 PM Rajat Jain <rajatja@google.com> wrote:
    > On Mon, Sep 24, 2018 at 7:54 AM Banik, Subrata <subrata.banik@intel.com> wrote:

    > > First of all, this is pre-production chip, so, I don't think there is a bug in the driver (yet) discovered.
    > >
    > > Looking to the above ASL code I may conclude that is definitely is *not* from our reference BIOS.
    > > I have checked two versions of it and found that in both we have the following mapping:
    > > for LP variant: there are only 4 communities are exported for H variant: there are only 5 communities are exported
    > >
    > > So, I guess the problem is in ASL code you provided. It simple should not export that community at all.
    > >
    > > In case you need to do so, there are ways:
    > > - contact Intel and ask for a change in reference BIOS
    > > - acquire another ACPI ID for the case, or, perhaps use special constants like
    > > _HRV for that purpose (also need to contact Intel while doing that)
    >
    > As Subrata clarified (Subrata & Aamir are from Intel's Coreboot team),
    > that is not *the* Intel reference BIOS that you are using, but it is
    > *an* Intel generated BIOS/Coreboot image.

    Ah, interesting.

    > Andy, can you please let us know in what order are the resources laid
    > out in your reference BIOS for the case when it exports 5 communities
    > (i.e. community-0-2, 4-5)?

    We have no hardware with such PCH to answer to this question. LP
    version exports only 4 communities and order of them as per existing
    driver.

    --
    With Best Regards,
    Andy Shevchenko

    \
     
     \ /
      Last update: 2018-09-24 23:10    [W:3.516 / U:0.000 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site