Message-ID: <1107bc10-9b14-98f4-3e47-f87188453ce7@collabora.com>
Date:   Tue, 7 Mar 2023 21:53:48 +0500
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.7.2
Cc:     Muhammad Usama Anjum <usama.anjum@collabora.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Ariel Elior <aelior@marvell.com>,
        Manish Chopra <manishc@marvell.com>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Paolo Abeni <pabeni@redhat.com>, kernel@collabora.com,
        kernel-janitors@vger.kernel.org, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] qede: remove linux/version.h and linux/compiler.h
Content-Language: en-US
To:     Simon Horman <simon.horman@corigine.com>
References: <20230303185351.2825900-1-usama.anjum@collabora.com>
 <20230303155436.213ee2c0@kernel.org>
 <df8a446a-e8a9-3b3d-fd0f-791f0d01a0c9@collabora.com>
 <ZAdoivY94Y5dfOa4@corigine.com>
From:   Muhammad Usama Anjum <usama.anjum@collabora.com>
In-Reply-To: <ZAdoivY94Y5dfOa4@corigine.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Precedence: bulk

On 3/7/23 9:38 PM, Simon Horman wrote:
> On Tue, Mar 07, 2023 at 06:39:20PM +0500, Muhammad Usama Anjum wrote:
>> On 3/4/23 4:54 AM, Jakub Kicinski wrote:
>>> On Fri,  3 Mar 2023 23:53:50 +0500 Muhammad Usama Anjum wrote:
>>>> make versioncheck reports the following:
>>>> ./drivers/net/ethernet/qlogic/qede/qede.h: 10 linux/version.h not needed.
>>>> ./drivers/net/ethernet/qlogic/qede/qede_ethtool.c: 7 linux/version.h not needed.
>>>>
>>>> So remove linux/version.h from both of these files. Also remove
>>>> linux/compiler.h while at it as it is also not being used.
>>>>
>>>> Signed-off-by: Muhammad Usama Anjum <usama.anjum@collabora.com>
>>>
>>> # Form letter - net-next is closed
>>>
>>> The merge window for v6.3 has begun and therefore net-next is closed
>>> for new drivers, features, code refactoring and optimizations.
>>> We are currently accepting bug fixes only.
>>>
>>> Please repost when net-next reopens after Mar 6th.
>> It is Mar 7th. Please review.
> 
> I think that the way it works is that you need to repost the patch.
> Probably with REPOST in the subject if it is unchanged:Sorry, I didn't know. I'll repost it.

> 
> Subject: [PATCH net-next repost v2] ...
> 
> Or bumped to v3 if there are changes.
> 
> Subject: [PATCH net-next v3] ...
> 
> Also, as per the examples above, the target tree, in this case
> 'net-next' should be included in the subject.
I don't know much about net tree and its location. This is why people use
linux-next for sending patches. I'm not sure about the networking sub
system. Would it be accepted if I send it against linux-next as in [PATCH
linux-next repost v2]?


-- 
BR,
Muhammad Usama Anjum