Messages in this thread | | | Date | Fri, 8 Sep 2017 10:32:57 +0100 | From | Jonathan Cameron <> | Subject | Re: [PATCH] Staging: iio: adc: Added Space around binary op. |
| |
On Fri, 8 Sep 2017 07:29:06 +0100 Jonathan Cameron <jic23@jic23.retrosnub.co.uk> wrote:
> On 8 September 2017 05:47:52 BST, Himanshi Jain <himshijain.hj@gmail.com> wrote: > >Added space around(one on each side of) binary > >operator(-) as preferred according to kernel > >coding style. > > > >Signed-off-by: Himanshi Jain <himshijain.hj@gmail.com> > > Take a closer look at that macro. It isn't doing what you think... > To give a hint, changing this breaks userspace.
Ok, I'm bored of this particular one coming up. When you have worked out what is going on Himanshi, would you mind putting together a patch adding a comment describing why it is a bad idea to 'fix' this? That would be a very useful patch as far as I'm concerned :)
There aren't that many cases of this in IIO so adding a comment on each of them is probably reasonable just to avoid wasting people's time on fixing them! (I think we have had more than 5 such goes this year so far...)
Jonathan
> > Jonathan > > > >--- > > drivers/staging/iio/adc/ad7192.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > >diff --git a/drivers/staging/iio/adc/ad7192.c > >b/drivers/staging/iio/adc/ad7192.c > >index d11c6de..1aee662 100644 > >--- a/drivers/staging/iio/adc/ad7192.c > >+++ b/drivers/staging/iio/adc/ad7192.c > >@@ -341,7 +341,7 @@ static int ad7192_setup(struct ad7192_state *st, > > } > > > > static IIO_DEVICE_ATTR_NAMED(in_v_m_v_scale_available, > >- in_voltage-voltage_scale_available, > >+ in_voltage - voltage_scale_available, > > 0444, ad7192_show_scale_available, NULL, 0); > > > > static IIO_DEVICE_ATTR(in_voltage_scale_available, 0444, >
| |