Re: [PATCH v3 2/3] iio: accel: mma8452: Rename time step look up struct to generic name as the values are same for all the events.

From: Jonathan Cameron
Date: Sat Sep 30 2017 - 14:05:21 EST

Next message: harinath Nampally: "Re: [PATCH v3 2/3] iio: accel: mma8452: Rename time step look up struct to generic name as the values are same for all the events."
Previous message: Jonathan Cameron: "Re: [PATCH v3 3/3] iio: accel: mma8452: Rename read/write event value callbacks to generic function name."
In reply to: Martin Kepplinger: "Re: [PATCH v3 2/3] iio: accel: mma8452: Rename time step look up struct to generic name as the values are same for all the events."
Next in thread: harinath Nampally: "Re: [PATCH v3 2/3] iio: accel: mma8452: Rename time step look up struct to generic name as the values are same for all the events."
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Wed, 27 Sep 2017 08:51:26 +0200
Martin Kepplinger <martink@xxxxxxxxx> wrote:

> Am 25.09.2017 12:40 schrieb Harinath Nampally:
> > Improves code readability, no impact on functionality.
> >
> > Signed-off-by: Harinath Nampally <harinath922@xxxxxxxxx>
>
> Please make the headline shorter and put some of it in the git commit
> message.
> (And please just resend it "--in-reply-to" this conversation, this patch
> nr 2 of 3)

>From a patch management point of view I actually disagree with this.
I would prefer to see a clean fresh series. Otherwise it very rapidly
gets hard to be sure that I am picking up the latest versions.

Obviously drop any patches that have already been taken.
In this case it will be a v4 series containing patches 1 and 2 only.

Thanks

Jonathan
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html