Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Oleksii Moisieiev <Oleksii_Moisieiev@epam.com>
To:     Stefano Stabellini <sstabellini@kernel.org>
CC:     Sudeep Holla <sudeep.holla@arm.com>,
        Cristian Marussi <cristian.marussi@arm.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        Vincent Guittot <vincent.guittot@linaro.org>,
        Souvik Chakravarty <Souvik.Chakravarty@arm.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [RFC PATCH 0/1] dt-bindings: arm: Add scmi_devid paramter for
Thread-Topic: [RFC PATCH 0/1] dt-bindings: arm: Add scmi_devid paramter for
Thread-Index: AQHYJ0gzI98NnKSQaEaZikv/1tAERayfaB6AgABVqICAABNVgIACyvOAgAC3jwCAB2uVAA==
Date:   Tue, 1 Mar 2022 16:10:34 +0000
Message-ID: <20220301161034.GA78066@EPUAKYIW015D>
References: <cover.1645460043.git.oleksii_moisieiev@epam.com>
 <20220222110003.GC21915@e120937-lin> <20220222160637.yn6pru4nfgwih23j@bogus>
 <20220222171549.GA2194063@EPUAKYIW015D>
 <20220224115443.fwhczfvm3cfwoim7@bogus>
 <alpine.DEB.2.22.394.2202241424110.239973@ubuntu-linux-20-04-desktop>
In-Reply-To: <alpine.DEB.2.22.394.2202241424110.239973@ubuntu-linux-20-04-desktop>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?X9m4e2+LKvU/O9bca9yUeqXbxtuPgB6TxqFqiMkE98ZXInvQch/2jiSQ/cZq?=
 =?us-ascii?Q?oqiMO6pxPodg5eqZMzKxcbHrbaUrQA8Ka8UvQ8gXNaztGss77lYi5xmzr9kC?=
 =?us-ascii?Q?+Ho+C6kdYx01IngzEK3v0hOC0td7eeaOaskQDBVYcYYGbwjgUhFEaHhNzT4P?=
 =?us-ascii?Q?1Z4jTTcfxpQs0u/JJ22lj3Xehs0ii1jjkYxp2Z5MA1dS+Jwv6lLGdoqGzUG4?=
 =?us-ascii?Q?xXFNiPihLkiDlLkn14OQctunrKVFSGPIcIYFFb+cCwQNlh8+OG2EDt42+31G?=
 =?us-ascii?Q?eZXJA57oiZOWSYG6KWqXsR/qfRx6s7wCgiZchjW7dob1L3DigdmTNy8UQI3M?=
 =?us-ascii?Q?w0CPFWrSr5LJVouPQKgzEFkFTciAl72axwBHSReVl0uEBEszn515k7WEVEIC?=
 =?us-ascii?Q?PH6wnwGtL1tw3EpyJuBvl6uGdWX8a5z/kWjbleM4v1B98kwFTBhzxShwOsDg?=
 =?us-ascii?Q?ZQh5lAeF77PD78XQV67KS7Dh2k2ECazTZJDMPoVuXxvi+TT2I/sYtRolRQ/+?=
 =?us-ascii?Q?SC0QH+a6vLsBvtNuZNLUsbvt1xcx5uw19coTV4R9ljHsemeVwyLg+Ajy8rcj?=
 =?us-ascii?Q?ObDLqouyC5H97KuVBFkDeJw02WFZQ/0r2mHa04+I75wwzt2171d3hko83Y9b?=
 =?us-ascii?Q?rRFiV09Ooztl5JmJZgAd+ihjfa6LqQNEIV0VbW7Hpq5E4TQDBMNCrzJqS1Sw?=
 =?us-ascii?Q?L8Eeu6oVmxcGlG5/JEdQOEF8Ktw1OPgCLarWjveAv53V/vfHdldtVlxTUv75?=
 =?us-ascii?Q?tr4Ji717oUjCaaHvpJMEWAZQZ8HGNBKVPCDlrX4UdNJNDtLhAYB2sGldMlC8?=
 =?us-ascii?Q?sKmlMAiw3ASzfk/w175UO88d/W6exxxaFlI2CNaKftqhMupDMfShl2u2cglJ?=
 =?us-ascii?Q?1OgCOOR4NMPq7ZZxZGCdFdfFFqrjorwySen6RY5AXg/BxNQSShP0IV0M4pLI?=
 =?us-ascii?Q?eIW/F0PYLb0yVYLuUMGym6w2/R15ZKKmy3Hxbyt6Z1b828OW4rGXUjph3E6b?=
 =?us-ascii?Q?QcSTaDIw+Js/ubPoCM1hjjupZ3+E6onxXW784Lh+hrmcaV7EUWLr4eNzhhLd?=
 =?us-ascii?Q?qGYQdPtg/tiYCN+6gwDtwAWLzyhZDifwI6Z8bpLIznorq5jyyd5AIcr+E5NC?=
 =?us-ascii?Q?QfKBaRVm9zuHb7icsP2uMDvR5PWrrB4U+DQGIb9Ohv7ha+HM3h+YxPYlye1U?=
 =?us-ascii?Q?hQjWjv6gsQFHcW+7z91Ek7i6AXBnzhCziYoZwSkBpmF2L17ejKGm4YsFQIJm?=
 =?us-ascii?Q?RyZzqD1OHsqr6jrTX4FXZHCpnZV7h5A2X+pIOL7dfmXSWbyxYPvMytJXSq38?=
 =?us-ascii?Q?E5R8EVksUxNGMEVEaqyM272obMxrdC1Y5hz3H3BfhZRCWW6T5+E1zikk9j6l?=
 =?us-ascii?Q?91zXtDZ8cxt9KhXjbmvJpCRCS3bZ1k9avw92zpaxiQ6hDFQasuD2LLizX5Tl?=
 =?us-ascii?Q?j2haNhV1azb+uccbr3Nm7NywGckJH5lZZqsHsAlPz2ZUtcFsf5SyLS9shWpY?=
 =?us-ascii?Q?K6VLCq4QWpE8RrWjIm/emJ5nv7R/Qp8dxckTC0eqF0ESp5SsUnTCeWZdhanS?=
 =?us-ascii?Q?/tLnsMR5QWDZoXhewQCwuhISRGSqTyPkGt/MVkSXb/ooeOHlow1eBbuZdoqV?=
 =?us-ascii?Q?bmM0YtumIhCt9MbRXUO/mWUgQyvrmS+c+JmmK9ViHqYg/ZQWy+HZWy3YOa7f?=
 =?us-ascii?Q?nrNBJRrmTSFG+tnFEQlGWQOA0Yg=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <424EF99A9B8AA04B91771A3356958C75@eurprd03.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: epam.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PA4PR03MB7136.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 042bb5a1-003c-45d4-2b85-08d9fb9e0487
