[media] firedtv: remove obsolete ieee1394 backend code
authorHans Verkuil <hverkuil@xs4all.nl>
Sat, 5 Feb 2011 13:07:39 +0000 (10:07 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 21 Mar 2011 23:32:12 +0000 (20:32 -0300)
drivers/ieee1394/ has been removed in Linux 2.6.37.  The corresponding
backend code in firedtv is no longer built in now and can be deleted.
Firedtv continues to work with drivers/firewire/.

Also, fix a Kconfig menu comment:  Removal of CONFIG_IEEE1394 made the
"Supported FireWire (IEEE 1394) Adapters" comment disappear; bring it back
with corrected dependency.

Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Reviewed-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/Kconfig
drivers/media/dvb/firewire/Kconfig
drivers/media/dvb/firewire/Makefile
drivers/media/dvb/firewire/firedtv-1394.c [deleted file]
drivers/media/dvb/firewire/firedtv-dvb.c
drivers/media/dvb/firewire/firedtv.h

index 161ccfd..ee214c3 100644 (file)
@@ -65,7 +65,7 @@ comment "Supported SDMC DM1105 Adapters"
 source "drivers/media/dvb/dm1105/Kconfig"
 
 comment "Supported FireWire (IEEE 1394) Adapters"
-       depends on DVB_CORE && IEEE1394
+       depends on DVB_CORE && FIREWIRE
 source "drivers/media/dvb/firewire/Kconfig"
 
 comment "Supported Earthsoft PT1 Adapters"
index 4afa292..f3e9448 100644 (file)
@@ -1,6 +1,6 @@
 config DVB_FIREDTV
        tristate "FireDTV and FloppyDTV"
-       depends on DVB_CORE && (FIREWIRE || IEEE1394)
+       depends on DVB_CORE && FIREWIRE
        help
          Support for DVB receivers from Digital Everywhere
          which are connected via IEEE 1394 (FireWire).
@@ -13,12 +13,6 @@ config DVB_FIREDTV
 
 if DVB_FIREDTV
 
-config DVB_FIREDTV_FIREWIRE
-       def_bool FIREWIRE = y || (FIREWIRE = m && DVB_FIREDTV = m)
-
-config DVB_FIREDTV_IEEE1394
-       def_bool IEEE1394 = y || (IEEE1394 = m && DVB_FIREDTV = m)
-
 config DVB_FIREDTV_INPUT
        def_bool INPUT = y || (INPUT = m && DVB_FIREDTV = m)
 
index da84203..357b3aa 100644 (file)
@@ -1,9 +1,6 @@
 obj-$(CONFIG_DVB_FIREDTV) += firedtv.o
 
-firedtv-y := firedtv-avc.o firedtv-ci.o firedtv-dvb.o firedtv-fe.o
-firedtv-$(CONFIG_DVB_FIREDTV_FIREWIRE) += firedtv-fw.o
-firedtv-$(CONFIG_DVB_FIREDTV_IEEE1394) += firedtv-1394.o
+firedtv-y := firedtv-avc.o firedtv-ci.o firedtv-dvb.o firedtv-fe.o firedtv-fw.o
 firedtv-$(CONFIG_DVB_FIREDTV_INPUT)    += firedtv-rc.o
 
 ccflags-y += -Idrivers/media/dvb/dvb-core
-ccflags-$(CONFIG_DVB_FIREDTV_IEEE1394) += -Idrivers/ieee1394
diff --git a/drivers/media/dvb/firewire/firedtv-1394.c b/drivers/media/dvb/firewire/firedtv-1394.c
deleted file mode 100644 (file)
index b34ca7a..0000000
+++ /dev/null
@@ -1,300 +0,0 @@
-/*
- * FireDTV driver -- ieee1394 I/O backend
- *
- * Copyright (C) 2004 Andreas Monitzer <andy@monitzer.com>
- * Copyright (C) 2007-2008 Ben Backx <ben@bbackx.com>
- * Copyright (C) 2008 Henrik Kurelid <henrik@kurelid.se>
- *
- *     This program is free software; you can redistribute it and/or
- *     modify it under the terms of the GNU General Public License as
- *     published by the Free Software Foundation; either version 2 of
- *     the License, or (at your option) any later version.
- */
-
-#include <linux/device.h>
-#include <linux/errno.h>
-#include <linux/kernel.h>
-#include <linux/list.h>
-#include <linux/slab.h>
-#include <linux/spinlock.h>
-#include <linux/types.h>
-
-#include <dma.h>
-#include <csr1212.h>
-#include <highlevel.h>
-#include <hosts.h>
-#include <ieee1394.h>
-#include <iso.h>
-#include <nodemgr.h>
-
-#include <dvb_demux.h>
-
-#include "firedtv.h"
-
-static LIST_HEAD(node_list);
-static DEFINE_SPINLOCK(node_list_lock);
-
-#define CIP_HEADER_SIZE                        8
-#define MPEG2_TS_HEADER_SIZE           4
-#define MPEG2_TS_SOURCE_PACKET_SIZE    (4 + 188)
-
-static void rawiso_activity_cb(struct hpsb_iso *iso)
-{
-       struct firedtv *f, *fdtv = NULL;
-       unsigned int i, num, packet;
-       unsigned char *buf;
-       unsigned long flags;
-       int count;
-
-       spin_lock_irqsave(&node_list_lock, flags);
-       list_for_each_entry(f, &node_list, list)
-               if (f->backend_data == iso) {
-                       fdtv = f;
-                       break;
-               }
-       spin_unlock_irqrestore(&node_list_lock, flags);
-
-       packet = iso->first_packet;
-       num = hpsb_iso_n_ready(iso);
-
-       if (!fdtv) {
-               pr_err("received at unknown iso channel\n");
-               goto out;
-       }
-
-       for (i = 0; i < num; i++, packet = (packet + 1) % iso->buf_packets) {
-               buf = dma_region_i(&iso->data_buf, unsigned char,
-                       iso->infos[packet].offset + CIP_HEADER_SIZE);
-               count = (iso->infos[packet].len - CIP_HEADER_SIZE) /
-                       MPEG2_TS_SOURCE_PACKET_SIZE;
-
-               /* ignore empty packet */
-               if (iso->infos[packet].len <= CIP_HEADER_SIZE)
-                       continue;
-
-               while (count--) {
-                       if (buf[MPEG2_TS_HEADER_SIZE] == 0x47)
-                               dvb_dmx_swfilter_packets(&fdtv->demux,
-                                               &buf[MPEG2_TS_HEADER_SIZE], 1);
-                       else
-                               dev_err(fdtv->device,
-                                       "skipping invalid packet\n");
-                       buf += MPEG2_TS_SOURCE_PACKET_SIZE;
-               }
-       }
-out:
-       hpsb_iso_recv_release_packets(iso, num);
-}
-
-static inline struct node_entry *node_of(struct firedtv *fdtv)
-{
-       return container_of(fdtv->device, struct unit_directory, device)->ne;
-}
-
-static int node_lock(struct firedtv *fdtv, u64 addr, void *data)
-{
-       quadlet_t *d = data;
-       int ret;
-
-       ret = hpsb_node_lock(node_of(fdtv), addr,
-                            EXTCODE_COMPARE_SWAP, &d[1], d[0]);
-       d[0] = d[1];
-
-       return ret;
-}
-
-static int node_read(struct firedtv *fdtv, u64 addr, void *data)
-{
-       return hpsb_node_read(node_of(fdtv), addr, data, 4);
-}
-
-static int node_write(struct firedtv *fdtv, u64 addr, void *data, size_t len)
-{
-       return hpsb_node_write(node_of(fdtv), addr, data, len);
-}
-
-#define FDTV_ISO_BUFFER_PACKETS 256
-#define FDTV_ISO_BUFFER_SIZE (FDTV_ISO_BUFFER_PACKETS * 200)
-
-static int start_iso(struct firedtv *fdtv)
-{
-       struct hpsb_iso *iso_handle;
-       int ret;
-
-       iso_handle = hpsb_iso_recv_init(node_of(fdtv)->host,
-                               FDTV_ISO_BUFFER_SIZE, FDTV_ISO_BUFFER_PACKETS,
-                               fdtv->isochannel, HPSB_ISO_DMA_DEFAULT,
-                               -1, /* stat.config.irq_interval */
-                               rawiso_activity_cb);
-       if (iso_handle == NULL) {
-               dev_err(fdtv->device, "cannot initialize iso receive\n");
-               return -ENOMEM;
-       }
-       fdtv->backend_data = iso_handle;
-
-       ret = hpsb_iso_recv_start(iso_handle, -1, -1, 0);
-       if (ret != 0) {
-               dev_err(fdtv->device, "cannot start iso receive\n");
-               hpsb_iso_shutdown(iso_handle);
-               fdtv->backend_data = NULL;
-       }
-       return ret;
-}
-
-static void stop_iso(struct firedtv *fdtv)
-{
-       struct hpsb_iso *iso_handle = fdtv->backend_data;
-
-       if (iso_handle != NULL) {
-               hpsb_iso_stop(iso_handle);
-               hpsb_iso_shutdown(iso_handle);
-       }
-       fdtv->backend_data = NULL;
-}
-
-static const struct firedtv_backend fdtv_1394_backend = {
-       .lock           = node_lock,
-       .read           = node_read,
-       .write          = node_write,
-       .start_iso      = start_iso,
-       .stop_iso       = stop_iso,
-};
-
-static void fcp_request(struct hpsb_host *host, int nodeid, int direction,
-                       int cts, u8 *data, size_t length)
-{
-       struct firedtv *f, *fdtv = NULL;
-       unsigned long flags;
-       int su;
-
-       if (length == 0 || (data[0] & 0xf0) != 0)
-               return;
-
-       su = data[1] & 0x7;
-
-       spin_lock_irqsave(&node_list_lock, flags);
-       list_for_each_entry(f, &node_list, list)
-               if (node_of(f)->host == host &&
-                   node_of(f)->nodeid == nodeid &&
-                   (f->subunit == su || (f->subunit == 0 && su == 0x7))) {
-                       fdtv = f;
-                       break;
-               }
-       spin_unlock_irqrestore(&node_list_lock, flags);
-
-       if (fdtv)
-               avc_recv(fdtv, data, length);
-}
-
-static int node_probe(struct device *dev)
-{
-       struct unit_directory *ud =
-                       container_of(dev, struct unit_directory, device);
-       struct firedtv *fdtv;
-       int kv_len, err;
-       void *kv_str;
-
-       if (ud->model_name_kv) {
-               kv_len = (ud->model_name_kv->value.leaf.len - 2) * 4;
-               kv_str = CSR1212_TEXTUAL_DESCRIPTOR_LEAF_DATA(ud->model_name_kv);
-       } else {
-               kv_len = 0;
-               kv_str = NULL;
-       }
-       fdtv = fdtv_alloc(dev, &fdtv_1394_backend, kv_str, kv_len);
-       if (!fdtv)
-               return -ENOMEM;
-
-       /*
-        * Work around a bug in udev's path_id script:  Use the fw-host's dev
-        * instead of the unit directory's dev as parent of the input device.
-        */
-       err = fdtv_register_rc(fdtv, dev->parent->parent);
-       if (err)
-               goto fail_free;
-
-       spin_lock_irq(&node_list_lock);
-       list_add_tail(&fdtv->list, &node_list);
-       spin_unlock_irq(&node_list_lock);
-
-       err = avc_identify_subunit(fdtv);
-       if (err)
-               goto fail;
-
-       err = fdtv_dvb_register(fdtv);
-       if (err)
-               goto fail;
-
-       avc_register_remote_control(fdtv);
-
-       return 0;
-fail:
-       spin_lock_irq(&node_list_lock);
-       list_del(&fdtv->list);
-       spin_unlock_irq(&node_list_lock);
-       fdtv_unregister_rc(fdtv);
-fail_free:
-       kfree(fdtv);
-
-       return err;
-}
-
-static int node_remove(struct device *dev)
-{
-       struct firedtv *fdtv = dev_get_drvdata(dev);
-
-       fdtv_dvb_unregister(fdtv);
-
-       spin_lock_irq(&node_list_lock);
-       list_del(&fdtv->list);
-       spin_unlock_irq(&node_list_lock);
-
-       fdtv_unregister_rc(fdtv);
-       kfree(fdtv);
-
-       return 0;
-}
-
-static int node_update(struct unit_directory *ud)
-{
-       struct firedtv *fdtv = dev_get_drvdata(&ud->device);
-
-       if (fdtv->isochannel >= 0)
-               cmp_establish_pp_connection(fdtv, fdtv->subunit,
-                                           fdtv->isochannel);
-       return 0;
-}
-
-static struct hpsb_protocol_driver fdtv_driver = {
-       .name           = "firedtv",
-       .id_table       = fdtv_id_table,
-       .update         = node_update,
-       .driver         = {
-               .probe  = node_probe,
-               .remove = node_remove,
-       },
-};
-
-static struct hpsb_highlevel fdtv_highlevel = {
-       .name           = "firedtv",
-       .fcp_request    = fcp_request,
-};
-
-int __init fdtv_1394_init(void)
-{
-       int ret;
-
-       hpsb_register_highlevel(&fdtv_highlevel);
-       ret = hpsb_register_protocol(&fdtv_driver);
-       if (ret) {
-               printk(KERN_ERR "firedtv: failed to register protocol\n");
-               hpsb_unregister_highlevel(&fdtv_highlevel);
-       }
-       return ret;
-}
-
-void __exit fdtv_1394_exit(void)
-{
-       hpsb_unregister_protocol(&fdtv_driver);
-       hpsb_unregister_highlevel(&fdtv_highlevel);
-}
index 079e8c5..86349db 100644 (file)
@@ -351,16 +351,11 @@ static int __init fdtv_init(void)
        if (ret < 0)
                return ret;
 
-       ret = fdtv_1394_init();
-       if (ret < 0)
-               fdtv_fw_exit();
-
        return ret;
 }
 
 static void __exit fdtv_exit(void)
 {
-       fdtv_1394_exit();
        fdtv_fw_exit();
 }
 
