* [PATCH] chcr: remove unused variables net_device, pi, adap and cntrl
@ 2017-11-30 14:11 Colin King
2017-12-11 11:46 ` Herbert Xu
0 siblings, 1 reply; 3+ messages in thread
From: Colin King @ 2017-11-30 14:11 UTC (permalink / raw)
To: Harsh Jain, Herbert Xu, David S . Miller, linux-crypto
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
Variables adap, pi and cntrl are assigned but are never read, hence
they are redundant and can be removed.
Cleans up various clang build warnings.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
drivers/crypto/chelsio/chcr_ipsec.c | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/drivers/crypto/chelsio/chcr_ipsec.c b/drivers/crypto/chelsio/chcr_ipsec.c
index f90f991c133f..a0f003ffd1f4 100644
--- a/drivers/crypto/chelsio/chcr_ipsec.c
+++ b/drivers/crypto/chelsio/chcr_ipsec.c
@@ -199,14 +199,9 @@ static inline int chcr_ipsec_setkey(struct xfrm_state *x,
*/
static int chcr_xfrm_add_state(struct xfrm_state *x)
{
- struct net_device *netdev = x->xso.dev;
- struct port_info *pi = netdev_priv(netdev);
struct ipsec_sa_entry *sa_entry;
- struct adapter *adap;
int res = 0;
- adap = pi->adapter;
-
if (x->props.aalgo != SADB_AALG_NONE) {
pr_debug("CHCR: Cannot offload authenticated xfrm states\n");
return -EINVAL;
@@ -570,7 +565,7 @@ int chcr_ipsec_xmit(struct sk_buff *skb, struct net_device *dev)
{
struct xfrm_state *x = xfrm_input_state(skb);
struct ipsec_sa_entry *sa_entry;
- u64 *pos, *end, *before, cntrl, *sgl;
+ u64 *pos, *end, *before, *sgl;
int qidx, left, credits;
unsigned int flits = 0, ndesc, kctx_len;
struct adapter *adap;
@@ -596,7 +591,6 @@ out_free: dev_kfree_skb_any(skb);
q = &adap->sge.ethtxq[qidx + pi->first_qset];
cxgb4_reclaim_completed_tx(adap, &q->q, true);
- cntrl = TXPKT_L4CSUM_DIS_F | TXPKT_IPCSUM_DIS_F;
flits = calc_tx_sec_flits(skb, sa_entry->kctx_len);
ndesc = flits_to_desc(flits);
--
2.14.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] chcr: remove unused variables net_device, pi, adap and cntrl
2017-11-30 14:11 [PATCH] chcr: remove unused variables net_device, pi, adap and cntrl Colin King
@ 2017-12-11 11:46 ` Herbert Xu
0 siblings, 0 replies; 3+ messages in thread
From: Herbert Xu @ 2017-12-11 11:46 UTC (permalink / raw)
To: Colin King
Cc: Harsh Jain, David S . Miller, linux-crypto, kernel-janitors,
linux-kernel
On Thu, Nov 30, 2017 at 02:11:44PM +0000, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> Variables adap, pi and cntrl are assigned but are never read, hence
> they are redundant and can be removed.
>
> Cleans up various clang build warnings.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
Patch applied. Thanks.
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] chcr: remove unused variables net_device, pi, adap and cntrl
@ 2017-12-11 11:46 ` Herbert Xu
0 siblings, 0 replies; 3+ messages in thread
From: Herbert Xu @ 2017-12-11 11:46 UTC (permalink / raw)
To: Colin King
Cc: Harsh Jain, David S . Miller, linux-crypto, kernel-janitors,
linux-kernel
On Thu, Nov 30, 2017 at 02:11:44PM +0000, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> Variables adap, pi and cntrl are assigned but are never read, hence
> they are redundant and can be removed.
>
> Cleans up various clang build warnings.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
Patch applied. Thanks.
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-12-11 11:47 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-30 14:11 [PATCH] chcr: remove unused variables net_device, pi, adap and cntrl Colin King
2017-12-11 11:46 ` Herbert Xu
2017-12-11 11:46 ` Herbert Xu
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.