X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Mar 2022 16:10:34.8070
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b41b72d0-4e9f-4c26-8a69-f949f367c91d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: aM+g1MD6KbyB7aUxitJVZYvC3rnoDCKsJcVxDQT1EP82U4fuTbwjbe5BqyuFWntieJd/zcG4P/JbxsNj3f1QQKOkgqvOjY62TGEoyI401xI=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR03MB4256
X-Proofpoint-GUID: zbueCTtkv7wctb0BZcnm-yJfypE3pQve
X-Proofpoint-ORIG-GUID: zbueCTtkv7wctb0BZcnm-yJfypE3pQve
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.816,Hydra:6.0.425,FMLib:17.11.64.514
 definitions=2022-03-01_07,2022-02-26_01,2022-02-23_01
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 adultscore=0
 priorityscore=1501 mlxscore=0 malwarescore=0 spamscore=0 bulkscore=0
 phishscore=0 clxscore=1015 suspectscore=0 impostorscore=0 mlxlogscore=999
 lowpriorityscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2201110000 definitions=main-2203010086
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,SPF_HELO_NONE,
        SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Stefano,

On Thu, Feb 24, 2022 at 02:51:42PM -0800, Stefano Stabellini wrote:
> On Thu, 24 Feb 2022, Sudeep Holla wrote:
> > On Tue, Feb 22, 2022 at 05:15:49PM +0000, Oleksii Moisieiev wrote:
> > > Hi Sudeep,
> > >=20
> > > On Tue, Feb 22, 2022 at 04:06:37PM +0000, Sudeep Holla wrote:
> > > > Hi Oleksii,
> > > >=20
> > > > My initial feedback on this. And thanks Cristian for making it so e=
asy as
> > > > you have covered most of the things in depth(which I might have not=
 done
> > > > myself that well)
> > > >=20
> > > > On Tue, Feb 22, 2022 at 11:00:03AM +0000, Cristian Marussi wrote:
> > > > > On Mon, Feb 21, 2022 at 05:26:46PM +0000, Oleksii Moisieiev wrote=
:
> > > > > > Introducing new parameter called scmi_devid to the device-tree =
bindings.
> > > > > > This parameter should be set for the device nodes, which has
> > > > > > clocks/power-domains/resets working through SCMI.
> > > >=20
> > > > I prefer you had given more details on your usage model here instea=
d of
> > > > pointing to the other Xen thread as it helps for someone without mu=
ch
> > > > background on Xen or your use-case to review this.
> > > >=20
> > > Let me describe the process in few words:
> > > We implemented a new feature, called SCI-mediator in Xen.
> > > The proposed implementation allows Guests to communicate with the Fir=
mware using SCMI
> > > protocol with SMC as a transport. Other implementation are also
> > > possible, such as SCMI-Mailbox, SCPI-mailbox etc.
> > >=20
> > > In this feature Xen is the Trusted Agent, which receives the followin=
g
> > > information in Xen device-tree:
> > > 1) All channels should be described, each channel defined as
> > > arm,scmi-shmem node;
> > > 2) Scmi node arm,scmi-smc with protocols description;
> >=20
> > Sounds good so far.
> >=20
> > > 3) scmi-devid should be set in nodes, which works through SCMI.
> > >
> >=20
> > Why is this needed for Guest OS, you need not populate this if Guest OS
> > is not required to use it, right ? If it is needed just by Xen hypervis=
or,
> > lets talk about that and why it is bad idea to mix that with general
> > SCMI bindings.
>=20
> I'll try to help Oleksii by answering here, I hope I am not off the mark
> :-)

I appreciate it.

>=20
> I think Sudeep is right, scmi-devid is not needed by the guest OS.
>=20

I agree. I think this should be handled by Xen.

Best regards,
Oleksii.=