Commit a72319bf authored by Jérôme Pouiller's avatar Jérôme Pouiller Committed by Greg Kroah-Hartman

staging: wfx: apply 80-columns rule to strings

Strings are allowed to exceed 80 columns but, in this case, the format
arguments should be placed on a new line. Apply this rule to the whole
code of the driver.
Signed-off-by: default avatarJérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20200515083325.378539-3-Jerome.Pouiller@silabs.comSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 9773cf1c
...@@ -180,7 +180,8 @@ static int wfx_sdio_probe(struct sdio_func *func, ...@@ -180,7 +180,8 @@ static int wfx_sdio_probe(struct sdio_func *func,
int ret; int ret;
if (func->num != 1) { if (func->num != 1) {
dev_err(&func->dev, "SDIO function number is %d while it should always be 1 (unsupported chip?)\n", func->num); dev_err(&func->dev, "SDIO function number is %d while it should always be 1 (unsupported chip?)\n",
func->num);
return -ENODEV; return -ENODEV;
} }
......
...@@ -550,8 +550,8 @@ void wfx_tx_confirm_cb(struct wfx_vif *wvif, const struct hif_cnf_tx *arg) ...@@ -550,8 +550,8 @@ void wfx_tx_confirm_cb(struct wfx_vif *wvif, const struct hif_cnf_tx *arg)
} }
} }
if (tx_count) if (tx_count)
dev_dbg(wvif->wdev->dev, dev_dbg(wvif->wdev->dev, "%d more retries than expected\n",
"%d more retries than expected\n", tx_count); tx_count);
skb_trim(skb, skb->len - wfx_tx_get_icv_len(tx_priv->hw_key)); skb_trim(skb, skb->len - wfx_tx_get_icv_len(tx_priv->hw_key));
// From now, you can touch to tx_info->status, but do not touch to // From now, you can touch to tx_info->status, but do not touch to
......
...@@ -107,8 +107,8 @@ static int get_firmware(struct wfx_dev *wdev, u32 keyset_chip, ...@@ -107,8 +107,8 @@ static int get_firmware(struct wfx_dev *wdev, u32 keyset_chip,
const char *data; const char *data;
int ret; int ret;
snprintf(filename, sizeof(filename), "%s_%02X.sec", wdev->pdata.file_fw, snprintf(filename, sizeof(filename), "%s_%02X.sec",
keyset_chip); wdev->pdata.file_fw, keyset_chip);
ret = firmware_request_nowarn(fw, filename, wdev->dev); ret = firmware_request_nowarn(fw, filename, wdev->dev);
if (ret) { if (ret) {
dev_info(wdev->dev, "can't load %s, falling back to %s.sec\n", dev_info(wdev->dev, "can't load %s, falling back to %s.sec\n",
...@@ -325,8 +325,8 @@ static int init_gpr(struct wfx_dev *wdev) ...@@ -325,8 +325,8 @@ static int init_gpr(struct wfx_dev *wdev)
gpr_init[i].value); gpr_init[i].value);
if (ret < 0) if (ret < 0)
return ret; return ret;
dev_dbg(wdev->dev, " index %02x: %08x\n", gpr_init[i].index, dev_dbg(wdev->dev, " index %02x: %08x\n",
gpr_init[i].value); gpr_init[i].index, gpr_init[i].value);
} }
return 0; return 0;
} }
......
...@@ -192,8 +192,8 @@ int hif_read_mib(struct wfx_dev *wdev, int vif_id, u16 mib_id, ...@@ -192,8 +192,8 @@ int hif_read_mib(struct wfx_dev *wdev, int vif_id, u16 mib_id,
ret = wfx_cmd_send(wdev, hif, reply, buf_len, false); ret = wfx_cmd_send(wdev, hif, reply, buf_len, false);
if (!ret && mib_id != le16_to_cpu(reply->mib_id)) { if (!ret && mib_id != le16_to_cpu(reply->mib_id)) {
dev_warn(wdev->dev, dev_warn(wdev->dev, "%s: confirmation mismatch request\n",
"%s: confirmation mismatch request\n", __func__); __func__);
ret = -EIO; ret = -EIO;
} }
if (ret == -ENOMEM) if (ret == -ENOMEM)
......
...@@ -192,12 +192,12 @@ struct gpio_desc *wfx_get_gpio(struct device *dev, ...@@ -192,12 +192,12 @@ struct gpio_desc *wfx_get_gpio(struct device *dev,
if (!ret || PTR_ERR(ret) == -ENOENT) if (!ret || PTR_ERR(ret) == -ENOENT)
dev_warn(dev, "gpio %s is not defined\n", label); dev_warn(dev, "gpio %s is not defined\n", label);
else else
dev_warn(dev, dev_warn(dev, "error while requesting gpio %s\n",
"error while requesting gpio %s\n", label); label);
ret = NULL; ret = NULL;
} else { } else {
dev_dbg(dev, dev_dbg(dev, "using gpio %d for %s\n",
"using gpio %d for %s\n", desc_to_gpio(ret), label); desc_to_gpio(ret), label);
} }
return ret; return ret;
} }
...@@ -230,15 +230,18 @@ int wfx_send_pds(struct wfx_dev *wdev, u8 *buf, size_t len) ...@@ -230,15 +230,18 @@ int wfx_send_pds(struct wfx_dev *wdev, u8 *buf, size_t len)
ret = hif_configuration(wdev, buf + start, ret = hif_configuration(wdev, buf + start,
i - start + 1); i - start + 1);
if (ret > 0) { if (ret > 0) {
dev_err(wdev->dev, "PDS bytes %d to %d: invalid data (unsupported options?)\n", start, i); dev_err(wdev->dev, "PDS bytes %d to %d: invalid data (unsupported options?)\n",
start, i);
return -EINVAL; return -EINVAL;
} }
if (ret == -ETIMEDOUT) { if (ret == -ETIMEDOUT) {
dev_err(wdev->dev, "PDS bytes %d to %d: chip didn't reply (corrupted file?)\n", start, i); dev_err(wdev->dev, "PDS bytes %d to %d: chip didn't reply (corrupted file?)\n",
start, i);
return ret; return ret;
} }
if (ret) { if (ret) {
dev_err(wdev->dev, "PDS bytes %d to %d: chip returned an unknown error\n", start, i); dev_err(wdev->dev, "PDS bytes %d to %d: chip returned an unknown error\n",
start, i);
return -EIO; return -EIO;
} }
buf[i] = ','; buf[i] = ',';
......
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