Francois Romieu wrote:
Arnaldo Carvalho de Melo <acme@xxxxxxxxxxxxxxxx> :
[dscc4.c sets skb->mac.raw after hdlc_type_trans]
Francois, is this really what is intended? I left it as
In the pre-eth_type_trans area, yes :o/ I did not test dscc4 in
a bridged setup. Please fix the driver in your patch.
OK, I'll do that later today, i.e. just remove
the skb->mac.raw = skb->data after the call to hdlc_type_trans as
it already, like eth_type_trans, sets skb->mac.raw to skb->data
prior to calling any hdlc layer type_trans pointer.
For those paying attention to this lowly stuff, what do you
think of renaming all those foo_type_trans to
foo_setup_rx_skb (suggestions for a better name than foo_type_trans
are welcome) and make it set skb->protocol, making it return just
void? I think it makes the whole thing clearer...
- Arnaldo
|