[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <c82a3843-1b58-46ea-994a-8350544aa30d@microchip.com>
Date: Fri, 9 Feb 2024 15:02:25 +0000
From: <Dharma.B@...rochip.com>
To: <krzk@...nel.org>, <andrzej.hajda@...el.com>, <neil.armstrong@...aro.org>,
<rfoss@...nel.org>, <Laurent.pinchart@...asonboard.com>, <jonas@...boo.se>,
<jernej.skrabec@...il.com>, <maarten.lankhorst@...ux.intel.com>,
<mripard@...nel.org>, <tzimmermann@...e.de>, <airlied@...il.com>,
<daniel@...ll.ch>, <robh+dt@...nel.org>, <krzysztof.kozlowski+dt@...aro.org>,
<conor+dt@...nel.org>, <Manikandan.M@...rochip.com>,
<linux-kernel@...r.kernel.org>, <dri-devel@...ts.freedesktop.org>,
<linux@...linux.org.uk>, <Nicolas.Ferre@...rochip.com>,
<alexandre.belloni@...tlin.com>, <claudiu.beznea@...on.dev>,
<geert+renesas@...der.be>, <arnd@...db.de>, <palmer@...osinc.com>,
<akpm@...ux-foundation.org>, <gerg@...ux-m68k.org>, <rdunlap@...radead.org>,
<vbabka@...e.cz>, <linux-arm-kernel@...ts.infradead.org>
CC: <robh@...nel.org>
Subject: Re: [PATCH v3 1/4] dt-bindings: display: bridge: add sam9x75-lvds
compatible
On 09/02/24 7:50 pm, Dharma B wrote:
> On 08/02/24 2:31 pm, Krzysztof Kozlowski wrote:
>> EXTERNAL EMAIL: Do not click links or open attachments unless you know
>> the content is safe
>>
>> On 07/02/2024 11:27, Dharma Balasubiramani wrote:
>>> Add the 'sam9x75-lvds' compatible binding, which describes the Low
>>> Voltage
>>> Differential Signaling (LVDS) Controller found on some Microchip's
>>> sam9x7
>>> series System-on-Chip (SoC) devices. This binding will be used to define
>>> the properties and configuration for the LVDS Controller in DT.
>>>
>>> Signed-off-by: Dharma Balasubiramani <dharma.b@...rochip.com>
>>
>> Not tested...
>>
>> Please use scripts/get_maintainers.pl to get a list of necessary people
>> and lists to CC. It might happen, that command when run on an older
>> kernel, gives you outdated entries. Therefore please be sure you base
>> your patches on recent Linux kernel.
>>
>> Tools like b4 or scripts/get_maintainer.pl provide you proper list of
>> people, so fix your workflow. Tools might also fail if you work on some
>> ancient tree (don't, instead use mainline), work on fork of kernel
>> (don't, instead use mainline) or you ignore some maintainers (really
>> don't). Just use b4 and everything should be fine, although remember
>> about `b4 prep --auto-to-cc` if you added new patches to the patchset.
>>
>> You missed at least devicetree list (maybe more), so this won't be
>> tested by automated tooling. Performing review on untested code might be
>> a waste of time.
>
> Apologies for the oversight, somehow it got missed.
The get_maintainer.pl seems to be inconsistent with the results.
linux$ ./scripts/get_maintainer.pl *patch | wc -l
./scripts/get_maintainer.pl: file '0000-cover-letter.patch' doesn't
appear to be a patch. Add -f to options?
31
linux$ ./scripts/get_maintainer.pl *patch | wc -l
./scripts/get_maintainer.pl: file '0000-cover-letter.patch' doesn't
appear to be a patch. Add -f to options?
29
linux$ ./scripts/get_maintainer.pl *patch | wc -l
./scripts/get_maintainer.pl: file '0000-cover-letter.patch' doesn't
appear to be a patch. Add -f to options?
30
linux$ ./scripts/get_maintainer.pl *patch | wc -l
./scripts/get_maintainer.pl: file '0000-cover-letter.patch' doesn't
appear to be a patch. Add -f to options?
30
Anyway, I will add the largest recipients list in v4.
>>
>> Please kindly resend and include all necessary To/Cc entries.
>
> I will resend the series including all necessary To/Cc entries.
>
> Thanks.
>>
>>
>> Best regards,
>> Krzysztof
>>
>
--
With Best Regards,
Dharma B.
Powered by blists - more mailing lists