Thanks Andy, > -----Original Message----- > From: Andy Shevchenko [mailto:andy.shevchenko@xxxxxxxxx] > Sent: Monday, February 01, 2016 4:54 PM > To: Chen, Yu C > Cc: linux-acpi@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Rafael J. > Wysocki; Len Brown; matthew.garrett@xxxxxxxxxx; 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@xxxxxxxxx> 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; > > +} > ��.n��������+%������w��{.n�����{�����ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f