DPDK patches and discussions
 help / color / mirror / Atom feed
From: 王颢 <howard_wang@realsil.com.cn>
To: Ferruh Yigit <ferruh.yigit@amd.com>, "dev@dpdk.org" <dev@dpdk.org>
Cc: "pro_nic_dpdk@realtek.com" <pro_nic_dpdk@realtek.com>
Subject: 答复: [PATCH v5 00/18] modify code as suggested by the maintainer
Date: Mon, 4 Nov 2024 02:05:34 +0000	[thread overview]
Message-ID: <1829a53dee504477a9cdb09e1568d133@realsil.com.cn> (raw)
In-Reply-To: <f35ede55-3600-4119-ba4c-b3a413604233@amd.com>

Dear Ferruh,

Thank you for your comments. I will make the changes you suggested. Our DPDK PMD is developed based on our company’s Linux kernel drivers r8125 and r8126, which are more comprehensive and detailed. To facilitate synchronized maintenance, I have retained the CamelCase.

Best Regards,
Howard Wang

-----邮件原件-----
发件人: Ferruh Yigit <ferruh.yigit@amd.com> 
发送时间: 2024年11月3日 10:24
收件人: 王颢 <howard_wang@realsil.com.cn>; dev@dpdk.org
抄送: pro_nic_dpdk@realtek.com
主题: Re: [PATCH v5 00/18] modify code as suggested by the maintainer


External mail.



On 10/28/2024 7:30 AM, Howard Wang wrote:
> Fix some warning issues.
>
> Howard Wang (18):
>   net/r8169: add PMD driver skeleton
>   net/r8169: add logging structure
>   net/r8169: add hardware registers access routines
>   net/r8169: implement core logic for Tx/Rx
>   net/r8169: add support for hw config
>   net/r8169: add phy registers access routines
>   net/r8169: add support for hardware operations
>   net/r8169: add support for phy configuration
>   net/r8169: add support for hw initialization
>   net/r8169: add link status and interrupt management
>   net/r8169: implement Rx path
>   net/r8169: implement Tx path
>   net/r8169: implement device statistics
>   net/r8169: implement promisc and allmulti modes
>   net/r8169: implement MTU configuration
>   net/r8169: add support for getting fw version
>   net/r8169: add driver_start and driver_stop
>   doc/guides/nics: add documents for r8169 pmd
>

Hi Howard,

Thanks for restructuring the patch series, it looks better now, I put some comments.

Btw, there are many "Avoid CamelCase" checkpatch warning, if this code is not shared (but developed for DPDK), can you please address the warnings?

  reply	other threads:[~2024-11-04  2:05 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-28  7:30 Howard Wang
2024-10-28  7:30 ` [PATCH v5 01/18] net/r8169: add PMD driver skeleton Howard Wang
2024-11-03  2:17   ` Ferruh Yigit
2024-11-04  9:43     ` 答复: " 王颢
2024-11-05 17:30       ` Ferruh Yigit
2024-10-28  7:30 ` [PATCH v5 02/18] net/r8169: add logging structure Howard Wang
2024-10-28  7:30 ` [PATCH v5 03/18] net/r8169: add hardware registers access routines Howard Wang
2024-11-03  2:18   ` Ferruh Yigit
2024-10-28  7:30 ` [PATCH v5 04/18] net/r8169: implement core logic for Tx/Rx Howard Wang
2024-10-28  7:30 ` [PATCH v5 05/18] net/r8169: add support for hw config Howard Wang
2024-10-28  7:31 ` [PATCH v5 06/18] net/r8169: add phy registers access routines Howard Wang
2024-10-28  7:31 ` [PATCH v5 07/18] net/r8169: add support for hardware operations Howard Wang
2024-10-28  7:31 ` [PATCH v5 08/18] net/r8169: add support for phy configuration Howard Wang
2024-10-28  7:31 ` [PATCH v5 09/18] net/r8169: add support for hw initialization Howard Wang
2024-11-03  2:19   ` Ferruh Yigit
2024-10-28  7:31 ` [PATCH v5 10/18] net/r8169: add link status and interrupt management Howard Wang
2024-10-28  7:31 ` [PATCH v5 11/18] net/r8169: implement Rx path Howard Wang
2024-10-28  7:31 ` [PATCH v5 12/18] net/r8169: implement Tx path Howard Wang
2024-10-28  7:31 ` [PATCH v5 13/18] net/r8169: implement device statistics Howard Wang
2024-10-28  7:31 ` [PATCH v5 14/18] net/r8169: implement promisc and allmulti modes Howard Wang
2024-10-28  7:31 ` [PATCH v5 15/18] net/r8169: implement MTU configuration Howard Wang
2024-10-28  7:31 ` [PATCH v5 16/18] net/r8169: add support for getting fw version Howard Wang
2024-10-28  7:31 ` [PATCH v5 17/18] net/r8169: add driver_start and driver_stop Howard Wang
2024-10-28  7:31 ` [PATCH v5 18/18] doc/guides/nics: add documents for r8169 pmd Howard Wang
2024-11-03  2:20   ` Ferruh Yigit
2024-11-03  2:23 ` [PATCH v5 00/18] modify code as suggested by the maintainer Ferruh Yigit
2024-11-04  2:05   ` 王颢 [this message]
2024-11-04  8:50     ` 答复: " Ferruh Yigit

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=1829a53dee504477a9cdb09e1568d133@realsil.com.cn \
    --to=howard_wang@realsil.com.cn \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@amd.com \
    --cc=pro_nic_dpdk@realtek.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).