On Wed, Jan 13, 2016 at 12:59:36AM +0100, Ricardo Ribalda Delgado wrote: > On Wed, Jan 13, 2016 at 12:30 AM, Greg Kroah-Hartman > <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > On Tue, Jan 12, 2016 at 11:53:49PM +0100, Ricardo Ribalda Delgado wrote: > >> Hi Greg > >> > >> On Tue, Jan 12, 2016 at 11:38 PM, Greg Kroah-Hartman > >> <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > >> >> + depends on SERIAL_8250 && PNP && !SERIAL_8250_PNP > >> >> + default n > >> > > >> > This is only going to help at build time, not run time, which I thought > >> > you wanted to help resolve. In other words, this isn't going to help > >> > 99% of the users out there as they use distro kernels that enable > >> > everything. > >> > > >> > Is that ok? > >> > >> The great advantage of the fintek driver against the standard pnp is > >> the rs485 functionality, which is something I don't expect to be used > >> by the distro users. > > > > Why not? Are you saying that 99.9% of Linux users will never be able to > > use your driver? > > > >> This patch buys me some time until I figure out a good way for both > >> drivers to coexist. > > > > Let's just mark it as broken, as Peter originally did, if they can't > > coexist today, until that gets fixed. That should give people more of a > > reason to work on it :) > > What about removing the .id_table from the struct pnp_driver of 8250_fintek? > > By default it will load with the pnp driver. > > The user can later bind/unbind the pnp driver in favor of the fintek > driver via sysfs if needed. That will never happen, let's just mark it broken until it is fixed "properly". thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
