diff options
| author | Christian Gromm <christian.gromm@microchip.com> | 2019-07-01 15:49:33 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-07-03 18:28:28 +0200 |
| commit | 8f9e3a519d56f420d2d4cee6521bae48b8273abf (patch) | |
| tree | 153b96f42957476cb5ae6917837fe1888565d492 | |
| parent | 23a4388f24f5356fa0bef07911c7b4fbc97619ac (diff) | |
| download | lwn-8f9e3a519d56f420d2d4cee6521bae48b8273abf.tar.gz lwn-8f9e3a519d56f420d2d4cee6521bae48b8273abf.zip | |
staging: most: remove data sanity check
This patch removes the data check in the set_cfg_* functions, because
the modules infacing the hardware (usb, i2c, ...) already have it.
Signed-off-by: Christian Gromm <christian.gromm@microchip.com>
Link: https://lore.kernel.org/r/1561988973-301-1-git-send-email-christian.gromm@microchip.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/most/configfs.c | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/drivers/staging/most/configfs.c b/drivers/staging/most/configfs.c index 1d8bf29e0ffb..025495657b68 100644 --- a/drivers/staging/most/configfs.c +++ b/drivers/staging/most/configfs.c @@ -35,56 +35,42 @@ static struct list_head mdev_link_list; static int set_cfg_buffer_size(struct mdev_link *link) { - if (!link->buffer_size) - return -ENODATA; return most_set_cfg_buffer_size(link->device, link->channel, link->buffer_size); } static int set_cfg_subbuffer_size(struct mdev_link *link) { - if (!link->subbuffer_size) - return -ENODATA; return most_set_cfg_subbuffer_size(link->device, link->channel, link->subbuffer_size); } static int set_cfg_dbr_size(struct mdev_link *link) { - if (!link->dbr_size) - return -ENODATA; return most_set_cfg_dbr_size(link->device, link->channel, link->dbr_size); } static int set_cfg_num_buffers(struct mdev_link *link) { - if (!link->num_buffers) - return -ENODATA; return most_set_cfg_num_buffers(link->device, link->channel, link->num_buffers); } static int set_cfg_packets_xact(struct mdev_link *link) { - if (!link->packets_per_xact) - return -ENODATA; return most_set_cfg_packets_xact(link->device, link->channel, link->packets_per_xact); } static int set_cfg_direction(struct mdev_link *link) { - if (!strlen(link->direction)) - return -ENODATA; return most_set_cfg_direction(link->device, link->channel, link->direction); } static int set_cfg_datatype(struct mdev_link *link) { - if (!strlen(link->datatype)) - return -ENODATA; return most_set_cfg_datatype(link->device, link->channel, link->datatype); } |