Re: [PATCH net-next v6] ethtool: ice: Support for RSS settings to GTP from ethtool
From: Takeru Hayasaka <hidden>
Date: 2024-02-08 07:35:02
Also in:
intel-wired-lan, linux-doc, lkml
From: Takeru Hayasaka <hidden>
Date: 2024-02-08 07:35:02
Also in:
intel-wired-lan, linux-doc, lkml
Hello Jakub-san, and all reviewers and maintainers, It has been a week since I last received any feedback on this patch. I apologize for the inconvenience during this busy period, but I would greatly appreciate it if you could continue the review. Sincerely, Takeru 2024年2月1日(木) 9:59 takeru hayasaka [off-list ref]:
Hi Jakub-san Thank you for your review. I apologize for the delay in resubmitting the patch.quoted
Could you split the uAPI changes and the driver changes to two separate commits? You should post them as a patch series, but combining them into a single commit makes the uAPI addition to easy to miss.Understood. I will split the patches and submit them separately in v7. Thanks Takeru 2024年2月1日(木) 6:13 Jakub Kicinski [off-list ref]:quoted
On Wed, 31 Jan 2024 13:46:22 +0000 Takeru Hayasaka wrote:quoted
.../device_drivers/ethernet/intel/ice.rst | 21 ++++- drivers/net/ethernet/intel/ice/ice_ethtool.c | 82 +++++++++++++++++++ drivers/net/ethernet/intel/ice/ice_flow.h | 31 +++++-- drivers/net/ethernet/intel/ice/ice_lib.c | 37 +++++++++ include/uapi/linux/ethtool.h | 48 +++++++++++Could you split the uAPI changes and the driver changes to two separate commits? You should post them as a patch series, but combining them into a single commit makes the uAPI addition to easy to miss. -- pw-bot: cr