Commit 9d5164da authored by Roel Van Nyen's avatar Roel Van Nyen Committed by Mauro Carvalho Chehab

V4L/DVB: tm6000: fix coding style issues of tm6000-cards.c

fix coding style issues

[mchehab@redhat.com: Fix conflicts with other patches]
Signed-off-by: default avatarRoel Van Nyen <roel.vannyen@gmail.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent e18f58c6
...@@ -299,7 +299,7 @@ struct tm6000_board tm6000_boards[] = { ...@@ -299,7 +299,7 @@ struct tm6000_board tm6000_boards[] = {
}; };
/* table of devices that work with this driver */ /* table of devices that work with this driver */
struct usb_device_id tm6000_id_table [] = { struct usb_device_id tm6000_id_table[] = {
{ USB_DEVICE(0x6000, 0x0001), .driver_info = TM5600_BOARD_10MOONS_UT821 }, { USB_DEVICE(0x6000, 0x0001), .driver_info = TM5600_BOARD_10MOONS_UT821 },
{ USB_DEVICE(0x6000, 0x0002), .driver_info = TM6010_BOARD_GENERIC }, { USB_DEVICE(0x6000, 0x0002), .driver_info = TM6010_BOARD_GENERIC },
{ USB_DEVICE(0x06e1, 0xf332), .driver_info = TM6000_BOARD_ADSTECH_DUAL_TV }, { USB_DEVICE(0x06e1, 0xf332), .driver_info = TM6000_BOARD_ADSTECH_DUAL_TV },
...@@ -324,23 +324,23 @@ struct usb_device_id tm6000_id_table [] = { ...@@ -324,23 +324,23 @@ struct usb_device_id tm6000_id_table [] = {
int tm6000_tuner_callback(void *ptr, int component, int command, int arg) int tm6000_tuner_callback(void *ptr, int component, int command, int arg)
{ {
int rc=0; int rc = 0;
struct tm6000_core *dev = ptr; struct tm6000_core *dev = ptr;
if (dev->tuner_type!=TUNER_XC2028) if (dev->tuner_type != TUNER_XC2028)
return 0; return 0;
switch (command) { switch (command) {
case XC2028_RESET_CLK: case XC2028_RESET_CLK:
tm6000_set_reg (dev, REQ_04_EN_DISABLE_MCU_INT, tm6000_set_reg(dev, REQ_04_EN_DISABLE_MCU_INT,
0x02, arg); 0x02, arg);
msleep(10); msleep(10);
rc=tm6000_set_reg (dev, REQ_03_SET_GET_MCU_PIN, rc = tm6000_set_reg(dev, REQ_03_SET_GET_MCU_PIN,
TM6000_GPIO_CLK, 0); TM6000_GPIO_CLK, 0);
if (rc<0) if (rc < 0)
return rc; return rc;
msleep(10); msleep(10);
rc=tm6000_set_reg (dev, REQ_03_SET_GET_MCU_PIN, rc = tm6000_set_reg(dev, REQ_03_SET_GET_MCU_PIN,
TM6000_GPIO_CLK, 1); TM6000_GPIO_CLK, 1);
break; break;
case XC2028_TUNER_RESET: case XC2028_TUNER_RESET:
...@@ -389,24 +389,24 @@ int tm6000_tuner_callback(void *ptr, int component, int command, int arg) ...@@ -389,24 +389,24 @@ int tm6000_tuner_callback(void *ptr, int component, int command, int arg)
} }
break; break;
case 1: case 1:
tm6000_set_reg (dev, REQ_04_EN_DISABLE_MCU_INT, tm6000_set_reg(dev, REQ_04_EN_DISABLE_MCU_INT,
0x02, 0x01); 0x02, 0x01);
msleep(10); msleep(10);
break; break;
case 2: case 2:
rc=tm6000_set_reg (dev, REQ_03_SET_GET_MCU_PIN, rc = tm6000_set_reg(dev, REQ_03_SET_GET_MCU_PIN,
TM6000_GPIO_CLK, 0); TM6000_GPIO_CLK, 0);
if (rc<0) if (rc < 0)
return rc; return rc;
msleep(100); msleep(100);
rc=tm6000_set_reg (dev, REQ_03_SET_GET_MCU_PIN, rc = tm6000_set_reg(dev, REQ_03_SET_GET_MCU_PIN,
TM6000_GPIO_CLK, 1); TM6000_GPIO_CLK, 1);
msleep(100); msleep(100);
break; break;
} }
} }
return (rc); return rc;
} }
int tm6000_cards_setup(struct tm6000_core *dev) int tm6000_cards_setup(struct tm6000_core *dev)
...@@ -507,13 +507,13 @@ int tm6000_cards_setup(struct tm6000_core *dev) ...@@ -507,13 +507,13 @@ int tm6000_cards_setup(struct tm6000_core *dev)
return 0; return 0;
}; };
static void tm6000_config_tuner (struct tm6000_core *dev) static void tm6000_config_tuner(struct tm6000_core *dev)
{ {
struct tuner_setup tun_setup; struct tuner_setup tun_setup;
/* Load tuner module */ /* Load tuner module */
v4l2_i2c_new_subdev(&dev->v4l2_dev, &dev->i2c_adap, v4l2_i2c_new_subdev(&dev->v4l2_dev, &dev->i2c_adap,
"tuner", "tuner",dev->tuner_addr, NULL); "tuner", "tuner", dev->tuner_addr, NULL);
memset(&tun_setup, 0, sizeof(tun_setup)); memset(&tun_setup, 0, sizeof(tun_setup));
tun_setup.type = dev->tuner_type; tun_setup.type = dev->tuner_type;
...@@ -528,7 +528,7 @@ static void tm6000_config_tuner (struct tm6000_core *dev) ...@@ -528,7 +528,7 @@ static void tm6000_config_tuner (struct tm6000_core *dev)
struct xc2028_ctrl ctl; struct xc2028_ctrl ctl;
memset(&xc2028_cfg, 0, sizeof(xc2028_cfg)); memset(&xc2028_cfg, 0, sizeof(xc2028_cfg));
memset (&ctl,0,sizeof(ctl)); memset(&ctl, 0, sizeof(ctl));
ctl.input1 = 1; ctl.input1 = 1;
ctl.read_not_reliable = 0; ctl.read_not_reliable = 0;
...@@ -539,7 +539,7 @@ static void tm6000_config_tuner (struct tm6000_core *dev) ...@@ -539,7 +539,7 @@ static void tm6000_config_tuner (struct tm6000_core *dev)
xc2028_cfg.tuner = TUNER_XC2028; xc2028_cfg.tuner = TUNER_XC2028;
xc2028_cfg.priv = &ctl; xc2028_cfg.priv = &ctl;
switch(dev->model) { switch (dev->model) {
case TM6010_BOARD_HAUPPAUGE_900H: case TM6010_BOARD_HAUPPAUGE_900H:
case TM6010_BOARD_TERRATEC_CINERGY_HYBRID_XE: case TM6010_BOARD_TERRATEC_CINERGY_HYBRID_XE:
case TM6010_BOARD_TWINHAN_TU501: case TM6010_BOARD_TWINHAN_TU501:
...@@ -580,8 +580,8 @@ static int tm6000_init_dev(struct tm6000_core *dev) ...@@ -580,8 +580,8 @@ static int tm6000_init_dev(struct tm6000_core *dev)
dev->caps = tm6000_boards[dev->model].caps; dev->caps = tm6000_boards[dev->model].caps;
/* initialize hardware */ /* initialize hardware */
rc=tm6000_init (dev); rc = tm6000_init(dev);
if (rc<0) if (rc < 0)
goto err; goto err;
rc = v4l2_device_register(&dev->udev->dev, &dev->v4l2_dev); rc = v4l2_device_register(&dev->udev->dev, &dev->v4l2_dev);
...@@ -589,8 +589,8 @@ static int tm6000_init_dev(struct tm6000_core *dev) ...@@ -589,8 +589,8 @@ static int tm6000_init_dev(struct tm6000_core *dev)
goto err; goto err;
/* register i2c bus */ /* register i2c bus */
rc=tm6000_i2c_register(dev); rc = tm6000_i2c_register(dev);
if (rc<0) if (rc < 0)
goto err; goto err;
/* Default values for STD and resolutions */ /* Default values for STD and resolutions */
...@@ -599,7 +599,7 @@ static int tm6000_init_dev(struct tm6000_core *dev) ...@@ -599,7 +599,7 @@ static int tm6000_init_dev(struct tm6000_core *dev)
dev->norm = V4L2_STD_PAL_M; dev->norm = V4L2_STD_PAL_M;
/* Configure tuner */ /* Configure tuner */
tm6000_config_tuner (dev); tm6000_config_tuner(dev);
/* Set video standard */ /* Set video standard */
v4l2_device_call_all(&dev->v4l2_dev, 0, core, s_std, dev->norm); v4l2_device_call_all(&dev->v4l2_dev, 0, core, s_std, dev->norm);
...@@ -616,20 +616,20 @@ static int tm6000_init_dev(struct tm6000_core *dev) ...@@ -616,20 +616,20 @@ static int tm6000_init_dev(struct tm6000_core *dev)
"tvaudio", "tvaudio", I2C_ADDR_TDA9874, NULL); "tvaudio", "tvaudio", I2C_ADDR_TDA9874, NULL);
/* register and initialize V4L2 */ /* register and initialize V4L2 */
rc=tm6000_v4l2_register(dev); rc = tm6000_v4l2_register(dev);
if (rc<0) if (rc < 0)
goto err; goto err;
if(dev->caps.has_dvb) { if (dev->caps.has_dvb) {
dev->dvb = kzalloc(sizeof(*(dev->dvb)), GFP_KERNEL); dev->dvb = kzalloc(sizeof(*(dev->dvb)), GFP_KERNEL);
if(!dev->dvb) { if (!dev->dvb) {
rc = -ENOMEM; rc = -ENOMEM;
goto err2; goto err2;
} }
#ifdef CONFIG_VIDEO_TM6000_DVB #ifdef CONFIG_VIDEO_TM6000_DVB
rc = tm6000_dvb_register(dev); rc = tm6000_dvb_register(dev);
if(rc < 0) { if (rc < 0) {
kfree(dev->dvb); kfree(dev->dvb);
dev->dvb = NULL; dev->dvb = NULL;
goto err2; goto err2;
...@@ -668,7 +668,7 @@ static void get_max_endpoint(struct usb_device *udev, ...@@ -668,7 +668,7 @@ static void get_max_endpoint(struct usb_device *udev,
tm_ep->bInterfaceNumber = alt->desc.bInterfaceNumber; tm_ep->bInterfaceNumber = alt->desc.bInterfaceNumber;
tm_ep->bAlternateSetting = alt->desc.bAlternateSetting; tm_ep->bAlternateSetting = alt->desc.bAlternateSetting;
printk("tm6000: %s endpoint: 0x%02x (max size=%u bytes)\n", printk(KERN_INFO "tm6000: %s endpoint: 0x%02x (max size=%u bytes)\n",
msgtype, curr_e->desc.bEndpointAddress, msgtype, curr_e->desc.bEndpointAddress,
size); size);
} }
...@@ -683,22 +683,21 @@ static int tm6000_usb_probe(struct usb_interface *interface, ...@@ -683,22 +683,21 @@ static int tm6000_usb_probe(struct usb_interface *interface,
{ {
struct usb_device *usbdev; struct usb_device *usbdev;
struct tm6000_core *dev = NULL; struct tm6000_core *dev = NULL;
int i,rc=0; int i, rc = 0;
int nr=0; int nr = 0;
char *speed; char *speed;
usbdev = usb_get_dev(interface_to_usbdev(interface));
usbdev=usb_get_dev(interface_to_usbdev(interface));
/* Selects the proper interface */ /* Selects the proper interface */
rc=usb_set_interface(usbdev,0,1); rc = usb_set_interface(usbdev, 0, 1);
if (rc<0) if (rc < 0)
goto err; goto err;
/* Check to see next free device and mark as used */ /* Check to see next free device and mark as used */
nr=find_first_zero_bit(&tm6000_devused,TM6000_MAXBOARDS); nr = find_first_zero_bit(&tm6000_devused, TM6000_MAXBOARDS);
if (nr >= TM6000_MAXBOARDS) { if (nr >= TM6000_MAXBOARDS) {
printk ("tm6000: Supports only %i tm60xx boards.\n",TM6000_MAXBOARDS); printk(KERN_ERR "tm6000: Supports only %i tm60xx boards.\n", TM6000_MAXBOARDS);
usb_put_dev(usbdev); usb_put_dev(usbdev);
return -ENOMEM; return -ENOMEM;
} }
...@@ -706,24 +705,23 @@ static int tm6000_usb_probe(struct usb_interface *interface, ...@@ -706,24 +705,23 @@ static int tm6000_usb_probe(struct usb_interface *interface,
/* Create and initialize dev struct */ /* Create and initialize dev struct */
dev = kzalloc(sizeof(*dev), GFP_KERNEL); dev = kzalloc(sizeof(*dev), GFP_KERNEL);
if (dev == NULL) { if (dev == NULL) {
printk ("tm6000" ": out of memory!\n"); printk(KERN_ERR "tm6000" ": out of memory!\n");
usb_put_dev(usbdev); usb_put_dev(usbdev);
return -ENOMEM; return -ENOMEM;
} }
spin_lock_init(&dev->slock); spin_lock_init(&dev->slock);
/* Increment usage count */ /* Increment usage count */
tm6000_devused|=1<<nr; tm6000_devused |= 1<<nr;
snprintf(dev->name, 29, "tm6000 #%d", nr); snprintf(dev->name, 29, "tm6000 #%d", nr);
dev->model=id->driver_info; dev->model = id->driver_info;
if ((card[nr]>=0) && (card[nr]<ARRAY_SIZE(tm6000_boards))) { if ((card[nr] >= 0) && (card[nr] < ARRAY_SIZE(tm6000_boards)))
dev->model=card[nr]; dev->model = card[nr];
}
INIT_LIST_HEAD(&dev->tm6000_corelist); INIT_LIST_HEAD(&dev->tm6000_corelist);
dev->udev= usbdev; dev->udev = usbdev;
dev->devno=nr; dev->devno = nr;
switch (usbdev->speed) { switch (usbdev->speed) {
case USB_SPEED_LOW: case USB_SPEED_LOW:
...@@ -755,7 +753,7 @@ static int tm6000_usb_probe(struct usb_interface *interface, ...@@ -755,7 +753,7 @@ static int tm6000_usb_probe(struct usb_interface *interface,
dir_out = ((e->desc.bEndpointAddress & dir_out = ((e->desc.bEndpointAddress &
USB_ENDPOINT_DIR_MASK) == USB_DIR_OUT); USB_ENDPOINT_DIR_MASK) == USB_DIR_OUT);
printk("tm6000: alt %d, interface %i, class %i\n", printk(KERN_INFO "tm6000: alt %d, interface %i, class %i\n",
i, i,
interface->altsetting[i].desc.bInterfaceNumber, interface->altsetting[i].desc.bInterfaceNumber,
interface->altsetting[i].desc.bInterfaceClass); interface->altsetting[i].desc.bInterfaceClass);
...@@ -792,7 +790,7 @@ static int tm6000_usb_probe(struct usb_interface *interface, ...@@ -792,7 +790,7 @@ static int tm6000_usb_probe(struct usb_interface *interface,
} }
printk("tm6000: New video device @ %s Mbps (%04x:%04x, ifnum %d)\n", printk(KERN_INFO "tm6000: New video device @ %s Mbps (%04x:%04x, ifnum %d)\n",
speed, speed,
le16_to_cpu(dev->udev->descriptor.idVendor), le16_to_cpu(dev->udev->descriptor.idVendor),
le16_to_cpu(dev->udev->descriptor.idProduct), le16_to_cpu(dev->udev->descriptor.idProduct),
...@@ -800,8 +798,8 @@ static int tm6000_usb_probe(struct usb_interface *interface, ...@@ -800,8 +798,8 @@ static int tm6000_usb_probe(struct usb_interface *interface,
/* check if the the device has the iso in endpoint at the correct place */ /* check if the the device has the iso in endpoint at the correct place */
if (!dev->isoc_in.endp) { if (!dev->isoc_in.endp) {
printk("tm6000: probing error: no IN ISOC endpoint!\n"); printk(KERN_ERR "tm6000: probing error: no IN ISOC endpoint!\n");
rc= -ENODEV; rc = -ENODEV;
goto err; goto err;
} }
...@@ -809,19 +807,19 @@ static int tm6000_usb_probe(struct usb_interface *interface, ...@@ -809,19 +807,19 @@ static int tm6000_usb_probe(struct usb_interface *interface,
/* save our data pointer in this interface device */ /* save our data pointer in this interface device */
usb_set_intfdata(interface, dev); usb_set_intfdata(interface, dev);
printk("tm6000: Found %s\n", tm6000_boards[dev->model].name); printk(KERN_INFO "tm6000: Found %s\n", tm6000_boards[dev->model].name);
rc=tm6000_init_dev(dev); rc = tm6000_init_dev(dev);
if (rc<0) if (rc < 0)
goto err; goto err;
return 0; return 0;
err: err:
printk("tm6000: Error %d while registering\n", rc); printk(KERN_ERR "tm6000: Error %d while registering\n", rc);
tm6000_devused&=~(1<<nr); tm6000_devused &= ~(1<<nr);
usb_put_dev(usbdev); usb_put_dev(usbdev);
kfree(dev); kfree(dev);
...@@ -841,12 +839,12 @@ static void tm6000_usb_disconnect(struct usb_interface *interface) ...@@ -841,12 +839,12 @@ static void tm6000_usb_disconnect(struct usb_interface *interface)
if (!dev) if (!dev)
return; return;
printk("tm6000: disconnecting %s\n", dev->name); printk(KERN_INFO "tm6000: disconnecting %s\n", dev->name);
mutex_lock(&dev->lock); mutex_lock(&dev->lock);
#ifdef CONFIG_VIDEO_TM6000_DVB #ifdef CONFIG_VIDEO_TM6000_DVB
if(dev->dvb) { if (dev->dvb) {
tm6000_dvb_unregister(dev); tm6000_dvb_unregister(dev);
kfree(dev->dvb); kfree(dev->dvb);
} }
...@@ -858,8 +856,6 @@ static void tm6000_usb_disconnect(struct usb_interface *interface) ...@@ -858,8 +856,6 @@ static void tm6000_usb_disconnect(struct usb_interface *interface)
v4l2_device_unregister(&dev->v4l2_dev); v4l2_device_unregister(&dev->v4l2_dev);
// wake_up_interruptible_all(&dev->open);
dev->state |= DEV_DISCONNECTED; dev->state |= DEV_DISCONNECTED;
usb_put_dev(dev->udev); usb_put_dev(dev->udev);
...@@ -886,7 +882,7 @@ static int __init tm6000_module_init(void) ...@@ -886,7 +882,7 @@ static int __init tm6000_module_init(void)
/* register this driver with the USB subsystem */ /* register this driver with the USB subsystem */
result = usb_register(&tm6000_usb_driver); result = usb_register(&tm6000_usb_driver);
if (result) if (result)
printk("tm6000" printk(KERN_ERR "tm6000"
" usb_register failed. Error number %d.\n", result); " usb_register failed. Error number %d.\n", result);
return result; return result;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment