Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 4 Apr 2023 09:33:53 +0700
From:   Bagas Sanjaya <bagasdotme@gmail.com>
To:     Jorge Lopez <jorgealtxwork@gmail.com>, hdegoede@redhat.com,
        Linux x86 Platform Drivers 
        <platform-driver-x86@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Documentation <linux-doc@vger.kernel.org>
Subject: Re: [PATCH v7] Introduction-of-HP-BIOSCFG-driver-documentation
Message-ID: <ZCuMkdb6jeL4S8hz@debian.me>
References: <20230403211548.6253-1-jorge.lopez2@hp.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="wZWQ5ERks+J5+e0a"
Content-Disposition: inline
In-Reply-To: <20230403211548.6253-1-jorge.lopez2@hp.com>
Precedence: bulk


--wZWQ5ERks+J5+e0a
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, Apr 03, 2023 at 04:15:48PM -0500, Jorge Lopez wrote:
> HP BIOS Configuration driver purpose is to provide a driver supporting
> the latest sysfs class firmware attributes framework allowing the user
> to change BIOS settings and security solutions on HP Inc.=E2=80=99s comme=
rcial
> notebooks.
>=20
> Many features of HP Commercial notebooks can be managed using Windows
> Management Instrumentation (WMI). WMI is an implementation of Web-Based
> Enterprise Management (WBEM) that provides a standards-based interface
> for changing and monitoring system settings. HP BIOSCFG driver provides
> a native Linux solution and the exposed features facilitates the
> migration to Linux environments.
>=20
> The Linux security features to be provided in hp-bioscfg driver enables
> managing the BIOS settings and security solutions via sysfs, a virtual
> filesystem that can be used by user-mode applications. The new=20
> documentation cover features such Secure Platform Management and Sure=20
> Start. Each section provides security feature description and identifies=
=20
> sysfs directories and files exposed by the driver.
>=20
> Many HP Commercial notebooks include a feature called Secure Platform
> Management (SPM), which replaces older password-based BIOS settings
> management with public key cryptography. PC secure product management
> begins when a target system is provisioned with cryptographic keys
> that are used to ensure the integrity of communications between system
> management utilities and the BIOS.
>=20
> HP Commercial notebooks have several BIOS settings that control its=20
> behaviour and capabilities, many of which are related to security.=20
> To prevent unauthorized changes to these settings, the system can be=20
> configured to use a cryptographic signature-based authorization string=20
> that the BIOS will use to verify authorization to modify the setting.

If this is single patch, I'd like to write the patch subject as
"Documentation: sysfs: document HP-specific firmware attributes".

And also, adjust the patch description accordingly, since as it is
written above, it looks like general documentation of HP-specific feature
(which should be in actual diff).

> Version 7
> 	Includes only sysfs-class-firmware-attributes documentation

Where is the rest of patches if this is a series? Had they been merged?

Thanks.

--=20
An old man doll... just what I always wanted! - Clara

--wZWQ5ERks+J5+e0a
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHQEABYKAB0WIQSSYQ6Cy7oyFNCHrUH2uYlJVVFOowUCZCuMjAAKCRD2uYlJVVFO
ow7OAP9gRcmB0bGzGPNpYcIbQOXyVkzMwcRvfFZI8DazZekgfwD1F0pLNN9sqHpv
EwDI6/5ooWivsvoDHirVWJkj0mdpCw==
=mUcv
-----END PGP SIGNATURE-----

--wZWQ5ERks+J5+e0a--