Skip to content

K
Kernel

openeuler / Kernel
1 年多 前同步成功

通知 8
Star 0
Fork 0
K
Kernel
切换分支/标签
  1. 11 1月, 2012 8 次提交
    • M
      [media] cx231xx: fix device disconnect checks · 990862a2
      由 Mauro Carvalho Chehab 提交于 
      The driver were using DEV_MISCONFIGURED on some places, and
DEV_DISCONNECTED on others. In a matter of fact, DEV_MISCONFIGURED
were set only during the usb disconnect callback, with
was confusing.

Also, the alsa driver never checks if the device is present,
before doing some dangerous things.

Remove DEV_MISCONFIGURED, replacing it by DEV_DISCONNECTED.

Also, fixes the other usecases for DEV_DISCONNECTED.
Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      990862a2
    • M
      [media] cx231xx: cx231xx_devused is racy · db702a7a
      由 Mauro Carvalho Chehab 提交于 
      cx231xx_devused is racy. Re-implement it in a proper way,
to remove the risk of mangling it.
Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      db702a7a
    • M
      [media] cx231xx: Fix unregister logic · 7528cd27
      由 Mauro Carvalho Chehab 提交于 
      There are several weirdness at the unregister logic.

First of all, IR has a poll thread. This thread needs to be
removed, as it uses some resources associated to the main driver.
So, the driver needs to explicitly unregister the I2C client for
ir-kbd-i2c.

If, for some reason, the driver needs to wait for a close()
to happen, not all memories will be freed, because the free
logic were in the wrong place.

Also, v4l2_device_unregister() seems to be called too early,
as devices are still using it.

Finally, even with the device disconnected, there is one
USB function call that will still try to talk with it.
Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      7528cd27
    • M
      [media] mb86a20s: implement get_frontend() · 959a119f
      由 Mauro Carvalho Chehab 提交于 
      Reports the auto-detected parameters to userspace.
Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      959a119f
    • M
      [media] cx231xx-input: stop polling if the device got removed. · 90bf3aab
      由 Mauro Carvalho Chehab 提交于 
      If the device got removed, stops polling it. Also, un-registers
it at input/evdev, as it won't work anymore. We can't free the
IR structure yet, as the ir_remove method will be called later.
Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      90bf3aab
    • T
      [media] cx231xx: remove useless 'lif' variable in cx231xx_usb_probe() · c53a8e95
      由 Thomas Petazzoni 提交于 
      Now that we set the intfdata on the right interface, the 'lif'
variable is useless.
Signed-off-by: NThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      c53a8e95
    • T
      [media] cx231xx: fix crash after load/unload/load of module · a116a05c
      由 Thomas Petazzoni 提交于 
      The following sequence of commands was triggering a kernel crash in
cdev_get():

 modprobe cx231xx
 rmmod cx231xx
 modprobe cx231xx
 v4l2grab -n 1

The problem was that cx231xx_usb_disconnect() was not doing anything
because the test:

	if (!dev->udev)
		return;

was reached (i.e, dev->udev was NULL).

This is due to the fact that the 'dev' pointer placed as intfdata into
the usb_interface structure had the wrong value, because
cx231xx_probe() was doing the usb_set_intfdata() on the wrong
usb_interface structure. For some reason, cx231xx_probe() was doing
the following:

static int cx231xx_usb_probe(struct usb_interface *interface,
			     const struct usb_device_id *id)
{
        struct usb_interface *lif = NULL;
	[...]
        /* store the current interface */
        lif = interface;
	[...]
        /* store the interface 0 back */
        lif = udev->actconfig->interface[0];
	[...]
	usb_set_intfdata(lif, dev);
	[...]
	retval = v4l2_device_register(&interface->dev, &dev->v4l2_dev);
	[...]
}

So, the usb_set_intfdata() was done on udev->actconfig->interface[0]
and not on the 'interface' passed as argument to the ->probe() and
->disconnect() hooks. Later on, v4l2_device_register() was
initializing the intfdata of the correct usb_interface structure as a
pointer to the v4l2_device structure.

Upon unregistration, the ->disconnect() hook was getting the intfdata
of the usb_interface passed as argument... and casted it to a 'struct
cx231xx *' while it was in fact a 'struct v4l2_device *'.

The correct fix seems to just be to set the intfdata on the proper
interface from the beginning. Now, loading/unloading/reloading the
driver allows to use the device properly.
Signed-off-by: NThomas Petazzoni <thomas.petazzoni@free-electrons.com>
Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      a116a05c
    • M
      [media] dvb_ca_en50221: fix compilation breakage · 624f0c18
      由 Mauro Carvalho Chehab 提交于 
      As reported by Toralf:

the build failed with :
  CC [M]  drivers/media/dvb/dvb-core/dvb_ca_en50221.o
In file included from arch/x86/include/asm/uaccess.h:573:0,
                 from include/linux/poll.h:14,
                 from drivers/media/dvb/dvb-core/dvbdev.h:27,
                 from drivers/media/dvb/dvb-core/dvb_ca_en50221.h:27,
                 from drivers/media/dvb/dvb-core/dvb_ca_en50221.c:41:
In function "copy_from_user", inlined from "dvb_ca_en50221_io_write" at drivers/media/dvb/dvb-core/dvb_ca_en50221.c:1314:26: arch/x86/include/asm/uaccess_32.h:211:26: error: call to "copy_from_user_overflow" declared with attribute error: copy_from_user() buffer size is not provably correct
Reported-by: NToralf Foerster <toralf.foerster@gmx.de>
Signed-off-by: NMauro Carvalho Chehab <mchehab@redhat.com>
      624f0c18
  3. 07 1月, 2012 15 次提交
  5. 06 1月, 2012 17 次提交