Re: [PATCH v2 7/8] auxdisplay: Add Titanmec TM16xx 7-segment display controllers driver

From: Andy Shevchenko
Date: Mon Jun 30 2025 - 03:28:23 EST


On Mon, Jun 30, 2025 at 08:12:16AM +0200, Krzysztof Kozlowski wrote:
> On 29/06/2025 15:18, Jean-François Lessard wrote:

...

> > + display->leds =
> > + devm_kcalloc(dev, display->num_leds, sizeof(*display->leds), GFP_KERNEL);
>
> Wrong wrapping. Use kernel style, not clang style.
>
>
> > + if (!display->leds)
> > + return -ENOMEM;

Just wondering how .clang-format is official? Note some of the maintainers even
prefer (ugly in some cases in my opinion) style because it's generated by the
clang-format.

--
With Best Regards,
Andy Shevchenko