On Sun, Dec 27, 2020 at 09:56:21AM -0700, Rob Herring wrote:
> On Fri, 25 Dec 2020 19:01:09 +0800, Xin Ji wrote:
> > Add DPI flag for distinguish MIPI input signal type, DSI or DPI. Add
> > swing setting for adjusting DP tx PHY swing
> > 
> > Signed-off-by: Xin Ji <x...@analogixsemi.com>
> > ---
> >  .../bindings/display/bridge/analogix,anx7625.yaml     | 19 
> > +++++++++++++++++++
> >  1 file changed, 19 insertions(+)
> > 
> 
> My bot found errors running 'make dt_binding_check' on your patch:
Hi Rob Herring, I'll fix this issue in the next serial, thanks!
> 
> yamllint warnings/errors:
> 
> dtschema/dtc warnings/errors:
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.example.dt.yaml:
>  encoder@58: anx,swing-setting: 'anyOf' conditional failed, one must be fixed:
>       [[0, 20], [1, 84], [2, 100], [3, 116], [4, 41], [5, 123], [6, 119], [7, 
> 91], [8, 127], [12, 32], [13, 96], [16, 96], [18, 64], [19, 96], [20, 20], 
> [21, 84], [22, 100], [23, 116], [24, 41], [25, 123], [26, 119], [27, 91], 
> [28, 127], [32, 32], [33, 96], [36, 96], [38, 64], [39, 96]] is too long
>       [0, 20] is too long
>       [1, 84] is too long
>       [2, 100] is too long
>       [3, 116] is too long
>       [4, 41] is too long
>       [5, 123] is too long
>       [6, 119] is too long
>       [7, 91] is too long
>       [8, 127] is too long
>       [12, 32] is too long
>       [13, 96] is too long
>       [16, 96] is too long
>       [18, 64] is too long
>       [19, 96] is too long
>       [20, 20] is too long
>       [21, 84] is too long
>       [22, 100] is too long
>       [23, 116] is too long
>       [24, 41] is too long
>       [25, 123] is too long
>       [26, 119] is too long
>       [27, 91] is too long
>       [28, 127] is too long
>       [32, 32] is too long
>       [33, 96] is too long
>       [36, 96] is too long
>       [38, 64] is too long
>       [39, 96] is too long
>       From schema: 
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.example.dt.yaml:
>  encoder@58: 'anx,mipi-dpi-in', 'anx,swing-setting' do not match any of the 
> regexes: '^#.*', 
> '^(at25|bm|devbus|dmacap|dsa|exynos|fsi[ab]|gpio-fan|gpio-key|gpio|gpmc|hdmi|i2c-gpio),.*',
>  '^(keypad|m25p|max8952|max8997|max8998|mpmc),.*', 
> '^(pinctrl-single|#pinctrl-single|PowerPC),.*', 
> '^(pl022|pxa-mmc|rcar_sound|rotary-encoder|s5m8767|sdhci),.*', 
> '^(simple-audio-card|st-plgpio|st-spics|ts),.*', '^70mai,.*', '^GEFanuc,.*', 
> '^ORCL,.*', '^SUNW,.*', '^[a-zA-Z0-9#_][a-zA-Z0-9+\\-._@]{0,63}$', 
> '^[a-zA-Z0-9+\\-._]*@[0-9a-zA-Z,]*$', '^abb,.*', '^abilis,.*', '^abracon,.*', 
> '^abt,.*', '^acer,.*', '^acme,.*', '^actions,.*', '^active-semi,.*', 
> '^ad,.*', '^adafruit,.*', '^adapteva,.*', '^adaptrum,.*', '^adh,.*', 
> '^adi,.*', '^advantech,.*', '^aeroflexgaisler,.*', '^al,.*', '^allegro,.*', 
> '^allo,.*', '^allwinner,.*', '^alphascale,.*', '^alps,.*', '^alt,.*', 
> '^altr,.*', '^amarula,.*', 
 '^amazon,.*', '^amcc,.*', '^amd,.*', '^amediatech,.*', '^amlogic,.*', 
