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

Groups > linux.kernel > #1322929

RE: [PATCH][v2] ACPI: Do not report _OSI("Darwin") when acpi_osi=!Darwin provided

Show all headers | View raw

Thanks Andy,

> -----Original Message-----
> From: Andy Shevchenko [mailto:andy.shevchenko@gmail.com]
> Sent: Monday, February 01, 2016 4:54 PM
> To: Chen, Yu C
> Cc: linux-acpi@vger.kernel.org; linux-kernel@vger.kernel.org; Rafael J.
> Wysocki; Len Brown; matthew.garrett@nebula.com; Zhang, Rui
> Subject: Re: [PATCH][v2] ACPI: Do not report _OSI("Darwin") when
> acpi_osi=!Darwin provided
> 
> On Mon, Feb 1, 2016 at 4:26 AM, Chen Yu <yu.c.chen@intel.com> wrote:
> > Commit 7bc5a2bad0b8 ("ACPI: Support _OSI("Darwin") correctly") always
> > reports positive value when Apple hardware queries _OSI("Darwin").
> > But sometimes the users might want to tell the hardware they don't
> > need the Darwin feature, for example, users may leverage the hardware
> > to power off the Thunderbolt, by appending acpi_osi=!Darwin in command
> > line, thus Apple hardware regards it as an incompatible OS X system,
> > hence turns off the Thunderbolt.
> 
> 
> > +static bool acpi_osi_setup_disabled(char *str) {
> > +       int i;
> > +       struct osi_setup_entry *osi;
> > +
> > +       if (!str)
> > +               return false;
> > +
> > +       for (i = 0; i < OSI_STRING_ENTRIES_MAX; i++) {
> > +               osi = &osi_setup_entries[i];
> > +               if (!strcmp(osi->string, str)) {
> 
> Seems like we have one more user for
> http://www.spinics.net/lists/kernel/msg2157535.html
[Yu] This is nice, I can convert this patch to use match_string, and
may I add my patch into your patchset? I saw there are some modifications
on drivers in your patchset above. 
> 
> > +                       if (!osi->enable)
> > +                               return true;
> > +                       else
> > +                               return false;
> 
> return !osi->enable;
[Yu] OK.
> 
> > +               }
> > +       }
> > +
> > +       return false;
> > +}
> 

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH][v2] ACPI: Do not report _OSI("Darwin") when acpi_osi=!Darwin provided Chen Yu <yu.c.chen@intel.com> - 2016-02-01 03:30 +0100
  Re: [PATCH][v2] ACPI: Do not report _OSI("Darwin") when  acpi_osi=!Darwin provided Andy Shevchenko <andy.shevchenko@gmail.com> - 2016-02-01 10:00 +0100
    RE: [PATCH][v2] ACPI: Do not report _OSI("Darwin") when  acpi_osi=!Darwin provided "Chen, Yu C" <yu.c.chen@intel.com> - 2016-02-01 10:20 +0100
      Re: [PATCH][v2] ACPI: Do not report _OSI("Darwin") when  acpi_osi=!Darwin provided Andy Shevchenko <andy.shevchenko@gmail.com> - 2016-02-01 10:40 +0100
        RE: [PATCH][v2] ACPI: Do not report _OSI("Darwin") when  acpi_osi=!Darwin provided "Chen, Yu C" <yu.c.chen@intel.com> - 2016-02-01 15:10 +0100

csiph-web