Commit 7aed13d1 authored by Christian Gromm's avatar Christian Gromm Committed by Greg Kroah-Hartman

staging: most: cdev: remove overcautious parameter checking

The interface pointer passed to a component API function cannot be NULL.
This patch removes unnecessary sanity checks of the pointer.
Signed-off-by: default avatarChristian Gromm <christian.gromm@microchip.com>
Link: https://lore.kernel.org/r/1592815695-19305-4-git-send-email-christian.gromm@microchip.comSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 08839388
...@@ -324,9 +324,6 @@ static int comp_disconnect_channel(struct most_interface *iface, int channel_id) ...@@ -324,9 +324,6 @@ static int comp_disconnect_channel(struct most_interface *iface, int channel_id)
{ {
struct comp_channel *c; struct comp_channel *c;
if (!iface)
return -EINVAL;
c = get_channel(iface, channel_id); c = get_channel(iface, channel_id);
if (!c) if (!c)
return -ENXIO; return -ENXIO;
...@@ -391,9 +388,6 @@ static int comp_tx_completion(struct most_interface *iface, int channel_id) ...@@ -391,9 +388,6 @@ static int comp_tx_completion(struct most_interface *iface, int channel_id)
{ {
struct comp_channel *c; struct comp_channel *c;
if (!iface)
return -EINVAL;
c = get_channel(iface, channel_id); c = get_channel(iface, channel_id);
if (!c) if (!c)
return -ENXIO; return -ENXIO;
...@@ -426,7 +420,7 @@ static int comp_probe(struct most_interface *iface, int channel_id, ...@@ -426,7 +420,7 @@ static int comp_probe(struct most_interface *iface, int channel_id,
int retval; int retval;
int current_minor; int current_minor;
if ((!iface) || (!cfg) || (!name)) if ((!cfg) || (!name))
return -EINVAL; return -EINVAL;
c = get_channel(iface, channel_id); c = get_channel(iface, channel_id);
......
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