mbox series

[v3,0/4] add NETCMIX block control support on i.MX95

Message ID 20240829011849.364987-1-wei.fang@nxp.com (mailing list archive)
Headers show
Series add NETCMIX block control support on i.MX95 | expand

Message

Wei Fang Aug. 29, 2024, 1:18 a.m. UTC
The NETCMIX block control consists of registers for configuration of
peripherals in the NETC domain, such as MQS, SAI and NETC. So add the
NETCMIX block control support on i.MX95 platform.

---
V2:
1. Patch 2 is new patch.
2. Add rmii reference clock mux for ENETC0 and ENETC1 in patch 3.
Link to V1: https://lore.kernel.org/all/20240709073603.1967609-3-wei.fang@nxp.com/
V3:
Respin the patch set due to merge conflicts.
Link to V2: https://lore.kernel.org/all/20240729012756.3686758-1-wei.fang@nxp.com/
---

Wei Fang (4):
  dt-bindings: clock: add i.MX95 NETCMIX block control
  dt-bindings: clock: add RMII clock selection
  clk: imx95: enable the clock of NETCMIX block control
  arm64: dts: imx95: Add NETCMIX block control support

 .../bindings/clock/nxp,imx95-blk-ctl.yaml     |  1 +
 arch/arm64/boot/dts/freescale/imx95.dtsi      | 12 ++++++++
 drivers/clk/imx/clk-imx95-blk-ctl.c           | 30 +++++++++++++++++++
 include/dt-bindings/clock/nxp,imx95-clock.h   |  3 ++
 4 files changed, 46 insertions(+)

Comments

Abel Vesa Aug. 29, 2024, 7:50 a.m. UTC | #1
On Thu, 29 Aug 2024 09:18:45 +0800, Wei Fang wrote:
> The NETCMIX block control consists of registers for configuration of
> peripherals in the NETC domain, such as MQS, SAI and NETC. So add the
> NETCMIX block control support on i.MX95 platform.
> 

Applied, thanks!

[1/4] dt-bindings: clock: add i.MX95 NETCMIX block control
      commit: 4b78b54762dbfc2f22f28655fa3cf6f5d50de197
[2/4] dt-bindings: clock: add RMII clock selection
      commit: b4f62001ccd3fa953769ccbd313c9a7a4f5f8f3d
[3/4] clk: imx95: enable the clock of NETCMIX block control
      commit: 42dc425fa8b5be982bcc2025d5bf30be8b26da86
[4/4] arm64: dts: imx95: Add NETCMIX block control support
      (no commit info)

Best regards,
Peng Fan Aug. 29, 2024, 7:52 a.m. UTC | #2
Hi Abel,

> Subject: Re: [Patch v3 0/4] add NETCMIX block control support on
> i.MX95
> 
> 
> On Thu, 29 Aug 2024 09:18:45 +0800, Wei Fang wrote:
> > The NETCMIX block control consists of registers for configuration of
> > peripherals in the NETC domain, such as MQS, SAI and NETC. So add
> the
> > NETCMIX block control support on i.MX95 platform.
> >
> 
> Applied, thanks!
> 
> [1/4] dt-bindings: clock: add i.MX95 NETCMIX block control
>       commit: 4b78b54762dbfc2f22f28655fa3cf6f5d50de197
> [2/4] dt-bindings: clock: add RMII clock selection
>       commit: b4f62001ccd3fa953769ccbd313c9a7a4f5f8f3d
> [3/4] clk: imx95: enable the clock of NETCMIX block control
>       commit: 42dc425fa8b5be982bcc2025d5bf30be8b26da86
> [4/4] arm64: dts: imx95: Add NETCMIX block control support
>       (no commit info)

Patch 4 needs to go through shawn's tree to avoid merge
conflict

Regards
Peng.

