search for: max_brightness

....c b/drivers/gpu/drm/gma500/mdfld_device.c >> index e2ab858122f9..2e24ecccfd39 100644 >> --- a/drivers/gpu/drm/gma500/mdfld_device.c >> +++ b/drivers/gpu/drm/gma500/mdfld_device.c >> @@ -130,7 +130,8 @@ static int mdfld_backlight_init(struct drm_device *dev) >> props.max_brightness = BRIGHTNESS_MAX_LEVEL; >> props.type = BACKLIGHT_PLATFORM; >> mdfld_backlight_device = backlight_device_register("mdfld-bl", >> - NULL, (void *)dev, &mdfld_ops, &props); >> + NULL, (void *)dev, &mdfld_ops, &props, >> + BACKLIGHT_...

...f --git a/drivers/gpu/drm/gma500/mdfld_device.c b/drivers/gpu/drm/gma500/mdfld_device.c index e2ab858122f9..2e24ecccfd39 100644 --- a/drivers/gpu/drm/gma500/mdfld_device.c +++ b/drivers/gpu/drm/gma500/mdfld_device.c @@ -130,7 +130,8 @@ static int mdfld_backlight_init(struct drm_device *dev) props.max_brightness = BRIGHTNESS_MAX_LEVEL; props.type = BACKLIGHT_PLATFORM; mdfld_backlight_device = backlight_device_register("mdfld-bl", - NULL, (void *)dev, &mdfld_ops, &props); + NULL, (void *)dev, &mdfld_ops, &props, + BACKLIGHT_REGISTER_FB_CLIENT); if (IS_ERR(mdfld_bac...

.../gma500/mdfld_device.c b/drivers/gpu/drm/gma500/mdfld_device.c > index e2ab858122f9..2e24ecccfd39 100644 > --- a/drivers/gpu/drm/gma500/mdfld_device.c > +++ b/drivers/gpu/drm/gma500/mdfld_device.c > @@ -130,7 +130,8 @@ static int mdfld_backlight_init(struct drm_device *dev) > props.max_brightness = BRIGHTNESS_MAX_LEVEL; > props.type = BACKLIGHT_PLATFORM; > mdfld_backlight_device = backlight_device_register("mdfld-bl", > - NULL, (void *)dev, &mdfld_ops, &props); > + NULL, (void *)dev, &mdfld_ops, &props, > + BACKLIGHT_REGISTER_FB_CLIENT);...

...00/mdfld_device.c > >> index e2ab858122f9..2e24ecccfd39 100644 > >> --- a/drivers/gpu/drm/gma500/mdfld_device.c > >> +++ b/drivers/gpu/drm/gma500/mdfld_device.c > >> @@ -130,7 +130,8 @@ static int mdfld_backlight_init(struct drm_device *dev) > >> props.max_brightness = BRIGHTNESS_MAX_LEVEL; > >> props.type = BACKLIGHT_PLATFORM; > >> mdfld_backlight_device = backlight_device_register("mdfld-bl", > >> - NULL, (void *)dev, &mdfld_ops, &props); > >> + NULL, (void *)dev, &mdfld_ops, &props, >...

https://bugs.freedesktop.org/show_bug.cgi?id=31920 Summary: Brightness control is erratic (/sys/class/backlight/nv_backlight/max_brightness is wrong) Product: xorg Version: git Platform: Other OS/Version: Linux (All) Status: NEW Severity: normal Priority: medium Component: Driver/nouveau AssignedTo: nouveau at lists.freedesktop....

...bd; + char* backlight_name = NULL; if (!(nvif_rd32(device, NV40_PMC_BACKLIGHT) & NV40_PMC_BACKLIGHT_MASK)) return 0; @@ -81,8 +86,14 @@ nv40_backlight_init(struct drm_connector *connector) memset(&props, 0, sizeof(struct backlight_properties)); props.type = BACKLIGHT_RAW; props.max_brightness = 31; - bd = backlight_device_register("nv_backlight", connector->kdev, drm, + backlight_name = nouveau_get_backlight_name(); + bd = backlight_device_register(backlight_name , connector->kdev, drm, &nv40_bl_ops, &props); + + // backlight_device_register() makes a...

...evice, NV40_PMC_BACKLIGHT) & NV40_PMC_BACKLIGHT_MASK)) > return 0; > @@ -81,8 +86,14 @@ nv40_backlight_init(struct drm_connector *connector) > memset(&props, 0, sizeof(struct backlight_properties)); > props.type = BACKLIGHT_RAW; > props.max_brightness = 31; > - bd = backlight_device_register("nv_backlight", connector->kdev, drm, > + backlight_name = nouveau_get_backlight_name(); > + bd = backlight_device_register(backlight_name , connector->kdev, drm, > &nv4...

...ar backlight_name[BL_NAME_SIZE]; if (!(nvif_rd32(device, NV40_PMC_BACKLIGHT) & NV40_PMC_BACKLIGHT_MASK)) return 0; @@ -81,10 +104,16 @@ nv40_backlight_init(struct drm_connector *connector) memset(&props, 0, sizeof(struct backlight_properties)); props.type = BACKLIGHT_RAW; props.max_brightness = 31; - bd = backlight_device_register("nv_backlight", connector->kdev, drm, + nouveau_get_backlight_name(backlight_name, &bl_connector); + bd = backlight_device_register(backlight_name , connector->kdev, drm, &nv40_bl_ops, &props); - if (IS_ERR(bd)) + + if (...

...for digits + 1 for '\0' if (!(nvif_rd32(device, NV40_PMC_BACKLIGHT) & NV40_PMC_BACKLIGHT_MASK)) return 0; @@ -81,10 +103,16 @@ nv40_backlight_init(struct drm_connector *connector) memset(&props, 0, sizeof(struct backlight_properties)); props.type = BACKLIGHT_RAW; props.max_brightness = 31; - bd = backlight_device_register("nv_backlight", connector->kdev, drm, + nouveau_get_backlight_name(backlight_name, &bl_connector); + bd = backlight_device_register(backlight_name , connector->kdev, drm, &nv40_bl_ops, &props); - if (IS_ERR(bd)) + + if (...

Next version of https://patchwork.freedesktop.org/series/48596/ . Made some important changes to the refactoring patch, but everything else is the same. Lyude Paul (5): drm/nouveau: Check backlight IDs are >= 0, not > 0 drm/nouveau: Move backlight device into nouveau_connector drm/nouveau: s/nouveau_backlight_exit/nouveau_backlight_fini/ drm/nouveau: Cleanup indenting in

This series fixes some issues with nouveau's backlight support that were causing kernel panics on module reloads, specifically on systems with nouveau handling the backlight of one of the displays. While we're at it, let's cleanup nouveau_backlight.c as well Lyude Paul (5): drm/nouveau: Check backlight IDs are >= 0, not > 0 drm/nouveau: Move backlight device into

...ar backlight_name[BL_NAME_SIZE]; if (!(nvif_rd32(device, NV40_PMC_BACKLIGHT) & NV40_PMC_BACKLIGHT_MASK)) return 0; @@ -81,10 +107,19 @@ nv40_backlight_init(struct drm_connector *connector) memset(&props, 0, sizeof(struct backlight_properties)); props.type = BACKLIGHT_RAW; props.max_brightness = 31; - bd = backlight_device_register("nv_backlight", connector->kdev, drm, + if (!nouveau_get_backlight_name(backlight_name, &bl_connector)) { + NV_ERROR(drm, "Failed to retrieve a unique name for the backlight interface\n"); + return 0; + } + bd = backlight_device_re...

Next version of https://patchwork.freedesktop.org/series/48596/ . Added NV_INFO_ONCE and made "Move backlight device into nouveau_connector" use that instead so we don't print the GMUX warning more then once. Lyude Paul (5): drm/nouveau: Add NV_PRINTK_ONCE and variants drm/nouveau: Move backlight device into nouveau_connector drm/nouveau:

...t; > if (!(nvif_rd32(device, NV40_PMC_BACKLIGHT) & NV40_PMC_BACKLIGHT_MASK)) > return 0; > @@ -81,10 +104,16 @@ nv40_backlight_init(struct drm_connector *connector) > memset(&props, 0, sizeof(struct backlight_properties)); > props.type = BACKLIGHT_RAW; > props.max_brightness = 31; > - bd = backlight_device_register("nv_backlight", connector->kdev, drm, > + nouveau_get_backlight_name(backlight_name, &bl_connector); > + bd = backlight_device_register(backlight_name , connector->kdev, drm, > &nv40_bl_ops, &props); > -...

Forgot to send 6 patches instead of five since there's one new one now, whoops! No actual changes, next version of https://patchwork.freedesktop.org/series/48596/ Lyude Paul (6): drm/nouveau: Check backlight IDs are >= 0, not > 0 drm/nouveau: Add NV_PRINTK_ONCE and variants drm/nouveau: Move backlight device into nouveau_connector drm/nouveau:

Refactor for Ben, hopefully this time this should apply to his tree. Next version of https://patchwork.freedesktop.org/series/48596/ No changes otherwise. Lyude Paul (6): drm/nouveau: Check backlight IDs are >= 0, not > 0 drm/nouveau: Add NV_PRINTK_ONCE and variants drm/nouveau: Move backlight device into nouveau_connector drm/nouveau:

...d; + + if (!(NV_READ(NV40_PMC_BACKLIGHT) & NV40_PMC_BACKLIGHT_MASK)) + return 0; + + bd = backlight_device_register("nv_backlight", &dev->pdev->dev, dev, + &nv40_bl_ops); + if (IS_ERR(bd)) + return PTR_ERR(bd); + + dev_priv->backlight = bd; + bd->props.max_brightness = 31; + bd->props.brightness = nv40_get_intensity(bd); + backlight_update_status(bd); + + return 0; +} + +static int nouveau_nv50_backlight_init(struct drm_device *dev) +{ + struct drm_nouveau_private *dev_priv = dev->dev_private; + struct backlight_device *bd; + + if (!NV_READ(NV50_PDISPLAY_...

...) & NV40_PMC_BACKLIGHT_MASK)) > > return 0; > > @@ -81,8 +86,14 @@ nv40_backlight_init(struct drm_connector *connector) > > memset(&props, 0, sizeof(struct backlight_properties)); > > props.type = BACKLIGHT_RAW; > > props.max_brightness = 31; > > - bd = backlight_device_register("nv_backlight", connector->kdev, drm, > > + backlight_name = nouveau_get_backlight_name(); > > + bd = backlight_device_register(backlight_name , connector->kdev, drm, > >...

If have a couple patches I found while looking at a panic I was seeing while unloading the nouveau module. Unloading the nouveau module on my optimus notebook machine causes the system to panic. This started occuring when moving from 4.4 to 4.14. These patches make it such that the system does not panic when unloading the module. 4.14 also requires commit 34112bf4935d ("drm/nouveau/fbcon:

https://bugs.freedesktop.org/show_bug.cgi?id=31122 Summary: Cannot control backlight intensity on Powerbook Product: xorg Version: unspecified Platform: PowerPC OS/Version: Linux (All) Status: NEW Severity: enhancement Priority: medium Component: Driver/nouveau AssignedTo: nouveau at