U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
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


             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