Re: [PATCH 0/5] Update and move Dell drivers dell_rbu and dcdbas

From: Andy Shevchenko
Date: Wed Sep 26 2018 - 07:04:33 EST


On Tue, Sep 25, 2018 at 10:34 PM Stuart Hayes <stuart.w.hayes@xxxxxxxxx> wrote:
>
> From: Stuart Hayes <stuart.w.hayes@xxxxxxxxx>
>
> The dell_rbu and dcdbas drivers need some changes, and should be moved to
> drivers/platform/x86. Additionally, dell_rbu needs a maintainer, and the
> listed maintainer for dcdbas is inactive and needs to be changed.

I pushed this series to my review and testing queue to allow bots play with.

But.
Please, add commit messages to patches 4 and 5 (I can't take w/o them).
Collect Doug's tag that proves he is okay with the change (I remember
he answered something to the old thread, though I don't remember the
details except annoying footer which should not be up in open source
mailing lists).

So, while bots are playing with the code I'm waiting for v2 of this
with above addressed.
Thanks!

>
> Stuart Hayes (5):
> firmware: dell_rbu: Make payload memory uncachable
> firmware: dcdbas: Add support for WSMT ACPI table
> firmware: dell_rbu: Move dell_rbu to drivers/platform/x86
> firmware: dcdbas: Move dcdbas to drivers/platform/x86
> MAINTAINERS: Update maintainer for dcdbas and dell_rbu
>
> MAINTAINERS | 11 +-
> drivers/firmware/Kconfig | 28 ----
> drivers/firmware/Makefile | 2 -
> drivers/platform/x86/Kconfig | 28 ++++
> drivers/platform/x86/Makefile | 2 +
> drivers/{firmware => platform/x86}/dcdbas.c | 123 +++++++++++++++++-
> drivers/{firmware => platform/x86}/dcdbas.h | 10 ++
> drivers/platform/x86/dell-smbios-smm.c | 2 +-
> drivers/{firmware => platform/x86}/dell_rbu.c | 8 ++
> 9 files changed, 175 insertions(+), 39 deletions(-)
> rename drivers/{firmware => platform/x86}/dcdbas.c (82%)
> rename drivers/{firmware => platform/x86}/dcdbas.h (93%)
> rename drivers/{firmware => platform/x86}/dell_rbu.c (98%)
>
> --
> 2.19.0.221.g150f307af
>


--
With Best Regards,
Andy Shevchenko