[PATCH skeleton v2] Refactored LED control dbus methods
Benjamin Herrenschmidt
benh at au1.ibm.com
Thu Jan 28 10:25:29 AEDT 2016
On Wed, 2016-01-27 at 14:54 +1100, Cyril Bur wrote:
> +const char *power_ctrl = "brightness";
> > +const char *blink_ctrl = "trigger";
> > +const char *duty_on = "delay_on";
> > +const char *duty_off = "delay_off";
>
> Why can't these be #defined?
Ugh ? A const string is better, more strongly typed and guarantees
there is going to be only one (though compilers are smart these days).
Cheers,
Ben.
More information about the openbmc
mailing list