Martin Blumenstingl <[email protected]> writes:

> +     if (of_property_read_bool(np, "qca,clk-25mhz"))
> +             ah->is_clk_25mhz = true;
> +
> +     if (of_property_read_bool(np, "qca,disable-2ghz"))
> +             ah->disable_2ghz = true;
> +
> +     if (of_property_read_bool(np, "qca,disable-5ghz"))
> +             ah->disable_5ghz = true;

This is bike-shedding, but how about

        ah->is_clk_25mhz = of_property_read_bool(np, "qca,clk-25mhz");
        ah->disable_2ghz = of_property_read_bool(np, "qca,disable-2ghz");
        ah->disable_5ghz = of_property_read_bool(np, "qca,disable-5ghz");

instead?


Bjørn
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to