mbox series

[v4,0/2] create patches for nxp-ls1046a platform watchdog feature

Message ID 20250604034246.3234837-1-Meng.Li@windriver.com
Headers show
Series create patches for nxp-ls1046a platform watchdog feature | expand

Message

Meng Li June 4, 2025, 3:42 a.m. UTC 
v4:
 - Improve commit log of patch0002

v3:
 - merge patch 0002 into patch 0001
 - improve the commit log of patches

v2:
 - improve the commit log of patch 0001
 - add another 2 patches to fix below warning
 arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dtb: watchdog@2ad0000(fsl,imx21-wdt): big-endian: False schema does not allow True

v1:
 - create patch 0001-arch-arm64-dts-add-big-endian-property-back-into-wat.patch

Meng Li (2):
  dt-bindings: watchdog: fsl-imx-wdt: add compatible string
    fsl,ls1046a-wdt
  arch: arm64: dts: add big-endian property back into watchdog node

 Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.yaml | 2 ++
 arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi              | 3 ++-
 2 files changed, 4 insertions(+), 1 deletion(-)

Comments

Rob Herring (Arm) June 5, 2025, 5:40 p.m. UTC | #1 
On Wed, Jun 04, 2025 at 11:42:45AM +0800, Meng Li wrote:
> Add compatible string fsl,ls1046a-wdt for ls1046a SoC. fsl,ls1046a-wdt
> allows big-endian property.
> 
> Signed-off-by: Meng Li <Meng.Li@windriver.com>
> ---
>  Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.yaml | 2 ++
>  1 file changed, 2 insertions(+)

You missed Cc'ing the watchdog maintainer if you want this applied.

Acked-by: Rob Herring (Arm) <robh@kernel.org>
Meng Li June 6, 2025, 1:46 a.m. UTC | #2 
Add watchdog binding maintainer for the patch-0001.

Thanks,
Limeng

> -----Original Message-----
> From: Li, Meng <Meng.Li@windriver.com>
> Sent: Wednesday, June 4, 2025 11:43 AM
> To: linux@roeck-us.net; s.hauer@pengutronix.de; kernel@pengutronix.de;
> linux-watchdog@vger.kernel.org; imx@lists.linux.dev; shawnguo@kernel.org;
> robh@kernel.org; conor+dt@kernel.org; Frank.Li@nxp.com
> Cc: linux-arm-kernel@lists.infradead.org; devicetree@vger.kernel.org; linux-
> kernel@vger.kernel.org; Li, Meng <Meng.Li@windriver.com>
> Subject: [v4 PATCH 0/2] create patches for nxp-ls1046a platform watchdog
> feature
> 
> v4:
>  - Improve commit log of patch0002
> 
> v3:
>  - merge patch 0002 into patch 0001
>  - improve the commit log of patches
> 
> v2:
>  - improve the commit log of patch 0001
>  - add another 2 patches to fix below warning
>  arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dtb:
> watchdog@2ad0000(fsl,imx21-wdt): big-endian: False schema does not allow
> True
> 
> v1:
>  - create patch 0001-arch-arm64-dts-add-big-endian-property-back-into-
> wat.patch
> 
> Meng Li (2):
>   dt-bindings: watchdog: fsl-imx-wdt: add compatible string
>     fsl,ls1046a-wdt
>   arch: arm64: dts: add big-endian property back into watchdog node
> 
>  Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.yaml | 2 ++
>  arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi              | 3 ++-
>  2 files changed, 4 insertions(+), 1 deletion(-)
> 
> --
> 2.34.1
Frank Li June 6, 2025, 3:34 p.m. UTC | #3 
On Fri, Jun 06, 2025 at 01:46:56AM +0000, Li, Meng wrote:
> Add watchdog binding maintainer for the patch-0001.

You may need resend whole patches, which included linux-watchdog, they may
use patchwork to track patches and miss this one.

Frank

