On Thu, 2023-10-05 at 16:58 +0300, Dan Carpenter wrote: > We know that "rc" is zero so there is no need to check. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > --- > drivers/dax/bus.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/dax/bus.c b/drivers/dax/bus.c > index 1d818401103b..ea7298d8da99 100644 > --- a/drivers/dax/bus.c > +++ b/drivers/dax/bus.c > @@ -1300,7 +1300,7 @@ static ssize_t memmap_on_memory_store(struct device *dev, > dev_dax->memmap_on_memory = val; > > device_unlock(dax_region->dev); > - return rc == 0 ? len : rc; > + return len; > } > static DEVICE_ATTR_RW(memmap_on_memory); > Hi Dan, thanks for the report - since this function is only in mm- unstable currently, and I have a new revision about to go out, I'll just include this fixup in it.
