On Tue, Mar 10, 2015 at 04:55:56PM +0200, Tomi Valkeinen wrote: > On 10/03/15 16:46, Russell King - ARM Linux wrote: > > > In which case, let me propose that the exynos fbdev driver needs to be > > moved to drivers/staging, and stay there until this stuff gets fixed. > > drivers/staging is supposed to be for stuff which isn't up to the mark, > > and which is potentially unstable. And that's what this driver exactly > > is. > > There is drivers/gpu/drm/exynos/ which is getting a lot of updates. So... > > I'd propose removing the exynos fbdev driver if the exynos drm driver > offers the same functionality. I don't know if that's the case. Does the > drm driver support all the devices the fbdev supports? > > Also, I'm not sure if and how we can remove drivers. If exynos fbdev > driver is dropped, that would perhaps break boards that have exynos > fbdev in their .dts file. And if the drm driver doesn't offer the exact > same /dev/fbX interface, it would break the userspace. > > So I don't know if that's possible. But that's what I'd like to do, > eventually, for all the fbdev drivers. Implement drm driver, remove the > fbdev one. That's why I suggested moving it to drivers/staging - it's a hint that the driver needs a serious amount of work, and when built as a module, it also provides users with the hint that the module they're loading is of questionable quality (which is definitely the case here.) Others have done that kind of thing before - we've had drivers which have fallen by the way side, and at some point the decision has been made to move them to drivers/staging, and if nothing happens to fix them up (showing that no one cares about them), they've eventually been dropped. Of course, us talking about this might be enough to spur some effort to get the thing properly fixed. :) -- FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up according to speedtest.net. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |