On Mon, 2017-06-05 at 11:32 +0100, Lee Jones wrote: > On Fri, 02 Jun 2017, Richard Fitzgerald wrote: [] > > if (IS_ERR(madera->reset_gpio)) > > + ret = PTR_ERR(madera->reset_gpio); > > + else > > + ret = 0; > > If you pre-initialise ret, you can drop the else. Better to use: ret = PTR_ERR_OR_ZERO(madera->reset_gpio); _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