'^ampere,.*', '^ampire,.*', '^ams,.*', '^amstaos,.*', '^analogix,.*', 
'^andestech,.*', '^anvo,.*', '^apm,.*', '^aptina,.*', '^arasan,.*', 
'^archermind,.*', '^arctic,.*', '^arcx,.*', '^aries,.*', '^arm,.*', 
'^armadeus,.*', '^arrow,.*', '^artesyn,.*', '^asahi-kasei,.*', '^asc,.*', 
'^aspeed,.*', '^asus,.*', '^atlas,.*', '^atmel,.*', '^auo,.*', '^auvidea,.*', 
'^avago,.*', '^avia,.*', '^avic,.*', '^avnet,.*', '^awinic,.*', '^axentia,.*', 
'^axis,.*', '^azoteq,.*', '^azw,.*', '^baikal,.*', '^bananapi,.*', 
'^beacon,.*', '^beagle,.*', '^bhf,.*', '^bitmain,.*', '^boe,.*', '^bosch,.*', 
'^boundary,.*', '^brcm,.*', '^broadmobi,.*', '^bticino,.*', '^buffalo,.*', 
'^bur,.*', '^calaosystems,.*', '^calxeda,.*', '^caninos,.*', '^capella,.*', 
'^cascoda,.*', '^catalyst,.*', '^cavium,.*', '^cdns,.*', '^cdtech,.*', 
'^cellwise,.*', '^ceva,.*', '^checkpoint,.*', '^chefree,.*', '^chipidea,.*', 
'^chipone,.*', '^chipspark,.*', '^chrontel,.*'
 , '^chrp,.*', '^chunghwa,.*', '^chuwi,.*', '^ciaa,.*', '^cirrus,.*', 
'^cloudengines,.*', '^cnm,.*', '^cnxt,.*', '^colorfly,.*', '^compulab,.*', 
'^coreriver,.*', '^corpro,.*', '^cortina,.*', '^cosmic,.*', '^crane,.*', 
'^creative,.*', '^crystalfontz,.*', '^csky,.*', '^csq,.*', '^cubietech,.*', 
'^cypress,.*', '^cznic,.*', '^dallas,.*', '^dataimage,.*', '^davicom,.*', 
'^dell,.*', '^delta,.*', '^denx,.*', '^devantech,.*', '^dfi,.*', '^dh,.*', 
'^difrnce,.*', '^digi,.*', '^digilent,.*', '^dioo,.*', '^dlc,.*', '^dlg,.*', 
'^dlink,.*', '^dmo,.*', '^domintech,.*', '^dongwoon,.*', '^dptechnics,.*', 
'^dragino,.*', '^dserve,.*', '^dynaimage,.*', '^ea,.*', '^ebs-systart,.*', 
'^ebv,.*', '^eckelmann,.*', '^edt,.*', '^eeti,.*', '^einfochips,.*', 
'^elan,.*', '^element14,.*', '^elgin,.*', '^elida,.*', '^elimo,.*', 
'^embest,.*', '^emlid,.*', '^emmicro,.*', '^empire-electronix,.*', 
'^emtrion,.*', '^endless,.*', '^ene,.*', '^energymicro,.*', '^engicam,.*', 
'^epcos,.*', '^epfl,.*', '^epson,.*', '^esp,.*', 
 '^est,.*', '^ettus,.*', '^eukrea,.*', '^everest,.*', '^everspin,.*', 
'^evervision,.*', '^exar,.*', '^excito,.*', '^ezchip,.*', '^facebook,.*', 
'^fairphone,.*', '^faraday,.*', '^fastrax,.*', '^fcs,.*', '^feixin,.*', 
'^feiyang,.*', '^fii,.*', '^firefly,.*', '^focaltech,.*', '^frida,.*', 
'^friendlyarm,.*', '^fsl,.*', '^fujitsu,.*', '^gardena,.*', '^gateworks,.*', 
'^gcw,.*', '^ge,.*', '^geekbuying,.*', '^gef,.*', '^gemei,.*', '^geniatech,.*', 
'^giantec,.*', '^giantplus,.*', '^globalscale,.*', '^globaltop,.*', '^gmt,.*', 
'^goodix,.*', '^google,.*', '^grinn,.*', '^grmn,.*', '^gumstix,.*', '^gw,.*', 
'^hannstar,.*', '^haoyu,.*', '^hardkernel,.*', '^hideep,.*', '^himax,.*', 
'^hirschmann,.*', '^hisilicon,.*', '^hit,.*', '^hitex,.*', '^holt,.*', 
'^holtek,.*', '^honestar,.*', '^honeywell,.*', '^hoperun,.*', '^hp,.*', 
'^hsg,.*', '^hugsun,.*', '^hwacom,.*', '^hydis,.*', '^hyundai,.*', '^i2se,.*', 
'^ibm,.*', '^icplus,.*', '^idt,.*', '^ifi,.*', '^ilitek,.*', '^img,.*', 
'^imi,.*', '^incircuit,.*', '
 ^inet-tek,.*', '^infineon,.*', '^inforce,.*', '^ingenic,.*', '^innolux,.*', 