index 78cc28f..4a87049 100644 (file)
@@ -118,15 +118,6 @@ struct firedtv {
        u8                      avc_data[512];
 };
 
-/* firedtv-1394.c */
-#ifdef CONFIG_DVB_FIREDTV_IEEE1394
-int fdtv_1394_init(void);
-void fdtv_1394_exit(void);
-#else
-static inline int fdtv_1394_init(void) { return 0; }
-static inline void fdtv_1394_exit(void) {}
-#endif
-
 /* firedtv-avc.c */
 int avc_recv(struct firedtv *fdtv, void *data, size_t length);
 int avc_tuner_status(struct firedtv *fdtv, struct firedtv_tuner_status *stat);
@@ -170,13 +161,8 @@ extern const struct ieee1394_device_id fdtv_id_table[];
 void fdtv_frontend_init(struct firedtv *fdtv);
 
 /* firedtv-fw.c */
-#ifdef CONFIG_DVB_FIREDTV_FIREWIRE
 int fdtv_fw_init(void);
 void fdtv_fw_exit(void);
-#else
-static inline int fdtv_fw_init(void) { return 0; }
-static inline void fdtv_fw_exit(void) {}
-#endif
 
 /* firedtv-rc.c */
 #ifdef CONFIG_DVB_FIREDTV_INPUT