>
> Thanks,
> Limeng
>
> > -----Original Message-----
> > From: Li, Meng <Meng.Li@windriver.com>
> > Sent: Wednesday, June 4, 2025 11:43 AM
> > To: linux@roeck-us.net; s.hauer@pengutronix.de; kernel@pengutronix.de;
> > linux-watchdog@vger.kernel.org; imx@lists.linux.dev; shawnguo@kernel.org;
> > robh@kernel.org; conor+dt@kernel.org; Frank.Li@nxp.com
> > Cc: linux-arm-kernel@lists.infradead.org; devicetree@vger.kernel.org; linux-
> > kernel@vger.kernel.org; Li, Meng <Meng.Li@windriver.com>
> > Subject: [v4 PATCH 0/2] create patches for nxp-ls1046a platform watchdog
> > feature
> >
> > v4:
> >  - Improve commit log of patch0002
> >
> > v3:
> >  - merge patch 0002 into patch 0001
> >  - improve the commit log of patches
> >
> > v2:
> >  - improve the commit log of patch 0001
> >  - add another 2 patches to fix below warning
> >  arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dtb:
> > watchdog@2ad0000(fsl,imx21-wdt): big-endian: False schema does not allow
> > True
> >
> > v1:
> >  - create patch 0001-arch-arm64-dts-add-big-endian-property-back-into-
> > wat.patch
> >
> > Meng Li (2):
> >   dt-bindings: watchdog: fsl-imx-wdt: add compatible string
> >     fsl,ls1046a-wdt
> >   arch: arm64: dts: add big-endian property back into watchdog node
> >
> >  Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.yaml | 2 ++
> >  arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi              | 3 ++-
> >  2 files changed, 4 insertions(+), 1 deletion(-)
> >
> > --
> > 2.34.1
>
Meng Li June 8, 2025, 3:08 a.m. UTC | #4 
> -----Original Message-----
> From: Frank Li <Frank.li@nxp.com>
> Sent: Friday, June 6, 2025 11:34 PM
> To: Li, Meng <Meng.Li@windriver.com>
> Cc: linux@roeck-us.net; s.hauer@pengutronix.de; kernel@pengutronix.de;
> linux-watchdog@vger.kernel.org; imx@lists.linux.dev; shawnguo@kernel.org;
> robh@kernel.org; conor+dt@kernel.org; krzk+dt@kernel.org; linux-arm-
> kernel@lists.infradead.org; devicetree@vger.kernel.org; linux-
> kernel@vger.kernel.org
> Subject: Re: [v4 PATCH 0/2] create patches for nxp-ls1046a platform
> watchdog feature
> 
> CAUTION: This email comes from a non Wind River email account!
> Do not click links or open attachments unless you recognize the sender and
> know the content is safe.
> 
> On Fri, Jun 06, 2025 at 01:46:56AM +0000, Li, Meng wrote:
> > Add watchdog binding maintainer for the patch-0001.
> 
> You may need resend whole patches, which included linux-watchdog, they
> may use patchwork to track patches and miss this one.
> 

Thanks for reminding me.
I have sent the whole patches.

Regards,
Limeng

> Frank
> 
> >
> > Thanks,
> > Limeng
> >
> > > -----Original Message-----
> > > From: Li, Meng <Meng.Li@windriver.com>
> > > Sent: Wednesday, June 4, 2025 11:43 AM
> > > To: linux@roeck-us.net; s.hauer@pengutronix.de;
> > > kernel@pengutronix.de; linux-watchdog@vger.kernel.org;
> > > imx@lists.linux.dev; shawnguo@kernel.org; robh@kernel.org;
> > > conor+dt@kernel.org; Frank.Li@nxp.com
> > > Cc: linux-arm-kernel@lists.infradead.org;
> > > devicetree@vger.kernel.org; linux- kernel@vger.kernel.org; Li, Meng
> > > <Meng.Li@windriver.com>
> > > Subject: [v4 PATCH 0/2] create patches for nxp-ls1046a platform
> > > watchdog feature
> > >
> > > v4:
> > >  - Improve commit log of patch0002
> > >
> > > v3:
> > >  - merge patch 0002 into patch 0001
> > >  - improve the commit log of patches
> > >
> > > v2:
> > >  - improve the commit log of patch 0001
> > >  - add another 2 patches to fix below warning
> > >  arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dtb:
> > > watchdog@2ad0000(fsl,imx21-wdt): big-endian: False schema does not
> > > allow True
> > >
> > > v1:
> > >  - create patch
> > > 0001-arch-arm64-dts-add-big-endian-property-back-into-
> > > wat.patch
> > >
> > > Meng Li (2):
> > >   dt-bindings: watchdog: fsl-imx-wdt: add compatible string
> > >     fsl,ls1046a-wdt
> > >   arch: arm64: dts: add big-endian property back into watchdog node
> > >
> > >  Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.yaml | 2 ++
> > >  arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi              | 3 ++-
> > >  2 files changed, 4 insertions(+), 1 deletion(-)
> > >
> > > --
> > > 2.34.1
> >