]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
media: ttpci: coding style fixes: comments
authorStefan Herdler <herdler@nurfuerspam.de>
Mon, 15 Apr 2024 02:24:04 +0000 (04:24 +0200)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Mon, 15 Apr 2024 22:02:53 +0000 (00:02 +0200)
This patch fixes the following checkpatch warnings:

WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
WARNING: It's generally not useful to have the filename in the file

Signed-off-by: Stefan Herdler <herdler@nurfuerspam.de>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/media/pci/ttpci/budget-av.c
drivers/media/pci/ttpci/budget-ci.c
drivers/media/pci/ttpci/budget-core.c
drivers/media/pci/ttpci/budget.c

index b9efcd3ccfdb54c5ce99b82ef3f7a28fd1143058..dbd4ef40e86780a2026879ea98694e28eb7cbed6 100644 (file)
@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: GPL-2.0-or-later
 /*
- * budget-av.c: driver for the SAA7146 based Budget DVB cards
- *              with analog video in
+ * budget-av.ko: driver for the SAA7146 based Budget DVB cards
+ *               with analog video input (and optionally with CI)
  *
  * Compiled from various sources by Michael Hunold <michael@mihu.de>
  *
@@ -63,8 +63,8 @@ struct budget_av {
 
 static int ciintf_slot_shutdown(struct dvb_ca_en50221 *ca, int slot);
 
-
-/* GPIO Connections:
+/*
+ * GPIO Connections:
  * 0 - Vcc/Reset (Reset is controlled by capacitor). Resets the frontend *AS WELL*!
  * 1 - CI memory select 0=>IO memory, 1=>Attribute Memory
  * 2 - CI Card Enable (Active Low)
@@ -267,8 +267,10 @@ static int ciintf_poll_slot_status(struct dvb_ca_en50221 *ca, int slot, int open
        if (slot != 0)
                return -EINVAL;
 
-       /* test the card detect line - needs to be done carefully
-        * since it never goes high for some CAMs on this interface (e.g. topuptv) */
+       /*
+        * test the card detect line - needs to be done carefully
+        * since it never goes high for some CAMs on this interface (e.g. topuptv)
+        */
        if (budget_av->slot_status == SLOTSTATUS_NONE) {
                saa7146_setgpio(saa, 3, SAA7146_GPIO_INPUT);
                udelay(1);
@@ -281,12 +283,14 @@ static int ciintf_poll_slot_status(struct dvb_ca_en50221 *ca, int slot, int open
                saa7146_setgpio(saa, 3, SAA7146_GPIO_OUTLO);
        }
 
-       /* We also try and read from IO memory to work round the above detection bug. If
+       /*
+        * We also try and read from IO memory to work round the above detection bug. If
         * there is no CAM, we will get a timeout. Only done if there is no cam
         * present, since this test actually breaks some cams :(
         *
         * if the CI interface is not open, we also do the above test since we
-        * don't care if the cam has problems - we'll be resetting it on open() anyway */
+        * don't care if the cam has problems - we'll be resetting it on open() anyway
+        */
        if ((budget_av->slot_status == SLOTSTATUS_NONE) || (!open)) {
                saa7146_setgpio(budget_av->budget.dev, 1, SAA7146_GPIO_OUTLO);
                result = ttpci_budget_debiread(&budget_av->budget, DEBICICAM, 0, 1, 0, 1);
index 2859b8ab85aa83f9b66db33b2bab348c465f758c..ebf340417de0621c6bde70e4425d214b08414893 100644 (file)
@@ -1,6 +1,7 @@
 // SPDX-License-Identifier: GPL-2.0-or-later
 /*
- * budget-ci.c: driver for the SAA7146 based Budget DVB cards
+ * budget-ci.ko: driver for the SAA7146 based Budget DVB cards
+ *               with CI (but without analog video input)
  *
  * Compiled from various sources by Michael Hunold <michael@mihu.de>
  *
index ffa659be10c269824dcd2f51694e3c513c0c335c..f41f4eea7c0829d5f3a11328c48f62a85b9bd99e 100644 (file)
@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: GPL-2.0-or-later
 /*
- * budget-core.c: driver for the SAA7146 based Budget DVB cards
+ * budget-core.ko: base-driver for the SAA7146 based Budget DVB cards
  *
  * Compiled from various sources by Michael Hunold <michael@mihu.de>
  *
@@ -491,8 +491,10 @@ int ttpci_budget_init(struct budget *budget, struct saa7146_dev *dev,
        spin_lock_init(&budget->feedlock);
        spin_lock_init(&budget->debilock);
 
-       /* the Siemens DVB needs this if you want to have the i2c chips
-          get recognized before the main driver is loaded */
+       /*
+        * the Siemens DVB needs this if you want to have the i2c chips
+        * get recognized before the main driver is loaded
+        */
        if (bi->type != BUDGET_FS_ACTIVY)
                saa7146_write(dev, GPIO_CTRL, 0x500000);        /* GPIO 3 = 1 */
 
index faa2a3561899e96caca59027112ffaa67041f553..9fe087addcc656a97495095d454a153d032f76a9 100644 (file)
@@ -1,6 +1,7 @@
 // SPDX-License-Identifier: GPL-2.0-or-later
 /*
- * budget.c: driver for the SAA7146 based Budget DVB cards
+ * budget.ko: driver for the SAA7146 based Budget DVB cards
+ *            without analog video input or CI
  *
  * Compiled from various sources by Michael Hunold <michael@mihu.de>
  *
@@ -50,9 +51,11 @@ static void Set22K(struct budget *budget, int state)
        saa7146_setgpio(dev, 3, (state ? SAA7146_GPIO_OUTHI : SAA7146_GPIO_OUTLO));
 }
 
-/* Diseqc functions only for TT Budget card */
-/* taken from the Skyvision DVB driver by
-   Ralph Metzler <rjkm@metzlerbros.de> */
+/*
+ * Diseqc functions only for TT Budget card
+ * taken from the Skyvision DVB driver by
+ * Ralph Metzler <rjkm@metzlerbros.de>
+ */
 
 static void DiseqcSendBit(struct budget *budget, int data)
 {
@@ -675,9 +678,11 @@ static void frontend_init(struct budget *budget)
                                        tt1600_stv090x_config.tuner_set_refclk    = ctl->tuner_set_refclk;
                                        tt1600_stv090x_config.tuner_get_status    = ctl->tuner_get_status;
 
-                                       /* call the init function once to initialize
-                                          tuner's clock output divider and demod's
-                                          master clock */
+                                       /*
+                                        * call the init function once to initialize
+                                        * tuner's clock output divider and demod's
+                                        * master clock
+                                        */
                                        if (budget->dvb_frontend->ops.init)
                                                budget->dvb_frontend->ops.init(budget->dvb_frontend);
 
@@ -730,9 +735,11 @@ static void frontend_init(struct budget *budget)
                                        tt1600_stv090x_config.tuner_set_refclk    = ctl->tuner_set_refclk;
                                        tt1600_stv090x_config.tuner_get_status    = ctl->tuner_get_status;
 
-                                       /* call the init function once to initialize
-                                          tuner's clock output divider and demod's
-                                          master clock */
+                                       /*
+                                        * call the init function once to initialize
+                                        * tuner's clock output divider and demod's
+                                        * master clock
+                                        */
                                        if (budget->dvb_frontend->ops.init)
                                                budget->dvb_frontend->ops.init(budget->dvb_frontend);