On Tue, Aug 27, 2024 at 10:45:17AM +0800, Hongbo Li wrote:
> The helper str_false_true is introduced to reback "false/true"
> string literal. We can simplify this format by str_false_true.

This seems unnecessarily verbose & complex.  How about:

        dprintk("%s: link support=%s\n", __func__, strbool(*res != 0));

> -     dprintk("%s: link support=%s\n", __func__, *res == 0 ? "false" : 
> "true");
> +     dprintk("%s: link support=%s\n", __func__, str_false_true(*res == 0));

(do we have a convention for the antonym of kstrtoX?)

Reply via email to