On Wednesday, March 18, 2015 02:38:26 PM Krzysztof Kozlowski wrote: > 2015-03-18 13:51 GMT+01:00 Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx>: > > The patchset itself looks good... but it's missing commit message. > What benefits does the AFTR bring? AFTR support brings reduced energy consumption and is a prerequisite for more advanced W-AFTR/LPA power saving modes. AFTR has been already supported on other Exynos SoCs for few years so there is really no need to explain its purpose with every new SoC support addition. [ Moreover you know this all really well since we've worked together on many Exynos Power Management issues :). ] I understand that you are complaining about skimpy commit message and I can certainly improve it (however personally I think that it will not bring much benefit because the patch is rather straightforward one). Please also note that while I appreciate review comments please try to make them more substantial and bring them earlier (this patchset hasn't really changed since v1 which was posted in October 2014). Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |