From: "Tomas Winkler" <tomasw@gmail.com>
To: "Rami Rosen" <ramirose@gmail.com>
Cc: "John W. Linville" <linville@tuxdriver.com>,
linux-wireless@vger.kernel.org, reinette.chatre@intel.com
Subject: Re: [PATCH] [v4] iwlwifi: iwl-tx.c cleanup (remove unused parameter and unused local variable).
Date: Thu, 18 Dec 2008 18:27:17 +0200 [thread overview]
Message-ID: <1ba2fa240812180827nf1c27d1jb20ee8b0b1f04b62@mail.gmail.com> (raw)
In-Reply-To: <eb3ff54b0812180804i3a782509oa009ff138e3f055c@mail.gmail.com>
On Thu, Dec 18, 2008 at 6:04 PM, Rami Rosen <ramirose@gmail.com> wrote:
> This patch removes unused parameter and unused local variable in
> methods in iwl-tx.c:
>
> - Remove a parameter (is_unicast) from iwl_tx_cmd_build_basic().
>
> - Remove an unused variable name unicast from iwl_tx_skb().
>
> (wireless-testing).
>
> Signed-off-by: Rami Rosen <ramirose@gmail.com>
Ack
Tomas
>
> diff --git a/drivers/net/wireless/iwlwifi/iwl-tx.c
> b/drivers/net/wireless/iwlwifi/iwl-tx.c
> index 3cb4363..43dcfe3 100644
> --- a/drivers/net/wireless/iwlwifi/iwl-tx.c
> +++ b/drivers/net/wireless/iwlwifi/iwl-tx.c
> @@ -645,7 +645,7 @@ static void iwl_tx_cmd_build_basic(struct iwl_priv *priv,
> struct iwl_tx_cmd *tx_cmd,
> struct ieee80211_tx_info *info,
> struct ieee80211_hdr *hdr,
> - int is_unicast, u8 std_id)
> + u8 std_id)
> {
> __le16 fc = hdr->frame_control;
> __le32 tx_flags = tx_cmd->tx_flags;
> @@ -834,7 +834,7 @@ int iwl_tx_skb(struct iwl_priv *priv, struct sk_buff *skb)
> u16 len, len_org;
> u16 seq_number = 0;
> __le16 fc;
> - u8 hdr_len, unicast;
> + u8 hdr_len;
> u8 sta_id;
> u8 wait_write_ptr = 0;
> u8 tid = 0;
> @@ -854,8 +854,6 @@ int iwl_tx_skb(struct iwl_priv *priv, struct sk_buff *skb)
> goto drop_unlock;
> }
>
> - unicast = !is_multicast_ether_addr(hdr->addr1);
> -
> fc = hdr->frame_control;
>
> #ifdef CONFIG_IWLWIFI_DEBUG
> @@ -996,7 +994,7 @@ int iwl_tx_skb(struct iwl_priv *priv, struct sk_buff *skb)
> len = (u16)skb->len;
> tx_cmd->len = cpu_to_le16(len);
> /* TODO need this for burst mode later on */
> - iwl_tx_cmd_build_basic(priv, tx_cmd, info, hdr, unicast, sta_id);
> + iwl_tx_cmd_build_basic(priv, tx_cmd, info, hdr, sta_id);
>
> /* set is_hcca to 0; it probably will never be implemented */
> iwl_tx_cmd_build_rate(priv, tx_cmd, info, fc, sta_id, 0);
>
>
> On Thu, Dec 18, 2008 at 5:31 PM, John W. Linville
> <linville@tuxdriver.com> wrote:
>> On Thu, Dec 18, 2008 at 12:42:37AM +0200, Tomas Winkler wrote:
>>> On Wed, Dec 17, 2008 at 11:51 PM, Rami Rosen <ramirose@gmail.com> wrote:
>>> > This patch removes unused parameters in methods in iwl-tx.c:
>>> > remove a parameter (is_unicast) from iwl_tx_cmd_build_basic()
>>> > remove a parameter (sta_id) from iwl_tx_cmd_build_rate()
>>> > remove a parameter (sta_id ) from iwl_tx_cmd_build_hwcrypto()
>>>
>>> Unicast can go sta_id has to stay, we need it for merging 3945 back
>>
>> OK, I'll drop this patch and wait for a new submission.
>>
>> John
>> --
>> John W. Linville Linux should be at the core
>> linville@tuxdriver.com of your literate lifestyle.
>>
>
prev parent reply other threads:[~2008-12-18 16:27 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-12-18 16:04 [PATCH] [v4] iwlwifi: iwl-tx.c cleanup (remove unused parameter and unused local variable) Rami Rosen
2008-12-18 16:27 ` Tomas Winkler [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1ba2fa240812180827nf1c27d1jb20ee8b0b1f04b62@mail.gmail.com \
--to=tomasw@gmail.com \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=ramirose@gmail.com \
--cc=reinette.chatre@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox