lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <f3a7f3ba-c7c8-48d8-89d6-45d454d77183@quicinc.com>
Date: Mon, 29 Jul 2024 17:59:04 +0800
From: Tengfei Fan <quic_tengfan@...cinc.com>
To: "Aiqun Yu (Maria)" <quic_aiquny@...cinc.com>,
        Andrew Lunn
	<andrew@...n.ch>, Andrew Halaney <ahalaney@...hat.com>
CC: Vinod Koul <vkoul@...nel.org>, "David S. Miller" <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>,
        "Paolo
 Abeni" <pabeni@...hat.com>, Rob Herring <robh@...nel.org>,
        Krzysztof
 Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Bhupesh
 Sharma <bhupesh.sharma@...aro.org>,
        Alexandre Torgue
	<alexandre.torgue@...s.st.com>,
        Giuseppe Cavallaro <peppe.cavallaro@...com>,
        Jose Abreu <joabreu@...opsys.com>,
        Maxime Coquelin
	<mcoquelin.stm32@...il.com>, <kernel@...cinc.com>,
        <netdev@...r.kernel.org>, <linux-arm-msm@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <linux-stm32@...md-mailman.stormreply.com>,
        <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH v2 2/2] net: stmmac: dwmac-qcom-ethqos: add support for
 emac4 on qcs9100 platforms



On 7/10/2024 11:02 AM, Aiqun Yu (Maria) wrote:
> 
> 
> On 7/9/2024 11:33 PM, Andrew Lunn wrote:
>> On Tue, Jul 09, 2024 at 09:40:55AM -0500, Andrew Halaney wrote:
>>> These patches are for netdev, so you need to follow the netdev
>>> rules, i.e. the subject should be have [PATCH net-next] in it, etc as
>>> documented over here:
>>>
>>>      https://docs.kernel.org/process/maintainer-netdev.html#tl-dr
>>>
>>> On Tue, Jul 09, 2024 at 10:13:18PM GMT, Tengfei Fan wrote:
>>>> QCS9100 uses EMAC version 4, add the relevant defines, rename the
>>>> has_emac3 switch to has_emac_ge_3 (has emac greater-or-equal than 3)
>>>> and add the new compatible.
>>>
>>> This blurb isn't capturing what's done in this change, please make it
>>> reflect the patch.
>>
>> Hi Tengfei
>>
>> If i remember correctly, there was a similar comment made to one of
>> the patches in the huge v1 series.
>>
>> The commit messages are very important, just as important as the code
>> itself. Please review them all and fixup issues like this before you
>> repost.
> 
> Thx Andrew L for the info.
>>
>>      Andrew
>>
>> ---
>> pw-bot: cr
> 

After considering the feedback provided on the subject, We have decided
to keep current SA8775p compatible and ABI compatibility in drivers.
Let's close this session and ignore all the current patches here.
Thank you for your input.

-- 
Thx and BRs,
Tengfei Fan

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