Changeset View
Changeset View
Standalone View
Standalone View
sys/net/if.c
Show First 20 Lines • Show All 4,935 Lines • ▼ Show 20 Lines | |||||
} | } | ||||
uint8_t | uint8_t | ||||
if_getpcp(if_t ifp) | if_getpcp(if_t ifp) | ||||
{ | { | ||||
return (ifp->if_pcp); | return (ifp->if_pcp); | ||||
} | } | ||||
void * | |||||
melifaro: Maybe it's worth returning `struct fw_com *` so we don't need an explicit typecast afterwards? | |||||
Done Inline ActionsIt would need to be a more specific name then, because if_l2com is general L2 private data used elsewhere directly by the netstack. Firewire is using it for its own L2 bits. jhibbits: It would need to be a more specific name then, because if_l2com is general L2 private data used… | |||||
Not Done Inline ActionsMakes sense. Anyway, It'll probably be entirely eliminated in 15, so I guess it doesn't matter much. melifaro: Makes sense. Anyway, It'll probably be entirely eliminated in 15, so I guess it doesn't matter… | |||||
if_getl2com(if_t ifp) | |||||
{ | |||||
return (ifp->if_l2com); | |||||
} | |||||
#ifdef DDB | #ifdef DDB | ||||
static void | static void | ||||
if_show_ifnet(struct ifnet *ifp) | if_show_ifnet(struct ifnet *ifp) | ||||
{ | { | ||||
if (ifp == NULL) | if (ifp == NULL) | ||||
return; | return; | ||||
db_printf("%s:\n", ifp->if_xname); | db_printf("%s:\n", ifp->if_xname); | ||||
▲ Show 20 Lines • Show All 70 Lines • Show Last 20 Lines |
Maybe it's worth returning struct fw_com * so we don't need an explicit typecast afterwards?