From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E3936C4332F for ; Sat, 4 Nov 2023 16:11:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230461AbjKDQLM (ORCPT ); Sat, 4 Nov 2023 12:11:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44552 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229453AbjKDQLK (ORCPT ); Sat, 4 Nov 2023 12:11:10 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DC711FB for ; Sat, 4 Nov 2023 09:11:07 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5DE55C433C7; Sat, 4 Nov 2023 16:10:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1699114267; bh=QwQFuRelGSR+6v2zVom1dHUhiePAW6pQq8oZghmzFm0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=laAuGyzQQlfuQNG/s0CF2wmFpT2EHOgpjWwvij0aepg8Nc1bYhXbTeIuRRZl6Zn0+ RnfU6uV8QSpmaaizLV1SpnH5AjJK0VxnjTA3RTbzJnjnTlvySwJNcn8CgHBdwetv+F FW0JzUosevzlhhcwa0KXkN+Fj6mCHfvdUnwHtVhcUiBVCzV1q5R9gFUV3FN1U1hgw5 o3tmN1OwsigGvMY5Z8/XDufT3i9RulgHS/AGOaDPbij6KYA7mNinP/cmuaNbuE43yk MCTw0aX+6Mv7XGeH3MFtL+/lz3tn9I3zEZF0NkBV5qIH9tQUfj+iDbJYHFawMFzef8 /L4UBdbWSeBoQ== Date: Sat, 4 Nov 2023 12:10:52 -0400 From: Simon Horman To: Ronald Wahl Cc: linux-kernel@vger.kernel.org, netdev@vger.kernel.org, Grygorii Strashko , Dan Carpenter , Siddharth Vadapalli , Jakub Kicinski , "David S . Miller" , Roger Quadros , Ronald Wahl Subject: Re: [PATCH] net: ethernet: ti: am65-cpsw: rx_pause/tx_pause controls wrong direction Message-ID: <20231104161052.GL891380@kernel.org> References: <20231031122005.13368-1-rwahl@gmx.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231031122005.13368-1-rwahl@gmx.de> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Oct 31, 2023 at 01:20:05PM +0100, Ronald Wahl wrote: > From: Ronald Wahl > > The rx_pause flag says that whether we support receiving Pause frames. > When a Pause frame is received TX is delayed for some time. This is TX > flow control. In the same manner tx_pause is actually RX flow control. > > Signed-off-by: Ronald Wahl Hi Ronald, I am a little unclear if this patch addresses a user-visible bug, or is adding a new feature. If it is fixing a bug then it should be targeted at the net tree. It should apply cleanly there, and the tree should be noted in the subject. Subject: [PATCH net] net: ethernet: ... Also, if it is a bug fix, it should have a fixes tag, indicating the revision(s) where the problem was introduced. This to assist in backporting fixes. In this case perhaps the following is appropriate: Fixes: e8609e69470f ("net: ethernet: ti: am65-cpsw: Convert to PHYLINK") It is probably not necessary to repost to address the above. But please keep it in mind for future Networking patches. On the other hand, if this is a new feature, then it should be targeted at net-next: Subject: [PATCH net-next] net: ethernet: ... And in that case the following applies. ## Form letter - net-next-closed The merge window for v6.7 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 November 12th. RFC patches sent for review only are obviously welcome at any time. See: https://www.kernel.org/doc/html/next/process/maintainer-netdev.html#development-cycle