Commit b4e17054 authored by Sven Eckelmann's avatar Sven Eckelmann

batman-adv: Reduce usage of char

char was used in different places to store information without really
using the characteristics of that data type or by ignoring the fact that
char has not a well defined signedness.
Signed-off-by: default avatarSven Eckelmann <sven@narfation.org>
parent b2c44a53
...@@ -195,7 +195,7 @@ static void aggregate(struct forw_packet *forw_packet_aggr, ...@@ -195,7 +195,7 @@ static void aggregate(struct forw_packet *forw_packet_aggr,
void add_bat_packet_to_list(struct bat_priv *bat_priv, void add_bat_packet_to_list(struct bat_priv *bat_priv,
unsigned char *packet_buff, int packet_len, unsigned char *packet_buff, int packet_len,
struct hard_iface *if_incoming, char own_packet, struct hard_iface *if_incoming, int own_packet,
unsigned long send_time) unsigned long send_time)
{ {
/** /**
......
...@@ -35,7 +35,7 @@ static inline int aggregated_packet(int buff_pos, int packet_len, int num_tt) ...@@ -35,7 +35,7 @@ static inline int aggregated_packet(int buff_pos, int packet_len, int num_tt)
void add_bat_packet_to_list(struct bat_priv *bat_priv, void add_bat_packet_to_list(struct bat_priv *bat_priv,
unsigned char *packet_buff, int packet_len, unsigned char *packet_buff, int packet_len,
struct hard_iface *if_incoming, char own_packet, struct hard_iface *if_incoming, int own_packet,
unsigned long send_time); unsigned long send_time);
void receive_aggr_bat_packet(const struct ethhdr *ethhdr, void receive_aggr_bat_packet(const struct ethhdr *ethhdr,
unsigned char *packet_buff, int packet_len, unsigned char *packet_buff, int packet_len,
......
...@@ -127,8 +127,8 @@ static void bit_reset_window(unsigned long *seq_bits) ...@@ -127,8 +127,8 @@ static void bit_reset_window(unsigned long *seq_bits)
* 1 if the window was moved (either new or very old) * 1 if the window was moved (either new or very old)
* 0 if the window was not moved/shifted. * 0 if the window was not moved/shifted.
*/ */
char bit_get_packet(void *priv, unsigned long *seq_bits, int bit_get_packet(void *priv, unsigned long *seq_bits,
int32_t seq_num_diff, int8_t set_mark) int32_t seq_num_diff, int set_mark)
{ {
struct bat_priv *bat_priv = priv; struct bat_priv *bat_priv = priv;
......
...@@ -35,8 +35,8 @@ void bit_mark(unsigned long *seq_bits, int32_t n); ...@@ -35,8 +35,8 @@ void bit_mark(unsigned long *seq_bits, int32_t n);
/* receive and process one packet, returns 1 if received seq_num is considered /* receive and process one packet, returns 1 if received seq_num is considered
* new, 0 if old */ * new, 0 if old */
char bit_get_packet(void *priv, unsigned long *seq_bits, int bit_get_packet(void *priv, unsigned long *seq_bits,
int32_t seq_num_diff, int8_t set_mark); int32_t seq_num_diff, int set_mark);
/* count the hamming weight, how many good packets did we receive? */ /* count the hamming weight, how many good packets did we receive? */
int bit_packet_count(const unsigned long *seq_bits); int bit_packet_count(const unsigned long *seq_bits);
......
...@@ -360,7 +360,7 @@ void gw_node_purge(struct bat_priv *bat_priv) ...@@ -360,7 +360,7 @@ void gw_node_purge(struct bat_priv *bat_priv)
struct gw_node *gw_node, *curr_gw; struct gw_node *gw_node, *curr_gw;
struct hlist_node *node, *node_tmp; struct hlist_node *node, *node_tmp;
unsigned long timeout = 2 * PURGE_TIMEOUT * HZ; unsigned long timeout = 2 * PURGE_TIMEOUT * HZ;
char do_deselect = 0; int do_deselect = 0;
curr_gw = gw_get_selected_gw_node(bat_priv); curr_gw = gw_get_selected_gw_node(bat_priv);
......
...@@ -61,9 +61,9 @@ static void kbit_to_gw_bandwidth(int down, int up, long *gw_srv_class) ...@@ -61,9 +61,9 @@ static void kbit_to_gw_bandwidth(int down, int up, long *gw_srv_class)
/* returns the up and downspeeds in kbit, calculated from the class */ /* returns the up and downspeeds in kbit, calculated from the class */
void gw_bandwidth_to_kbit(uint8_t gw_srv_class, int *down, int *up) void gw_bandwidth_to_kbit(uint8_t gw_srv_class, int *down, int *up)
{ {
char sbit = (gw_srv_class & 0x80) >> 7; int sbit = (gw_srv_class & 0x80) >> 7;
char dpart = (gw_srv_class & 0x78) >> 3; int dpart = (gw_srv_class & 0x78) >> 3;
char upart = (gw_srv_class & 0x07); int upart = (gw_srv_class & 0x07);
if (!gw_srv_class) { if (!gw_srv_class) {
*down = 0; *down = 0;
......
...@@ -151,7 +151,7 @@ int debug_log(struct bat_priv *bat_priv, const char *fmt, ...) __printf(2, 3); ...@@ -151,7 +151,7 @@ int debug_log(struct bat_priv *bat_priv, const char *fmt, ...) __printf(2, 3);
while (0) while (0)
#else /* !CONFIG_BATMAN_ADV_DEBUG */ #else /* !CONFIG_BATMAN_ADV_DEBUG */
__printf(3, 4) __printf(3, 4)
static inline void bat_dbg(char type __always_unused, static inline void bat_dbg(int type __always_unused,
struct bat_priv *bat_priv __always_unused, struct bat_priv *bat_priv __always_unused,
const char *fmt __always_unused, ...) const char *fmt __always_unused, ...)
{ {
......
...@@ -163,7 +163,7 @@ static int is_bidirectional_neigh(struct orig_node *orig_node, ...@@ -163,7 +163,7 @@ static int is_bidirectional_neigh(struct orig_node *orig_node,
struct bat_priv *bat_priv = netdev_priv(if_incoming->soft_iface); struct bat_priv *bat_priv = netdev_priv(if_incoming->soft_iface);
struct neigh_node *neigh_node = NULL, *tmp_neigh_node; struct neigh_node *neigh_node = NULL, *tmp_neigh_node;
struct hlist_node *node; struct hlist_node *node;
unsigned char total_count; uint8_t total_count;
uint8_t orig_eq_count, neigh_rq_count, tq_own; uint8_t orig_eq_count, neigh_rq_count, tq_own;
int tq_asym_penalty, ret = 0; int tq_asym_penalty, ret = 0;
...@@ -602,9 +602,9 @@ void receive_bat_packet(const struct ethhdr *ethhdr, ...@@ -602,9 +602,9 @@ void receive_bat_packet(const struct ethhdr *ethhdr,
struct orig_node *orig_neigh_node, *orig_node; struct orig_node *orig_neigh_node, *orig_node;
struct neigh_node *router = NULL, *router_router = NULL; struct neigh_node *router = NULL, *router_router = NULL;
struct neigh_node *orig_neigh_router = NULL; struct neigh_node *orig_neigh_router = NULL;
char has_directlink_flag; int has_directlink_flag;
char is_my_addr = 0, is_my_orig = 0, is_my_oldorig = 0; int is_my_addr = 0, is_my_orig = 0, is_my_oldorig = 0;
char is_broadcast = 0, is_bidirectional, is_single_hop_neigh; int is_broadcast = 0, is_bidirectional, is_single_hop_neigh;
int is_duplicate; int is_duplicate;
uint32_t if_incoming_seqno; uint32_t if_incoming_seqno;
......
...@@ -165,7 +165,7 @@ static void send_packet(struct forw_packet *forw_packet) ...@@ -165,7 +165,7 @@ static void send_packet(struct forw_packet *forw_packet)
struct bat_priv *bat_priv; struct bat_priv *bat_priv;
struct batman_packet *batman_packet = struct batman_packet *batman_packet =
(struct batman_packet *)(forw_packet->skb->data); (struct batman_packet *)(forw_packet->skb->data);
unsigned char directlink = (batman_packet->flags & DIRECTLINK ? 1 : 0); int directlink = (batman_packet->flags & DIRECTLINK ? 1 : 0);
if (!forw_packet->if_incoming) { if (!forw_packet->if_incoming) {
pr_err("Error - can't forward packet: incoming iface not " pr_err("Error - can't forward packet: incoming iface not "
...@@ -307,12 +307,12 @@ void schedule_own_packet(struct hard_iface *hard_iface) ...@@ -307,12 +307,12 @@ void schedule_own_packet(struct hard_iface *hard_iface)
void schedule_forward_packet(struct orig_node *orig_node, void schedule_forward_packet(struct orig_node *orig_node,
const struct ethhdr *ethhdr, const struct ethhdr *ethhdr,
struct batman_packet *batman_packet, struct batman_packet *batman_packet,
uint8_t directlink, int tt_buff_len, int directlink, int tt_buff_len,
struct hard_iface *if_incoming) struct hard_iface *if_incoming)
{ {
struct bat_priv *bat_priv = netdev_priv(if_incoming->soft_iface); struct bat_priv *bat_priv = netdev_priv(if_incoming->soft_iface);
struct neigh_node *router; struct neigh_node *router;
unsigned char in_tq, in_ttl, tq_avg = 0; uint8_t in_tq, in_ttl, tq_avg = 0;
unsigned long send_time; unsigned long send_time;
if (batman_packet->ttl <= 1) { if (batman_packet->ttl <= 1) {
......
...@@ -28,7 +28,7 @@ void schedule_own_packet(struct hard_iface *hard_iface); ...@@ -28,7 +28,7 @@ void schedule_own_packet(struct hard_iface *hard_iface);
void schedule_forward_packet(struct orig_node *orig_node, void schedule_forward_packet(struct orig_node *orig_node,
const struct ethhdr *ethhdr, const struct ethhdr *ethhdr,
struct batman_packet *batman_packet, struct batman_packet *batman_packet,
uint8_t directlink, int tt_buff_len, int directlink, int tt_buff_len,
struct hard_iface *if_outgoing); struct hard_iface *if_outgoing);
int add_bcast_packet_to_list(struct bat_priv *bat_priv, int add_bcast_packet_to_list(struct bat_priv *bat_priv,
const struct sk_buff *skb); const struct sk_buff *skb);
......
...@@ -380,7 +380,7 @@ void softif_neigh_purge(struct bat_priv *bat_priv) ...@@ -380,7 +380,7 @@ void softif_neigh_purge(struct bat_priv *bat_priv)
struct softif_neigh *softif_neigh, *curr_softif_neigh; struct softif_neigh *softif_neigh, *curr_softif_neigh;
struct softif_neigh_vid *softif_neigh_vid; struct softif_neigh_vid *softif_neigh_vid;
struct hlist_node *node, *node_tmp, *node_tmp2; struct hlist_node *node, *node_tmp, *node_tmp2;
char do_deselect; int do_deselect;
rcu_read_lock(); rcu_read_lock();
hlist_for_each_entry_rcu(softif_neigh_vid, node, hlist_for_each_entry_rcu(softif_neigh_vid, node,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment