On Fri, Jan 31, 2025 at 12:58:13PM +0000, Anatoly Burakov wrote: > Intel IGC and E1000 drivers are distinct, but they are actually generated > from the same base code. This patchset will merge together all e1000-derived > drivers into one common base, with three different ethdev driver > frontends (EM, IGB, and IGC). > > After the merge is done, base code is also updated to latest snapshot. > <snip> > drivers/net/intel/e1000/base/README | 8 +- > .../net/intel/e1000/base/e1000_80003es2lan.c | 10 +- > drivers/net/intel/e1000/base/e1000_82571.c | 4 +- <snip> > drivers/net/intel/igc/base/igc_phy.c | 4420 ----------------- > drivers/net/intel/igc/base/igc_phy.h | 337 -- > drivers/net/intel/igc/base/igc_regs.h | 732 --- > drivers/net/intel/igc/base/meson.build | 19 - > drivers/net/intel/igc/igc_logs.h | 43 - > drivers/net/intel/igc/meson.build | 21 - > drivers/net/meson.build | 1 - > 64 files changed, 3300 insertions(+), 18218 deletions(-)
And I thought my deduplication patchset was good, cutting the codebase down by 1000 lines! Shame you couldn't quite get it to a 15000 line even reduction - 14,918 lines just doesn't have that same ring about it! :-) More seriously, though: thanks, Anatoly. I'll start reviewing this patchset next week. /Bruce