[PATCH v3 2/2] rtc: Add Raspberry Pi 5 RTC driver
Alexandre Belloni
alexandre.belloni at bootlin.com
Mon Sep 21 11:59:49 PDT 2026
On 18/09/2026 13:47:22+0200, Krzysztof Kozlowski wrote:
> On Thu, Sep 17, 2026 at 03:58:41PM +0200, Sander Speetjens wrote:
> > + return devm_rtc_register_device(vrtc->rtc);
> > +}
> > +
> > +static const struct of_device_id rpi_rtc_dt_match[] = {
> > + { .compatible = "raspberrypi,firmware-rtc"},
>
> Please fixup white space issues, see existing drivers (missing space
> before }
>
> > + {},
>
> and unnecessary ,.
>
> Just take existing latest code as starting point, otherwise you
> reintroduce discouraged patterns.
>
> More important: undocumented ABI.
>
This was my comment on the previous revision, I'm not sure why it was
ignored.
--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the linux-arm-kernel
mailing list