On Tue, 8 Sep 2020 at 21:45, Rob Herring <r...@kernel.org> wrote: > > 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 <k...@kernel.org> > > --- > > .../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
My bad, I'll send v3. Best regards, Krzysztof
