Re: [PATCH 11/14] media: wl128x-radio: fix skb debug printing

From: Pavel Machek
Date: Sat Dec 22 2018 - 14:30:49 EST


On Fri 2018-12-21 02:17:49, Sebastian Reichel wrote:
> From: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxx>
>
> This fixes incorrect code in the TX/RX skb debug print
> function and add stubs in receive/transmit packet path.
>
> Signed-off-by: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxx>

Acked-by: Pavel Machek <pavel@xxxxxx>


> @@ -228,7 +228,7 @@ inline void dump_rx_skb_data(struct sk_buff *skb)
>
> evt_hdr = (struct fm_event_msg_hdr *)skb->data;
> printk(KERN_INFO ">> hdr:%02x len:%02x sts:%02x numhci:%02x opcode:%02x type:%s dlen:%02x",
> - evt_hdr->hdr, evt_hdr->len,
> + evt_hdr->header, evt_hdr->len,
> evt_hdr->status, evt_hdr->num_fm_hci_cmds, evt_hdr->op,
> (evt_hdr->rd_wr) ? "RD" : "WR", evt_hdr->dlen);

Would conversion to dev_info() make sense?

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Attachment: signature.asc
Description: Digital signature