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;
Feedback-ID: i56a14606:Fastmail
User-Agent: Cyrus-JMAP/3.9.0-alpha0-624-g7714e4406d-fm-20230801.001-g7714e440
Mime-Version: 1.0
Message-Id: <ae2ad056-96de-41b7-8df4-1d9c0f5c469b@app.fastmail.com>
In-Reply-To: <5ade5d29-5d9d-0d68-c3d8-de61ca90bb81@linaro.org>
References: <20230804061811.3999129-1-danishanwar@ti.com>
 <5ade5d29-5d9d-0d68-c3d8-de61ca90bb81@linaro.org>
Date:   Fri, 04 Aug 2023 11:20:44 +0200
From:   "Arnd Bergmann" <arnd@arndb.de>
To:     "Konrad Dybcio" <konrad.dybcio@linaro.org>,
        "MD Danish Anwar" <danishanwar@ti.com>,
        "Peng Fan" <peng.fan@nxp.com>,
        =?UTF-8?Q?Rafa=C5=82_Mi=C5=82ecki?= <rafal@milecki.pl>,
        =?UTF-8?Q?N=C3=ADcolas_F=2E_R=2E_A=2E_Prado?= 
        <nfraprado@collabora.com>,
        "Krzysztof Kozlowski" <krzysztof.kozlowski@linaro.org>,
        "Geert Uytterhoeven" <geert+renesas@glider.be>,
        "Bjorn Andersson" <quic_bjorande@quicinc.com>,
        "Will Deacon" <will@kernel.org>,
        "Catalin Marinas" <catalin.marinas@arm.com>, srk@ti.com,
        "Nishanth Menon" <nm@ti.com>,
        "Vignesh Raghavendra" <vigneshr@ti.com>
Cc:     linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        Linux-OMAP <linux-omap@vger.kernel.org>
Subject: Re: [PATCH v2] arm64: defconfig: Enable PRUSS as module
Content-Type: text/plain
Precedence: bulk

On Fri, Aug 4, 2023, at 11:11, Konrad Dybcio wrote:
> On 4.08.2023 08:18, MD Danish Anwar wrote:
>> Enables PRUSS as kernel module for TI SoCs.
>> 
>> Reviewed-by: Christian Gmeiner <christian.gmeiner@gmail.com>
>> Signed-off-by: MD Danish Anwar <danishanwar@ti.com>
>> ---
> As an outsider, I have no idea what this does, and the Kconfig help
> message doesn't say much more either.
>
> Could you please add a short summary about what sort of hardware
> is driven by this driver?

Any additional information would also help, but more importantly,
Danish please use a recipient list that has only the people on 'To:'
that would actually apply the patch (Nishanth and Vignesh) and
ask them to merge it, while having everyone else on Cc.

The problem with addressing a trivial patch to a dozen people
is that often everyone will think that someone else will take
care of it. If you address a patch to just one or two maintainers,
they are more likely to either apply it or tell you if you got
the wrong person instead of ignoring.

For defconfig patches adding platform specific lines, there are
two ways this can be handled:

a) the K3 maintainers pick it up into a branch and send a
   pull request to soc@kernel.org with all the defconfig
   changes, from where I pick that up. This usually works
   best.

b) you send it to soc@kernel.org yourself and I wait for
   an Ack from the K3 maintainers so I know they agree with
   the contents as well as being bypassed as maintainers on
   this occasion.

     Arnd