mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2024-12-21 08:46:49 +07:00
brcmfmac: Combine protocol push hdr and bus txdata.
For the transmission of data a protocol push hdr is performed followed by a bus txdata call. For the new protocol msgbuf this is not workable. Since they are already "loosely" coupled for bcdc protocol they are combined. This means that txdata will go "through" the protocol layer and a seperate protocol push hdr will not be needed anymore. Reviewed-by: Arend Van Spriel <arend@broadcom.com> Reviewed-by: Franky (Zhenhui) Lin <frankyl@broadcom.com> Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: Hante Meuleman <meuleman@broadcom.com> Signed-off-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
parent
943258b6a3
commit
7b8a466e7c
@ -329,6 +329,15 @@ brcmf_proto_bcdc_hdrpull(struct brcmf_pub *drvr, bool do_fws, u8 *ifidx,
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static int
|
||||||
|
brcmf_proto_bcdc_txdata(struct brcmf_pub *drvr, int ifidx, u8 offset,
|
||||||
|
struct sk_buff *pktbuf)
|
||||||
|
{
|
||||||
|
brcmf_proto_bcdc_hdrpush(drvr, ifidx, offset, pktbuf);
|
||||||
|
return brcmf_bus_txdata(drvr->bus_if, pktbuf);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
int brcmf_proto_bcdc_attach(struct brcmf_pub *drvr)
|
int brcmf_proto_bcdc_attach(struct brcmf_pub *drvr)
|
||||||
{
|
{
|
||||||
struct brcmf_bcdc *bcdc;
|
struct brcmf_bcdc *bcdc;
|
||||||
@ -343,10 +352,10 @@ int brcmf_proto_bcdc_attach(struct brcmf_pub *drvr)
|
|||||||
goto fail;
|
goto fail;
|
||||||
}
|
}
|
||||||
|
|
||||||
drvr->proto->hdrpush = brcmf_proto_bcdc_hdrpush;
|
|
||||||
drvr->proto->hdrpull = brcmf_proto_bcdc_hdrpull;
|
drvr->proto->hdrpull = brcmf_proto_bcdc_hdrpull;
|
||||||
drvr->proto->query_dcmd = brcmf_proto_bcdc_query_dcmd;
|
drvr->proto->query_dcmd = brcmf_proto_bcdc_query_dcmd;
|
||||||
drvr->proto->set_dcmd = brcmf_proto_bcdc_set_dcmd;
|
drvr->proto->set_dcmd = brcmf_proto_bcdc_set_dcmd;
|
||||||
|
drvr->proto->txdata = brcmf_proto_bcdc_txdata;
|
||||||
drvr->proto->pd = bcdc;
|
drvr->proto->pd = bcdc;
|
||||||
|
|
||||||
drvr->hdrlen += BCDC_HEADER_LEN + BRCMF_PROT_FW_SIGNAL_MAX_TXBYTES;
|
drvr->hdrlen += BCDC_HEADER_LEN + BRCMF_PROT_FW_SIGNAL_MAX_TXBYTES;
|
||||||
|
@ -838,7 +838,7 @@ static void brcmf_fws_cleanup(struct brcmf_fws_info *fws, int ifidx)
|
|||||||
brcmf_fws_hanger_cleanup(fws, matchfn, ifidx);
|
brcmf_fws_hanger_cleanup(fws, matchfn, ifidx);
|
||||||
}
|
}
|
||||||
|
|
||||||
static int brcmf_fws_hdrpush(struct brcmf_fws_info *fws, struct sk_buff *skb)
|
static u8 brcmf_fws_hdrpush(struct brcmf_fws_info *fws, struct sk_buff *skb)
|
||||||
{
|
{
|
||||||
struct brcmf_fws_mac_descriptor *entry = brcmf_skbcb(skb)->mac;
|
struct brcmf_fws_mac_descriptor *entry = brcmf_skbcb(skb)->mac;
|
||||||
u8 *wlh;
|
u8 *wlh;
|
||||||
@ -887,9 +887,7 @@ static int brcmf_fws_hdrpush(struct brcmf_fws_info *fws, struct sk_buff *skb)
|
|||||||
if (fillers)
|
if (fillers)
|
||||||
memset(wlh, BRCMF_FWS_TYPE_FILLER, fillers);
|
memset(wlh, BRCMF_FWS_TYPE_FILLER, fillers);
|
||||||
|
|
||||||
brcmf_proto_hdrpush(fws->drvr, brcmf_skb_if_flags_get_field(skb, INDEX),
|
return (u8)(data_offset >> 2);
|
||||||
data_offset >> 2, skb);
|
|
||||||
return 0;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static bool brcmf_fws_tim_update(struct brcmf_fws_info *fws,
|
static bool brcmf_fws_tim_update(struct brcmf_fws_info *fws,
|
||||||
@ -897,10 +895,11 @@ static bool brcmf_fws_tim_update(struct brcmf_fws_info *fws,
|
|||||||
int fifo, bool send_immediately)
|
int fifo, bool send_immediately)
|
||||||
{
|
{
|
||||||
struct sk_buff *skb;
|
struct sk_buff *skb;
|
||||||
struct brcmf_bus *bus;
|
|
||||||
struct brcmf_skbuff_cb *skcb;
|
struct brcmf_skbuff_cb *skcb;
|
||||||
s32 err;
|
s32 err;
|
||||||
u32 len;
|
u32 len;
|
||||||
|
u8 data_offset;
|
||||||
|
int ifidx;
|
||||||
|
|
||||||
/* check delayedQ and suppressQ in one call using bitmap */
|
/* check delayedQ and suppressQ in one call using bitmap */
|
||||||
if (brcmu_pktq_mlen(&entry->psq, 3 << (fifo * 2)) == 0)
|
if (brcmu_pktq_mlen(&entry->psq, 3 << (fifo * 2)) == 0)
|
||||||
@ -928,13 +927,11 @@ static bool brcmf_fws_tim_update(struct brcmf_fws_info *fws,
|
|||||||
skcb->state = BRCMF_FWS_SKBSTATE_TIM;
|
skcb->state = BRCMF_FWS_SKBSTATE_TIM;
|
||||||
skcb->htod = 0;
|
skcb->htod = 0;
|
||||||
skcb->htod_seq = 0;
|
skcb->htod_seq = 0;
|
||||||
bus = fws->drvr->bus_if;
|
data_offset = brcmf_fws_hdrpush(fws, skb);
|
||||||
err = brcmf_fws_hdrpush(fws, skb);
|
ifidx = brcmf_skb_if_flags_get_field(skb, INDEX);
|
||||||
if (err == 0) {
|
|
||||||
brcmf_fws_unlock(fws);
|
brcmf_fws_unlock(fws);
|
||||||
err = brcmf_bus_txdata(bus, skb);
|
err = brcmf_proto_txdata(fws->drvr, ifidx, data_offset, skb);
|
||||||
brcmf_fws_lock(fws);
|
brcmf_fws_lock(fws);
|
||||||
}
|
|
||||||
if (err)
|
if (err)
|
||||||
brcmu_pkt_buf_free_skb(skb);
|
brcmu_pkt_buf_free_skb(skb);
|
||||||
return true;
|
return true;
|
||||||
@ -1719,7 +1716,7 @@ int brcmf_fws_hdrpull(struct brcmf_pub *drvr, int ifidx, s16 signal_len,
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static void brcmf_fws_precommit_skb(struct brcmf_fws_info *fws, int fifo,
|
static u8 brcmf_fws_precommit_skb(struct brcmf_fws_info *fws, int fifo,
|
||||||
struct sk_buff *p)
|
struct sk_buff *p)
|
||||||
{
|
{
|
||||||
struct brcmf_skbuff_cb *skcb = brcmf_skbcb(p);
|
struct brcmf_skbuff_cb *skcb = brcmf_skbcb(p);
|
||||||
@ -1737,7 +1734,7 @@ static void brcmf_fws_precommit_skb(struct brcmf_fws_info *fws, int fifo,
|
|||||||
flags |= BRCMF_FWS_HTOD_FLAG_PKT_REQUESTED;
|
flags |= BRCMF_FWS_HTOD_FLAG_PKT_REQUESTED;
|
||||||
}
|
}
|
||||||
brcmf_skb_htod_tag_set_field(p, FLAGS, flags);
|
brcmf_skb_htod_tag_set_field(p, FLAGS, flags);
|
||||||
brcmf_fws_hdrpush(fws, p);
|
return brcmf_fws_hdrpush(fws, p);
|
||||||
}
|
}
|
||||||
|
|
||||||
static void brcmf_fws_rollback_toq(struct brcmf_fws_info *fws,
|
static void brcmf_fws_rollback_toq(struct brcmf_fws_info *fws,
|
||||||
@ -1805,20 +1802,21 @@ static int brcmf_fws_commit_skb(struct brcmf_fws_info *fws, int fifo,
|
|||||||
{
|
{
|
||||||
struct brcmf_skbuff_cb *skcb = brcmf_skbcb(skb);
|
struct brcmf_skbuff_cb *skcb = brcmf_skbcb(skb);
|
||||||
struct brcmf_fws_mac_descriptor *entry;
|
struct brcmf_fws_mac_descriptor *entry;
|
||||||
struct brcmf_bus *bus = fws->drvr->bus_if;
|
|
||||||
int rc;
|
int rc;
|
||||||
u8 ifidx;
|
u8 ifidx;
|
||||||
|
u8 data_offset;
|
||||||
|
|
||||||
entry = skcb->mac;
|
entry = skcb->mac;
|
||||||
if (IS_ERR(entry))
|
if (IS_ERR(entry))
|
||||||
return PTR_ERR(entry);
|
return PTR_ERR(entry);
|
||||||
|
|
||||||
brcmf_fws_precommit_skb(fws, fifo, skb);
|
data_offset = brcmf_fws_precommit_skb(fws, fifo, skb);
|
||||||
entry->transit_count++;
|
entry->transit_count++;
|
||||||
if (entry->suppressed)
|
if (entry->suppressed)
|
||||||
entry->suppr_transit_count++;
|
entry->suppr_transit_count++;
|
||||||
|
ifidx = brcmf_skb_if_flags_get_field(skb, INDEX);
|
||||||
brcmf_fws_unlock(fws);
|
brcmf_fws_unlock(fws);
|
||||||
rc = brcmf_bus_txdata(bus, skb);
|
rc = brcmf_proto_txdata(fws->drvr, ifidx, data_offset, skb);
|
||||||
brcmf_fws_lock(fws);
|
brcmf_fws_lock(fws);
|
||||||
brcmf_dbg(DATA, "%s flags %X htod %X bus_tx %d\n", entry->name,
|
brcmf_dbg(DATA, "%s flags %X htod %X bus_tx %d\n", entry->name,
|
||||||
skcb->if_flags, skcb->htod, rc);
|
skcb->if_flags, skcb->htod, rc);
|
||||||
@ -1979,10 +1977,9 @@ static void brcmf_fws_dequeue_worker(struct work_struct *worker)
|
|||||||
&skb, true);
|
&skb, true);
|
||||||
ifidx = brcmf_skb_if_flags_get_field(skb,
|
ifidx = brcmf_skb_if_flags_get_field(skb,
|
||||||
INDEX);
|
INDEX);
|
||||||
brcmf_proto_hdrpush(drvr, ifidx, 0, skb);
|
/* Use proto layer to send data frame */
|
||||||
/* Use bus module to send data frame */
|
|
||||||
brcmf_fws_unlock(fws);
|
brcmf_fws_unlock(fws);
|
||||||
ret = brcmf_bus_txdata(drvr->bus_if, skb);
|
ret = brcmf_proto_txdata(drvr, ifidx, 0, skb);
|
||||||
brcmf_fws_lock(fws);
|
brcmf_fws_lock(fws);
|
||||||
if (ret < 0)
|
if (ret < 0)
|
||||||
brcmf_txfinalize(drvr, skb, false);
|
brcmf_txfinalize(drvr, skb, false);
|
||||||
|
@ -39,7 +39,7 @@ int brcmf_proto_attach(struct brcmf_pub *drvr)
|
|||||||
if (brcmf_proto_bcdc_attach(drvr))
|
if (brcmf_proto_bcdc_attach(drvr))
|
||||||
goto fail;
|
goto fail;
|
||||||
|
|
||||||
if ((proto->hdrpush == NULL) || (proto->hdrpull == NULL) ||
|
if ((proto->txdata == NULL) || (proto->hdrpull == NULL) ||
|
||||||
(proto->query_dcmd == NULL) || (proto->set_dcmd == NULL)) {
|
(proto->query_dcmd == NULL) || (proto->set_dcmd == NULL)) {
|
||||||
brcmf_err("Not all proto handlers have been installed\n");
|
brcmf_err("Not all proto handlers have been installed\n");
|
||||||
goto fail;
|
goto fail;
|
||||||
|
@ -17,14 +17,14 @@
|
|||||||
#define BRCMFMAC_PROTO_H
|
#define BRCMFMAC_PROTO_H
|
||||||
|
|
||||||
struct brcmf_proto {
|
struct brcmf_proto {
|
||||||
void (*hdrpush)(struct brcmf_pub *drvr, int ifidx, u8 offset,
|
|
||||||
struct sk_buff *skb);
|
|
||||||
int (*hdrpull)(struct brcmf_pub *drvr, bool do_fws, u8 *ifidx,
|
int (*hdrpull)(struct brcmf_pub *drvr, bool do_fws, u8 *ifidx,
|
||||||
struct sk_buff *skb);
|
struct sk_buff *skb);
|
||||||
int (*query_dcmd)(struct brcmf_pub *drvr, int ifidx, uint cmd,
|
int (*query_dcmd)(struct brcmf_pub *drvr, int ifidx, uint cmd,
|
||||||
void *buf, uint len);
|
void *buf, uint len);
|
||||||
int (*set_dcmd)(struct brcmf_pub *drvr, int ifidx, uint cmd, void *buf,
|
int (*set_dcmd)(struct brcmf_pub *drvr, int ifidx, uint cmd, void *buf,
|
||||||
uint len);
|
uint len);
|
||||||
|
int (*txdata)(struct brcmf_pub *drvr, int ifidx, u8 offset,
|
||||||
|
struct sk_buff *skb);
|
||||||
void *pd;
|
void *pd;
|
||||||
};
|
};
|
||||||
|
|
||||||
@ -32,11 +32,6 @@ struct brcmf_proto {
|
|||||||
int brcmf_proto_attach(struct brcmf_pub *drvr);
|
int brcmf_proto_attach(struct brcmf_pub *drvr);
|
||||||
void brcmf_proto_detach(struct brcmf_pub *drvr);
|
void brcmf_proto_detach(struct brcmf_pub *drvr);
|
||||||
|
|
||||||
static inline void brcmf_proto_hdrpush(struct brcmf_pub *drvr, int ifidx,
|
|
||||||
u8 offset, struct sk_buff *skb)
|
|
||||||
{
|
|
||||||
drvr->proto->hdrpush(drvr, ifidx, offset, skb);
|
|
||||||
}
|
|
||||||
static inline int brcmf_proto_hdrpull(struct brcmf_pub *drvr, bool do_fws,
|
static inline int brcmf_proto_hdrpull(struct brcmf_pub *drvr, bool do_fws,
|
||||||
u8 *ifidx, struct sk_buff *skb)
|
u8 *ifidx, struct sk_buff *skb)
|
||||||
{
|
{
|
||||||
@ -52,6 +47,11 @@ static inline int brcmf_proto_set_dcmd(struct brcmf_pub *drvr, int ifidx,
|
|||||||
{
|
{
|
||||||
return drvr->proto->set_dcmd(drvr, ifidx, cmd, buf, len);
|
return drvr->proto->set_dcmd(drvr, ifidx, cmd, buf, len);
|
||||||
}
|
}
|
||||||
|
static inline int brcmf_proto_txdata(struct brcmf_pub *drvr, int ifidx,
|
||||||
|
u8 offset, struct sk_buff *skb)
|
||||||
|
{
|
||||||
|
return drvr->proto->txdata(drvr, ifidx, offset, skb);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
#endif /* BRCMFMAC_PROTO_H */
|
#endif /* BRCMFMAC_PROTO_H */
|
||||||
|
Loading…
Reference in New Issue
Block a user