On Sun, 06 Sep 2020 17:36:46 +0200, Krzysztof Kozlowski wrote:
> Convert the Samsung S3FWRN5 NCI NFC controller bindings to dtschema.
> This is conversion only so it includes properties with invalid prefixes
> (s3fwrn5,en-gpios) which should be addressed later.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> .../devicetree/bindings/net/nfc/s3fwrn5.txt | 25 --------
> .../bindings/net/nfc/samsung,s3fwrn5.yaml | 61 +++++++++++++++++++
> MAINTAINERS | 1 +
> 3 files changed, 62 insertions(+), 25 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/net/nfc/s3fwrn5.txt
> create mode 100644
> Documentation/devicetree/bindings/net/nfc/samsung,s3fwrn5.yaml
>
My bot found errors running 'make dt_binding_check' on your patch:
./Documentation/devicetree/bindings/net/nfc/samsung,s3fwrn5.yaml: $id: relative
path/filename doesn't match actual path or filename
expected: http://devicetree.org/schemas/net/nfc/samsung,s3fwrn5.yaml#
/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/nfc/samsung,s3fwrn5.example.dt.yaml:
s3fwrn5@27: 's3fwrn5,en-gpios', 's3fwrn5,fw-gpios' do not match any of the
regexes: '^#.*',
'^(at25|devbus|dmacap|dsa|exynos|fsi[ab]|gpio-fan|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,]*$', '^abilis,.*', '^abracon,.*', '^acer,.*',
'^acme,.*', '^actions,.*', '^active-semi,.*', '^ad,.*', '^adafruit,.*',
'^adapteva,.*', '^adaptrum,.*', '^adh,.*', '^adi,.*', '^advantech,.*',
'^aeroflexgaisler,.*', '^al,.*', '^allegro,.*', '^allo,.*', '^allwinner,.*',
'^alphascale,.*', '^alps,.*', '^altr,.*', '^amarula,.*', '^amazon,.*',
'^amcc,.*', '^amd,.*', '^amediatech,.*', '^amlogic,.*', '^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,.*', '^capella,.*', '^cascoda,.*', '^catalyst,.*', '^cavium,.*',
'^cdns,.*', '^cdtech,.*', '^cellwise,.*', '^ceva,.*', '^checkpoint,.*',
'^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,.*', '^dh,.*', '^difrnce,.*', '^digi,.*',
'^digilent,.*', '^dioo,.*', '^dlc,.*', '^dlg,.*', '^dlink,.*', '^dmo,.*',
'^domintech,.*', '^dongwoon,.*', '^dptechnics,.*', '^dragino,.*', '^dserve,.*',
'^dynaimage,.*', '^ea,.*', '^ebs-systart,.*', '^ebv,.*', '^eckelmann,.*',
'^edt,.*', '^eeti,.*', '^einfochips,.*', '^elan,.*', '^elgin,.*', '^elida,.*',
'^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,.*', '^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,.*', '^hisilicon,.*', '^hit,.*',
'^hitex,.*', '^holt,.*', '^holtek,.*', '^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,.*', '^koe,.*', '^kontron,.*', '^kosagi,.*', '^kyo,.*',
'^lacie,.*', '^laird,.*', '^lamobo,.*', '^lantiq,.*', '^lattice,.*',
'^leadtek,.*', '^leez,.*', '^lego,.*', '^lemaker,.*', '^lenovo,.*', '^lg,.*',
'^lgphilips,.*', '^libretech,.*', '^licheepi,.*', '^linaro,.*',
'^linksprite,.*', '^linksys,.*', '^linutronix,.*', '^linux,.*', '^linx,.*',
'^lltc,.*', '^logicpd,.*', '^logictechno,.*', '^longcheer,.*', '^loongson,.*',
'^lsi,.*', '^lwn,.*', '^lxa,.*', '^macnica,.*', '^mapleboard,.*',
'^marvell,.*', '^maxbotix,.*', '^maxim,.*', '^mbvl,.*', '^mcube,.*',
'^meas,.*', '^mecer,.*', '^mediatek,.*', '^megachips,.*', '^mele,.*',
'^melexis,.*', '^melfas,.*', '^mellanox,.*', '^memsic,.*', '^menlo,.*',
'^merrii,.*', '^micrel,.*', '^microchip,.*', '^microcrystal,.*', '^micron,.*',
'^microsoft,.*', '^mikroe,.*', '^mikrotik,.*', '^miniand,.*', '^minix,.*',
'^miramems,.*', '^mitsubishi,.*', '^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,.*', '^okaya,.*',
'^oki,.*', '^olimex,.*', '^olpc,.*', '^onion,.*', '^onnn,.*', '^ontat,.*',
'^opalkelly,.*', '^opencores,.*', '^openrisc,.*', '^option,.*', '^oranth,.*',
'^orisetech,.*', '^ortustech,.*', '^osddisplays,.*', '^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,.*', '^powervr,.*',
'^primux,.*', '^probox2,.*', '^prt,.*', '^pulsedlight,.*', '^purism,.*',
'^qca,.*', '^qcom,.*', '^qemu,.*', '^qi,.*', '^qiaodian,.*', '^qihua,.*',
'^qnap,.*', '^radxa,.*', '^raidsonic,.*', '^ralink,.*', '^ramtron,.*',
'^raspberrypi,.*', '^raydium,.*', '^rda,.*', '^realtek,.*', '^renesas,.*',
'^rervision,.*', '^richtek,.*', '^ricoh,.*', '^rikomagic,.*', '^riscv,.*',
'^rockchip,.*', '^rocktech,.*', '^rohm,.*', '^ronbo,.*', '^roofull,.*',
'^samsung,.*', '^samtec,.*', '^sancloud,.*', '^sandisk,.*', '^satoz,.*',
'^sbs,.*', '^schindler,.*', '^seagate,.*', '^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,.*', '^technexion,.*',
'^technologic,.*', '^techstar,.*', '^tempo,.*', '^terasic,.*', '^tfc,.*',
'^thine,.*', '^thingyjp,.*', '^ti,.*', '^tianma,.*', '^tlm,.*', '^tmt,.*',
'^topeet,.*', '^toppoly,.*', '^topwise,.*', '^toradex,.*', '^toshiba,.*',
'^toumaz,.*', '^tpk,.*', '^tplink,.*', '^tpo,.*', '^tq,.*', '^tronfy,.*',
'^tronsmart,.*', '^truly,.*', '^tsd,.*', '^tyan,.*', '^u-blox,.*',
'^u-boot,.*', '^ubnt,.*', '^ucrobotics,.*', '^udoo,.*', '^ugoos,.*',
'^uniwest,.*', '^upisemi,.*', '^urt,.*', '^usi,.*', '^utoo,.*', '^v3,.*',
'^vaisala,.*', '^vamrs,.*', '^variscite,.*', '^via,.*', '^videostrong,.*',
'^virtio,.*', '^vishay,.*', '^visionox,.*', '^vitesse,.*', '^vivante,.*',
'^vocore,.*', '^voipac,.*', '^vot,.*', '^vxt,.*', '^waveshare,.*', '^wd,.*',
'^we,.*', '^wetek,.*', '^wexler,.*', '^whwave,.*', '^wi2wi,.*', '^winbond,.*',
'^winstar,.*', '^wits,.*', '^wlf,.*', '^wm,.*', '^wobo,.*', '^x-powers,.*',
'^xes,.*', '^xiaomi,.*', '^xillybus,.*', '^xingbangda,.*', '^xinpeng,.*',
'^xlnx,.*', '^xnano,.*', '^xunlong,.*', '^xylon,.*', '^ylm,.*', '^yna,.*',
'^yones-toptech,.*', '^ysoft,.*', '^zarlink,.*', '^zeitec,.*', '^zidoo,.*',
'^zii,.*', '^zte,.*', '^zyxel,.*'
From schema:
/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/vendor-prefixes.yaml
See https://patchwork.ozlabs.org/patch/1358325
If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure dt-schema is up to date:
pip3 install git+https://github.com/devicetree-org/dt-schema.git@master
--upgrade
Please check and re-submit.