'^inside-secure,.*', '^inspur,.*', '^intel,.*', '^intercontrol,.*', 
'^invensense,.*', '^inversepath,.*', '^iom,.*', '^isee,.*', '^isil,.*', 
'^issi,.*', '^ite,.*', '^itead,.*', '^ivo,.*', '^iwave,.*', '^jdi,.*', 
'^jedec,.*', '^jesurun,.*', '^jianda,.*', '^kam,.*', '^karo,.*', 
'^keithkoep,.*', '^keymile,.*', '^khadas,.*', '^kiebackpeter,.*', 
'^kinetic,.*', '^kingdisplay,.*', '^kingnovel,.*', '^kionix,.*', '^kobo,.*', 
'^kobol,.*', '^koe,.*', '^kontron,.*', '^kosagi,.*', '^kyo,.*', '^lacie,.*', 
'^laird,.*', '^lamobo,.*', '^lantiq,.*', '^lattice,.*', '^leadtek,.*', 
'^leez,.*', '^lego,.*', '^lemaker,.*', '^lenovo,.*', '^lg,.*', '^lgphilips,.*', 
'^libretech,.*', '^licheepi,.*', '^linaro,.*', '^linksprite,.*', '^linksys,.*', 
'^linutronix,.*', '^linux,.*', '^linx,.*', '^litex,.*', '^lltc,.*', 
'^logicpd,.*', '^logictechno,.*', '^longcheer,.*', '^lontium,.*', 
'^loongson,.*', '^lsi,.*', '^lwn,.*', '^lxa,.*', '^macnica,.
 *', '^mantix,.*', '^mapleboard,.*', '^marvell,.*', '^maxbotix,.*', 
'^maxim,.*', '^mbvl,.*', '^mcube,.*', '^meas,.*', '^mecer,.*', '^mediatek,.*', 
'^megachips,.*', '^mele,.*', '^melexis,.*', '^melfas,.*', '^mellanox,.*', 
'^memsic,.*', '^menlo,.*', '^mentor,.*', '^meraki,.*', '^merrii,.*', 
'^micrel,.*', '^microchip,.*', '^microcrystal,.*', '^micron,.*', 
'^microsoft,.*', '^microsys,.*', '^mikroe,.*', '^mikrotik,.*', '^miniand,.*', 
'^minix,.*', '^miramems,.*', '^mitsubishi,.*', '^modtronix,.*', 
'^mosaixtech,.*', '^motorola,.*', '^moxa,.*', '^mpl,.*', '^mps,.*', 
'^mqmaker,.*', '^mrvl,.*', '^mscc,.*', '^msi,.*', '^mstar,.*', '^mti,.*', 
'^multi-inno,.*', '^mundoreader,.*', '^murata,.*', '^mxicy,.*', '^myir,.*', 
'^national,.*', '^nec,.*', '^neonode,.*', '^netgear,.*', '^netlogic,.*', 
'^netron-dy,.*', '^netxeon,.*', '^neweast,.*', '^newhaven,.*', '^nexbox,.*', 
'^nextthing,.*', '^ni,.*', '^nintendo,.*', '^nlt,.*', '^nokia,.*', 
'^nordic,.*', '^novtech,.*', '^nutsboard,.*', '^nuvoton,.*', '^nvd
 ,.*', '^nvidia,.*', '^nxp,.*', '^oceanic,.*', '^oct,.*', '^okaya,.*', 
