Re: [PATCH 0/9] google: Add support for Laguna CPM

[Krzysztof Kozlowski]

On Mon, Oct 26, 2026 at 02:43:38PM -0700, Brian Norris wrote:
> The Central Power Manager (CPM) is a co-processor on the Google Tensor
> SoCs, as used on Pixel phones. CPM supports power-management and a
> number of other features.
> 
> The Application Processor communicates with CPM over a pair of mailbox
> channels. CPM provides clock, power-domain, reset, and interrupt
> services.
> 
> This series adds device tree bindings, DTS, and drivers to support CPM,
> and several of the basic services provided by CPM. These services are a
> minimal set needed to bootstrap many other components on the SoC --
> e.g., I have a working proof-of-concept PCIe driver running on top of
> this series + pinctrl (not included here). CPM provides some other
> components not yet included in this series (e.g., PMIC, RTC).
> 
> This series depends on the bindings and mailbox-payload-definitions from
> this series:
> 
>   https://lore.kernel.org/all/20261023235408.3063350-1-dianders@xxxxxxxxxxxx/
>   [PATCH v2 0/4] mailbox: Improve the mbox core then introduce the goog-mba driver
> 
> So this series cannot be applied as-is yet. However, v1 of that series
> was under review for about 3 months, so I'd like to at least:
> 
>   1) demonstrate its clients (this came up at
>      https://lore.kernel.org/all/CAD=FV=VW2h1zWDYWQ6LknX3O+3=EY665jde35Ly+UQzuscXy=Q@xxxxxxxxxxxxxx/,
>      for one); and
> 
>   2) start review for anything that isn't significantly impacted by
>      potential changes in the above mailbox series.
> 
> With luck, the general design and DT bindings don't need to change, but
> even if so, I expect any changes should not significantly affect:
> 
>  (a) the CPM DT binding or DTS (apart from #mbox-cells)
> 
>  (b) the CPM interface (cpm_request_client(), cpm_send_message())
> 
>  (c) any of the CPM clients (irqchip, reset, clk, pmdomain)
> 
> Thus, I think this falls somewhere between "PATCH" and "RFC".

Either you expect review or not. RFC means no. If that's patch, then at
the top level of cover letter, as the most important thing, you explain
the dependency and merging proposal of this. You combined here like 5
different subsystens...

Best regards,
Krzysztof