Re: [PATCH v2] iio: adc: ad7944: Add support for "3-wire mode"

[Jonathan Cameron <Jonathan.Cameron@Huawei.com>]

> 
> >  struct ad7944_adc {
> >  	struct spi_device *spi;
> > +	enum ad7944_spi_mode spi_mode;
> >  	/* Chip-specific timing specifications. */
> >  	const struct ad7944_timing_spec *timing_spec;
> >  	/* GPIO connected to CNV pin. */
> > @@ -58,6 +75,9 @@ struct ad7944_adc {
> >  	 } sample __aligned(IIO_DMA_MINALIGN);
> >  };  
> 
> Have you run `pahole` to see if there is a better place for a new member?

I know this matters for structures where we see lots of them, but do we actually
care for one offs?  Whilst it doesn't matter here I'd focus much more
on readability and like parameter grouping for cases like this than wasting
a few bytes.

Jonathan