Messages in this thread | | | Subject | Re: [PATCH] vt6656: Coding style fixes | From | Joe Perches <> | Date | Thu, 04 May 2017 09:08:51 -0700 |
| |
On Thu, 2017-05-04 at 17:33 +0200, Simo Koskinen wrote: > Fixed coding style warnings reported by checkpatch.pl.
Please strive to do more than shut up checkpatch.
Think a little about what this code is doing. Spend the time to analyze the code and improve it.
For instance, this function is currently:
static u32 vnt_get_rsvtime(struct vnt_private *priv, u8 pkt_type, u32 frame_length, u16 rate, int need_ack) { u32 data_time, ack_time;
data_time = vnt_get_frame_time(priv->preamble_type, pkt_type, frame_length, rate);
if (pkt_type == PK_TYPE_11B) ack_time = vnt_get_frame_time(priv->preamble_type, pkt_type, 14, (u16)priv->top_cck_basic_rate); else ack_time = vnt_get_frame_time(priv->preamble_type, pkt_type, 14, (u16)priv->top_ofdm_basic_rate);
if (need_ack) return data_time + priv->sifs + ack_time;
return data_time; }
It is computing how long it takes to do something.
It also is doing the ack_time calculation unnecessarily when need_ack is not set.
Better code might be something like:
static u32 vnt_get_rsvtime(struct vnt_private *priv, u8 pkt_type, u32 frame_length, u16 rate, int need_ack) { u32 data_time, ack_time;
data_time = vnt_get_frame_time(priv->preamble_type, pkt_type, frame_length, rate);
if (!need_ack) return data_time;
if (pkt_type == PK_TYPE_11B) rate = priv->top_cck_basic_rate; else rate = priv->top_ofdm_basic_rate;
ack_time = vnt_get_frame_time(priv->preamble_type, pkt_type, 14, rate);
return data_time + priv->sifs + ack_time; }
where the return of data_time is done when !need_ack and rate is calculated and a single vnt_get_frame_time call is used for ack_time only when necessary.
It's slightly smaller object code and faster to execute when need_ack is not set.
Ideally the hard-coded 14 value in the ack_time calculation should be a #define or a sizeof.
| |