On 2021/07/23 11:18, Thomas Monjalon wrote: > 16/07/2021 10:35, Heinrich Kuhn: >> This patch set restructures the NFP PMD, aligning it more with the >> common layout adopted by most other PMD's. Although the changes look >> fairly large, functionally nothing is added or removed from the driver >> and the existing code is mostly just reorganized into the familiar >> structure seen in other PMD's. > > It seems this refactoring is destroying this firmware improvement: > https://git.dpdk.org/dpdk/commit/?id=40edb9c0d36b781 > > Please rebase carefully, keeping the behaviour really intact. > >> Apart form adopting the common PMD layout >> this change should also aid in future feature development to the NFP >> PMD. The previous layout where most of the logic resided in a single >> file (nfp_net.c) would have become tedious to support going forward. > > Thanks Thomas, I actually finished this work earlier in the month and just missed your firmware helper addition. I posted v3 of this rework
- [dpdk-dev] [PATCH 7/7] net/nfp: batch file rename for cons... Heinrich Kuhn
- [dpdk-dev] [PATCH v2 0/7] Refactor the NFP PMD Heinrich Kuhn
- [dpdk-dev] [PATCH v2 1/7] net/nfp: split rxtx headers... Heinrich Kuhn
- [dpdk-dev] [PATCH v2 2/7] net/nfp: move rxtx function... Heinrich Kuhn
- [dpdk-dev] [PATCH v2 3/7] net/nfp: move CPP bridge to... Heinrich Kuhn
- [dpdk-dev] [PATCH v2 4/7] net/nfp: prototype common f... Heinrich Kuhn
- [dpdk-dev] [PATCH v2 5/7] net/nfp: move VF functions ... Heinrich Kuhn
- [dpdk-dev] [PATCH v2 6/7] net/nfp: move PF functions ... Heinrich Kuhn
- [dpdk-dev] [PATCH v2 7/7] net/nfp: batch file rename ... Heinrich Kuhn
- Re: [dpdk-dev] [PATCH v2 0/7] Refactor the NFP PMD Thomas Monjalon
- Re: [dpdk-dev] [PATCH v2 0/7] Refactor the NFP PM... Heinrich Kuhn
- [dpdk-dev] [PATCH v3 0/7] Refactor the NFP PMD Heinrich Kuhn
- [dpdk-dev] [PATCH v3 1/7] net/nfp: split rxtx hea... Heinrich Kuhn
- [dpdk-dev] [PATCH v3 2/7] net/nfp: move rxtx func... Heinrich Kuhn
- [dpdk-dev] [PATCH v3 3/7] net/nfp: move CPP bridg... Heinrich Kuhn
- [dpdk-dev] [PATCH v3 4/7] net/nfp: prototype comm... Heinrich Kuhn
- Re: [dpdk-dev] [PATCH v3 4/7] net/nfp: protot... Ferruh Yigit
- Re: [dpdk-dev] [PATCH v3 4/7] net/nfp: pr... Ferruh Yigit
- [dpdk-dev] [PATCH v3 5/7] net/nfp: move VF functi... Heinrich Kuhn
- [dpdk-dev] [PATCH v3 6/7] net/nfp: move PF functi... Heinrich Kuhn
- [dpdk-dev] [PATCH v3 7/7] net/nfp: batch file ren... Heinrich Kuhn