On Thu, Jun 04, 2015 at 01:01:17PM +0200, Alexander Aring wrote:
> On Thu, Jun 04, 2015 at 11:20:42AM +0200, Christoffer Holmstedt wrote:
> > Signed-off-by: Christoffer Holmstedt <christoffer@xxxxxxxxxxxxxxxxxxxxxxx>
> > ---
> > src/info.c | 80 ++++++++++++++++++++++++++++++++++++++++++++++++--------------
> > 1 file changed, 62 insertions(+), 18 deletions(-)
> >
> > diff --git a/src/info.c b/src/info.c
> > index 2434bef6a87f..d2e88febd20c 100644
> > --- a/src/info.c
> > +++ b/src/info.c
> > @@ -146,6 +146,41 @@ static void print_freq_handler(int channel_page, int channel)
> > }
> > }
> >
> > +static void print_cca_mode_handler(enum nl802154_cca_modes cca_mode,
> > + enum nl802154_cca_opts cca_opt)
> > +{
> > + switch (cca_mode) {
> > + case NL802154_CCA_ENERGY:
> > + printf("energy above threshold");
> > + break;
> > + case NL802154_CCA_CARRIER:
> > + printf("carrier sense only");
> > + break;
> > + case NL802154_CCA_ENERGY_CARRIER:
> > + printf("carrier sense with energy above threshold");
> > + switch (cca_opt) {
> > + case NL802154_CCA_OPT_ENERGY_CARRIER_AND:
> > + printf(" (logical operator is 'and')");
> > + break;
> > + case NL802154_CCA_OPT_ENERGY_CARRIER_OR:
> > + printf(" (logical operator is 'or')");
> > + break;
> > + default:
> > + printf(" (logical operator is unknown)");
> > + }
> > + break;
> > + case NL802154_CCA_ALOHA:
> > + printf("ALOHA");
> > + break;
> > + case NL802154_CCA_UWB_SHR:
> > + printf("UWB preamble sense based on the SHR of a frame");
> > + break;
> > + case NL802154_CCA_UWB_MULTIPEXED:
> > + printf("UWB preamble sense based on the packet with the multiplexed preamble");
> > + break;
> > + }
> > +}
> > +
>
> mhhh, there exists now two different styles to making some "enum" ->
> string mapping. The first one was (and this is the original way which
> was grabbed from wireless) to have some temp buffer and use sprintf to
> put the string in there.
>
> This is a more generic way. After calling the function we should use
> printf to printout the string with some format string. You know what I
> mean? We should change this.
I think I do, just like the default case in iftype_name() function in
interface.c and is later on printed out with printf in print_iface_handler() in
interface.c
I will fix that.
>
> So what I mean is here use in this function sprintf and put the mapping
> string into a local buffer, not directly printf. You can do that after
> calling this function.
>
> > static const char *commands[NL802154_CMD_MAX + 1] = {
> > [NL802154_CMD_UNSPEC] = "unspec",
> > [NL802154_CMD_GET_WPAN_PHY] = "get_wpan_phy",
> > @@ -235,23 +270,14 @@ static int print_phy_handler(struct nl_msg *msg, void *arg)
> > }
> >
> > if (tb_msg[NL802154_ATTR_CCA_MODE]) {
> > + enum nl802154_cca_opts cca_opt;
>
> init this with a invalid value, so default case will occur if not
> NL802154_ATTR_CCA_OPT is there.
>
Right. I will fix that.
Concerning setting invalid values, for cca_modes we have an enum value for
invalid but for cca_opts we don't. Should I include limits.h and set MAX_INT or
just set a high random int value like 9999?
> > + if (tb_msg[NL802154_ATTR_CCA_OPT])
> > + cca_opt = nla_get_u32(tb_msg[NL802154_ATTR_CCA_OPT]);
> > +
> > cca_mode = nla_get_u32(tb_msg[NL802154_ATTR_CCA_MODE]);
> > - printf("cca_mode: %d", cca_mode);
> > - if (cca_mode == NL802154_CCA_ENERGY_CARRIER) {
> > - enum nl802154_cca_opts cca_opt;
> >
> > - cca_opt = nla_get_u32(tb_msg[NL802154_ATTR_CCA_OPT]);
> > - switch (cca_opt) {
> > - case NL802154_CCA_OPT_ENERGY_CARRIER_AND:
> > - printf(" logical and ");
> > - break;
> > - case NL802154_CCA_OPT_ENERGY_CARRIER_OR:
> > - printf(" logical or ");
> > - break;
> > - default:
> > - printf(" logical op mode unkown ");
> > - }
> > - }
> > + printf("cca_mode: %d, ", cca_mode);
> > + print_cca_mode_handler(cca_mode, cca_opt);
> > printf("\n");
>
> - Alex
--
Christoffer Holmstedt
--
To unsubscribe from this list: send the line "unsubscribe linux-wpan" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
