On Tue, Dec 10, 2019 at 11:48:49PM +0200, Aaro Koskinen wrote: > On Tue, Dec 10, 2019 at 12:15:15PM -0800, Guenter Roeck wrote: > > On Tue, Dec 10, 2019 at 09:46:59PM +0200, Aaro Koskinen wrote: > > > On Tue, Dec 10, 2019 at 01:01:20PM +0100, Greg Kroah-Hartman wrote: > > > > I have no idea :( > > > > > > It's stated in the TODO file you are deleting (visible in your > > > patch): "This driver is functional and supports Ethernet on > > > OCTEON+/OCTEON2/OCTEON3 chips at least up to CN7030." > > > > > > This includes e.g. some D-Link routers and Uniquiti EdgeRouters. You > > > can check from /proc/cpuinfo if you are running on this MIPS SoC. > > > > It also results in "mips:allmodconfig" build failures in mainline > > and is for that reason being marked as BROKEN. Unfortunately, > > misguided attempts to clean it up had the opposite effect. > > This was because of stubs hack added by someone - people who do not run > or care about the hardware can now break it for others with their > silly x86 "compile test"s. > Thast was the first breakage. The second was to replace typedefs with structures without considering that those typedefs are still used throughout the Cavium code, creating conflicts between "mystruct_t" and "struct mystruct" in various API calls. It may well be that this "improvement" was tested with x86_64:allmodconfig - if it was tested in the first place. It was most definitely not tested with cavium_octeon_defconfig, much less with real hardware. Pretty much none of the changes made to the driver in the recent past have improved it. On the contrary, it is getting worse. With no one committed to get the driver out of staging, I don't think there is a reasonable alternative to removing it. For my part I am for sure not looking forward having to deal with it breaking over and over again and having to spend time tracking down the breakage. Guenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
