Messages in this thread | | | Date | Mon, 25 Sep 2017 08:23:50 +0200 | From | Martin Kepplinger <> | Subject | Re: [PATCH 0/3] This patchset refactors event related functions |
| |
Am 25.09.2017 05:15 schrieb Harinath Nampally: > Harinath Nampally (3): > Following 2 patches are for refactor: > iio: accel: mma8452: Rename time step look up struct to generic > name as the values are same for all the events. > iio: accel: mma8452: Rename read/write event value callbacks to > generic function name. > Following patch adds new feature: > iio: accel: mma8452: Add single pulse/tap event detection feature > for fxls8471. >
I guess it's fine for now, but please don't call your patchset "This patchset does...". Just say "Refactor..." or whatever like you do in your patches.
| |