[Public] > -----Original Message----- > From: Mark Pearson <markpearson@xxxxxxxxxx> > Sent: Tuesday, October 26, 2021 13:50 > To: Limonciello, Mario <Mario.Limonciello@xxxxxxx>; Hans de Goede > <hdegoede@xxxxxxxxxx>; Mark Gross <mgross@xxxxxxxxxxxxxxx>; Rafael J . > Wysocki <rjw@xxxxxxxxxxxxx> > Cc: open list:X86 PLATFORM DRIVERS <platform-driver-x86@xxxxxxxxxxxxxxx>; > linux-acpi@xxxxxxxxxxxxxxx > Subject: Re: [External] [PATCH v3 3/3] ACPI: platform_profile: Add support for > notification chains > > > Hi Mario > > On 2021-10-26 14:38, Limonciello, Mario wrote: > > [Public] > > > >> -----Original Message----- > >> From: Mark Pearson <markpearson@xxxxxxxxxx> > >> Sent: Tuesday, October 26, 2021 13:36 > >> To: Limonciello, Mario <Mario.Limonciello@xxxxxxx>; Hans de Goede > >> <hdegoede@xxxxxxxxxx>; Mark Gross <mgross@xxxxxxxxxxxxxxx>; Rafael J . > >> Wysocki <rjw@xxxxxxxxxxxxx> > >> Cc: open list:X86 PLATFORM DRIVERS <platform-driver- > x86@xxxxxxxxxxxxxxx>; > >> linux-acpi@xxxxxxxxxxxxxxx > >> Subject: Re: [External] [PATCH v3 3/3] ACPI: platform_profile: Add support > for > >> notification chains > >> > >> Thanks Mario, > >> > >> Patch series looks good. One minor suggestion below. > > > > Thanks Mark. What do you think of the other idea I had in my cover letter? > > I think that's another way to do this, that might mean less surgery to this > > source file and other function call. > > > > I'll re-spin it either to accept your suggestion below or the other idea I put > > in the cover letter. > > I don't have a strong opinion either I'm afraid :) Getting the > profile when you register seems neat to me - I have no objections. > > Mark OK I slightly lean towards keeping it as is to prevent racing around driver load order. I'll adopt your suggestion and re-send.
