Received-SPF: pass (google.com: domain of linux-kernel+bounces-28838-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <cf895498-ef54-4667-a205-cc3150be9a22@collabora.com>
Date: Wed, 17 Jan 2024 11:29:03 +0100
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2 0/7] Allow coreboot modules to autoload and enable
 cbmem in the arm64 defconfig
Content-Language: en-US
To: =?UTF-8?B?TsOtY29sYXMgRi4gUi4gQS4gUHJhZG8=?= <nfraprado@collabora.com>,
 Tzung-Bi Shih <tzungbi@kernel.org>
Cc: kernel@collabora.com, chrome-platform@lists.linux.dev,
 Abhijit Gangurde <abhijit.gangurde@amd.com>,
 Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
 Arnd Bergmann <arnd@arndb.de>, Bjorn Andersson <quic_bjorande@quicinc.com>,
 Brian Norris <briannorris@chromium.org>,
 Catalin Marinas <catalin.marinas@arm.com>,
 Geert Uytterhoeven <geert+renesas@glider.be>,
 Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
 Julius Werner <jwerner@chromium.org>,
 Konrad Dybcio <konrad.dybcio@linaro.org>,
 Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
 Marek Szyprowski <m.szyprowski@samsung.com>,
 Masahiro Yamada <masahiroy@kernel.org>, Nathan Chancellor
 <nathan@kernel.org>, Neil Armstrong <neil.armstrong@linaro.org>,
 Nicolas Schier <nicolas@fjasle.eu>, Nipun Gupta <nipun.gupta@amd.com>,
 Pieter Jansen van Vuuren <pieter.jansen-van-vuuren@amd.com>,
 U mang Jain <umang.jain@ideasonboard.com>, Will Deacon <will@kernel.org>,
 linux-arm-kernel@lists.infradead.org, linux-kbuild@vger.kernel.org,
 linux-kernel@vger.kernel.org
References: <20240112131857.900734-1-nfraprado@collabora.com>
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
In-Reply-To: <20240112131857.900734-1-nfraprado@collabora.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit

Il 12/01/24 14:18, Nícolas F. R. A. Prado ha scritto:
> 
> This series adds the missing pieces to the coreboot bus and the module
> alias generation to allow coreboot modules to be automatically loaded
> when matching devices are detected.
> 
> The configs for cbmem coreboot entries are then enabled in the arm64
> defconfig, as modules, to allow reading logs from coreboot on arm64
> Chromebooks, which is useful for debugging the boot process.
> 

For the entire series:

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

> Changes in v2:
> - Added commits for vpd, memconsole and framebuffer drivers to add them
>    to the module device table
> 
> Nícolas F. R. A. Prado (7):
>    firmware: coreboot: Generate modalias uevent for devices
>    firmware: coreboot: Generate aliases for coreboot modules
>    firmware: google: cbmem: Add to module device table
>    firmware: google: vpd: Add to module device table
>    firmware: google: memconsole: Add to module device table
>    firmware: google: framebuffer: Add to module device table
>    arm64: defconfig: Enable support for cbmem entries in the coreboot
>      table
> 
>   arch/arm64/configs/defconfig                   |  3 +++
>   drivers/firmware/google/cbmem.c                |  7 +++++++
>   drivers/firmware/google/coreboot_table.c       |  9 +++++++++
>   drivers/firmware/google/framebuffer-coreboot.c |  7 +++++++
>   drivers/firmware/google/memconsole-coreboot.c  |  7 +++++++
>   drivers/firmware/google/vpd.c                  |  7 +++++++
>   include/linux/mod_devicetable.h                |  8 ++++++++
>   scripts/mod/devicetable-offsets.c              |  3 +++
>   scripts/mod/file2alias.c                       | 10 ++++++++++
>   9 files changed, 61 insertions(+)
>