Thread (16 messages) 16 messages, 5 authors, 2016-09-05

[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

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 unusual
So 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)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help