[U-Boot] [PATCH] drivers: core: use strcmp when find device by name
`if (!strncmp(dev->name, name, strlen(name)))` might find out the wrong device, it might find out `dram_pll_ref_sel`, when name is `dram_pll`. So use strcmp to avoid such issue.
Signed-off-by: Peng Fan peng.fan@nxp.com --- drivers/core/uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/core/uclass.c b/drivers/core/uclass.c index fc3157de39..e2f35393a9 100644 --- a/drivers/core/uclass.c +++ b/drivers/core/uclass.c @@ -260,7 +260,7 @@ int uclass_find_device_by_name(enum uclass_id id, const char *name, return ret;
uclass_foreach_dev(dev, uc) { - if (!strncmp(dev->name, name, strlen(name))) { + if (!strcmp(dev->name, name)) { *devp = dev; return 0; }
On Sun, 28 Apr 2019 at 03:43, Peng Fan peng.fan@nxp.com wrote:
`if (!strncmp(dev->name, name, strlen(name)))` might find out the wrong device, it might find out `dram_pll_ref_sel`, when name is `dram_pll`. So use strcmp to avoid such issue.
Signed-off-by: Peng Fan peng.fan@nxp.com
drivers/core/uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org
This seems to match the function description in the header file, too.
Hi Simon,
Subject: Re: [PATCH] drivers: core: use strcmp when find device by name
On Sun, 28 Apr 2019 at 03:43, Peng Fan peng.fan@nxp.com wrote:
`if (!strncmp(dev->name, name, strlen(name)))` might find out the wrong device, it might find out `dram_pll_ref_sel`, when name is `dram_pll`. So use strcmp to avoid such issue.
Signed-off-by: Peng Fan peng.fan@nxp.com
drivers/core/uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org
This seems to match the function description in the header file, too.
Will you pick up this patch? Not see this patch in your tree.
Thanks, Peng.
Hi Peng,
On Wed, 15 May 2019 at 05:57, Peng Fan peng.fan@nxp.com wrote:
Hi Simon,
Subject: Re: [PATCH] drivers: core: use strcmp when find device by name
On Sun, 28 Apr 2019 at 03:43, Peng Fan peng.fan@nxp.com wrote:
`if (!strncmp(dev->name, name, strlen(name)))` might find out the wrong device, it might find out `dram_pll_ref_sel`, when name is `dram_pll`. So use strcmp to avoid such issue.
Signed-off-by: Peng Fan peng.fan@nxp.com
drivers/core/uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org
This seems to match the function description in the header file, too.
Will you pick up this patch? Not see this patch in your tree.
I found that this causes a test failure - can you try 'make qcheck' and see if you can figure out what is going on?
Regards, Simon
Hi Simon,
Subject: Re: [PATCH] drivers: core: use strcmp when find device by name
Hi Peng,
On Wed, 15 May 2019 at 05:57, Peng Fan peng.fan@nxp.com wrote:
Hi Simon,
Subject: Re: [PATCH] drivers: core: use strcmp when find device by name
On Sun, 28 Apr 2019 at 03:43, Peng Fan peng.fan@nxp.com wrote:
`if (!strncmp(dev->name, name, strlen(name)))` might find out the wrong device, it might find out `dram_pll_ref_sel`, when name is `dram_pll`. So use strcmp to avoid such issue.
Signed-off-by: Peng Fan peng.fan@nxp.com
drivers/core/uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org
This seems to match the function description in the header file, too.
Will you pick up this patch? Not see this patch in your tree.
I found that this causes a test failure - can you try 'make qcheck' and see if you can figure out what is going on?
With this patch applied. Test/dm/adc.c and usb.c needs some change to use real device name. I just posted out new patchset including this one.
Pease review.
Thanks, Peng
Regards, Simon
participants (2)
-
Peng Fan -
Simon Glass