Date:   Mon, 3 Jan 2022 21:46:32 +0200
From:   Abel Vesa <abel.vesa@nxp.com>
To:     Alexandre Belloni <alexandre.belloni@bootlin.com>
Cc:     Rob Herring <robh@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Shawn Guo <shawnguo@kernel.org>,
        Fabio Estevam <festevam@gmail.com>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        NXP Linux Team <linux-imx@nxp.com>,
        devicetree@vger.kernel.org,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        linux-arm-kernel@lists.infradead.org, linux-rtc@vger.kernel.org
Subject: Re: [PATCH 1/3] dt-bindings: fsl: scu: Add i.MX8QM rtc binding
Message-ID: <YdNSmA2wXINLXvFx@abelvesa>
References: <1640088948-3690-1-git-send-email-abel.vesa@nxp.com>
 <1640088948-3690-2-git-send-email-abel.vesa@nxp.com>
 <YczU26vFxiO0wWKa@piout.net>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YczU26vFxiO0wWKa@piout.net>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?7u3M8maKHNRlMB/2XmEf0XqgZ9LYqFP0mFy6p67YCDI3wvDX1ET4gDBc9Yp6?=
 =?us-ascii?Q?2lPGsgS5Wkk7Cc3FxUomCdgQNx0LGKiVvA8zPUXCnFkv9XVhPy91L7wO662N?=
 =?us-ascii?Q?HQBUj0pjqq8vJxb4vYYCnZwtbCyPEuTSqr8zPgTEytBkbJwaRc+pR9UBqH6E?=
 =?us-ascii?Q?FzDDFqDyPj4UdV4fTPzgmBh4DQSFIkoi88A1YXcknSgtsz+56A5J3v63oMyy?=
 =?us-ascii?Q?nFvufJOxhUisLz+/RDBfeyK2J89SJcrKZQ4gmuTyaN9KRkmdi9wRiCJ9jaKB?=
 =?us-ascii?Q?z9aXaPyaK0MIjxVMLDDK2qg5Uy7j8xjcxcaBYY0K0YBj8bMjTf2iVWqjBndE?=
 =?us-ascii?Q?LNMKMgIN3C2yrD9VEEa8oxgYmGnFA3itQp+UYoUombm+k6ysE3QHdZ9yMUwx?=
 =?us-ascii?Q?tiPce7MM0+5ogg/Cjnm34rivTchhc5yB17glqzL/fnDYbrngOsSNZ57hknxL?=
 =?us-ascii?Q?e3UXfSoBbx8vHhbJ5fVKHGHWSYY8GL8Mw37cv0B1oWHJ/zJfrWONf1/afxdK?=
 =?us-ascii?Q?Ecg8CLNiQRbFBK1e0MHewuq1oRLEhnYysJNuyoITLu0ViwkkP03Fzrup5k9Z?=
 =?us-ascii?Q?4arq/UYMcw/zPtiXay0T5lziRO+UFp1e8BSv7PAMdGTVzgPukUksYJUydxPi?=
 =?us-ascii?Q?ZADPFTusfK5XqU2XYrDijKfcKIJ2Xpzp1qa3VWgHa1Wxe8nQrzyQtCOoPp0f?=
 =?us-ascii?Q?/xE64xGxw9tb1ZWIYzgTdD0t6CxHXdZ2BPUZF611W3pCzrzBxYxKMDQgYfHA?=
 =?us-ascii?Q?9VcrcDqmBdbf4x1te8qCeJR6JzSRXBcR8SB94UC2q80o9Y6TTzrDz71RinGu?=
 =?us-ascii?Q?ic8o52k3g9bcuRUFSb3Wb8HuzYgBxq3nsfGLqDCt6cOoW1zUV15BM0ac7fNo?=
 =?us-ascii?Q?67WZTu1jBa8tbv52ub0wwZYEIrJbO0SzpVh/vjX6NmE9zB1KU05xydVUU5sw?=
 =?us-ascii?Q?ZFSivD50/jQPzxfayXPdXSNzNqwZ+69MLP2dGz04Pr/HgPBjeoAfEEljmsN7?=
 =?us-ascii?Q?jgAb31Uc8N20lzGFFT0cVt2haCRibyPb55j5OMeF7akWaHYeN5LB3vW+WiSx?=
 =?us-ascii?Q?1m1byxWqINKpcWy8cFlndKZqGxdqU/sc8h5X46P3GYTVUq1EmXINnZf5YBwq?=
 =?us-ascii?Q?uPLri623a2JVJgE54XPBh26kCgViW9pKakSLZC9gUaWKC1eLZbKKnC6moPbS?=
 =?us-ascii?Q?JrcksGyPQllGeo+x59KkURPVEFmbfg4TATA2Gyvbs7NW8+O02haRwGPhcG2y?=
 =?us-ascii?Q?faCRHQluZSYL0NM7O0Oo7XPTkaS0up8cAHZvMWA9w5vYSFbzaarfcPOX7kYN?=
 =?us-ascii?Q?D9xLKp3qUiL8tCY+p9CW7diSOAQ/uLRdFNBOEeJhOD0LMfFd0p2uL1tDtH+N?=
 =?us-ascii?Q?iQLGTxUv8RvbNz+aMSZ0oFM0bnQ54ujWxx4vpJVAICcdlHY40njWrM8rfS2w?=
 =?us-ascii?Q?6DJcNgy9JtE8J5ygEueRfsVVTniVyGIQ4BKCfbu+VHRlyTlDZ9vrd/OmWdqO?=
 =?us-ascii?Q?lhgrcJ0Ex/P7vIshWAz74zSW+Kwxwr3Ef0H4nUZKsksyu66m6MTddCXGwug+?=
 =?us-ascii?Q?M8oxtxIIpinjJXY/GYMenBVI2fBI+cKYuBu1TFH38F9M6GfZqnLehUG3Xla/?=
 =?us-ascii?Q?ShSZaBiewIrrP5DNyb9+UrA=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 964e1b52-03cf-43dc-649f-08d9cef1bf1d
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB4688.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Jan 2022 19:46:34.0019
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Pe6d7X0Lqzdr/reBlS6B6fpr+9uyGpE6qpferqxopLNkq++kmHiRgdiekOosXUMaOfRdbY+s3ACQOj5NCPWAGg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB5296
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 21-12-29 22:36:27, Alexandre Belloni wrote:
> On 21/12/2021 14:15:46+0200, Abel Vesa wrote:
> > Add i.MX8QM rtc compatible to the SCU bindings documentation.
> > 
> > Signed-off-by: Abel Vesa <abel.vesa@nxp.com>
> > ---
> >  Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt b/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt
> > index f1470d562156..4633be2182dd 100644
> > --- a/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt
> > +++ b/Documentation/devicetree/bindings/arm/freescale/fsl,scu.txt
> > @@ -134,7 +134,9 @@ RTC bindings based on SCU Message Protocol
> >  ------------------------------------------------------------
> >  
> >  Required properties:
> > -- compatible: should be "fsl,imx8qxp-sc-rtc";
> > +- compatible:		Should be one of:
> > +			"fsl,imx8qm-sc-rtc",
> > +			"fsl,imx8qxp-sc-rtc".
> 
> Are the imx8qm and imx8qxp SCU actually different? I would hope you
> won't had a third messaging protocol for the RTC...
> 

Sorry, my bad.

No, there is no difference between imx8mq and imx8qxp.

I'll replace the compatible in the imx8qm.dtsi with fsl,imx8qxp-sc-rtc,
drop this patch and the one that adds the compatible to the rtc driver.

> 
> -- 
> Alexandre Belloni, co-owner and COO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com/