Re: [PATCH 3/4] arm64: defconfig: add S32G RTC module support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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'.

Best Regards,
Ciprian




[Index of Archives]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux