On 13/04/2023 10:36, Fabrizio Lamarque wrote: > Added undocumented properties: Use imperative. https://elixir.bootlin.com/linux/v5.17.1/source/Documentation/process/submitting-patches.rst#L95 > > - adi,clock-xtal > - adi,int-clock-output-enable > > Removed clocks from required properties. Why? > Renamed avdd-supply to vreg-supply, while keeping backward compatibility > (deprecated avdd-supply). Why? Use subject prefixes matching the subsystem (which you can get for example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory your patch is touching). Please use scripts/get_maintainers.pl to get a list of necessary people and lists to CC. It might happen, that command when run on an older kernel, gives you outdated entries. Therefore please be sure you base your patches on recent Linux kernel. You decided to ignore quite a lot of entries, but most important - also lists, so it won't even be tested. > > Signed-off-by: Fabrizio Lamarque <fl.scratchpad@xxxxxxxxx> > --- > .../bindings/iio/adc/adi,ad7192.yaml | 28 +++++++++++++++---- Bindings are always separate patches. > drivers/iio/adc/ad7192.c | 18 ++++++------ > 2 files changed, 32 insertions(+), 14 deletions(-) > > diff --git a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml > b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml > index d521d516088b..5dc7a7eea5f9 100644 > --- a/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml > +++ b/Documentation/devicetree/bindings/iio/adc/adi,ad7192.yaml > @@ -32,7 +32,9 @@ properties: > > clocks: > maxItems: 1 > - description: phandle to the master clock (mclk) > + description: | > + phandle to the external master clock (mclk). If not defined, internal > + clock is selected. > > clock-names: > items: > @@ -45,7 +47,23 @@ properties: > description: DVdd voltage supply > > avdd-supply: > - description: AVdd voltage supply > + description: Phandle to reference voltage regulator. Use > vref-supply instead. Corrupted patch. Run checkpatch, test your patches with dt_binding_check. This really needs a lot of work. Best regards, Krzysztof
