Messages in this thread | | | Date | Mon, 11 Oct 2021 00:04:52 +0200 | From | David Heidelberg <> | Subject | Re: [PATCH] dt-bindings: reserved-memory: ramoops: Convert txt bindings to yaml |
| |
On Sun, Oct 10 2021 at 14:31:56 -0500, Rob Herring <robh@kernel.org> wrote: > On Sat, 09 Oct 2021 13:19:27 +0200, David Heidelberg wrote: >> Convert ramoops driver to the YAML syntax. >> >> Signed-off-by: David Heidelberg <david@ixit.cz> >> --- >> .../bindings/reserved-memory/ramoops.txt | 66 --------- >> .../bindings/reserved-memory/ramoops.yaml | 126 >> ++++++++++++++++++ >> 2 files changed, 126 insertions(+), 66 deletions(-) >> delete mode 100644 >> Documentation/devicetree/bindings/reserved-memory/ramoops.txt >> create mode 100644 >> Documentation/devicetree/bindings/reserved-memory/ramoops.yaml >> > > Running 'make dtbs_check' with the schema in this patch gives the > following warnings. Consider if they are expected or the schema is > incorrect. These may not be new warnings. > > Note that it is not yet a requirement to have 0 warnings for > dtbs_check. > This will change in the future. > > Full log is available here: https://patchwork.ozlabs.org/patch/1538758 > > > ramoops@ac300000: 'devinfo-size' does not match any of the regexes: > 'pinctrl-[0-9]+' > arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dt.yaml > arch/arm64/boot/dts/qcom/sdm845-oneplus-fajita.dt.yaml > doesn't exist besides oneplus bindings
> ramoops@ffc00000: 'msg-size' does not match any of the regexes: > 'pinctrl-[0-9]+' > arch/arm64/boot/dts/qcom/sm6125-sony-xperia-seine-pdx201.dt.yaml
doesn't exist > > ramoops@ffc00000: 'msg-size', 'no-map' do not match any of the > regexes: 'pinctrl-[0-9]+' > arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano-bahamut.dt.yaml > arch/arm64/boot/dts/qcom/sm8150-sony-xperia-kumano-griffin.dt.yaml > arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo-pdx203.dt.yaml > arch/arm64/boot/dts/qcom/sm8250-sony-xperia-edo-pdx206.dt.yaml
doesn't exist and I'm not sure if no-map make sense for pstore ramoops, since it HAS to be mapped (if I understood right).
Binding should be right, I can send patches for fired warnings.
David
| |