[RESEND PATCH linux dev-6.6 v1] clk: npcm8xx: add clock controller

Andrew Jeffery andrew at codeconstruct.com.au
Thu Jun 27 12:11:05 AEST 2024


Hi Tomer,

On Wed, 2024-06-26 at 16:42 +0300, Tomer Maimon wrote:
> Nuvoton Arbel BMC NPCM8XX contains an integrated clock controller which
> generates and supplies clocks to all modules within the BMC.
> 
> Signed-off-by: Tomer Maimon <tmaimon77 at gmail.com>

How does this relate to the series that you've been iterating on
upstream? Is this v25 of the patches?

It help me if you clarified this in the commit message (ideally with a
link to the upstream series in question if it's a direct backport).

If it is v25 then there seem to be some concerns from the CI bots:

- https://lore.kernel.org/lkml/202406191439.3NcnExKM-lkp@intel.com/
- https://lore.kernel.org/lkml/202406201328.SGrN27to-lkp@intel.com/

It looks like you also need to fix the dts to address the new
constraints you've added to the binding (and also address the build
failure).

Andrew


More information about the openbmc mailing list