On Fri, Jun 20, 2014 at 02:14:19PM +0800, Sean Cross wrote: > This adds an initial machine driver for the ES8328 audio codec on Freescale > boards. The driver supports headphones and an audio regulator for an onboard > speaker amp. > > Signed-off-by: Sean Cross <xobs@xxxxxxxxxx> > --- <snip> > +static int imx_set_frequency(struct imx_es8328_data *data, int freq) > +{ > + int ret; > + > + return ret; > +} This function returns an uninitialised variable and is not used anywhere, so I assume can be removed. Thanks, Charles -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html