Re: [PATCH v2] cpufreq: Use str_enable_disable-like helpers

From: Rafael J. Wysocki
Date: Fri Jan 17 2025 - 12:56:56 EST


On Wed, Jan 15, 2025 at 8:12 AM Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
>
> On 14-01-25, 20:06, Krzysztof Kozlowski wrote:
> > Replace ternary (condition ? "enable" : "disable") syntax with helpers
> > from string_choices.h because:
> > 1. Simple function call with one argument is easier to read. Ternary
> > operator has three arguments and with wrapping might lead to quite
> > long code.
> > 2. Is slightly shorter thus also easier to read.
> > 3. It brings uniformity in the text - same string.
> > 4. Allows deduping by the linker, which results in a smaller binary
> > file.
> >
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>
> Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>

Applied as 6.14 material, thanks!