Re: [PATCH v4 2/6] dt-bindings: ata: sata-brcm: Describe the NSP IDM registers

[Krzysztof Kozlowski]

On Thu, Oct 29, 2026 at 02:25:09PM -0700, Florian Fainelli wrote:
> Northstar Plus uses the SATA IDM block to enable the interface clock and
> deassert the core reset before accessing the controller registers.
> 
> Allow an optional third register region named "idm" for the NSP
> controller. Keep other compatibles limited to their existing two
> register regions and allow older NSP descriptions without the IDM.
> 
> Signed-off-by: Florian Fainelli <florian.fainelli@xxxxxxxxxxxx>
> ---
>  .../bindings/ata/brcm,sata-brcm.yaml          | 57 ++++++++++++-------
>  1 file changed, 37 insertions(+), 20 deletions(-)

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>

Best regards,
Krzysztof