On Fri, Nov 16, 2018 at 05:28:56PM +0800, Cheng-Yi Chiang wrote: > The default implementation of regulator_set_load returns > REGULATOR_MODE_NORMAL, which is positive. The stub does indeed do that however that just looks like a bug - the actual implementation returns 0 on success and the other users don't seem to expect to get a mode value back. > rt5663_i2c_probe should only do error handling when return value of > regulator_set_load is negative. > In this case, rt5663_i2c_probe should return error. However that's also true so the patch is valid.
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
