On Thu, Aug 19, 2021 at 08:46:14PM +0200, Hans de Goede wrote: > On 8/19/21 6:37 PM, Andy Shevchenko wrote: > > This is v4 of the Intel drivers move on in the source tree. > > v3 has been done by Kate: > > https://lore.kernel.org/platform-driver-x86/20210819033001.20136-1-hpa@xxxxxxxxxx/ > > > > I have taken the initial set from review-hans branch and removed Hans' SoB > > along with Link, while leaving others' tags. > > > > Changelog v4: > > - spelled fully the driver names in the commit messages > > - fixed SCU *.ko module names > > Actually I checked this during review and > CONFIG_INTEL_SCU_IPC, CONFIG_INTEL_SCU_PCI and CONFIG_INTEL_SCU_WDT > are booleans, iow if enabled they are always builtin so the > drivers/platform/x86/intel/scu/Makefile from v3 was correct. > > Since the v3 Makefile is more simple I prefer that version, > but if you prefer to keep the v4 version that is fine too. I prefer mine, but we need to fix what kbuild bot complained about. > > - dropped extra prefix in PMC files > > - dropped stray changes within the series > > - removed confusing comments in Makefile and Kconfig files > > - embedded a few of Kconfig improvements here and there (ordering, spelling) > > - split miscellaneous group > > - added a patch to fix kernel doc issue in SCU IPC code > > Thanks, I've not (re)reviewed things, but this all sounds good. > > > So, I have noticed the report about SCU and taking into account IPS > > header deferred move, I think those two should be excluded from the > > series and sent after rc1, it will also eliminate trampoline move for > > IPS header, since we may send one patch which includes DRM changes. > > > > Hans, what do you think? > > Fixing the SCU thing will require coordination with / an > ack from the MFD maintainer (Lee) so yeah dropping that for > now and doing the SCU move next cycle is probably the best. > > Dropping the IPS move for now is fine with me too. > > Can you send a v5 with those 2 patches dropped ? Yes. > We can still fix the SCU doc, just in the old place... And yes, definitely. -- With Best Regards, Andy Shevchenko