On Sat, Jul 23, 2022 at 10:41:53PM +0200, Krzysztof Kozlowski wrote: > On 21/07/2022 22:02, Stephan Gerhold wrote: > > What remains is maybe: > > > > - "qcom,smem-states", which is already used in several other schemas > > and could be possibly defined together with #qcom,smem-state-cells > > in some generic schema(?) > > > > - "qcom,halt-regs", "firmware-name", "smd-edge" are used by different > > Qualcomm remoteproc drivers, so they could possibly be defined in > > some common "qcom-remoteproc.yaml" schema(?) > > smd-edge and glink is already in remoteproc/qcom,smd-edge.yaml > qcom,glink-edge.yaml > Yep, I'm actually using it already in this patch. I think what I meant is that the smd-edge property $ref could be already defined in a common "qcom-remoteproc.yaml" schema and inherited via allOf, since all the qcom remoteprocs should have either a smd-edge or glink-edge. But all in all I'm still unsure which "common properties" Rob is referring to here, in which file to place them, and if this is worth it at all for the few I have listed above... :-) Thanks, Stephan
