On Thu, 2023-04-13 at 10:33 +0200, Fabrizio Lamarque wrote: > Fixed wrong selection of internal clock when mclk is defined. > > Resolved a logical inversion introduced in c9ec2cb328e3. > > Fixes: c9ec2cb328e3 iio: adc: ad7192: use devm_clk_get_optional() for mclk ditto... > Signed-off-by: Fabrizio Lamarque <fl.scratchpad@xxxxxxxxx> > --- Reviewed-by: Nuno Sa <nuno.sa@xxxxxxxxxx> > > drivers/iio/adc/ad7192.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/adc/ad7192.c b/drivers/iio/adc/ad7192.c > index 94a9cf34a255..5a9c8898f8af 100644 > --- a/drivers/iio/adc/ad7192.c > +++ b/drivers/iio/adc/ad7192.c > @@ -367,7 +367,7 @@ static int ad7192_of_clock_select(struct ad7192_state *st) > clock_sel = AD7192_CLK_INT; > > /* use internal clock */ > - if (st->mclk) { > + if (!st->mclk) { > if (of_property_read_bool(np, "adi,int-clock-output-enable")) > clock_sel = AD7192_CLK_INT_CO; > } else {