All of lore.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzk@kernel.org>
To: Yangfl <mmyangfl@gmail.com>
Cc: netdev@vger.kernel.org, Andrew Lunn <andrew@lunn.ch>,
	Vladimir Oltean <olteanv@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Simon Horman <horms@kernel.org>,
	Russell King <linux@armlinux.org.uk>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH net-next v6 1/3] dt-bindings: net: dsa: yt921x: Add Motorcomm YT921x switch support
Date: Mon, 25 Aug 2025 09:37:34 +0200	[thread overview]
Message-ID: <96b6f650-fa4d-461b-91f6-cb210cb1982d@kernel.org> (raw)
In-Reply-To: <CAAXyoMMej5XJmofiY_9cpzscxm2GyZw_v9hcSYu7NvhcNFFV2Q@mail.gmail.com>

On 25/08/2025 04:39, Yangfl wrote:
> On Mon, Aug 25, 2025 at 2:09 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>
>> On 24/08/2025 11:25, Yangfl wrote:
>>> On Sun, Aug 24, 2025 at 5:20 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>>>>
>>>> On Sun, Aug 24, 2025 at 08:51:09AM +0800, David Yang wrote:
>>>>> The Motorcomm YT921x series is a family of Ethernet switches with up to
>>>>> 8 internal GbE PHYs and up to 2 GMACs.
>>>>>
>>>>> Signed-off-by: David Yang <mmyangfl@gmail.com>
>>>>> ---
>>>>
>>>> <form letter>
>>>> This is a friendly reminder during the review process.
>>>>
>>>> It looks like you received a tag and forgot to add it.
>>>>
>>>> If you do not know the process, here is a short explanation:
>>>> Please add Acked-by/Reviewed-by/Tested-by tags when posting new
>>>> versions of patchset, under or above your Signed-off-by tag, unless
>>>> patch changed significantly (e.g. new properties added to the DT
>>>> bindings). Tag is "received", when provided in a message replied to you
>>>> on the mailing list. Tools like b4 can help here. However, there's no
>>>> need to repost patches *only* to add the tags. The upstream maintainer
>>>> will do that for tags received on the version they apply.
>>>>
>>>> Please read:
>>>> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577
>>>>
>>>> *If a tag was not added on purpose, please state why* and what changed.
>>>> </form letter>
>>>>
>>>>
>>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>>>
>>>> Best regards,
>>>> Krzysztof
>>>>
>>>
>>> Thanks.
>>>
>>>>  - use enum for reg in dt binding
>>>
>>> I made a change in dt binding. If you are fine with that change, I'll
>>> add the tag in the following versions (if any).
>>
>>
>> Cover letter must state the reason.
>>
>>
>> Best regards,
>> Krzysztof
> 
> as requested in the previous version
> 
> https://lore.kernel.org/r/f76df98e-f743-4dc2-9f10-93b97f69addb@lunn.ch

Hm? Where is there a statement that you remove review because of this
and that?


Best regards,
Krzysztof

  reply	other threads:[~2025-08-25  7:37 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-24  0:51 [PATCH net-next v6 0/3] net: dsa: yt921x: Add support for Motorcomm YT921x David Yang
2025-08-24  0:51 ` [PATCH net-next v6 1/3] dt-bindings: net: dsa: yt921x: Add Motorcomm YT921x switch support David Yang
2025-08-24  9:20   ` Krzysztof Kozlowski
2025-08-24  9:25     ` Yangfl
2025-08-24 17:16       ` Krzysztof Kozlowski
2025-08-25  2:39         ` Yangfl
2025-08-25  7:37           ` Krzysztof Kozlowski [this message]
2025-08-24  0:51 ` [PATCH net-next v6 2/3] net: dsa: tag_yt921x: add support for Motorcomm YT921x tags David Yang
2025-08-25 22:15   ` Vladimir Oltean
2025-08-26  1:47     ` Yangfl
2025-08-26  2:18       ` Andrew Lunn
2025-08-26  2:44         ` Yangfl
2025-08-26 14:38       ` Vladimir Oltean
2025-08-24  0:51 ` [PATCH net-next v6 3/3] net: dsa: yt921x: Add support for Motorcomm YT921x David Yang
2025-08-24  8:42   ` Russell King (Oracle)
2025-08-24  9:31     ` Yangfl
2025-08-24 15:22   ` Andrew Lunn
2025-08-24 15:26   ` Andrew Lunn
2025-08-24 15:34   ` Andrew Lunn
2025-08-24 15:51   ` Andrew Lunn
2025-08-24 16:38     ` Yangfl
2025-08-24 18:14       ` Russell King (Oracle)
2025-08-24 19:16         ` Andrew Lunn
2025-08-25 14:14         ` Yangfl
2025-08-26 19:07           ` Vladimir Oltean
2025-08-24 19:14       ` Andrew Lunn
2025-08-24 16:40   ` kernel test robot
2025-08-25 21:23   ` Vladimir Oltean
2025-08-26  5:19     ` Yangfl
2025-08-26 12:15       ` Andrew Lunn
2025-08-26 13:51       ` Vladimir Oltean

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=96b6f650-fa4d-461b-91f6-cb210cb1982d@kernel.org \
    --to=krzk@kernel.org \
    --cc=andrew@lunn.ch \
    --cc=conor+dt@kernel.org \
    --cc=davem@davemloft.net \
    --cc=devicetree@vger.kernel.org \
    --cc=edumazet@google.com \
    --cc=horms@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=kuba@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=mmyangfl@gmail.com \
    --cc=netdev@vger.kernel.org \
    --cc=olteanv@gmail.com \
    --cc=pabeni@redhat.com \
    --cc=robh@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.