diff options
| author | Stephen Hemminger <shemminger@vyatta.com> | 2011-04-11 06:43:50 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-04-11 18:08:47 -0400 |
| commit | 73d6ac633c6c0ca703f90db0b808d9593e46aef6 (patch) | |
| tree | 738fb16f97e24288f9db9c6b43b4bcee4f794dc6 /include/net/caif | |
| parent | 1c01a80cfec6f806246f31ff2680cd3639b30e67 (diff) | |
caif: code cleanup
Cleanup of new CAIF code.
* make local functions static
* remove code that is never used
* expand get_caif_conf() since wrapper is no longer needed
* make args to comparison functions const
* rename connect_req_to_link_param to keep exported names
consistent
Compile tested only.
Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Acked-by: Sjur Brændeland <sjur.brandeland@stericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/caif')
| -rw-r--r-- | include/net/caif/caif_dev.h | 27 | ||||
| -rw-r--r-- | include/net/caif/cfctrl.h | 12 | ||||
| -rw-r--r-- | include/net/caif/cfmuxl.h | 2 | ||||
| -rw-r--r-- | include/net/caif/cfpkt.h | 75 | ||||
| -rw-r--r-- | include/net/caif/cfsrvl.h | 3 |
5 files changed, 7 insertions, 112 deletions
diff --git a/include/net/caif/caif_dev.h b/include/net/caif/caif_dev.h index 8eff83b95366..7e3f7a6d2ba3 100644 --- a/include/net/caif/caif_dev.h +++ b/include/net/caif/caif_dev.h | |||
| @@ -74,19 +74,8 @@ int caif_connect_client(struct caif_connect_request *conn_req, | |||
| 74 | int caif_disconnect_client(struct cflayer *client_layer); | 74 | int caif_disconnect_client(struct cflayer *client_layer); |
| 75 | 75 | ||
| 76 | /** | 76 | /** |
| 77 | * caif_release_client - Release adaptation layer reference to client. | 77 | * caif_connect_req_to_link_param - Translate configuration parameters |
| 78 | * | 78 | * from socket format to internal format. |
| 79 | * @client_layer: Client layer. | ||
| 80 | * | ||
| 81 | * Releases a client/adaptation layer use of the caif stack. | ||
| 82 | * This function must be used after caif_disconnect_client to | ||
| 83 | * decrease the reference count of the service layer. | ||
| 84 | */ | ||
| 85 | void caif_release_client(struct cflayer *client_layer); | ||
| 86 | |||
| 87 | /** | ||
| 88 | * connect_req_to_link_param - Translate configuration parameters | ||
| 89 | * from socket format to internal format. | ||
| 90 | * @cnfg: Pointer to configuration handler | 79 | * @cnfg: Pointer to configuration handler |
| 91 | * @con_req: Configuration parameters supplied in function | 80 | * @con_req: Configuration parameters supplied in function |
| 92 | * caif_connect_client | 81 | * caif_connect_client |
| @@ -94,14 +83,8 @@ void caif_release_client(struct cflayer *client_layer); | |||
| 94 | * setting up channels. | 83 | * setting up channels. |
| 95 | * | 84 | * |
| 96 | */ | 85 | */ |
| 97 | int connect_req_to_link_param(struct cfcnfg *cnfg, | 86 | int caif_connect_req_to_link_param(struct cfcnfg *cnfg, |
| 98 | struct caif_connect_request *con_req, | 87 | struct caif_connect_request *con_req, |
| 99 | struct cfctrl_link_param *channel_setup_param); | 88 | struct cfctrl_link_param *setup_param); |
| 100 | |||
| 101 | /** | ||
| 102 | * get_caif_conf() - Get the configuration handler. | ||
| 103 | */ | ||
| 104 | struct cfcnfg *get_caif_conf(void); | ||
| 105 | |||
| 106 | 89 | ||
| 107 | #endif /* CAIF_DEV_H_ */ | 90 | #endif /* CAIF_DEV_H_ */ |
diff --git a/include/net/caif/cfctrl.h b/include/net/caif/cfctrl.h index e54f6396fa4c..d84416fa175a 100644 --- a/include/net/caif/cfctrl.h +++ b/include/net/caif/cfctrl.h | |||
| @@ -121,19 +121,9 @@ int cfctrl_linkup_request(struct cflayer *cfctrl, | |||
| 121 | struct cflayer *user_layer); | 121 | struct cflayer *user_layer); |
| 122 | int cfctrl_linkdown_req(struct cflayer *cfctrl, u8 linkid, | 122 | int cfctrl_linkdown_req(struct cflayer *cfctrl, u8 linkid, |
| 123 | struct cflayer *client); | 123 | struct cflayer *client); |
| 124 | void cfctrl_sleep_req(struct cflayer *cfctrl); | 124 | |
| 125 | void cfctrl_wake_req(struct cflayer *cfctrl); | ||
| 126 | void cfctrl_getstartreason_req(struct cflayer *cfctrl); | ||
| 127 | struct cflayer *cfctrl_create(void); | 125 | struct cflayer *cfctrl_create(void); |
| 128 | void cfctrl_set_dnlayer(struct cflayer *this, struct cflayer *dn); | ||
| 129 | void cfctrl_set_uplayer(struct cflayer *this, struct cflayer *up); | ||
| 130 | struct cfctrl_rsp *cfctrl_get_respfuncs(struct cflayer *layer); | 126 | struct cfctrl_rsp *cfctrl_get_respfuncs(struct cflayer *layer); |
| 131 | bool cfctrl_req_eq(struct cfctrl_request_info *r1, | ||
| 132 | struct cfctrl_request_info *r2); | ||
| 133 | void cfctrl_insert_req(struct cfctrl *ctrl, | ||
| 134 | struct cfctrl_request_info *req); | ||
| 135 | struct cfctrl_request_info *cfctrl_remove_req(struct cfctrl *ctrl, | ||
| 136 | struct cfctrl_request_info *req); | ||
| 137 | void cfctrl_cancel_req(struct cflayer *layr, struct cflayer *adap_layer); | 127 | void cfctrl_cancel_req(struct cflayer *layr, struct cflayer *adap_layer); |
| 138 | 128 | ||
| 139 | #endif /* CFCTRL_H_ */ | 129 | #endif /* CFCTRL_H_ */ |
diff --git a/include/net/caif/cfmuxl.h b/include/net/caif/cfmuxl.h index 4e1b4f33423e..5847a196b8ad 100644 --- a/include/net/caif/cfmuxl.h +++ b/include/net/caif/cfmuxl.h | |||
| @@ -16,7 +16,5 @@ int cfmuxl_set_uplayer(struct cflayer *layr, struct cflayer *up, u8 linkid); | |||
| 16 | struct cflayer *cfmuxl_remove_dnlayer(struct cflayer *layr, u8 phyid); | 16 | struct cflayer *cfmuxl_remove_dnlayer(struct cflayer *layr, u8 phyid); |
| 17 | int cfmuxl_set_dnlayer(struct cflayer *layr, struct cflayer *up, u8 phyid); | 17 | int cfmuxl_set_dnlayer(struct cflayer *layr, struct cflayer *up, u8 phyid); |
| 18 | struct cflayer *cfmuxl_remove_uplayer(struct cflayer *layr, u8 linkid); | 18 | struct cflayer *cfmuxl_remove_uplayer(struct cflayer *layr, u8 linkid); |
| 19 | bool cfmuxl_is_phy_inuse(struct cflayer *layr, u8 phyid); | ||
| 20 | u8 cfmuxl_get_phyid(struct cflayer *layr, u8 channel_id); | ||
| 21 | 19 | ||
| 22 | #endif /* CFMUXL_H_ */ | 20 | #endif /* CFMUXL_H_ */ |
diff --git a/include/net/caif/cfpkt.h b/include/net/caif/cfpkt.h index fbc681beff52..8b550f8950d7 100644 --- a/include/net/caif/cfpkt.h +++ b/include/net/caif/cfpkt.h | |||
| @@ -16,12 +16,6 @@ struct cfpkt; | |||
| 16 | */ | 16 | */ |
| 17 | struct cfpkt *cfpkt_create(u16 len); | 17 | struct cfpkt *cfpkt_create(u16 len); |
| 18 | 18 | ||
| 19 | /* Create a CAIF packet. | ||
| 20 | * data Data to copy. | ||
| 21 | * len Length of packet to be created | ||
| 22 | * @return New packet. | ||
| 23 | */ | ||
| 24 | struct cfpkt *cfpkt_create_uplink(const unsigned char *data, unsigned int len); | ||
| 25 | /* | 19 | /* |
| 26 | * Destroy a CAIF Packet. | 20 | * Destroy a CAIF Packet. |
| 27 | * pkt Packet to be destoyed. | 21 | * pkt Packet to be destoyed. |
| @@ -181,22 +175,6 @@ u16 cfpkt_iterate(struct cfpkt *pkt, | |||
| 181 | u16 (*iter_func)(u16 chks, void *buf, u16 len), | 175 | u16 (*iter_func)(u16 chks, void *buf, u16 len), |
| 182 | u16 data); | 176 | u16 data); |
| 183 | 177 | ||
| 184 | /* Append by giving user access to packet buffer | ||
| 185 | * cfpkt Packet to append to | ||
| 186 | * buf Buffer inside pkt that user shall copy data into | ||
| 187 | * buflen Length of buffer and number of bytes added to packet | ||
| 188 | * @return 0 on error, 1 on success | ||
| 189 | */ | ||
| 190 | int cfpkt_raw_append(struct cfpkt *cfpkt, void **buf, unsigned int buflen); | ||
| 191 | |||
| 192 | /* Extract by giving user access to packet buffer | ||
| 193 | * cfpkt Packet to extract from | ||
| 194 | * buf Buffer inside pkt that user shall copy data from | ||
| 195 | * buflen Length of buffer and number of bytes removed from packet | ||
| 196 | * @return 0 on error, 1 on success | ||
| 197 | */ | ||
| 198 | int cfpkt_raw_extract(struct cfpkt *cfpkt, void **buf, unsigned int buflen); | ||
| 199 | |||
| 200 | /* Map from a "native" packet (e.g. Linux Socket Buffer) to a CAIF packet. | 178 | /* Map from a "native" packet (e.g. Linux Socket Buffer) to a CAIF packet. |
| 201 | * dir - Direction indicating whether this packet is to be sent or received. | 179 | * dir - Direction indicating whether this packet is to be sent or received. |
| 202 | * nativepkt - The native packet to be transformed to a CAIF packet | 180 | * nativepkt - The native packet to be transformed to a CAIF packet |
| @@ -210,59 +188,6 @@ struct cfpkt *cfpkt_fromnative(enum caif_direction dir, void *nativepkt); | |||
| 210 | */ | 188 | */ |
| 211 | void *cfpkt_tonative(struct cfpkt *pkt); | 189 | void *cfpkt_tonative(struct cfpkt *pkt); |
| 212 | 190 | ||
| 213 | /* | ||
| 214 | * Insert a packet in the packet queue. | ||
| 215 | * pktq Packet queue to insert into | ||
| 216 | * pkt Packet to be inserted in queue | ||
| 217 | * prio Priority of packet | ||
| 218 | */ | ||
| 219 | void cfpkt_queue(struct cfpktq *pktq, struct cfpkt *pkt, | ||
| 220 | unsigned short prio); | ||
| 221 | |||
| 222 | /* | ||
| 223 | * Remove a packet from the packet queue. | ||
| 224 | * pktq Packet queue to fetch packets from. | ||
| 225 | * @return Dequeued packet. | ||
| 226 | */ | ||
| 227 | struct cfpkt *cfpkt_dequeue(struct cfpktq *pktq); | ||
| 228 | |||
| 229 | /* | ||
| 230 | * Peek into a packet from the packet queue. | ||
| 231 | * pktq Packet queue to fetch packets from. | ||
| 232 | * @return Peeked packet. | ||
| 233 | */ | ||
| 234 | struct cfpkt *cfpkt_qpeek(struct cfpktq *pktq); | ||
| 235 | |||
| 236 | /* | ||
| 237 | * Initiates the packet queue. | ||
| 238 | * @return Pointer to new packet queue. | ||
| 239 | */ | ||
| 240 | struct cfpktq *cfpktq_create(void); | ||
| 241 | |||
| 242 | /* | ||
| 243 | * Get the number of packets in the queue. | ||
| 244 | * pktq Packet queue to fetch count from. | ||
| 245 | * @return Number of packets in queue. | ||
| 246 | */ | ||
| 247 | int cfpkt_qcount(struct cfpktq *pktq); | ||
| 248 | |||
| 249 | /* | ||
| 250 | * Put content of packet into buffer for debuging purposes. | ||
| 251 | * pkt Packet to copy data from | ||
| 252 | * buf Buffer to copy data into | ||
| 253 | * buflen Length of data to copy | ||
| 254 | * @return Pointer to copied data | ||
| 255 | */ | ||
| 256 | char *cfpkt_log_pkt(struct cfpkt *pkt, char *buf, int buflen); | ||
| 257 | |||
| 258 | /* | ||
| 259 | * Clones a packet and releases the original packet. | ||
| 260 | * This is used for taking ownership of a packet e.g queueing. | ||
| 261 | * pkt Packet to clone and release. | ||
| 262 | * @return Cloned packet. | ||
| 263 | */ | ||
| 264 | struct cfpkt *cfpkt_clone_release(struct cfpkt *pkt); | ||
| 265 | |||
| 266 | 191 | ||
| 267 | /* | 192 | /* |
| 268 | * Returns packet information for a packet. | 193 | * Returns packet information for a packet. |
diff --git a/include/net/caif/cfsrvl.h b/include/net/caif/cfsrvl.h index b1fa87ee0992..6c8279c1ae9a 100644 --- a/include/net/caif/cfsrvl.h +++ b/include/net/caif/cfsrvl.h | |||
| @@ -22,7 +22,6 @@ struct cfsrvl { | |||
| 22 | struct kref ref; | 22 | struct kref ref; |
| 23 | }; | 23 | }; |
| 24 | 24 | ||
| 25 | void cfsrvl_release(struct kref *kref); | ||
| 26 | struct cflayer *cfvei_create(u8 linkid, struct dev_info *dev_info); | 25 | struct cflayer *cfvei_create(u8 linkid, struct dev_info *dev_info); |
| 27 | struct cflayer *cfdgml_create(u8 linkid, struct dev_info *dev_info); | 26 | struct cflayer *cfdgml_create(u8 linkid, struct dev_info *dev_info); |
| 28 | struct cflayer *cfutill_create(u8 linkid, struct dev_info *dev_info); | 27 | struct cflayer *cfutill_create(u8 linkid, struct dev_info *dev_info); |
| @@ -31,7 +30,7 @@ struct cflayer *cfrfml_create(u8 linkid, struct dev_info *dev_info, | |||
| 31 | int mtu_size); | 30 | int mtu_size); |
| 32 | struct cflayer *cfdbgl_create(u8 linkid, struct dev_info *dev_info); | 31 | struct cflayer *cfdbgl_create(u8 linkid, struct dev_info *dev_info); |
| 33 | bool cfsrvl_phyid_match(struct cflayer *layer, int phyid); | 32 | bool cfsrvl_phyid_match(struct cflayer *layer, int phyid); |
| 34 | void cfservl_destroy(struct cflayer *layer); | 33 | |
| 35 | void cfsrvl_init(struct cfsrvl *service, | 34 | void cfsrvl_init(struct cfsrvl *service, |
| 36 | u8 channel_id, | 35 | u8 channel_id, |
| 37 | struct dev_info *dev_info, | 36 | struct dev_info *dev_info, |
