On 05/11/13 17:53, Aaron Sierra wrote: > Martyn, > Can you please elaborate on why you feel it is not a fix? For instance, > will this type of solution not be accepted upstream? Is this solution not > complete enough? Do you feel that it doesn't resolve any issue? > > -Aaron
The vme_user driver is in the staging tree as it is not in a fit state to incorporate into the main kernel tree. As far as I can see, the below patch just causes the driver to be built at a subtly different time, from a subtly different location (outside of the staging tree). To be honest, I'm not sure how this is fixing anything. Martyn > >> >> On 31/10/13 23:47, Aaron Sierra wrote: >>> If VME bridge and vme_user modules are compiled into the kernel, then >>> vme_user will attempt to register itself before any VME buses have been >>> probed. This results in a kernel panic. >>> >>> This patch removes the staging VME devices build from the general >>> staging Makefile and moves the build to the general VME Makefile after >>> all VME buses are built. >>> >>> Signed-off-by: Aaron Sierra <asie...@xes-inc.com> >>> --- >>> drivers/staging/Makefile | 1 - >>> drivers/vme/Makefile | 1 + >>> 2 files changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/drivers/staging/Makefile b/drivers/staging/Makefile >>> index 415772e..8082bdb 100644 >>> --- a/drivers/staging/Makefile >>> +++ b/drivers/staging/Makefile >>> @@ -27,7 +27,6 @@ obj-$(CONFIG_USB_SERIAL_QUATECH2) += serqt_usb2/ >>> obj-$(CONFIG_OCTEON_ETHERNET) += octeon/ >>> obj-$(CONFIG_VT6655) += vt6655/ >>> obj-$(CONFIG_VT6656) += vt6656/ >>> -obj-$(CONFIG_VME_BUS) += vme/ >>> obj-$(CONFIG_DX_SEP) += sep/ >>> obj-$(CONFIG_IIO) += iio/ >>> obj-$(CONFIG_ZRAM) += zram/ >>> diff --git a/drivers/vme/Makefile b/drivers/vme/Makefile >>> index d7bfcb9..7a784ab 100644 >>> --- a/drivers/vme/Makefile >>> +++ b/drivers/vme/Makefile >>> @@ -5,3 +5,4 @@ obj-$(CONFIG_VME_BUS) += vme.o >>> >>> obj-y += bridges/ >>> obj-y += boards/ >>> +obj-$(CONFIG_STAGING) += ../staging/vme/ >>> -- Martyn Welch (Lead Software Engineer) | Registered in England and Wales GE Intelligent Platforms | (3828642) at 100 Barbirolli Square T +44(0)1327322748 | Manchester, M2 3AB E martyn.we...@ge.com | VAT:GB 927559189 _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
