| Message ID | 20210515072949.7151-1-thunder.leizhen@huawei.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 77a0989baa427dbd242c5784d05a53ca3d197d43 |
| Delegated to: | Kalle Valo |
| Headers | show |
| Series | [1/1] ssb: Fix error return code in ssb_bus_scan() | expand |
On Sat, 15 May 2021 15:29:49 +0800 Zhen Lei <thunder.leizhen@huawei.com> wrote: > Fix to return -EINVAL from the error handling case instead of 0, as done > elsewhere in this function. > @@ -325,6 +325,7 @@ int ssb_bus_scan(struct ssb_bus *bus, > if (bus->nr_devices > ARRAY_SIZE(bus->devices)) { > pr_err("More than %d ssb cores found (%d)\n", > SSB_MAX_NR_CORES, bus->nr_devices); > + err = -EINVAL; > goto err_unmap; > } > if (bus->bustype == SSB_BUSTYPE_SSB) { Acked-by: Michael Büsch <m@bues.ch>
Zhen Lei <thunder.leizhen@huawei.com> wrote: > Fix to return -EINVAL from the error handling case instead of 0, as done > elsewhere in this function. > > Fixes: 61e115a56d1a ("[SSB]: add Sonics Silicon Backplane bus support") > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com> > Acked-by: Michael Büsch <m@bues.ch> Patch applied to wireless-drivers-next.git, thanks. 77a0989baa42 ssb: Fix error return code in ssb_bus_scan()
diff --git a/drivers/ssb/scan.c b/drivers/ssb/scan.c index f49ab1aa2149ab2..4161e5d1f276e11 100644 --- a/drivers/ssb/scan.c +++ b/drivers/ssb/scan.c @@ -325,6 +325,7 @@ int ssb_bus_scan(struct ssb_bus *bus, if (bus->nr_devices > ARRAY_SIZE(bus->devices)) { pr_err("More than %d ssb cores found (%d)\n", SSB_MAX_NR_CORES, bus->nr_devices); + err = -EINVAL; goto err_unmap; } if (bus->bustype == SSB_BUSTYPE_SSB) {
Fix to return -EINVAL from the error handling case instead of 0, as done elsewhere in this function. Fixes: 61e115a56d1a ("[SSB]: add Sonics Silicon Backplane bus support") Reported-by: Hulk Robot <hulkci@huawei.com> Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com> --- drivers/ssb/scan.c | 1 + 1 file changed, 1 insertion(+)