The interface pointer passed to a component API function cannot be NULL.
This patch removes unnecessary sanity checks of the pointer.

Signed-off-by: Christian Gromm <christian.gr...@microchip.com>
---
 drivers/staging/most/cdev/cdev.c | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/drivers/staging/most/cdev/cdev.c b/drivers/staging/most/cdev/cdev.c
index 394c762..9fc84fa 100644
--- a/drivers/staging/most/cdev/cdev.c
+++ b/drivers/staging/most/cdev/cdev.c
@@ -324,9 +324,6 @@ static int comp_disconnect_channel(struct most_interface 
*iface, int channel_id)
 {
        struct comp_channel *c;
 
-       if (!iface)
-               return -EINVAL;
-
        c = get_channel(iface, channel_id);
        if (!c)
                return -ENXIO;
@@ -391,9 +388,6 @@ static int comp_tx_completion(struct most_interface *iface, 
int channel_id)
 {
        struct comp_channel *c;
 
-       if (!iface)
-               return -EINVAL;
-
        c = get_channel(iface, channel_id);
        if (!c)
                return -ENXIO;
@@ -426,7 +420,7 @@ static int comp_probe(struct most_interface *iface, int 
channel_id,
        int retval;
        int current_minor;
 
-       if ((!iface) || (!cfg) || (!name))
+       if ((!cfg) || (!name))
                return -EINVAL;
 
        c = get_channel(iface, channel_id);
-- 
2.7.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to