On 31/08/2022 01:25, Guru Das Srinagesh wrote: > For firmware that supports it, allow multiple SCM calls to be passed > down to it by removing the serialization lock in the SCM driver. > > This patch is based on this YAML conversion patch [1] that is in-flight > currently. > > [1] https://lore.kernel.org/lkml/20220708090431.30437-1-david@xxxxxxx/ > > Signed-off-by: Guru Das Srinagesh <quic_gurus@xxxxxxxxxxx> Thank you for your patch. There is something to discuss/improve. If you make a resend, at least be sure you Cc proper people. Use scripts/get_maintainers.pl to CC all maintainers and relevant mailing lists. This was not Cced to device tree maintainers, therefore it has to be NAK-ed. > --- > Documentation/devicetree/bindings/firmware/qcom,scm.yaml | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/Documentation/devicetree/bindings/firmware/qcom,scm.yaml b/Documentation/devicetree/bindings/firmware/qcom,scm.yaml > index 9fdeee0..e279fd2 100644 > --- a/Documentation/devicetree/bindings/firmware/qcom,scm.yaml > +++ b/Documentation/devicetree/bindings/firmware/qcom,scm.yaml > @@ -70,6 +70,11 @@ properties: > '#reset-cells': > const: 1 > > + allow-multi-call: > + description: > + Specify this flag to remove SCM call serialization. Need to ensure that > + the firmware being used supports this feature first. Missing vendor prefix, missing type. Isn't support for this obvious from compatible?\ Best regards, Krzysztof
