Commit 64294263 authored by Vladimir Oltean's avatar Vladimir Oltean Committed by David S. Miller

net: mscc: ocelot: rename variable 'cnt' in vcap_data_offset_get()

The 'cnt' variable is actually used for 2 purposes, to hold the number
of sub-words per VCAP entry, and the number of sub-words per VCAP
action.

In fact, I'm pretty sure these 2 numbers can never be different from one
another. By hardware definition, the entry (key) TCAM rows are divided
into the same number of sub-words as its associated action RAM rows.
But nonetheless, let's at least rename the variables such that
observations like this one are easier to make in the future.
Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 5963083a
...@@ -174,7 +174,8 @@ static void vcap_cache2action(struct ocelot *ocelot, ...@@ -174,7 +174,8 @@ static void vcap_cache2action(struct ocelot *ocelot,
static void vcap_data_offset_get(const struct vcap_props *vcap, static void vcap_data_offset_get(const struct vcap_props *vcap,
struct vcap_data *data, int ix) struct vcap_data *data, int ix)
{ {
int i, col, offset, num_entries_per_row, cnt, base; int num_subwords_per_entry, num_subwords_per_action;
int i, col, offset, num_entries_per_row, base;
u32 width = vcap->tg_width; u32 width = vcap->tg_width;
switch (data->tg_sw) { switch (data->tg_sw) {
...@@ -192,11 +193,12 @@ static void vcap_data_offset_get(const struct vcap_props *vcap, ...@@ -192,11 +193,12 @@ static void vcap_data_offset_get(const struct vcap_props *vcap,
} }
col = (ix % num_entries_per_row); col = (ix % num_entries_per_row);
cnt = (vcap->sw_count / num_entries_per_row); num_subwords_per_entry = (vcap->sw_count / num_entries_per_row);
base = (vcap->sw_count - col * cnt - cnt); base = (vcap->sw_count - col * num_subwords_per_entry -
num_subwords_per_entry);
data->tg_value = 0; data->tg_value = 0;
data->tg_mask = 0; data->tg_mask = 0;
for (i = 0; i < cnt; i++) { for (i = 0; i < num_subwords_per_entry; i++) {
offset = ((base + i) * width); offset = ((base + i) * width);
data->tg_value |= (data->tg_sw << offset); data->tg_value |= (data->tg_sw << offset);
data->tg_mask |= GENMASK(offset + width - 1, offset); data->tg_mask |= GENMASK(offset + width - 1, offset);
...@@ -205,12 +207,14 @@ static void vcap_data_offset_get(const struct vcap_props *vcap, ...@@ -205,12 +207,14 @@ static void vcap_data_offset_get(const struct vcap_props *vcap,
/* Calculate key/action/counter offsets */ /* Calculate key/action/counter offsets */
col = (num_entries_per_row - col - 1); col = (num_entries_per_row - col - 1);
data->key_offset = (base * vcap->entry_width) / vcap->sw_count; data->key_offset = (base * vcap->entry_width) / vcap->sw_count;
data->counter_offset = (cnt * col * vcap->counter_width); data->counter_offset = (num_subwords_per_entry * col *
vcap->counter_width);
i = data->type; i = data->type;
width = vcap->action_table[i].width; width = vcap->action_table[i].width;
cnt = vcap->action_table[i].count; num_subwords_per_action = vcap->action_table[i].count;
data->action_offset = (((cnt * col * width) / num_entries_per_row) + data->action_offset = ((num_subwords_per_action * col * width) /
vcap->action_type_width); num_entries_per_row);
data->action_offset += vcap->action_type_width;
} }
static void vcap_data_set(u32 *data, u32 offset, u32 len, u32 value) static void vcap_data_set(u32 *data, u32 offset, u32 len, u32 value)
......
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