Index: sys/dev/pwm/pwm_backlight.c =================================================================== --- sys/dev/pwm/pwm_backlight.c +++ sys/dev/pwm/pwm_backlight.c @@ -72,7 +72,8 @@ bool enabled; }; -static int pwm_backlight_find_level_per_percent(struct pwm_backlight_softc *sc, int percent); +static int pwm_backlight_find_level_per_percent(struct pwm_backlight_softc *sc, + int percent); static struct ofw_compat_data compat_data[] = { { "pwm-backlight", 1 }, @@ -123,20 +124,26 @@ if (OF_getencprop(node, "default-brightness-level", &sc->default_level, sizeof(uint32_t)) <= 0) { - device_printf(dev, "No default-brightness-level while brightness-levels is specified\n"); + device_printf(dev, "No default-brightness-level " \ + " while brightness-levels is specified\n"); return (ENXIO); } else { if (sc->default_level > sc->nlevels) { - device_printf(dev, "default-brightness-level isn't present in brightness-levels range\n"); + device_printf(dev, "default-brightness-level " \ + " isn't present in brightness-levels range\n"); return (ENXIO); } - sc->channel->duty = sc->channel->period * sc->levels[sc->default_level] / 100; + sc->channel->duty = sc->channel->period * + sc->levels[sc->default_level] / 100; } if (bootverbose) { - device_printf(dev, "Number of levels: %zd\n", sc->nlevels); - device_printf(dev, "Configured period time: %ju\n", (uintmax_t)sc->channel->period); - device_printf(dev, "Default duty cycle: %ju\n", (uintmax_t)sc->channel->duty); + device_printf(dev, "Number of levels: %zd\n", + sc->nlevels); + device_printf(dev, "Configured period time: %ju\n", + (uintmax_t)sc->channel->period); + device_printf(dev, "Default duty cycle: %ju\n", + (uintmax_t)sc->channel->duty); } } else { /* Get the current backlight level */ @@ -147,8 +154,10 @@ if (sc->channel->duty > sc->channel->period) sc->channel->duty = sc->channel->period; if (bootverbose) { - device_printf(dev, "Configured period time: %ju\n", (uintmax_t)sc->channel->period); - device_printf(dev, "Default duty cycle: %ju\n", (uintmax_t)sc->channel->duty); + device_printf(dev, "Configured period time: %ju\n", + (uintmax_t)sc->channel->period); + device_printf(dev, "Default duty cycle: %ju\n", + (uintmax_t)sc->channel->duty); } } @@ -156,11 +165,13 @@ sc->channel->enabled = true; PWMBUS_CHANNEL_CONFIG(sc->channel->dev, sc->channel->channel, sc->channel->period, sc->channel->duty); + PWMBUS_CHANNEL_SET_FLAGS(sc->channel->dev, sc->channel->channel, + sc->channel->flags); PWMBUS_CHANNEL_ENABLE(sc->channel->dev, sc->channel->channel, sc->channel->enabled); sc->current_level = pwm_backlight_find_level_per_percent(sc, - sc->channel->period / sc->channel->duty); + 100 * sc->channel->duty / sc->channel->period); sc->cdev = backlight_register("pwm_backlight", dev); if (sc->cdev == NULL) device_printf(dev, "Cannot register as a backlight\n"); @@ -265,7 +276,8 @@ for (i = 0; i < sc->nlevels; i++) props->levels[i] = sc->levels[i]; } else { - props->brightness = sc->channel->duty * 100 / sc->channel->period; + props->brightness = + sc->channel->duty * 100 / sc->channel->period; props->nlevels = 0; } return (0);