Commit c9638363 authored by Colin Ian King's avatar Colin Ian King Committed by Greg Kroah-Hartman

staging: wfx: fix a handful of spelling mistakes

There are various spelling mistakes in comments and error messages.
Fix these.
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Acked-by: default avatarRandy Dunlap <rdunlap@infradead.org>
Reviewed-by: default avatarJérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20200806104701.46123-1-colin.king@canonical.comSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent afa096c6
...@@ -73,7 +73,7 @@ void wfx_rx_cb(struct wfx_vif *wvif, ...@@ -73,7 +73,7 @@ void wfx_rx_cb(struct wfx_vif *wvif,
if (arg->rx_flags.encryp) if (arg->rx_flags.encryp)
hdr->flag |= RX_FLAG_DECRYPTED; hdr->flag |= RX_FLAG_DECRYPTED;
// Block ack negociation is offloaded by the firmware. However, // Block ack negotiation is offloaded by the firmware. However,
// re-ordering must be done by the mac80211. // re-ordering must be done by the mac80211.
if (ieee80211_is_action(frame->frame_control) && if (ieee80211_is_action(frame->frame_control) &&
mgmt->u.action.category == WLAN_CATEGORY_BACK && mgmt->u.action.category == WLAN_CATEGORY_BACK &&
......
...@@ -234,7 +234,7 @@ static void wfx_tx_fixup_rates(struct ieee80211_tx_rate *rates) ...@@ -234,7 +234,7 @@ static void wfx_tx_fixup_rates(struct ieee80211_tx_rate *rates)
int i; int i;
bool finished; bool finished;
// Firmware is not able to mix rates with differents flags // Firmware is not able to mix rates with different flags
for (i = 0; i < IEEE80211_TX_MAX_RATES; i++) { for (i = 0; i < IEEE80211_TX_MAX_RATES; i++) {
if (rates[0].flags & IEEE80211_TX_RC_SHORT_GI) if (rates[0].flags & IEEE80211_TX_RC_SHORT_GI)
rates[i].flags |= IEEE80211_TX_RC_SHORT_GI; rates[i].flags |= IEEE80211_TX_RC_SHORT_GI;
......
...@@ -267,7 +267,7 @@ static ssize_t wfx_send_hif_msg_write(struct file *file, ...@@ -267,7 +267,7 @@ static ssize_t wfx_send_hif_msg_write(struct file *file,
if (count < sizeof(struct hif_msg)) if (count < sizeof(struct hif_msg))
return -EINVAL; return -EINVAL;
// wfx_cmd_send() chekc that reply buffer is wide enough, but do not // wfx_cmd_send() checks that reply buffer is wide enough, but does not
// return precise length read. User have to know how many bytes should // return precise length read. User have to know how many bytes should
// be read. Filling reply buffer with a memory pattern may help user. // be read. Filling reply buffer with a memory pattern may help user.
memset(context->reply, 0xFF, sizeof(context->reply)); memset(context->reply, 0xFF, sizeof(context->reply));
...@@ -299,8 +299,8 @@ static ssize_t wfx_send_hif_msg_read(struct file *file, char __user *user_buf, ...@@ -299,8 +299,8 @@ static ssize_t wfx_send_hif_msg_read(struct file *file, char __user *user_buf,
return ret; return ret;
if (context->ret < 0) if (context->ret < 0)
return context->ret; return context->ret;
// Be carefull, write() is waiting for a full message while read() // Be careful, write() is waiting for a full message while read()
// only return a payload // only returns a payload
if (copy_to_user(user_buf, context->reply, count)) if (copy_to_user(user_buf, context->reply, count))
return -EFAULT; return -EFAULT;
......
...@@ -118,7 +118,7 @@ static int hif_keys_indication(struct wfx_dev *wdev, ...@@ -118,7 +118,7 @@ static int hif_keys_indication(struct wfx_dev *wdev,
// SL_PUB_KEY_EXCHANGE_STATUS_SUCCESS is used by legacy secure link // SL_PUB_KEY_EXCHANGE_STATUS_SUCCESS is used by legacy secure link
if (body->status && body->status != HIF_STATUS_SLK_NEGO_SUCCESS) if (body->status && body->status != HIF_STATUS_SLK_NEGO_SUCCESS)
dev_warn(wdev->dev, "secure link negociation error\n"); dev_warn(wdev->dev, "secure link negotiation error\n");
memcpy(pubkey, body->ncp_pub_key, sizeof(pubkey)); memcpy(pubkey, body->ncp_pub_key, sizeof(pubkey));
memreverse(pubkey, sizeof(pubkey)); memreverse(pubkey, sizeof(pubkey));
wfx_sl_check_pubkey(wdev, pubkey, body->ncp_pub_key_mac); wfx_sl_check_pubkey(wdev, pubkey, body->ncp_pub_key_mac);
......
...@@ -78,7 +78,7 @@ int wfx_cmd_send(struct wfx_dev *wdev, struct hif_msg *request, ...@@ -78,7 +78,7 @@ int wfx_cmd_send(struct wfx_dev *wdev, struct hif_msg *request,
wfx_bh_request_tx(wdev); wfx_bh_request_tx(wdev);
// NOTE: no timeout is catched async is enabled // NOTE: no timeout is caught async is enabled
if (async) if (async)
return 0; return 0;
...@@ -125,7 +125,7 @@ int wfx_cmd_send(struct wfx_dev *wdev, struct hif_msg *request, ...@@ -125,7 +125,7 @@ int wfx_cmd_send(struct wfx_dev *wdev, struct hif_msg *request,
// This function is special. After HIF_REQ_ID_SHUT_DOWN, chip won't reply to any // This function is special. After HIF_REQ_ID_SHUT_DOWN, chip won't reply to any
// request anymore. We need to slightly hack struct wfx_hif_cmd for that job. Be // request anymore. We need to slightly hack struct wfx_hif_cmd for that job. Be
// carefull to only call this funcion during device unregister. // careful to only call this function during device unregister.
int hif_shutdown(struct wfx_dev *wdev) int hif_shutdown(struct wfx_dev *wdev)
{ {
int ret; int ret;
......
...@@ -384,7 +384,7 @@ int wfx_probe(struct wfx_dev *wdev) ...@@ -384,7 +384,7 @@ int wfx_probe(struct wfx_dev *wdev)
err = wfx_sl_init(wdev); err = wfx_sl_init(wdev);
if (err && wdev->hw_caps.capabilities.link_mode == SEC_LINK_ENFORCED) { if (err && wdev->hw_caps.capabilities.link_mode == SEC_LINK_ENFORCED) {
dev_err(wdev->dev, dev_err(wdev->dev,
"chip require secure_link, but can't negociate it\n"); "chip require secure_link, but can't negotiate it\n");
goto err0; goto err0;
} }
......
...@@ -19,7 +19,7 @@ struct wfx_dev; ...@@ -19,7 +19,7 @@ struct wfx_dev;
struct hwbus_ops; struct hwbus_ops;
struct wfx_platform_data { struct wfx_platform_data {
/* Keyset and ".sec" extention will appended to this string */ /* Keyset and ".sec" extension will be appended to this string */
const char *file_fw; const char *file_fw;
const char *file_pds; const char *file_pds;
struct gpio_desc *gpio_wakeup; struct gpio_desc *gpio_wakeup;
......
...@@ -214,7 +214,7 @@ static int wfx_get_ps_timeout(struct wfx_vif *wvif, bool *enable_ps) ...@@ -214,7 +214,7 @@ static int wfx_get_ps_timeout(struct wfx_vif *wvif, bool *enable_ps)
if (chan0 && chan1 && chan0->hw_value != chan1->hw_value && if (chan0 && chan1 && chan0->hw_value != chan1->hw_value &&
wvif->vif->type != NL80211_IFTYPE_AP) { wvif->vif->type != NL80211_IFTYPE_AP) {
// It is necessary to enable powersave if channels // It is necessary to enable powersave if channels
// are differents. // are different.
if (enable_ps) if (enable_ps)
*enable_ps = true; *enable_ps = true;
if (wvif->wdev->force_ps_timeout > -1) if (wvif->wdev->force_ps_timeout > -1)
......
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