On 18/09/2024 10:02, Ciprian Marian Costea wrote: > On 9/17/2024 8:36 PM, Krzysztof Kozlowski wrote: >> On 11/09/2024 09:00, Ciprian Costea wrote: >>> From: Ciprian Marian Costea <ciprianmarian.costea@xxxxxxxxxxx> >>> >>> This patch enables CONFIG_RTC_DRV_S32G as module by default. >> >> We see this from the diff. We do not see why we would like this. >> >> Best regards, >> Krzysztof >> > > Hello Krzysztof, > > Thanks for your feedback. > > I can see how not providing any information in the commit message > regarding why this config is important for S32G2/S32G3 SoCs is not ideal. > > Considering this, if I would add such information, would it be > acceptable to enable this config as module ? Or should I drop this > commit instead ? > > My decision to enable it as module by default was also influenced by > other similar RTC related configs, which are present in 'defconfig'. > I don't object the patch contents itself. I want some sort of explanation in the commit msg. Each commit msg is supposed to answer to "why?" (unless it is obvious) and the answer is for example that platform foo bar uses it. See some examples in git history. Best regards, Krzysztof
