On Thu, Dec 09, 2021 at 12:06:23PM +0000, Srinivas Kandagatla wrote: > From: Jeya R <jeyr@xxxxxxxxxxxxxx> > > Add property to set DSP domain as non-secure. > > ADSP/MDSP/SDSP are by default secured, where as CDSP can be either be > secured/unsecured. Wouldn't it be easier to avoid the negation and add a "qcom,secure-domain" property instead? Given PATCH 8/8 ("arm64: dts: qcom: add non-secure domain property to fastrpc nodes") it looks like you are intentionally breaking DT compatibility here, but this patch does not justify why this is necessary. Thanks, Stephan