Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1359543 > unrolled thread

WARNING: DT compatible string vendor "mips" appears un-documented

Back to article view | Back to linux.kernel

Contents

  WARNING: DT compatible string vendor "mips" appears un-documented Antony Pavlov <antonynpavlov@gmail.com> - 2016-03-17 04:10 +0100
    Re: WARNING: DT compatible string vendor "mips" appears un-documented Rob Herring <robh@kernel.org> - 2016-03-17 14:10 +0100

#1359543 — WARNING: DT compatible string vendor "mips" appears un-documented

Hi all,

checkpatch.pl notes that DT compatible string vendor "mips" appears un-documented.

On the one hand there are several users of this vendor string:

    linux$ git grep \"mips,
    Documentation/devicetree/bindings/mips/img/xilfpga.txt: - compatible: Must be "mips,m14Kc".
    Documentation/devicetree/bindings/mips/img/xilfpga.txt:                 compatible = "mips,m14Kc";
    arch/mips/boot/dts/lantiq/danube.dtsi:                  compatible = "mips,mips24Kc";
    arch/mips/boot/dts/qca/ar9132.dtsi:                     compatible = "mips,mips24Kc";
    arch/mips/boot/dts/ralink/mt7620a.dtsi:                 compatible = "mips,mips24KEc";
    arch/mips/boot/dts/ralink/rt2880.dtsi:                  compatible = "mips,mips4KEc";
    arch/mips/boot/dts/ralink/rt3050.dtsi:                  compatible = "mips,mips24KEc";
    arch/mips/boot/dts/ralink/rt3883.dtsi:                  compatible = "mips,mips74Kc";
    arch/mips/boot/dts/xilfpga/microAptiv.dtsi:                     compatible = "mips,m14Kc";

On the other hand we already have the "mti" vendor string:

    linux$ grep MIPS Documentation/devicetree/bindings/vendor-prefixes.txt
    mti    Imagination Technologies Ltd. (formerly MIPS Technologies Inc.)

Can we add an another one vendor string for MIPS?

Any advice and suggestions will be greatly appreciated!

-- 
Best regards,
  Antony Pavlov

[toc] | [next] | [standalone]

#1359834

On Wed, Mar 16, 2016 at 10:02 PM, Antony Pavlov <antonynpavlov@gmail.com> wrote:
> Hi all,
>
> checkpatch.pl notes that DT compatible string vendor "mips" appears un-documented.
>
> On the one hand there are several users of this vendor string:
>
>     linux$ git grep \"mips,
>     Documentation/devicetree/bindings/mips/img/xilfpga.txt: - compatible: Must be "mips,m14Kc".
>     Documentation/devicetree/bindings/mips/img/xilfpga.txt:                 compatible = "mips,m14Kc";
>     arch/mips/boot/dts/lantiq/danube.dtsi:                  compatible = "mips,mips24Kc";
>     arch/mips/boot/dts/qca/ar9132.dtsi:                     compatible = "mips,mips24Kc";
>     arch/mips/boot/dts/ralink/mt7620a.dtsi:                 compatible = "mips,mips24KEc";
>     arch/mips/boot/dts/ralink/rt2880.dtsi:                  compatible = "mips,mips4KEc";
>     arch/mips/boot/dts/ralink/rt3050.dtsi:                  compatible = "mips,mips24KEc";
>     arch/mips/boot/dts/ralink/rt3883.dtsi:                  compatible = "mips,mips74Kc";
>     arch/mips/boot/dts/xilfpga/microAptiv.dtsi:                     compatible = "mips,m14Kc";
>
> On the other hand we already have the "mti" vendor string:
>
>     linux$ grep MIPS Documentation/devicetree/bindings/vendor-prefixes.txt
>     mti    Imagination Technologies Ltd. (formerly MIPS Technologies Inc.)
>
> Can we add an another one vendor string for MIPS?

No.

It is called checkpatch.pl, not checkfile.pl (yes, I know it can check
files too). There are many warnings if you look at the tree as a
whole. The main point of the checks (at least for DT) is to not
introduce new warnings.

So mips is deprecated and shouldn't be used for new bindings (perhaps
mti is too as img? should be used now). Continuing to use the existing
strings is fine.

Rob

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web