[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20220429213907.d2lwnukugoa7owxn@notapiano>
Date: Fri, 29 Apr 2022 17:39:07 -0400
From: Nícolas F. R. A. Prado
<nfraprado@...labora.com>
To: Allen-KH Cheng <allen-kh.cheng@...iatek.com>
Cc: Matthias Brugger <matthias.bgg@...il.com>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com>,
Project_Global_Chrome_Upstream_Group@...iatek.com,
devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org, linux-mediatek@...ts.infradead.org,
Chen-Yu Tsai <wenst@...omium.org>,
Ryder Lee <ryder.lee@...nel.org>, hsinyi@...omium.org
Subject: Re: [PATCH v8 2/2] arm64: dts: Add MediaTek SoC MT8186 dts and
evaluation board and Makefile
On Thu, Apr 28, 2022 at 01:27:41PM -0400, Nícolas F. R. A. Prado wrote:
> Hi Allen,
>
> On Thu, Apr 28, 2022 at 02:17:17PM +0800, Allen-KH Cheng wrote:
> > + scp: scp@...00000 {
> > + compatible = "mediatek,mt8186-scp";
> > + reg = <0 0x10500000 0 0x40000>,
> > + <0 0x105c0000 0 0x19080>;
> > + reg-names = "sram", "cfg";
>
> The mtk,scp dt-binding should be updated as it currently doesn't allow l1tcm to
> be missing. According to comments in the driver cfg is optional as well, so a
> patch to the binding making both optional would be great.
Hi Allen,
actually I've just sent a patch for this myself [1], as it was also required for
mt8192-asurada. Sorry for the noise.
Thanks,
Nícolas
[1] https://lore.kernel.org/all/20220429211111.2214119-2-nfraprado@collabora.com/
>
> Thanks,
> Nícolas
Powered by blists - more mailing lists