[PATCH 08/15] rtc: pcf2127: remove U-Boot specific compatible string

Vladimir Oltean vladimir.oltean at nxp.com
Tue Dec 7 15:06:29 CET 2021


On Tue, Dec 07, 2021 at 04:53:11AM +0000, Priyanka Jain wrote:
> >-----Original Message-----
> >From: Vladimir Oltean <vladimir.oltean at nxp.com>
> >Sent: Thursday, December 2, 2021 8:24 PM
> >To: u-boot at lists.denx.de
> >Cc: Michael Walle <michael at walle.cc>; Tom Rini <trini at konsulko.com>;
> >Priyanka Jain <priyanka.jain at nxp.com>; Leo Li <leoyang.li at nxp.com>; Simon
> >Glass <sjg at chromium.org>
> >Subject: [PATCH 08/15] rtc: pcf2127: remove U-Boot specific compatible string
> >
> >Now that all in-tree boards have been converted to the compatible strings
> >from Linux, delete the support for the ad-hoc "pcf2127-rtc" one.
> >
> >Cc: Simon Glass <sjg at chromium.org>
> >Signed-off-by: Vladimir Oltean <vladimir.oltean at nxp.com>
> >---
> > drivers/rtc/pcf2127.c | 1 -
> > 1 file changed, 1 deletion(-)
> >
> >diff --git a/drivers/rtc/pcf2127.c b/drivers/rtc/pcf2127.c index
> >291ef0329a3e..2f3fafb4968f 100644
> >--- a/drivers/rtc/pcf2127.c
> >+++ b/drivers/rtc/pcf2127.c
> >@@ -120,7 +120,6 @@ static const struct rtc_ops pcf2127_rtc_ops = {  };
> >
> > static const struct udevice_id pcf2127_rtc_ids[] = {
> >-	{ .compatible = "pcf2127-rtc" },
> > 	{ .compatible = "nxp,pcf2127" },
> > 	{ .compatible = "nxp,pcf2129" },
> > 	{ .compatible = "nxp,pca2129" },
> >--
> >2.25.1
> 
> Kindly confirm no existing platform uses this?
> Is build tested pass for all platforms?

I don't know how to trigger a build test for all platforms, sorry.
But I ran a "grep" across the whole tree for "pcf2127-rtc".
In fact I noticed just now that there is one extra occurrence at the
time of this patch in fsl-ls1028a-rdb.dts (the board itself whose dts is
synced with Linux in the next to last patch). I definitely meant to
change the fsl-ls1028a-rdb.dts RTC compatible string as part of a
separate patch as well, but it appears that I missed it and I'm doing it
as part of the dts sync patch. I'll change this in the next series.

Hopefully the fact that there aren't any remaining occurrences of the
string in the tree is sufficient proof?


More information about the U-Boot mailing list