This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: ddbridge/ci: further deduplicate code/logic in ddb_ci_attach()
Author:  Daniel Scheller <d.schel...@gmx.net>
Date:    Thu Feb 8 14:53:12 2018 -0500

Deduplicate the checks for a valid ptr in port->en, and also handle the
default case to also catch eventually yet unsupported CI hardware.

Signed-off-by: Daniel Scheller <d.schel...@gmx.net>
Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com>

 drivers/media/pci/ddbridge/ddbridge-ci.c | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

---

diff --git a/drivers/media/pci/ddbridge/ddbridge-ci.c 
b/drivers/media/pci/ddbridge/ddbridge-ci.c
index 5828111487b0..ed19890710d6 100644
--- a/drivers/media/pci/ddbridge/ddbridge-ci.c
+++ b/drivers/media/pci/ddbridge/ddbridge-ci.c
@@ -325,24 +325,20 @@ int ddb_ci_attach(struct ddb_port *port, u32 bitrate)
        case DDB_CI_EXTERNAL_SONY:
                cxd_cfg.bitrate = bitrate;
                port->en = cxd2099_attach(&cxd_cfg, port, &port->i2c->adap);
-               if (!port->en)
-                       return -ENODEV;
                break;
-
        case DDB_CI_EXTERNAL_XO2:
        case DDB_CI_EXTERNAL_XO2_B:
                ci_xo2_attach(port);
-               if (!port->en)
-                       return -ENODEV;
                break;
-
        case DDB_CI_INTERNAL:
                ci_attach(port);
-               if (!port->en)
-                       return -ENODEV;
                break;
+       default:
+               return -ENODEV;
        }
 
+       if (!port->en)
+               return -ENODEV;
        dvb_ca_en50221_init(port->dvb[0].adap, port->en, 0, 1);
        return 0;
 }

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to