From: Hans de Goede <hdegoede@redhat.com>
To: Jorge Lopez <jorgealtxwork@gmail.com>,
platform-driver-x86@vger.kernel.org
Subject: Re: [PATCH v4 0/6] Introduction of HP-BIOSCFG driver
Date: Tue, 8 Nov 2022 15:59:24 +0100 [thread overview]
Message-ID: <4a7d3347-4eff-1a56-5703-ff8bfe91981e@redhat.com> (raw)
In-Reply-To: <20221020201033.12790-1-jorge.lopez2@hp.com>
Hi Jorge,
On 10/20/22 22:10, Jorge Lopez wrote:
> Version 4 restructures the patches submitted in previous versions.
> Earlier hp-bioscfg patches were squashed together before creating
> the new split.
>
> Version 4.0 breaks down the changes as follows:
>
> 1. Moving existing HP drivers to a central location
I have merged this patch, so you can drop this for version 5
of the patchset.
> The driver files were broken down in 5 patches of 3 files each
> with exception of patch 6/6
>
> 2. Introduction of HP-BIOSCFG driver - Set 1
I've done a detailed review of this single patches. This has
found quite a few things to improve. Note that many of the
remarks and especially the remarks about enum-attributes.c
also apply to the other files (to the other *-attributes.c
files).
Please prepare a version 5 taking all remarks into account
for all files of the driver and then I will continue the
review from there.
One thing which I did already notice for the last patch
in the series, please drop the "depends on DMI" from the
Kconfig bits and drop "include/dmi.h", you are not using
any DMI functions so these are not necessary.
And please add a:
L: platform-driver-x86@vger.kernel.org
line to the MAINTAINERS entry.
Regards,
Hans
> 3. HP BIOSCFG driver - set 2
> 4. HP BIOSCFG driver - set 3
> 5. HP BIOSCFG driver - set 4
> 6. HP BIOSCFG driver - remaining components
>
> --
>
>
> Jorge Lopez (6):
> Moving existing HP drivers to a central location
> Introduction of HP-BIOSCFG driver
> HP BIOSCFG driver - set 2
> HP BIOSCFG driver - set 3
> HP BIOSCFG driver - set 4
> HP BIOSCFG driver - remaining components
>
> .../testing/sysfs-class-firmware-attributes | 181 ++-
> MAINTAINERS | 15 +-
> drivers/platform/x86/Kconfig | 80 +-
> drivers/platform/x86/Makefile | 4 +-
> drivers/platform/x86/hp/Kconfig | 81 ++
> drivers/platform/x86/hp/Makefile | 11 +
> drivers/platform/x86/hp/hp-bioscfg/Makefile | 19 +
> .../x86/hp/hp-bioscfg/biosattr-interface.c | 285 +++++
> drivers/platform/x86/hp/hp-bioscfg/bioscfg.c | 1064 +++++++++++++++++
> drivers/platform/x86/hp/hp-bioscfg/bioscfg.h | 671 +++++++++++
> .../x86/hp/hp-bioscfg/enum-attributes.c | 521 ++++++++
> .../x86/hp/hp-bioscfg/int-attributes.c | 478 ++++++++
> .../x86/hp/hp-bioscfg/ordered-attributes.c | 586 +++++++++
> .../x86/hp/hp-bioscfg/passwdattr-interface.c | 50 +
> .../x86/hp/hp-bioscfg/passwdobj-attributes.c | 647 ++++++++++
> .../x86/hp/hp-bioscfg/spmobj-attributes.c | 408 +++++++
> .../x86/hp/hp-bioscfg/string-attributes.c | 457 +++++++
> .../x86/hp/hp-bioscfg/sureadmin-attributes.c | 1014 ++++++++++++++++
> .../x86/hp/hp-bioscfg/surestart-attributes.c | 145 +++
> drivers/platform/x86/{ => hp}/hp-wmi.c | 0
> drivers/platform/x86/{ => hp}/hp_accel.c | 0
> drivers/platform/x86/{ => hp}/tc1100-wmi.c | 0
> 22 files changed, 6647 insertions(+), 70 deletions(-)
> create mode 100644 drivers/platform/x86/hp/Kconfig
> create mode 100644 drivers/platform/x86/hp/Makefile
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/Makefile
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/biosattr-interface.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/bioscfg.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/bioscfg.h
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/enum-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/int-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/ordered-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/passwdattr-interface.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/passwdobj-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/spmobj-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/string-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/sureadmin-attributes.c
> create mode 100644 drivers/platform/x86/hp/hp-bioscfg/surestart-attributes.c
> rename drivers/platform/x86/{ => hp}/hp-wmi.c (100%)
> rename drivers/platform/x86/{ => hp}/hp_accel.c (100%)
> rename drivers/platform/x86/{ => hp}/tc1100-wmi.c (100%)
>
> --
> 2.34.1
>
next prev parent reply other threads:[~2022-11-08 15:00 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-20 20:10 [PATCH v4 0/6] Introduction of HP-BIOSCFG driver Jorge Lopez
2022-10-20 20:10 ` [PATCH v4 1/6] Moving existing HP drivers to a central location Jorge Lopez
2022-11-07 13:48 ` Hans de Goede
2022-11-08 21:36 ` Jorge Lopez
2022-10-20 20:10 ` [PATCH v4 2/6] Introduction of HP-BIOSCFG driver Jorge Lopez
2022-11-08 14:51 ` Hans de Goede
2022-11-09 17:24 ` Jorge Lopez
2022-11-09 18:10 ` Hans de Goede
2022-11-09 20:00 ` Jorge Lopez
2022-11-09 20:05 ` Hans de Goede
2022-11-09 20:52 ` Jorge Lopez
2022-11-09 20:55 ` Hans de Goede
2022-11-09 21:04 ` Jorge Lopez
2022-11-09 21:11 ` Hans de Goede
2022-11-09 21:26 ` Jorge Lopez
2022-11-11 23:00 ` Jorge Lopez
2022-11-12 8:30 ` Hans de Goede
2022-11-14 14:13 ` Jorge Lopez
2022-10-20 20:10 ` [PATCH v4 3/6] HP BIOSCFG driver - set 2 Jorge Lopez
2022-10-20 20:10 ` [PATCH v4 4/6] HP BIOSCFG driver - set 3 Jorge Lopez
2022-10-20 20:10 ` [PATCH v4 5/6] HP BIOSCFG driver - set 4 Jorge Lopez
2022-10-20 20:10 ` [PATCH v4 6/6] HP BIOSCFG driver - remaining components Jorge Lopez
2022-11-08 14:59 ` Hans de Goede [this message]
2022-11-08 21:38 ` [PATCH v4 0/6] Introduction of HP-BIOSCFG driver Jorge Lopez
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4a7d3347-4eff-1a56-5703-ff8bfe91981e@redhat.com \
--to=hdegoede@redhat.com \
--cc=jorgealtxwork@gmail.com \
--cc=platform-driver-x86@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox