Re: [PATCH 0/5] Update and move Dell drivers dell_rbu and dcdbas
On Wed, Sep 26, 2018 at 5:50 PM wrote: > > > On Tue, Sep 25, 2018 at 10:34 PM Stuart Hayes > > wrote: > > > > > > From: Stuart Hayes > > > > > > 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! > > > > Just so it's extra clear, you want an "Acked-by" tag from Doug W in a public > forum? Or he should be CC to that patch and needs to give the acked-by > tag in the V2? > > He already did respond in a public forum indicating this although he didn't > "generate a tag" for that: > https://www.spinics.net/lists/platform-driver-x86/msg16336.html Thanks, that is what I need. > Can Stuart just add the "Acked-by" tag himself to the commit in V2 at this > point considering that? Yes. > Or do you need patchwork to pick it up and apply it? No need. Manual update would work to me. -- With Best Regards, Andy Shevchenko
Re: [PATCH 0/5] Update and move Dell drivers dell_rbu and dcdbas
On Wed, Sep 26, 2018 at 5:50 PM wrote: > > > On Tue, Sep 25, 2018 at 10:34 PM Stuart Hayes > > wrote: > > > > > > From: Stuart Hayes > > > > > > 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! > > > > Just so it's extra clear, you want an "Acked-by" tag from Doug W in a public > forum? Or he should be CC to that patch and needs to give the acked-by > tag in the V2? > > He already did respond in a public forum indicating this although he didn't > "generate a tag" for that: > https://www.spinics.net/lists/platform-driver-x86/msg16336.html Thanks, that is what I need. > Can Stuart just add the "Acked-by" tag himself to the commit in V2 at this > point considering that? Yes. > Or do you need patchwork to pick it up and apply it? No need. Manual update would work to me. -- With Best Regards, Andy Shevchenko
RE: [PATCH 0/5] Update and move Dell drivers dell_rbu and dcdbas
> On Tue, Sep 25, 2018 at 10:34 PM Stuart Hayes > wrote: > > > > From: Stuart Hayes > > > > 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! > Just so it's extra clear, you want an "Acked-by" tag from Doug W in a public forum? Or he should be CC to that patch and needs to give the acked-by tag in the V2? He already did respond in a public forum indicating this although he didn't "generate a tag" for that: https://www.spinics.net/lists/platform-driver-x86/msg16336.html Can Stuart just add the "Acked-by" tag himself to the commit in V2 at this point considering that? Or do you need patchwork to pick it up and apply it?
RE: [PATCH 0/5] Update and move Dell drivers dell_rbu and dcdbas
> On Tue, Sep 25, 2018 at 10:34 PM Stuart Hayes > wrote: > > > > From: Stuart Hayes > > > > 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! > Just so it's extra clear, you want an "Acked-by" tag from Doug W in a public forum? Or he should be CC to that patch and needs to give the acked-by tag in the V2? He already did respond in a public forum indicating this although he didn't "generate a tag" for that: https://www.spinics.net/lists/platform-driver-x86/msg16336.html Can Stuart just add the "Acked-by" tag himself to the commit in V2 at this point considering that? Or do you need patchwork to pick it up and apply it?
Re: [PATCH 0/5] Update and move Dell drivers dell_rbu and dcdbas
On Wed, Sep 26, 2018 at 2:04 PM Andy Shevchenko wrote: > > On Tue, Sep 25, 2018 at 10:34 PM Stuart Hayes > wrote: > > > > From: Stuart Hayes > > > > 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). Applying: MAINTAINERS: Update maintainer for dcdbas and dell_rbu .git/rebase-apply/patch:14: trailing whitespace. M: Stuart Hayes warning: 1 line adds whitespace errors. Fix the above as well. > > 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 -- With Best Regards, Andy Shevchenko
Re: [PATCH 0/5] Update and move Dell drivers dell_rbu and dcdbas
On Wed, Sep 26, 2018 at 2:04 PM Andy Shevchenko wrote: > > On Tue, Sep 25, 2018 at 10:34 PM Stuart Hayes > wrote: > > > > From: Stuart Hayes > > > > 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). Applying: MAINTAINERS: Update maintainer for dcdbas and dell_rbu .git/rebase-apply/patch:14: trailing whitespace. M: Stuart Hayes warning: 1 line adds whitespace errors. Fix the above as well. > > 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 -- With Best Regards, Andy Shevchenko
Re: [PATCH 0/5] Update and move Dell drivers dell_rbu and dcdbas
On Tue, Sep 25, 2018 at 10:34 PM Stuart Hayes wrote: > > From: Stuart Hayes > > 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
Re: [PATCH 0/5] Update and move Dell drivers dell_rbu and dcdbas
On Tue, Sep 25, 2018 at 10:34 PM Stuart Hayes wrote: > > From: Stuart Hayes > > 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
[PATCH 0/5] Update and move Dell drivers dell_rbu and dcdbas
From: Stuart Hayes 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. 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
[PATCH 0/5] Update and move Dell drivers dell_rbu and dcdbas
From: Stuart Hayes 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. 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