Re: [PATCH v2 0/7] AVB: cosmetic adjustments/improvements

From: Mattijs Korpershoek <mkorpershoek@baylibre.com>
To: u-boot@lists.denx.de, Igor Opaniuk <igor.opaniuk@foundries.io>
Cc: Igor Opaniuk <igor.opaniuk@gmail.com>,
	 Ilias Apalodimas <ilias.apalodimas@linaro.org>,
	 Ivan Khoronzhuk <ivan.khoronzhuk@gmail.com>,
	 Jens Wiklander <jens.wiklander@linaro.org>,
	 Michal Simek <michal.simek@amd.com>,
	 Neil Armstrong <neil.armstrong@linaro.org>,
	Stefan Roese <sr@denx.de>,  Tom Rini <trini@konsulko.com>
Subject: Re: [PATCH v2 0/7] AVB: cosmetic adjustments/improvements
Date: Tue, 13 Feb 2024 16:18:44 +0100	[thread overview]
Message-ID: <170783752432.651311.6466036162960002340.b4-ty@baylibre.com> (raw)
In-Reply-To: <20240209192045.3961832-1-igor.opaniuk@foundries.io>

Hi,

On Fri, 09 Feb 2024 20:20:38 +0100, Igor Opaniuk wrote:
> This is the first patch series for incoming major
> improvements for AVB implementation, that include:
> - Simplify and add more context for debug/error prints where it's needed.
> - Move SPDX license identifiers to the first line, so it conforms
>   to license placement rule.
> - Use mmc_switch_part() only for eMMC.
> - Rework do_avb_verify_part, take into account device lock state for setting correct
>   androidboot.verifiedbootstate.
> - Adjust AVB documentation, sync usage info with the one cmd/avb.c.
> 
> [...]

Thanks, Applied to https://source.denx.de/u-boot/custodians/u-boot-dfu (u-boot-dfu)

[1/7] common: avb_verify: don't call mmc_switch_part for SD
      https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/dbde93c12616bd6ba22e6bcb76363a6a3f253087
[2/7] avb: move SPDX license identifiers to the first line
      https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/bf7d47b4274568927adadeff8c9d1aaf2bd45ca6
[3/7] common: avb_verify: rework error/debug prints
      https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/bdcc8f8c8fd3dfb053cea7a0f62b34ef062411e0
[4/7] cmd: avb: rework prints
      https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/055462005fcc48a12a86edf2cbd3f7aa8712788b
[5/7] common: avb_verify: add str_avb_io_error/str_avb_slot_error
      https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/63736ddd4fc357768e31a6f10c10b1a28a10ccbc
[6/7] cmd: avb: rework do_avb_verify_part
      https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/07caf5febcfcfb5f2cdaade124ed8d11735a6029
[7/7] doc: android: avb: sync usage details
      https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/85af871dcca7e9cae074e3a635706052a57629a4

--
Mattijs