diff options
| author | Alex Elder <elder@linaro.org> | 2020-11-09 17:56:35 +0100 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2020-11-11 23:07:00 +0100 |
| commit | bf795af1d42a6e7711e39c4fc64f452cc83dde97 (patch) | |
| tree | c31a5a9a0327ad11e198f872796e9b78148c832b /drivers/net/ipa/gsi.c | |
| parent | net: ipa: change a warning to debug (diff) | |
| download | linux-bf795af1d42a6e7711e39c4fc64f452cc83dde97.tar.xz linux-bf795af1d42a6e7711e39c4fc64f452cc83dde97.zip | |
net: ipa: drop an error message
There is no need for gsi_modem_channel_halt() to report an error,
because gsi_generic_command() will already have done that if the
command times out. So get rid of the extra message.
Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'drivers/net/ipa/gsi.c')
| -rw-r--r-- | drivers/net/ipa/gsi.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/net/ipa/gsi.c b/drivers/net/ipa/gsi.c index 961a11d4fb27..3a5998a037da 100644 --- a/drivers/net/ipa/gsi.c +++ b/drivers/net/ipa/gsi.c @@ -1661,12 +1661,7 @@ static int gsi_modem_channel_alloc(struct gsi *gsi, u32 channel_id) static void gsi_modem_channel_halt(struct gsi *gsi, u32 channel_id) { - int ret; - - ret = gsi_generic_command(gsi, channel_id, GSI_GENERIC_HALT_CHANNEL); - if (ret) - dev_err(gsi->dev, "error %d halting modem channel %u\n", - ret, channel_id); + (void)gsi_generic_command(gsi, channel_id, GSI_GENERIC_HALT_CHANNEL); } /* Setup function for channels */ |