Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <ed853a8c-f1a5-d137-e2bb-df782d43b65e@gmail.com>
Date:   Mon, 16 May 2022 18:34:38 +0300
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.9.0
Subject: Re: [PATCH v3 2/3] dt-bindings: mtd: partitions: Extend
 fixed-partitions binding
Content-Language: en-US
To:     Miquel Raynal <miquel.raynal@bootlin.com>,
        Rob Herring <robh+dt@kernel.org>
Cc:     Richard Weinberger <richard@nod.at>,
        Vignesh Raghavendra <vigneshr@ti.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= <rafal@milecki.pl>,
        MTD Maling List <linux-mtd@lists.infradead.org>,
        devicetree@vger.kernel.org,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Karim <Karimdplay@gmail.com>, M <x1@disroot.org>
References: <20220503155007.2339847-1-csharper2005@gmail.com>
 <20220512152725.244872-1-miquel.raynal@bootlin.com>
 <CAL_JsqJdUm4p9qAq9dLeVTVC9PA2q2SP01kG2jyEb_f=Fo=bEQ@mail.gmail.com>
 <20220516102115.4ab6fada@xps13>
From:   Mikhail Zhilkin <csharper2005@gmail.com>
In-Reply-To: <20220516102115.4ab6fada@xps13>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

Hi, Miquel!

On 5/16/2022 11:21 AM, Miquel Raynal wrote:
> Hi,
>
> robh+dt@kernel.org wrote on Fri, 13 May 2022 09:12:03 -0500:
>
>> On Thu, May 12, 2022 at 10:27 AM Miquel Raynal
>> <miquel.raynal@bootlin.com> wrote:
>>> On Tue, 2022-05-03 at 15:50:07 UTC, Mikhail Zhilkin wrote:  
>>>> Extend fixed-partitions binding for support of Sercomm partition parser
>>>> (add "sercomm,sc-partitions" compatible).
>>>>
>>>> Signed-off-by: Mikhail Zhilkin <csharper2005@gmail.com>
>>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>  
>>> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/next, thanks.  
>> Patch 1 is a dependency, please apply it too. Without it, we now get
>> in linux-next:
> Only patch 3 was sent to me/the mtd ML. I'll drop the patch.
>
> Mikhail, please resend the series including us on all your patches.

Done! I sent the series to you and mtd
https://lore.kernel.org/linux-mtd/20220516151228.885222-1-csharper2005@gmail.com/T/#t

P.S. get_maintainer.pl didn't recommend to add you and mtd that and this
time. Do I need to send each patch from the series to all maintainers of
all the patches in the future?

$ scripts/get_maintainer.pl
v5-0001-dt-bindings-Add-Sercomm-Suzhou-Corporation-vendor.patch
Rob Herring <robh+dt@kernel.org> (maintainer:OPEN FIRMWARE AND FLATTENED
DEVICE TREE BINDINGS,commit_signer:59/65=91%,in file)
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org> (maintainer:OPEN
FIRMWARE AND FLATTENED DEVICE TREE
BINDINGS,commit_signer:12/65=18%,added_lines:7/139=5%,removed_lines:1/2=50%)
Shawn Guo <shawnguo@kernel.org> (commit_signer:7/65=11%)
Stanislav Jakubek <stano.jakubek@gmail.com>
(commit_signer:6/65=9%,authored:6/65=9%,added_lines:12/139=9%)
Sam Ravnborg <sam@ravnborg.org> (commit_signer:5/65=8%)
Linus Walleij <linus.walleij@linaro.org> (added_lines:10/139=7%)
Hao Fang <fanghao11@huawei.com> (removed_lines:1/2=50%)
devicetree@vger.kernel.org (open list:OPEN FIRMWARE AND FLATTENED DEVICE
TREE BINDINGS)
linux-kernel@vger.kernel.org (open list)

