Received-SPF: pass (google.com: domain of linux-kernel+bounces-66315-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Precedence: bulk
MIME-Version: 1.0
References: <20240212-coreboot-mod-defconfig-v4-0-d14172676f6d@collabora.com> <Zc2HBmDPJnHt0WJC@google.com>
In-Reply-To: <Zc2HBmDPJnHt0WJC@google.com>
From: Masahiro Yamada <masahiroy@kernel.org>
Date: Thu, 15 Feb 2024 14:52:00 +0900
Message-ID: <CAK7LNATc0KkstjU8ErryV-GckvUKTsjR8naTRSifjw_TZYdSig@mail.gmail.com>
Subject: Re: [PATCH v4 0/4] Allow coreboot modules to autoload and enable
 cbmem in the arm64 defconfig
To: Tzung-Bi Shih <tzungbi@kernel.org>
Cc: =?UTF-8?B?TsOtY29sYXMgRi4gUi4gQS4gUHJhZG8=?= <nfraprado@collabora.com>, 
	Arnd Bergmann <arnd@arndb.de>, Brian Norris <briannorris@chromium.org>, 
	Julius Werner <jwerner@chromium.org>, Nathan Chancellor <nathan@kernel.org>, 
	Nicolas Schier <nicolas@fjasle.eu>, Catalin Marinas <catalin.marinas@arm.com>, 
	Will Deacon <will@kernel.org>, 
	AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>, 
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>, 
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>, kernel@collabora.com, 
	chrome-platform@lists.linux.dev, linux-kernel@vger.kernel.org, 
	linux-kbuild@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, Feb 15, 2024 at 12:37=E2=80=AFPM Tzung-Bi Shih <tzungbi@kernel.org>=
 wrote:
>
> On Mon, Feb 12, 2024 at 09:50:04AM -0500, N=C3=ADcolas F. R. A. Prado wro=
te:
> > 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.
> >
> > [...]
> >
> > ---
> > N=C3=ADcolas F. R. A. Prado (4):
> >       firmware: coreboot: Generate modalias uevent for devices
> >       firmware: coreboot: Generate aliases for coreboot modules
> >       firmware: coreboot: Replace tag with id table in driver struct
> >       arm64: defconfig: Enable support for cbmem entries in the coreboo=
t table
> >
> >  arch/arm64/configs/defconfig                   |  3 +++
>
> Hi Catalin and Will,
>
> Is it OK to you if I pick the 4th patch (which touches the above files) t=
o
> chrome-platform-firmware tree for the next merge window?
>
> >  include/linux/mod_devicetable.h                | 10 ++++++++++
> >  scripts/mod/devicetable-offsets.c              |  3 +++
> >  scripts/mod/file2alias.c                       | 10 ++++++++++
>
> Hi Masahiro,
>
> Is it OK to you if I pick the 2nd patch (which touches the above files) t=
o
> chrome-platform-firmware tree for the next merge window?


Yes.


I gave Acked-by to 2/4 so it makes your life easier.


--=20
Best Regards
Masahiro Yamada