On Fri, Apr 26, 2013 at 05:34:45PM +0300, Dan Carpenter wrote:
>
> I hate these "mostly NUL terminated" strings. MAX_DESC_LEN
> is 100 characters so it's very rarely going to be filled all the
> way. The special case where ->nd_ps_desc isn't terminated will
> never be tested.
>
Yes, I think th
Dan Carpenter writes:
>
> On Wed, Apr 24, 2013 at 11:18:03AM +0800, Chen Gang wrote:
> >
> > For dgrp, the buffer length of 'nd->nd_ps_desc' is 'MAX_DESC_LEN + 1', so
> > the buffer length of 'getnode.nd_ps_desc' also need be 'MAX_DESC_LEN + 1',
> > then can fully copy from 'nd->nd_ps_desc' to 'g
Michal Marek writes:
>
> On 29.11.2012 03:12, Fengguang Wu wrote:
> > On Wed, Nov 28, 2012 at 02:16:49PM -0800, Greg KH wrote:
> >> On Thu, Nov 29, 2012 at 05:38:31AM +0800, kbuild test robot wrote:
> >>> tree:
> >>> git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git
> >>> dr
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/vt6656/main_usb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vt6656/main_usb.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/wlags49_h2/wl_pci.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wlags49_h2/wl_pci.c
b
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: Mauro Carvalho Chehab
Cc: linux-me...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/media/dt3155v4l/dt3155v4l.c | 4 ++--
1 file changed, 2 insertions(+), 2
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: Mauro Carvalho Chehab
Cc: linux-me...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/media/lirc/lirc_parallel.c | 2 +-
drivers/staging/media/lirc/lirc_serial.c
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: Mauro Carvalho Chehab
Cc: linux-me...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/media/solo6x10/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/tidspbridge/rmgr/drv_interface.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/tidspbridge/rmgr
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/et131x/et131x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/slicoss/slicoss.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/slicoss/slicoss.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/ste_rmi4/synaptics_i2c_rmi4.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ste_rmi4
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/xgifb/XGI_main_26.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/xgifb/XGI_main_26.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/crystalhd/crystalhd_cmds.c | 2 +-
drivers/staging/crystalhd/crystalhd_lnx.c | 6 +++---
2 files changed, 4 insertions(+), 4 deletions
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: Martyn Welch
Cc: Manohar Vanga
Cc: de...@driverdev.osuosl.org
---
drivers/staging/vme/devices/vme_pio2_core.c | 4 ++--
drivers/staging/vme/devices/vme_user.c | 4 ++--
2
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/sm7xxfb/sm7xxfb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/omap-thermal/omap-bandgap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/omap-thermal/omap
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/rtl8192u/r8192U_core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_core.c
b
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/vt6655/device_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vt6655/device_main.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devinitconst is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: Julian Andres Klode
Cc: Marc Dietrich
Cc: ac...@lists.launchpad.net
Cc: de...@driverdev.osuosl.org
---
drivers/staging/nvec/nvec.c | 2 +-
1 file changed, 1 insertion(+), 1
CONFIG_HOTPLUG is going away as an option so __devinitdata is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/wlags49_h2/wl_pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/wlags49_h2/wl_pci.c
b
CONFIG_HOTPLUG is going away as an option so __devinitconst is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/ipack/ipack.h | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/ipack/ipack.h b/drivers/staging
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl8192e
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: Julian Andres Klode
Cc: Marc Dietrich
Cc: ac...@lists.launchpad.net
Cc: de...@driverdev.osuosl.org
---
drivers/staging/nvec/nvec.c | 2 +-
drivers/staging/nvec/nvec_kbd.c
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/octeon/ethernet.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/octeon/ethernet.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/sbe-2t3e3/module.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/sbe-2t3e3/module.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/net/pc300_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/net/pc300_drv.c b/drivers/staging/net
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/rtl8187se/r8180_core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8187se/r8180_core.c
b
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: linux-...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/iio/accel/adis16201_core.c | 2 +-
drivers/staging/iio/accel/adis16203_core.c | 2
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/ipack/carriers/tpci200.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ipack/carriers/tpci200.c
b
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/imx-drm/ipu-v3/ipu-common.c | 2 +-
drivers/staging/imx-drm/ipuv3-crtc.c| 2 +-
drivers/staging/imx-drm/parallel-display.c | 2
CONFIG_HOTPLUG is going away as an option so __devexit is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/comedi/drivers/8255_pci.c | 2 +-
drivers/staging/comedi/drivers/addi_apci_035.c | 2 +-
drivers/staging/comedi/drivers
CONFIG_HOTPLUG is going away as an option so __devinitdata is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/sbe-2t3e3/module.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/sbe-2t3e3/module.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devinitdata is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192e/rtl8192e
CONFIG_HOTPLUG is going away as an option so __devinitdata is no
longer needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/rtl8187se/r8180_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8187se/r8180_core.c
b
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/imx-drm/ipu-v3/ipu-common.c | 2 +-
drivers/staging/imx-drm/ipuv3-crtc.c| 2 +-
drivers/staging/imx-drm/parallel-display.c | 2
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/octeon/ethernet.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/octeon/ethernet.c
b
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: Mauro Carvalho Chehab
Cc: linux-me...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/media/lirc/lirc_parallel.c | 2 +-
drivers/staging/media/lirc/lirc_serial.c
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/crystalhd/crystalhd_cmds.c | 2 +-
drivers/staging/crystalhd/crystalhd_lnx.c | 6 +++---
drivers/staging/crystalhd/crystalhd_misc.c | 2
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/sbe-2t3e3/module.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/sbe-2t3e3/module.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: Mauro Carvalho Chehab
Cc: linux-me...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/media/dt3155v4l/dt3155v4l.c | 6 +++---
1 file changed, 3 insertions(+), 3
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: Mauro Carvalho Chehab
Cc: linux-me...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/media/solo6x10/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/et131x/et131x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/omap-thermal/omap-bandgap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/omap-thermal/omap
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/rtl8187se/r8180_core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8187se/r8180_core.c
b
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/sep/sep_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/sep/sep_main.c b/drivers/staging/sep
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/comedi/drivers/8255_pci.c | 4 ++--
drivers/staging/comedi/drivers/addi-data/addi_common.c | 2 +-
drivers/staging/comedi
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/net/pc300_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/net/pc300_drv.c b/drivers/staging/net
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: linux-...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/iio/accel/adis16201_core.c | 2 +-
drivers/staging/iio/accel/adis16203_core.c | 2
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtl8192e
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/rtl8192u/r8192U_core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/rtl8192u/r8192U_core.c
b
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/sm7xxfb/sm7xxfb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: Julian Andres Klode
Cc: Marc Dietrich
Cc: ac...@lists.launchpad.net
Cc: de...@driverdev.osuosl.org
---
drivers/staging/nvec/nvec.c | 2 +-
drivers/staging/nvec/nvec_kbd.c
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/ste_rmi4/synaptics_i2c_rmi4.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ste_rmi4
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/slicoss/slicoss.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/slicoss/slicoss.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: linux-...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/iio/accel/adis16201_core.c | 2 +-
drivers/staging/iio/accel/adis16203_core.c | 2
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/tidspbridge/rmgr/drv_interface.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/tidspbridge/rmgr
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: Martyn Welch
Cc: Manohar Vanga
Cc: de...@driverdev.osuosl.org
---
drivers/staging/vme/devices/vme_pio2.h | 2 +-
drivers/staging/vme/devices/vme_pio2_core.c | 4 ++--
drivers
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/vt6656/main_usb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vt6656/main_usb.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/vt6655/device_main.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/vt6655/device_main.c
b
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/wlags49_h2/wl_pci.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wlags49_h2/wl_pci.c
b
CONFIG_HOTPLUG is going away as an option so __devinit is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/xgifb/XGI_main_26.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/xgifb/XGI_main_26.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/comedi/drivers/8255_pci.c | 2 +-
drivers/staging/comedi/drivers/addi_apci_035.c | 2 +-
drivers/staging/comedi/drivers
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/ste_rmi4/synaptics_i2c_rmi4.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ste_rmi4
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/ipack/carriers/tpci200.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ipack/carriers/tpci200.c
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/olpc_dcon/olpc_dcon.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/olpc_dcon/olpc_dcon.c
b
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: Martyn Welch
Cc: Manohar Vanga
Cc: de...@driverdev.osuosl.org
---
drivers/staging/vme/devices/vme_pio2_core.c | 2 +-
drivers/staging/vme/devices/vme_user.c | 2 +-
2 files
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/tidspbridge/rmgr/drv_interface.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/tidspbridge/rmgr
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: Matt Mooney
Cc: linux-...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/usbip/vhci_hcd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/slicoss/slicoss.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/slicoss/slicoss.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/rtl8192e/rtl8192e/rtl_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8192e/rtl8192e
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: Mauro Carvalho Chehab
Cc: linux-me...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/media/dt3155v4l/dt3155v4l.c | 2 +-
1 file changed, 1 insertion(+), 1
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: Julian Andres Klode
Cc: Marc Dietrich
Cc: ac...@lists.launchpad.net
Cc: de...@driverdev.osuosl.org
---
drivers/staging/nvec/nvec.c | 2 +-
drivers/staging/nvec/nvec_kbd.c
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/xgifb/XGI_main_26.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/xgifb/XGI_main_26.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/imx-drm/imx-drm-core.c | 2 +-
drivers/staging/imx-drm/ipu-v3/ipu-common.c | 2 +-
drivers/staging/imx-drm/ipuv3-crtc.c| 2
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/octeon/ethernet.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/octeon/ethernet.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/sm7xxfb/sm7xxfb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c
b/drivers
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/net/pc300_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/net/pc300_drv.c b/drivers/staging
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/rtl8187se/r8180_core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rtl8187se/r8180_core.c
b
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/crystalhd/crystalhd_lnx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/crystalhd/crystalhd_lnx.c
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: Mauro Carvalho Chehab
Cc: linux-me...@vger.kernel.org
Cc: de...@driverdev.osuosl.org
---
drivers/staging/media/lirc/lirc_parallel.c | 2 +-
drivers/staging/media/lirc
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed, remove it.
This also changes the syntax for the initialization of the wl_driver
struct to match convention.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/wlags49_h2/wl_pci.c | 15
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer
needed.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/et131x/et131x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging
Remove conditional code based on CONFIG_HOTPLUG being false. It's
always on now in preparation of it going away as an option.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/csr/firmware.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/dr
Remove conditional code based on CONFIG_HOTPLUG being false. It's
always on now in preparation of it going away as an option.
Signed-off-by: Bill Pemberton
Cc: de...@driverdev.osuosl.org
---
drivers/staging/ipack/ipack.c | 8
1 file changed, 8 deletions(-)
diff --git a/dr
The TIOCGSOFTCAR and TIOCSSOFTCAR ioctls are handled by the tty layer
so the dgrp driver shouldn't try to deal with them itself.
Signed-off-by: Bill Pemberton
---
drivers/staging/dgrp/dgrp_tty.c | 15 ---
1 file changed, 15 deletions(-)
diff --git a/drivers/staging/dgrp/dgrp_
alloc_tty_driver was always assumed to succeed. Add code to check the
return value and return -ENOMEM if alloc_tty_driver fails.
Reported-by: Fengguang Wu
Signed-off-by: Bill Pemberton
---
drivers/staging/dgrp/dgrp_tty.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers
register_proc_table and unregister_proc_table didn't deal with the
possibility that the *table pointer could be NULL. Check for this and
return if table is NULL.
Reported-by: Fengguang Wu
Signed-off-by: Bill Pemberton
---
drivers/staging/dgrp/dgrp_specproc.c | 6 ++
1 file chang
). Fix this to not make the strncpy
call and report an error if desclen is <= 0
Reported-by: Dan Carpenter
Signed-off-by: Bill Pemberton
---
drivers/staging/dgrp/dgrp_net_ops.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/staging/dgrp/dgrp_net_ops.c
b/drivers/staging/dgr
>
> I have a concern about the following code:
>
> drivers/staging/dgrp/dgrp_net_ops.c:3159 dgrp_receive()
> 3128 plen = get_unaligned_be16(b + 2);
> 3129
> 3130 if (plen < 4 || plen > 1000) {
>
Kconfig and Makefile changes to add dgrp to the build system.
Signed-off-by: Bill Pemberton
---
drivers/staging/Kconfig | 2 ++
drivers/staging/Makefile | 1 +
drivers/staging/dgrp/TODO | 12 +---
3 files changed, 12 insertions(+), 3 deletions(-)
diff --git a/drivers/staging
> > - if (msgInfo) {
> > - kfree(msgInfo->WaitEvent);
> > - kfree(msgInfo);
> > - }
> > + kfree(msgInfo->WaitEvent);
>
> If msgInfo was NULL, you just oopsed here :(
>
Sorry about that, getting too aggressive on that.
--
Bill
__
The return from k*alloc() should be checked and action taking for when
the allocation fails. In many places hv did this with an ASSERT()
call or didn't check the value at all.
Signed-off-by: Bill Pemberton
---
drivers/staging/hv/Channel.c |
kfree() knows how to deal with NULL, so there's no reason to check for
NULL before passing something to it.
Signed-off-by: Bill Pemberton
---
drivers/staging/hv/ChannelMgmt.c |6 ++
drivers/staging/hv/Hv.c | 10 +-
2 files changed, 7 insertions(+), 9 dele
There's not much point to make sure scmnd is not NULL after an assert
that would dereference scmnd. The ASSERT()'s should be removed, but
until they are at least they at least can be in the right order.
Signed-off-by: Bill Pemberton
---
drivers/staging/hv/storvsc_drv.c |2 +
95 matches
Mail list logo