>> /builds/robherring/linux-dt/Documentation/devicetree/bindings/mtd/partitions/fixed-partitions.example.dtb:
>> partition@0: 'sercomm,scpart-id' does 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),.*', '^100ask,.*',
>> '^70mai,.*', '^8dev,.*', '^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,.*', '^aesop,.*', '^airoha,.*', '^al,.*',
>> '^alcatel,.*', '^allegro,.*', '^allo,.*', '^allwinner,.*',
>> '^alphascale,.*', '^alps,.*', '^alt,.*', '^altr,.*', '^amarula,.*',
>> '^amazon,.*', '^amcc,.*', '^amd,.*', '^amediatech,.*', '^amlogic,.*',
>> '^ampere,.*', '^ampire,.*', '^ams,.*', '^amstaos,.*', '^analogix,.*',
>> '^andestech,.*', '^anvo,.*', '^apm,.*', '^apple,.*', '^aptina,.*',
>> '^arasan,.*', '^archermind,.*', '^arctic,.*', '^arcx,.*', '^aries,.*',
>> '^arm,.*', '^armadeus,.*', '^arrow,.*', '^artesyn,.*',
>> '^asahi-kasei,.*', '^asc,.*', '^asix,.*', '^aspeed,.*', '^asus,.*',
>> '^atlas,.*', '^atmel,.*', '^auo,.*', '^auvidea,.*', '^avago,.*',
>> '^avia,.*', '^avic,.*', '^avnet,.*', '^awinic,.*', '^axentia,.*',
>> '^axis,.*', '^azoteq,.*', '^azw,.*', '^baikal,.*', '^bananapi,.*',
>> '^beacon,.*', '^beagle,.*', '^bhf,.*', '^bitmain,.*', '^blutek,.*',
>> '^boe,.*', '^bosch,.*', '^boundary,.*', '^brcm,.*', '^broadmobi,.*',
>> '^bsh,.*', '^bticino,.*', '^buffalo,.*', '^bur,.*', '^calamp,.*',
>> '^calaosystems,.*', '^calxeda,.*', '^canaan,.*', '^caninos,.*',
>> '^capella,.*', '^cascoda,.*', '^catalyst,.*', '^cavium,.*',
>> '^cdns,.*', '^cdtech,.*', '^cellwise,.*', '^ceva,.*',
>> '^checkpoint,.*', '^chefree,.*', '^chipidea,.*', '^chipone,.*',
>> '^chipspark,.*', '^chrontel,.*', '^chrp,.*', '^chunghwa,.*',
>> '^chuwi,.*', '^ciaa,.*', '^cirrus,.*', '^cisco,.*',
>> '^cloudengines,.*', '^cnm,.*', '^cnxt,.*', '^colorfly,.*',
>> '^compulab,.*', '^congatec,.*', '^coreriver,.*', '^corpro,.*',
>> '^cortina,.*', '^cosmic,.*', '^crane,.*', '^creative,.*',
>> '^crystalfontz,.*', '^csky,.*', '^csq,.*', '^ctera,.*', '^ctu,.*',
>> '^cubietech,.*', '^cui,.*', '^cypress,.*', '^cyx,.*', '^cznic,.*',
>> '^dallas,.*', '^dataimage,.*', '^davicom,.*', '^dell,.*', '^delta,.*',
>> '^denx,.*', '^devantech,.*', '^dfi,.*', '^dh,.*', '^difrnce,.*',
>> '^digi,.*', '^digilent,.*', '^dioo,.*', '^dlc,.*', '^dlg,.*',
>> '^dlink,.*', '^dmo,.*', '^domintech,.*', '^dongwoon,.*',
>> '^dptechnics,.*', '^dragino,.*', '^ds,.*', '^dserve,.*',
>> '^dynaimage,.*', '^ea,.*', '^ebang,.*', '^ebs-systart,.*', '^ebv,.*',
>> '^eckelmann,.*', '^edimax,.*', '^edt,.*', '^eeti,.*',
>> '^einfochips,.*', '^eink,.*', '^elan,.*', '^element14,.*',
>> '^elgin,.*', '^elida,.*', '^elimo,.*', '^elpida,.*', '^embest,.*',
>> '^emlid,.*', '^emmicro,.*', '^empire-electronix,.*', '^emtrion,.*',
>> '^enclustra,.*', '^endless,.*', '^ene,.*', '^energymicro,.*',
>> '^engicam,.*', '^engleder,.*', '^epcos,.*', '^epfl,.*', '^epson,.*',
>> '^esp,.*', '^est,.*', '^ettus,.*', '^eukrea,.*', '^everest,.*',
>> '^everspin,.*', '^evervision,.*', '^exar,.*', '^excito,.*',
>> '^exegin,.*', '^ezchip,.*', '^facebook,.*', '^fairphone,.*',
>> '^faraday,.*', '^fastrax,.*', '^fcs,.*', '^feixin,.*', '^feiyang,.*',
>> '^fii,.*', '^firefly,.*', '^focaltech,.*', '^forlinx,.*', '^frida,.*',
>> '^friendlyarm,.*', '^fsl,.*', '^fujitsu,.*', '^fxtec,.*',
>> '^gardena,.*', '^gateworks,.*', '^gcw,.*', '^ge,.*', '^geekbuying,.*',
>> '^gef,.*', '^gemei,.*', '^geniatech,.*', '^giantec,.*',
>> '^giantplus,.*', '^globalscale,.*', '^globaltop,.*', '^gmt,.*',
>> '^goodix,.*', '^google,.*', '^grinn,.*', '^grmn,.*', '^gumstix,.*',
>> '^gw,.*', '^hannstar,.*', '^haochuangyi,.*', '^haoyu,.*',
>> '^hardkernel,.*', '^hideep,.*', '^himax,.*', '^hirschmann,.*',
>> '^hisi,.*', '^hisilicon,.*', '^hit,.*', '^hitex,.*', '^holt,.*',
>> '^holtek,.*', '^honestar,.*', '^honeywell,.*', '^hoperun,.*',
>> '^hp,.*', '^hpe,.*', '^hsg,.*', '^huawei,.*', '^hugsun,.*',
>> '^hwacom,.*', '^hycon,.*', '^hydis,.*', '^hynix,.*', '^hyundai,.*',
>> '^i2se,.*', '^ibm,.*', '^icplus,.*', '^idt,.*', '^ifi,.*',
>> '^ilitek,.*', '^imagis,.*', '^img,.*', '^imi,.*', '^incircuit,.*',
>> '^inet-tek,.*', '^infineon,.*', '^inforce,.*', '^ingenic,.*',
>> '^injoinic,.*', '^innolux,.*', '^inside-secure,.*', '^insignal,.*',
>> '^inspur,.*', '^intel,.*', '^intercontrol,.*', '^invensense,.*',
>> '^inversepath,.*', '^iom,.*', '^isee,.*', '^isil,.*', '^issi,.*',
>> '^ite,.*', '^itead,.*', '^itian,.*', '^ivo,.*', '^iwave,.*',
>> '^jdi,.*', '^jedec,.*', '^jesurun,.*', '^jethome,.*', '^jianda,.*',
>> '^joz,.*', '^kam,.*', '^karo,.*', '^keithkoep,.*', '^keymile,.*',
>> '^khadas,.*', '^kiebackpeter,.*', '^kinetic,.*', '^kingdisplay,.*',
>> '^kingnovel,.*', '^kionix,.*', '^kobo,.*', '^kobol,.*', '^koe,.*',
>> '^kontron,.*', '^kosagi,.*', '^kvg,.*', '^kyo,.*', '^lacie,.*',
>> '^laird,.*', '^lamobo,.*', '^lantiq,.*', '^lattice,.*', '^leadtek,.*',
>> '^leez,.*', '^lego,.*', '^lemaker,.*', '^lenovo,.*', '^lg,.*',
>> '^lgphilips,.*', '^libretech,.*', '^licheepi,.*', '^linaro,.*',
>> '^linksprite,.*', '^linksys,.*', '^linutronix,.*', '^linux,.*',
>> '^linx,.*', '^liteon,.*', '^litex,.*', '^lltc,.*', '^logicpd,.*',
>> '^logictechno,.*', '^longcheer,.*', '^lontium,.*', '^loongson,.*',
>> '^lsi,.*', '^lwn,.*', '^lxa,.*', '^m5stack,.*', '^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,.*', '^miyoo,.*', '^mntre,.*',
>> '^modtronix,.*', '^mosaixtech,.*', '^motorola,.*', '^moxa,.*',
>> '^mpl,.*', '^mps,.*', '^mqmaker,.*', '^mrvl,.*', '^mscc,.*',
>> '^msi,.*', '^mstar,.*', '^mti,.*', '^multi-inno,.*',
>> '^mundoreader,.*', '^murata,.*', '^mxic,.*', '^mxicy,.*', '^myir,.*',
>> '^national,.*', '^nec,.*', '^neonode,.*', '^netgear,.*',
>> '^netlogic,.*', '^netron-dy,.*', '^netronix,.*', '^netxeon,.*',
>> '^neweast,.*', '^newhaven,.*', '^nexbox,.*', '^nextthing,.*',
>> '^ni,.*', '^nintendo,.*', '^nlt,.*', '^nokia,.*', '^nordic,.*',
>> '^novtech,.*', '^nutsboard,.*', '^nuvoton,.*', '^nvd,.*',
>> '^nvidia,.*', '^nxp,.*', '^oceanic,.*', '^ocs,.*', '^oct,.*',
>> '^okaya,.*', '^oki,.*', '^olimex,.*', '^olpc,.*', '^oneplus,.*',
>> '^onion,.*', '^onnn,.*', '^ontat,.*', '^opalkelly,.*',
>> '^opencores,.*', '^openembed,.*', '^openrisc,.*', '^option,.*',
>> '^oranth,.*', '^orisetech,.*', '^ortustech,.*', '^osddisplays,.*',
>> '^osmc,.*', '^ouya,.*', '^overkiz,.*', '^ovti,.*', '^oxsemi,.*',
>> '^ozzmaker,.*', '^panasonic,.*', '^parade,.*', '^parallax,.*',
>> '^pda,.*', '^pericom,.*', '^pervasive,.*', '^phicomm,.*',
>> '^phytec,.*', '^picochip,.*', '^pine64,.*', '^pineriver,.*',
>> '^pixcir,.*', '^plantower,.*', '^plathome,.*', '^plda,.*', '^plx,.*',
>> '^ply,.*', '^pni,.*', '^pocketbook,.*', '^polaroid,.*',
>> '^portwell,.*', '^poslab,.*', '^pov,.*', '^powertip,.*',
>> '^powervr,.*', '^primux,.*', '^probox2,.*', '^prt,.*',
>> '^pulsedlight,.*', '^purism,.*', '^qca,.*', '^qcom,.*', '^qemu,.*',
>> '^qi,.*', '^qiaodian,.*', '^qihua,.*', '^qishenglong,.*', '^qnap,.*',
>> '^radxa,.*', '^raidsonic,.*', '^ralink,.*', '^ramtron,.*',
>> '^raspberrypi,.*', '^raydium,.*', '^rda,.*', '^realtek,.*',
>> '^remarkable,.*', '^renesas,.*', '^rervision,.*', '^revotics,.*',
>> '^rex,.*', '^richtek,.*', '^ricoh,.*', '^rikomagic,.*', '^riot,.*',
>> '^riscv,.*', '^rockchip,.*', '^rocktech,.*', '^rohm,.*', '^ronbo,.*',
>> '^roofull,.*', '^roseapplepi,.*', '^samsung,.*', '^samtec,.*',
>> '^sancloud,.*', '^sandisk,.*', '^satoz,.*', '^sbs,.*',
>> '^schindler,.*', '^seagate,.*', '^seeed,.*', '^seirobotics,.*',
>> '^semtech,.*', '^senseair,.*', '^sensirion,.*', '^sensortek,.*',
>> '^sff,.*', '^sgd,.*', '^sgmicro,.*', '^sgx,.*', '^sharp,.*',
>> '^shimafuji,.*', '^shiratech,.*', '^si-en,.*', '^si-linux,.*',
>> '^siemens,.*', '^sifive,.*', '^sigma,.*', '^sii,.*', '^sil,.*',
>> '^silabs,.*', '^silan,.*', '^silead,.*', '^silergy,.*',
>> '^silex-insight,.*', '^siliconfile,.*', '^siliconmitus,.*',
>> '^silvaco,.*', '^simtek,.*', '^sinlinx,.*', '^sinovoip,.*',
>> '^sinowealth,.*', '^sipeed,.*', '^sirf,.*', '^sis,.*', '^sitronix,.*',
>> '^skov,.*', '^skyworks,.*', '^smartlabs,.*', '^smsc,.*', '^snps,.*',
>> '^sochip,.*', '^socionext,.*', '^solidrun,.*', '^solomon,.*',
>> '^sony,.*', '^spansion,.*', '^sparkfun,.*', '^spinalhdl,.*',
>> '^sprd,.*', '^ssi,.*', '^sst,.*', '^sstar,.*', '^st,.*',
>> '^st-ericsson,.*', '^starfive,.*', '^starry,.*', '^startek,.*',
>> '^ste,.*', '^stericsson,.*', '^storlink,.*', '^storm,.*',
>> '^storopack,.*', '^summit,.*', '^sunchip,.*', '^sunplus,.*',
>> '^supermicro,.*', '^swir,.*', '^syna,.*', '^synology,.*',
>> '^synopsys,.*', '^tbs,.*', '^tbs-biometrics,.*', '^tcg,.*', '^tcl,.*',
>> '^tcs,.*', '^tdo,.*', '^team-source-display,.*', '^technexion,.*',
>> '^technologic,.*', '^techstar,.*', '^teltonika,.*', '^tempo,.*',
>> '^terasic,.*', '^tesla,.*', '^tfc,.*', '^thead,.*', '^thine,.*',
>> '^thingyjp,.*', '^thundercomm,.*', '^ti,.*', '^tianma,.*', '^tlm,.*',
>> '^tmt,.*', '^topeet,.*', '^topic,.*', '^toppoly,.*', '^topwise,.*',
>> '^toradex,.*', '^toshiba,.*', '^toumaz,.*', '^tpk,.*', '^tplink,.*',
>> '^tpo,.*', '^tq,.*', '^traverse,.*', '^tronfy,.*', '^tronsmart,.*',
>> '^truly,.*', '^tsd,.*', '^tyan,.*', '^u-blox,.*', '^u-boot,.*',
>> '^ubnt,.*', '^ucrobotics,.*', '^udoo,.*', '^ugoos,.*', '^uniwest,.*',
>> '^upisemi,.*', '^urt,.*', '^usi,.*', '^utoo,.*', '^v3,.*',
>> '^vaisala,.*', '^vamrs,.*', '^variscite,.*', '^vdl,.*',
>> '^vertexcom,.*', '^via,.*', '^vicor,.*', '^videostrong,.*',
>> '^virtio,.*', '^virtual,.*', '^vishay,.*', '^visionox,.*',
>> '^vitesse,.*', '^vivante,.*', '^vivax,.*', '^vocore,.*', '^voipac,.*',
>> '^vot,.*', '^vxt,.*', '^wanchanglong,.*', '^wand,.*', '^waveshare,.*',
>> '^wd,.*', '^we,.*', '^welltech,.*', '^wetek,.*', '^wexler,.*',
>> '^whwave,.*', '^wi2wi,.*', '^wiligear,.*', '^willsemi,.*',
>> '^winbond,.*', '^wingtech,.*', '^winlink,.*', '^winstar,.*',
>> '^wirelesstag,.*', '^wits,.*', '^wlf,.*', '^wm,.*', '^wobo,.*',
>> '^x-powers,.*', '^xen,.*', '^xes,.*', '^xiaomi,.*', '^xillybus,.*',
>> '^xingbangda,.*', '^xinpeng,.*', '^xiphera,.*', '^xlnx,.*',
>> '^xnano,.*', '^xunlong,.*', '^xylon,.*', '^yadro,.*', '^yamaha,.*',
>> '^yes-optoelectronics,.*', '^yic,.*', '^ylm,.*', '^yna,.*',
>> '^yones-toptech,.*', '^ys,.*', '^ysoft,.*', '^zarlink,.*',
>> '^zealz,.*', '^zeitec,.*', '^zidoo,.*', '^zii,.*', '^zinitix,.*',
>> '^zkmagic,.*', '^zte,.*', '^zyxel,.*'
>>   From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/vendor-prefixes.yaml
>>
>> Rob
>
> Thanks,
> Miquèl

-- 
Best regards,
Mikhail