Hi Zixun,
On Wed, 31 Jul 2024 at 07:43, Zixun LI <ad...@hifiphile.com> 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.
>
> Signed-off-by: Zixun LI <ad...@hifiphile.com>
> ---
> cmd/bind.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
I assume no tests need updating, but please do add a note to
doc/usage/cmd/bind.rst
> diff --git a/cmd/bind.c b/cmd/bind.c
> index 3a59eefd5c..30f1163e17 100644
> --- a/cmd/bind.c
> +++ b/cmd/bind.c
> @@ -31,7 +31,7 @@ static int bind_by_class_index(const char *uclass, int
> index,
> return -EINVAL;
> }
>
> - ret = uclass_find_device(uclass_id, index, &parent);
> + ret = uclass_find_device_by_seq(uclass_id, index, &parent);
> if (!parent || ret) {
> printf("Cannot find device %d of class %s\n", index, uclass);
> return ret;
> @@ -58,7 +58,7 @@ static int find_dev(const char *uclass, int index, struct
> udevice **devp)
> return -EINVAL;
> }
>
> - rc = uclass_find_device(uclass_id, index, devp);
> + rc = uclass_find_device_by_seq(uclass_id, index, devp);
> if (!*devp || rc) {
> printf("Cannot find device %d of class %s\n", index, uclass);
> return rc;
> --
> 2.45.2
>
Regards,
Simon
