2017-10-18 20:31 GMT+02:00 Thomas Gleixner <tglx@xxxxxxxxxxxxx>: > On Wed, 18 Oct 2017, Benjamin Gaignard wrote: > >> Convert driver to use timer_of helpers. This allow to remove >> custom proprietary structure. >> >> Increase min delta value because if it is too small it could >> generate too much interrupts and the system will not be able >> to catch them all. > > This does two completely independent changes at once. What the heck has > increasing min delta to do with converting it to timer_of() helpers? > > Nothing at all. So please split this into two distinct patches. Each doing > ONE thing. I will do that. > > See Documentation/process/submitting-patches.rst: > > 3) Separate your changes > ------------------------ > > Separate each **logical change** into a separate patch. > > Reading, understanding and complying with that document is not optional. > > Thanks, > > tglx -- Benjamin Gaignard Graphic Study Group Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
