[PATCH v4 3/3] iio: adc: add support for Allwinner SoCs ADC
From: Peter Meerwald-Stadler <hidden>
Date: 2016-09-05 09:02:26
Also in:
linux-iio, lkml
From: Peter Meerwald-Stadler <hidden>
Date: 2016-09-05 09:02:26
Also in:
linux-iio, lkml
quoted
quoted
quoted
quoted
+{ + return SUN6I_GPADC_CTRL1_ADC_CHAN_SELECT(chan); +} + +struct soc_specific { + const int temp_offset;wondering why you constify every member?Because they're supposed to be fixed values? It won't change in runtime. Is there any reason why I shouldn't do that?yes, but using the entire struct as const has the same effect; constifying individual members makes more sense if there are also non-const members nothing wrong, just unusualSo I would let all members non-const and then when using the struct soc_specific as a member in a struct or as a variable I would prefix it with const? That's what you mean by using the entire struct as const?
yes, exactly thanks, p. -- Peter Meerwald-Stadler +43-664-2444418 (mobile)