On mar., août 20, 2024 at 08:23, Mattijs Korpershoek <mkorpersh...@baylibre.com> wrote:
> Hi, > > On Fri, 02 Aug 2024 11:28:13 +0200, Zixun LI wrote: >> Currently uclass index is used for bind/unbind which ignores alias >> sequence numbering. Use device sequence number instead as it's >> the number explicitly set in the DT. >> >> Also update documentation to use sequence number. >> >> >> [...] > > Thanks, Applied to https://source.denx.de/u-boot/custodians/u-boot-dfu > (u-boot-dfu-next) > > [1/1] cmd: bind: Use device sequence instead for driver bind/unbind > > https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/0c48507eee08b5e7d4e8484727efe47cfabff0ee Somehow, b4 did the wrong thing here when running: $ b4 shazam -s -l --check 20240802092820.917450-1-ad...@hifiphile.com Grabbing thread from lore.kernel.org/all/20240802092820.917450-1-ad...@hifiphile.com/t.mbox.gz Checking for newer revisions Grabbing search results from lore.kernel.org Analyzing 12 messages in the thread Assuming new revision: v2 ([PATCH] dm: core: Show device sequence instead in dm_dump_tree()) Assuming new revision: v3 ([PATCH] cmd: bind: Use device sequence instead for driver bind/unbind) Analyzing 0 code-review messages Will use the latest revision: v3 You can pick other revisions using the -vN flag Checking attestation on all messages, may take a moment... --- ✗ [PATCH] cmd: bind: Use device sequence instead for driver bind/unbind + Reviewed-by: Simon Glass <s...@chromium.org> (✗ DKIM/chromium.org) + Link: https://lore.kernel.org/r/20240802092820.917450-3-ad...@hifiphile.com + Signed-off-by: Mattijs Korpershoek <mkorpersh...@baylibre.com> ● checkpatch.pl: passed all checks --- ✗ BADSIG: DKIM/hifiphile-com.20230601.gappssmtp.com --- Total patches: 1 --- Applying: cmd: bind: Use device sequence instead for driver bind/unbind It applied: https://patchwork.ozlabs.org/project/uboot/patch/20240802092820.917450-3-ad...@hifiphile.com/ Instead of: https://patchwork.ozlabs.org/project/uboot/patch/20240802092820.917450-1-ad...@hifiphile.com/ I have fixed this up manually, see: https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/73f5b54a589cb6ff97aaca55e7c97d68b43997ee > > -- > Mattijs
