lkml.org 
[lkml]   [2023]   [Aug]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v2] arm64: defconfig: Enable PRUSS as module
From
On 8/4/2023 2:50 PM, Arnd Bergmann wrote:
> 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.
>

Sure Arnd, I will only keep Nishant and Vignesh in to and move everyone
else to 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

--
Thanks and Regards,
Md Danish Anwar

\
 
 \ /
  Last update: 2023-08-04 11:47    [W:0.034 / U:1.368 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site