From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 6026513AF9 for ; Wed, 13 Mar 2024 11:04:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710327876; cv=none; b=kfGYJZHXQeBboUpOgFIYaKhZgVaUysnLVCZSrPKIbxCjDrs66h26uY5KakpspLDudKeHVc2tdKAwPXxaDpZrMYa4jb+Wr4oojAkOH9VZzDSILsRGW7Pt+GeNkxg0uWs+9DpJsvHWQS1wKQwz7Td/6DnipB04PiVOSoQsEJcnIzw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1710327876; c=relaxed/simple; bh=dAX+cA9zPdJofcNhWsOe3dFbFBgKyLHPr8FuKGSzUpw=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=WHCCuq5/LvdDDvTzeGInAHl9cPfKFWtGHNjBhTar1PJ9rkLgz2o06PIQeuG2nNInqkf/lodLm2hQik4YN2iBQuYauXccwJfJKiRw+/wFau47kYlKloCA4gzXWj0235DYyJS6mYSfJ3aKyy/5O1QjD8caiR/6vk8mDSmN5cnMbpI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 8A39A1007; Wed, 13 Mar 2024 04:05:10 -0700 (PDT) Received: from bogus (e103737-lin.cambridge.arm.com [10.1.197.49]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id C78853F73F; Wed, 13 Mar 2024 04:04:31 -0700 (PDT) Date: Wed, 13 Mar 2024 11:04:29 +0000 From: Sudeep Holla To: Nikunj Kela Cc: Ulf Hansson , Srinivas Kandagatla , Sudeep Holla , Krzysztof Kozlowski , Manivannan Sadhasivam , , Vincent Guittot , , , , "Prasad Sodagudi (QUIC)" , "Trilok Soni (QUIC)" Subject: Re: DT Query on "New Compatible vs New Property" Message-ID: References: <94a62a78-961a-4286-804c-fc0b9098b8a1@quicinc.com> <20240228140239.gkzcytw6cmb4opja@bogus> <799268ac-7ffb-4b99-b037-d5bb93d37f13@linaro.org> <20240228160925.fcitj2yz7hisidsl@bogus> <2b0a11f4-f54e-461c-91e7-8f313d91abe8@linaro.org> <321069a8-2c46-4871-b85a-5e9cbdda5b5d@quicinc.com> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <321069a8-2c46-4871-b85a-5e9cbdda5b5d@quicinc.com> On Tue, Mar 12, 2024 at 09:52:56AM -0700, Nikunj Kela wrote: > +Trilok > > On 3/4/2024 3:01 AM, Sudeep Holla wrote: > > arch/arm64/boot/dts/arm/juno-scmi.dts > > > > One is with old firmware interface and -scmi is with SCMI. No new top > > level compatible change is needed. I understand it was switch from one > > interface to the another and not like Qcom platforms which is moving > > from in-kernel solution to firmware solution. But the general rule applies > > here as well unless there are specific reasons for needing that exception. > > I am not against it or ruling that out, just curious to understand them. > > Thank you all for all your inputs on this. I discussed this with Srini and > he suggested that we could use a new optional DT property like "qcom, > fw-managed" to ascertain if we are running on firmware managed variant. Thus > each device node in the dts can add this. I did ask him if, instead of > putting it to each device node, we can use it at the board level however he > thinks that it would not be easy to update yaml documentation on DT nodes > with board level property. So if everyone here agrees with this approach, I > would like to close this thread. The counter argument from me for that is simple. If you are OK to manage with one board level compatible/property(doesn't matter for this discussion), then why can't that be the compatible of the firmware itself(SCMI and RPMI both must have their own compatible already). The main point is why do you need any extra information when they are already present. My comment is just for this one board level compatible/property. -- Regards, Sudeep