'^oki,.*', '^olimex,.*', '^olpc,.*', '^onion,.*', '^onnn,.*', '^ontat,.*', 
'^opalkelly,.*', '^opencores,.*', '^openrisc,.*', '^option,.*', '^oranth,.*', 
'^orisetech,.*', '^ortustech,.*', '^osddisplays,.*', '^ouya,.*', '^overkiz,.*', 
'^ovti,.*', '^oxsemi,.*', '^ozzmaker,.*', '^panasonic,.*', '^parade,.*', 
'^parallax,.*', '^pda,.*', '^pericom,.*', '^pervasive,.*', '^phicomm,.*', 
'^phytec,.*', '^picochip,.*', '^pine64,.*', '^pineriver,.*', '^pixcir,.*', 
'^plantower,.*', '^plathome,.*', '^plda,.*', '^plx,.*', '^pni,.*', 
'^pocketbook,.*', '^polaroid,.*', '^portwell,.*', '^poslab,.*', '^pov,.*', 
'^powertip,.*', '^powervr,.*', '^primux,.*', '^probox2,.*', '^prt,.*', 
'^pulsedlight,.*', '^purism,.*', '^qca,.*', '^qcom,.*', '^qemu,.*', '^qi,.*', 
'^qiaodian,.*', '^qihua,.*', '^qnap,.*', '^radxa,.*', '^raidsonic,.*', 
'^ralink,.*', '^ramtron,.*', '^raspberrypi,.*', '^raydium,.*', '^rda,.*', 
'^realtek,.*', '^renesas,.*', '^rerv
 ision,.*', '^revotics,.*', '^rex,.*', '^richtek,.*', '^ricoh,.*', 
'^rikomagic,.*', '^riot,.*', '^riscv,.*', '^rockchip,.*', '^rocktech,.*', 
'^rohm,.*', '^ronbo,.*', '^roofull,.*', '^roseapplepi,.*', '^samsung,.*', 
'^samtec,.*', '^sancloud,.*', '^sandisk,.*', '^satoz,.*', '^sbs,.*', 
'^schindler,.*', '^seagate,.*', '^seeed,.*', '^seirobotics,.*', '^semtech,.*', 
'^sensirion,.*', '^sensortek,.*', '^sff,.*', '^sgd,.*', '^sgmicro,.*', 
'^sgx,.*', '^sharp,.*', '^shimafuji,.*', '^shiratech,.*', '^si-en,.*', 
'^si-linux,.*', '^sifive,.*', '^sigma,.*', '^sii,.*', '^sil,.*', '^silabs,.*', 
'^silead,.*', '^silergy,.*', '^silex-insight,.*', '^siliconmitus,.*', 
'^simtek,.*', '^sinlinx,.*', '^sinovoip,.*', '^sipeed,.*', '^sirf,.*', 
'^sis,.*', '^sitronix,.*', '^skyworks,.*', '^smartlabs,.*', '^smsc,.*', 
'^snps,.*', '^sochip,.*', '^socionext,.*', '^solidrun,.*', '^solomon,.*', 
'^sony,.*', '^spansion,.*', '^sprd,.*', '^sst,.*', '^sstar,.*', '^st,.*', 
'^st-ericsson,.*', '^starry,.*', '^startek,.*', '^ste
 ,.*', '^stericsson,.*', '^summit,.*', '^sunchip,.*', '^swir,.*', '^syna,.*', 
'^synology,.*', '^tbs,.*', '^tbs-biometrics,.*', '^tcg,.*', '^tcl,.*', 
'^tdo,.*', '^technexion,.*', '^technologic,.*', '^techstar,.*', '^te
>       From schema: 
> /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/vendor-prefixes.yaml
> 
> See https://patchwork.ozlabs.org/patch/1420616
> 
> This check can fail if there are any dependencies. The base for a patch
> series is generally the most recent rc1.
> 
> If you already ran 'make dt_binding_check' and didn't see the above
> error(s), then make sure 'yamllint' is installed and dt-schema is up to
> date:
> 
> pip3 install dtschema --upgrade
> 
> Please check and re-submit.
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to