On Fri, Oct 25, 2013 at 11:31 PM, Hauke Mehrtens <hauke@xxxxxxxxxx> wrote: > On 10/25/2013 10:34 AM, Johannes Berg wrote: >> On Wed, 2013-10-23 at 20:27 +0200, Hauke Mehrtens wrote: >>> These drivers are depending on changes of the regulator core driver >>> which are not in the specific kernel versions. >> >> These dependencies seem to continue moving forward - is it even worth >> backporting? >> >> johannes >> > Yes moving the dependencies is easier than backporting the code which > does not work any more. ;-) The problem here is that the regulator core > can not be backported, because it is always build into the kernel and > not as a module. > > I am also think if it is worth backporting this stuff, I think it is > only used on embedded devices and there it is probably easier to just > patch the kernel with the driver updates you need, I am for dropping this. > > Are there any users of the regulator drivers? The regulator / media drivers were added more as a proof of concept. In fact I added the regulator drivers only after I added the media drivers as I found inherent dependencies on them on regulator calls. If we remove regulator drivers then we may need to remove some media drivers. As for the issues with some functionality on the regulator drivers requiring functionality from future kernels as its always built-in, that can be addressed if we add support for non-modular solutions on backports. This should be possible, in fact given that these drivers are typically only shipped on non modular kernels it does make sense to have that requirement for backporting some drivers. Mauro had expressed some interest on reusing some of the backporting we've done for media drivers, Mauro do you require the solution to be non-modular? If so they'd be a good first target for that solution, but note that that solution still requires work, right no we only have them as modular. Does modular work for you as well? If not we might as well rip the drivers out and consider adding them once we have in-kernel support. Luis -- To unsubscribe from this list: send the line "unsubscribe backports" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html