> 
> Best regards,
> --
> Abel Vesa <abel.vesa@linaro.org>
Abel Vesa Aug. 29, 2024, 7:53 a.m. UTC | #3
On 24-08-29 10:50:16, Abel Vesa wrote:
> 
> On Thu, 29 Aug 2024 09:18:45 +0800, Wei Fang wrote:
> > The NETCMIX block control consists of registers for configuration of
> > peripherals in the NETC domain, such as MQS, SAI and NETC. So add the
> > NETCMIX block control support on i.MX95 platform.
> > 
> 
> Applied, thanks!
> 
> [1/4] dt-bindings: clock: add i.MX95 NETCMIX block control
>       commit: 4b78b54762dbfc2f22f28655fa3cf6f5d50de197
> [2/4] dt-bindings: clock: add RMII clock selection
>       commit: b4f62001ccd3fa953769ccbd313c9a7a4f5f8f3d
> [3/4] clk: imx95: enable the clock of NETCMIX block control
>       commit: 42dc425fa8b5be982bcc2025d5bf30be8b26da86
> [4/4] arm64: dts: imx95: Add NETCMIX block control support
>       (no commit info)

For some reason, b4 ty thinks it's OK to mention this 4th patch as
applied, while it is obviously not.

I only applied the first 3 patches.

> 
> Best regards,
> -- 
> Abel Vesa <abel.vesa@linaro.org>
Wei Fang Aug. 29, 2024, 8 a.m. UTC | #4
> -----Original Message-----
> From: Abel Vesa <abel.vesa@linaro.org>
> Sent: 2024年8月29日 15:53
> To: mturquette@baylibre.com; sboyd@kernel.org; robh@kernel.org;
> conor+dt@kernel.org; shawnguo@kernel.org; s.hauer@pengutronix.de;
> festevam@gmail.com; abelvesa@kernel.org; Peng Fan <peng.fan@nxp.com>;
> Wei Fang <wei.fang@nxp.com>
> Cc: linux-clk@vger.kernel.org; devicetree@vger.kernel.org; imx@lists.linux.dev;
> linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org
> Subject: Re: [Patch v3 0/4] add NETCMIX block control support on i.MX95
> 
> On 24-08-29 10:50:16, Abel Vesa wrote:
> >
> > On Thu, 29 Aug 2024 09:18:45 +0800, Wei Fang wrote:
> > > The NETCMIX block control consists of registers for configuration of
> > > peripherals in the NETC domain, such as MQS, SAI and NETC. So add
> > > the NETCMIX block control support on i.MX95 platform.
> > >
> >
> > Applied, thanks!
> >
> > [1/4] dt-bindings: clock: add i.MX95 NETCMIX block control
> >       commit: 4b78b54762dbfc2f22f28655fa3cf6f5d50de197
> > [2/4] dt-bindings: clock: add RMII clock selection
> >       commit: b4f62001ccd3fa953769ccbd313c9a7a4f5f8f3d
> > [3/4] clk: imx95: enable the clock of NETCMIX block control
> >       commit: 42dc425fa8b5be982bcc2025d5bf30be8b26da86
> > [4/4] arm64: dts: imx95: Add NETCMIX block control support
> >       (no commit info)
> 
> For some reason, b4 ty thinks it's OK to mention this 4th patch as applied, while
> it is obviously not.
> 
> I only applied the first 3 patches.
> 
Many thanks.
I think Shawn will apply the 4th patch.
Konstantin Ryabitsev Aug. 29, 2024, 1:28 p.m. UTC | #5
On Thu, Aug 29, 2024 at 10:53:28AM GMT, Abel Vesa wrote:
> > [1/4] dt-bindings: clock: add i.MX95 NETCMIX block control
> >       commit: 4b78b54762dbfc2f22f28655fa3cf6f5d50de197
> > [2/4] dt-bindings: clock: add RMII clock selection
> >       commit: b4f62001ccd3fa953769ccbd313c9a7a4f5f8f3d
> > [3/4] clk: imx95: enable the clock of NETCMIX block control
> >       commit: 42dc425fa8b5be982bcc2025d5bf30be8b26da86
> > [4/4] arm64: dts: imx95: Add NETCMIX block control support
> >       (no commit info)
> 
> For some reason, b4 ty thinks it's OK to mention this 4th patch as
> applied, while it is obviously not.
> 
> I only applied the first 3 patches.

Sorry for the confusion.

It's because you did "b4 am" on the entire series. We couldn't find the
matching commit for 4/4, but this often happens because maintainers tweak the
title or the patch contents, so we can't always assume that the patch wasn't
applied just because we can't find the exact match.

If you know you're going to apply a subset, it's best to run "b4 am -P 1-3" so
that b4 knows for a fact that you only intended to apply a subset.

-K