From: Zixun LI <admin@hifiphile.com>
To: Simon Glass <sjg@chromium.org>, Tom Rini <trini@konsulko.com>,
Lukasz Majewski <lukma@denx.de>,
Mattijs Korpershoek <mkorpershoek@baylibre.com>,
Marek Vasut <marex@denx.de>
Cc: Zixun LI <admin@hifiphile.com>, u-boot@lists.denx.de
Subject: [RFC PATCH 0/4] dm: Duplicate uclass name fix and alias improvements
Date: Wed, 31 Jul 2024 15:42:15 +0200 [thread overview]
Message-ID: <20240731134257.686017-1-admin@hifiphile.com> (raw)
Patch 1 is a tentative fix for duplicate uclass name issue met in
https://lists.denx.de/pipermail/u-boot/2024-July/560189.html
The idea is to use orignal class name only for sequence alias to keep this
alias function working and rename the class something else.
Patch 2 and 3 make "dm tree", "bind" and "unbind" commands to take care
of alias sequence numbering. As the alias sequence numbering is more
meanful than uclass index.
Patch 4 is the actual fix for usb gadet class.
There are some situations where uclass index and device sequence number
are misused which is not convered by this patch set, for example in
drivers/net/sandbox.c, uclass_get_device() is used which is based on
uclass index, while in the comments it says "index - The alias index
(also DM seq number)"
Zixun LI (4):
dm: core: Add a way to specify an alt name for alias sequence
numbering
dm: core: Show device sequence instead in dm_dump_tree()
cmd: bind: Use device sequence instead for driver bind/unbind
usb: gadget: udc: Fix duplicate uclass name
cmd/bind.c | 4 ++--
drivers/core/device.c | 3 ++-
drivers/core/dump.c | 2 +-
drivers/core/read.c | 7 ++++++-
drivers/core/uclass.c | 11 +++++++++--
drivers/usb/gadget/udc/udc-uclass.c | 3 ++-
include/dm/read.h | 9 ++++++++-
include/dm/uclass.h | 2 ++
8 files changed, 32 insertions(+), 9 deletions(-)
--
2.45.2
next reply other threads:[~2024-07-31 21:11 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-31 13:42 Zixun LI [this message]
2024-07-31 13:42 ` [RFC PATCH 1/4] dm: core: Add a way to specify an alt name for alias sequence numbering Zixun LI
2024-07-31 13:42 ` [RFC PATCH 2/4] dm: core: Show device sequence instead in dm_dump_tree() Zixun LI
2024-08-01 14:42 ` Simon Glass
2024-07-31 13:42 ` [RFC PATCH 3/4] cmd: bind: Use device sequence instead for driver bind/unbind Zixun LI
2024-08-01 14:42 ` Simon Glass
2024-07-31 13:42 ` [RFC PATCH 4/4] usb: gadget: udc: Fix duplicate uclass name Zixun LI
2024-08-01 14:42 ` [RFC PATCH 0/4] dm: Duplicate uclass name fix and alias improvements Simon Glass
2024-08-01 18:37 ` Zixun LI
2024-08-01 19:28 ` Simon Glass
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240731134257.686017-1-admin@hifiphile.com \
--to=admin@hifiphile.com \
--cc=lukma@denx.de \
--cc=marex@denx.de \
--cc=mkorpershoek@baylibre.com \
--cc=sjg@chromium.org \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox