Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Date:   Tue, 17 May 2022 11:05:42 -0400
From:   =?utf-8?B?TsOtY29sYXMgRi4gUi4gQS4=?= Prado 
        <nfraprado@collabora.com>
To:     AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@collabora.com>
Cc:     robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
        matthias.bgg@gmail.com, sboyd@kernel.org,
        chun-jie.chen@mediatek.com, rex-bc.chen@mediatek.com,
        wenst@chromium.org, devicetree@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/6] dt-bindings: arm: MediaTek: Fix clock bindings
Message-ID: <20220517150542.cvvescaeohlfurkw@notapiano>
References: <20220517101514.21639-1-angelogioacchino.delregno@collabora.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20220517101514.21639-1-angelogioacchino.delregno@collabora.com>
Precedence: bulk

On Tue, May 17, 2022 at 12:15:08PM +0200, AngeloGioacchino Del Regno wrote:
> As per Rob Herring's review [1] on my mt6795 clocks bindings patches, for
> which I've used the already upstreamed ones as a base, it was found that
> these bindings have some issues.
> This series is addressing the issues that were found by Rob on my series,
> which are present on all of the already merged bindings.
> 
> [1]: https://patchwork.kernel.org/project/linux-mediatek/patch/20220513165050.500831-5-angelogioacchino.delregno@collabora.com/#24859953
> 
> AngeloGioacchino Del Regno (6):
>   dt-bindings: arm: mt8186-clock: Remove unnecessary 'items' and fix
>     formatting
>   dt-bindings: arm: mt8186: Set #clock-cells as required property
>   dt-bindings: arm: mt8195-clock: Remove unnecessary 'items' and fix
>     formatting
>   dt-bindings: arm: mt8195: Set #clock-cells as required property
>   dt-bindings: arm: mt8192-clock: Remove unnecessary 'items'
>   dt-bindings: arm: mt8192: Set #clock-cells as required property

For the whole series,

Reviewed-by: N?colas F. R. A. Prado <nfraprado@collabora.com>

Thanks,
N?colas

> 
>  .../arm/mediatek/mediatek,mt8186-clock.yaml   | 29 ++++-----
>  .../mediatek/mediatek,mt8186-sys-clock.yaml   |  1 +
>  .../arm/mediatek/mediatek,mt8192-clock.yaml   | 46 +++++++--------
>  .../mediatek/mediatek,mt8192-sys-clock.yaml   |  1 +
>  .../arm/mediatek/mediatek,mt8195-clock.yaml   | 59 ++++++++++---------
>  .../mediatek/mediatek,mt8195-sys-clock.yaml   |  1 +
>  6 files changed, 71 insertions(+), 66 deletions(-)
> 
> -- 
> 2.35.1
> 
>