Hi Jonathan, jic23@xxxxxxxxxx wrote on Sat, 15 Jan 2022 15:58:32 +0000: > On Wed, 15 Dec 2021 16:13:37 +0100 > Miquel Raynal <miquel.raynal@xxxxxxxxxxx> wrote: > > > This is an internal variable for the core, here it is set to a "default" > > value by the driver in order to later be able to perform checks against > > it. None of this is needed because this check actually cares about the > > buffers being enabled or not. So it is an unproper side-channel access > > to the information "are the buffers enabled?", returned officially by > > the iio_buffer_enabled() helper. Use this helper instead. > > > > Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx> > Hi Miquel, > > Make sure to +cc driver authors on v2. > > Whilst I think this is safe we should definitely give them visibility. > > Obviously some IIO drivers probably have authors who have long moved on > but this one is only 2018 vintage so Song Qiang might still have > access to hardware or be willing to do a review! Right, I've added Song Qiang into a Cc: tag for this patch for the next iteration. Thanks, Miquèl
