On 4/3/2018 7:51 PM, Daniel Mack wrote: > Bail out if the mapping fails. Even though this hasn't occured during > tests, this unlikely case should still be handled. > > Signed-off-by: Daniel Mack <[email protected]> > --- > drivers/net/wireless/ath/wcn36xx/dxe.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/net/wireless/ath/wcn36xx/dxe.c > b/drivers/net/wireless/ath/wcn36xx/dxe.c > index 6e9a3583c447..e8ad8f989ccd 100644 > --- a/drivers/net/wireless/ath/wcn36xx/dxe.c > +++ b/drivers/net/wireless/ath/wcn36xx/dxe.c > @@ -707,6 +707,11 @@ int wcn36xx_dxe_tx_frame(struct wcn36xx *wcn, > ctl->skb->data, > ctl->skb->len, > DMA_TO_DEVICE); > + if (dma_mapping_error(wcn->dev, desc->src_addr_l)) { > + dev_err(wcn->dev, "unable to DMA map src_addr_l\n"); > + ret = -ENOMEM; > + goto unlock; > + } > > desc->dst_addr_l = ch->dxe_wq; > desc->fr_len = ctl->skb->len; I have the exact patch ready for submission, you got a head of me :) Acked-by: Ramon Fried <[email protected]>
-- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project
