Messages in this thread | | | Date | Tue, 17 Apr 2012 14:43:52 +0100 | From | Mark Brown <> | Subject | Re: [alsa-devel] [PATCH v2] ASoC: da7210: Add support for PLL and SRM |
| |
On Tue, Apr 17, 2012 at 06:04:06PM +0530, Ashish Chavan wrote:
> + /* Search pll div array for correct divisors */ > + for (cnt = 0; cnt < ARRAY_SIZE(da7210_pll_div); cnt++) { > + /* check fref */ > + if (fref == da7210_pll_div[cnt].fref) { > + /* check mode */ > + if (da7210->master == da7210_pll_div[cnt].mode) { > + /* check fout */ > + if (fout == da7210_pll_div[cnt].fout) {
I've applied but this would be a lot clearer if it were writen as a single if statement with &&s. [unhandled content-type:application/pgp-signature] | |