* [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
@ 2019-07-25 14:32 ` Sam Ravnborg
0 siblings, 0 replies; 22+ messages in thread
From: Sam Ravnborg @ 2019-07-25 14:32 UTC (permalink / raw)
To: Lee Jones, Daniel Thompson, Bartlomiej Zolnierkiewicz
Cc: linux-fbdev, Jingoo Han, dri-devel, Michał Mirosław,
Gerd Hoffmann, Daniel Vetter, Sam Ravnborg, Peter Rosin
There was no users left - so drop the code to support EARLY_FB_BLANK.
This patch removes the support in backlight,
and drop the notifier in fbmem.
That EARLY_FB_BLANK is not used can be verified that no driver set any of:
lcd_ops.early_set_power()
lcd_ops.r_early_set_power()
Noticed while browsing backlight code for other reasons.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Daniel Thompson <daniel.thompson@linaro.org>
Cc: Jingoo Han <jingoohan1@gmail.com>
Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
Cc: Peter Rosin <peda@axentia.se>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: dri-devel@lists.freedesktop.org
Cc: linux-fbdev@vger.kernel.org
---
Build tested with various architectures, configs.
Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
maintained today?
Patch needs ack from Bartlomiej first of course.
Sam
drivers/video/backlight/lcd.c | 8 --------
drivers/video/fbdev/core/fbmem.c | 12 +-----------
include/linux/fb.h | 4 ----
include/linux/lcd.h | 10 ----------
4 files changed, 1 insertion(+), 33 deletions(-)
diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c
index d6b653aa4ee9..78b033358625 100644
--- a/drivers/video/backlight/lcd.c
+++ b/drivers/video/backlight/lcd.c
@@ -39,14 +39,6 @@ static int fb_notifier_callback(struct notifier_block *self,
if (event == FB_EVENT_BLANK) {
if (ld->ops->set_power)
ld->ops->set_power(ld, *(int *)evdata->data);
- } else if (event == FB_EARLY_EVENT_BLANK) {
- if (ld->ops->early_set_power)
- ld->ops->early_set_power(ld,
- *(int *)evdata->data);
- } else if (event == FB_R_EARLY_EVENT_BLANK) {
- if (ld->ops->r_early_set_power)
- ld->ops->r_early_set_power(ld,
- *(int *)evdata->data);
} else {
if (ld->ops->set_mode)
ld->ops->set_mode(ld, evdata->data);
diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c
index 00fe0efeaee9..e6a1c805064f 100644
--- a/drivers/video/fbdev/core/fbmem.c
+++ b/drivers/video/fbdev/core/fbmem.c
@@ -1058,7 +1058,7 @@ int
fb_blank(struct fb_info *info, int blank)
{
struct fb_event event;
- int ret = -EINVAL, early_ret;
+ int ret = -EINVAL;
if (blank > FB_BLANK_POWERDOWN)
blank = FB_BLANK_POWERDOWN;
@@ -1066,21 +1066,11 @@ fb_blank(struct fb_info *info, int blank)
event.info = info;
event.data = ␣
- early_ret = fb_notifier_call_chain(FB_EARLY_EVENT_BLANK, &event);
-
if (info->fbops->fb_blank)
ret = info->fbops->fb_blank(blank, info);
if (!ret)
fb_notifier_call_chain(FB_EVENT_BLANK, &event);
- else {
- /*
- * if fb_blank is failed then revert effects of
- * the early blank event.
- */
- if (!early_ret)
- fb_notifier_call_chain(FB_R_EARLY_EVENT_BLANK, &event);
- }
return ret;
}
diff --git a/include/linux/fb.h b/include/linux/fb.h
index 50948e519897..756706b666a1 100644
--- a/include/linux/fb.h
+++ b/include/linux/fb.h
@@ -135,10 +135,6 @@ struct fb_cursor_user {
/* A display blank is requested */
#define FB_EVENT_BLANK 0x09
-/* A hardware display blank early change occurred */
-#define FB_EARLY_EVENT_BLANK 0x10
-/* A hardware display blank revert early change occurred */
-#define FB_R_EARLY_EVENT_BLANK 0x11
struct fb_event {
struct fb_info *info;
diff --git a/include/linux/lcd.h b/include/linux/lcd.h
index 851eee8fff25..238fb1dfed98 100644
--- a/include/linux/lcd.h
+++ b/include/linux/lcd.h
@@ -41,16 +41,6 @@ struct lcd_ops {
/* Get the LCD panel power status (0: full on, 1..3: controller
power on, flat panel power off, 4: full off), see FB_BLANK_XXX */
int (*get_power)(struct lcd_device *);
- /*
- * Enable or disable power to the LCD(0: on; 4: off, see FB_BLANK_XXX)
- * and this callback would be called proir to fb driver's callback.
- *
- * P.S. note that if early_set_power is not NULL then early fb notifier
- * would be registered.
- */
- int (*early_set_power)(struct lcd_device *, int power);
- /* revert the effects of the early blank event. */
- int (*r_early_set_power)(struct lcd_device *, int power);
/* Enable or disable power to the LCD (0: on; 4: off, see FB_BLANK_XXX) */
int (*set_power)(struct lcd_device *, int power);
/* Get the current contrast setting (0-max_contrast) */
--
2.20.1
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
2019-07-25 14:32 ` Sam Ravnborg
@ 2019-07-25 14:39 ` Daniel Vetter
-1 siblings, 0 replies; 22+ messages in thread
From: Daniel Vetter @ 2019-07-25 14:39 UTC (permalink / raw)
To: Sam Ravnborg
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Lee Jones, Peter Rosin
On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> There was no users left - so drop the code to support EARLY_FB_BLANK.
> This patch removes the support in backlight,
> and drop the notifier in fbmem.
>
> That EARLY_FB_BLANK is not used can be verified that no driver set any of:
>
> lcd_ops.early_set_power()
> lcd_ops.r_early_set_power()
>
> Noticed while browsing backlight code for other reasons.
Ah I didn't grep hard enough, I didn't realize that no one sets the
lcd_ops->(r_)early_set_power hooks. Nice find!
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> Cc: Peter Rosin <peda@axentia.se>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-fbdev@vger.kernel.org
> ---
>
> Build tested with various architectures, configs.
>
> Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> maintained today?
backlight is separate from fbdev in Lee's own tree, not in drm-misc. I
think at least.
-Daniel
>
> Patch needs ack from Bartlomiej first of course.
>
> Sam
>
> drivers/video/backlight/lcd.c | 8 --------
> drivers/video/fbdev/core/fbmem.c | 12 +-----------
> include/linux/fb.h | 4 ----
> include/linux/lcd.h | 10 ----------
> 4 files changed, 1 insertion(+), 33 deletions(-)
>
> diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c
> index d6b653aa4ee9..78b033358625 100644
> --- a/drivers/video/backlight/lcd.c
> +++ b/drivers/video/backlight/lcd.c
> @@ -39,14 +39,6 @@ static int fb_notifier_callback(struct notifier_block *self,
> if (event = FB_EVENT_BLANK) {
> if (ld->ops->set_power)
> ld->ops->set_power(ld, *(int *)evdata->data);
> - } else if (event = FB_EARLY_EVENT_BLANK) {
> - if (ld->ops->early_set_power)
> - ld->ops->early_set_power(ld,
> - *(int *)evdata->data);
> - } else if (event = FB_R_EARLY_EVENT_BLANK) {
> - if (ld->ops->r_early_set_power)
> - ld->ops->r_early_set_power(ld,
> - *(int *)evdata->data);
> } else {
> if (ld->ops->set_mode)
> ld->ops->set_mode(ld, evdata->data);
> diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c
> index 00fe0efeaee9..e6a1c805064f 100644
> --- a/drivers/video/fbdev/core/fbmem.c
> +++ b/drivers/video/fbdev/core/fbmem.c
> @@ -1058,7 +1058,7 @@ int
> fb_blank(struct fb_info *info, int blank)
> {
> struct fb_event event;
> - int ret = -EINVAL, early_ret;
> + int ret = -EINVAL;
>
> if (blank > FB_BLANK_POWERDOWN)
> blank = FB_BLANK_POWERDOWN;
> @@ -1066,21 +1066,11 @@ fb_blank(struct fb_info *info, int blank)
> event.info = info;
> event.data = ␣
>
> - early_ret = fb_notifier_call_chain(FB_EARLY_EVENT_BLANK, &event);
> -
> if (info->fbops->fb_blank)
> ret = info->fbops->fb_blank(blank, info);
>
> if (!ret)
> fb_notifier_call_chain(FB_EVENT_BLANK, &event);
> - else {
> - /*
> - * if fb_blank is failed then revert effects of
> - * the early blank event.
> - */
> - if (!early_ret)
> - fb_notifier_call_chain(FB_R_EARLY_EVENT_BLANK, &event);
> - }
>
> return ret;
> }
> diff --git a/include/linux/fb.h b/include/linux/fb.h
> index 50948e519897..756706b666a1 100644
> --- a/include/linux/fb.h
> +++ b/include/linux/fb.h
> @@ -135,10 +135,6 @@ struct fb_cursor_user {
>
> /* A display blank is requested */
> #define FB_EVENT_BLANK 0x09
> -/* A hardware display blank early change occurred */
> -#define FB_EARLY_EVENT_BLANK 0x10
> -/* A hardware display blank revert early change occurred */
> -#define FB_R_EARLY_EVENT_BLANK 0x11
>
> struct fb_event {
> struct fb_info *info;
> diff --git a/include/linux/lcd.h b/include/linux/lcd.h
> index 851eee8fff25..238fb1dfed98 100644
> --- a/include/linux/lcd.h
> +++ b/include/linux/lcd.h
> @@ -41,16 +41,6 @@ struct lcd_ops {
> /* Get the LCD panel power status (0: full on, 1..3: controller
> power on, flat panel power off, 4: full off), see FB_BLANK_XXX */
> int (*get_power)(struct lcd_device *);
> - /*
> - * Enable or disable power to the LCD(0: on; 4: off, see FB_BLANK_XXX)
> - * and this callback would be called proir to fb driver's callback.
> - *
> - * P.S. note that if early_set_power is not NULL then early fb notifier
> - * would be registered.
> - */
> - int (*early_set_power)(struct lcd_device *, int power);
> - /* revert the effects of the early blank event. */
> - int (*r_early_set_power)(struct lcd_device *, int power);
> /* Enable or disable power to the LCD (0: on; 4: off, see FB_BLANK_XXX) */
> int (*set_power)(struct lcd_device *, int power);
> /* Get the current contrast setting (0-max_contrast) */
> --
> 2.20.1
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
@ 2019-07-25 14:39 ` Daniel Vetter
0 siblings, 0 replies; 22+ messages in thread
From: Daniel Vetter @ 2019-07-25 14:39 UTC (permalink / raw)
To: Sam Ravnborg
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Lee Jones, Peter Rosin
On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> There was no users left - so drop the code to support EARLY_FB_BLANK.
> This patch removes the support in backlight,
> and drop the notifier in fbmem.
>
> That EARLY_FB_BLANK is not used can be verified that no driver set any of:
>
> lcd_ops.early_set_power()
> lcd_ops.r_early_set_power()
>
> Noticed while browsing backlight code for other reasons.
Ah I didn't grep hard enough, I didn't realize that no one sets the
lcd_ops->(r_)early_set_power hooks. Nice find!
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> Cc: Peter Rosin <peda@axentia.se>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-fbdev@vger.kernel.org
> ---
>
> Build tested with various architectures, configs.
>
> Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> maintained today?
backlight is separate from fbdev in Lee's own tree, not in drm-misc. I
think at least.
-Daniel
>
> Patch needs ack from Bartlomiej first of course.
>
> Sam
>
> drivers/video/backlight/lcd.c | 8 --------
> drivers/video/fbdev/core/fbmem.c | 12 +-----------
> include/linux/fb.h | 4 ----
> include/linux/lcd.h | 10 ----------
> 4 files changed, 1 insertion(+), 33 deletions(-)
>
> diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c
> index d6b653aa4ee9..78b033358625 100644
> --- a/drivers/video/backlight/lcd.c
> +++ b/drivers/video/backlight/lcd.c
> @@ -39,14 +39,6 @@ static int fb_notifier_callback(struct notifier_block *self,
> if (event == FB_EVENT_BLANK) {
> if (ld->ops->set_power)
> ld->ops->set_power(ld, *(int *)evdata->data);
> - } else if (event == FB_EARLY_EVENT_BLANK) {
> - if (ld->ops->early_set_power)
> - ld->ops->early_set_power(ld,
> - *(int *)evdata->data);
> - } else if (event == FB_R_EARLY_EVENT_BLANK) {
> - if (ld->ops->r_early_set_power)
> - ld->ops->r_early_set_power(ld,
> - *(int *)evdata->data);
> } else {
> if (ld->ops->set_mode)
> ld->ops->set_mode(ld, evdata->data);
> diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c
> index 00fe0efeaee9..e6a1c805064f 100644
> --- a/drivers/video/fbdev/core/fbmem.c
> +++ b/drivers/video/fbdev/core/fbmem.c
> @@ -1058,7 +1058,7 @@ int
> fb_blank(struct fb_info *info, int blank)
> {
> struct fb_event event;
> - int ret = -EINVAL, early_ret;
> + int ret = -EINVAL;
>
> if (blank > FB_BLANK_POWERDOWN)
> blank = FB_BLANK_POWERDOWN;
> @@ -1066,21 +1066,11 @@ fb_blank(struct fb_info *info, int blank)
> event.info = info;
> event.data = ␣
>
> - early_ret = fb_notifier_call_chain(FB_EARLY_EVENT_BLANK, &event);
> -
> if (info->fbops->fb_blank)
> ret = info->fbops->fb_blank(blank, info);
>
> if (!ret)
> fb_notifier_call_chain(FB_EVENT_BLANK, &event);
> - else {
> - /*
> - * if fb_blank is failed then revert effects of
> - * the early blank event.
> - */
> - if (!early_ret)
> - fb_notifier_call_chain(FB_R_EARLY_EVENT_BLANK, &event);
> - }
>
> return ret;
> }
> diff --git a/include/linux/fb.h b/include/linux/fb.h
> index 50948e519897..756706b666a1 100644
> --- a/include/linux/fb.h
> +++ b/include/linux/fb.h
> @@ -135,10 +135,6 @@ struct fb_cursor_user {
>
> /* A display blank is requested */
> #define FB_EVENT_BLANK 0x09
> -/* A hardware display blank early change occurred */
> -#define FB_EARLY_EVENT_BLANK 0x10
> -/* A hardware display blank revert early change occurred */
> -#define FB_R_EARLY_EVENT_BLANK 0x11
>
> struct fb_event {
> struct fb_info *info;
> diff --git a/include/linux/lcd.h b/include/linux/lcd.h
> index 851eee8fff25..238fb1dfed98 100644
> --- a/include/linux/lcd.h
> +++ b/include/linux/lcd.h
> @@ -41,16 +41,6 @@ struct lcd_ops {
> /* Get the LCD panel power status (0: full on, 1..3: controller
> power on, flat panel power off, 4: full off), see FB_BLANK_XXX */
> int (*get_power)(struct lcd_device *);
> - /*
> - * Enable or disable power to the LCD(0: on; 4: off, see FB_BLANK_XXX)
> - * and this callback would be called proir to fb driver's callback.
> - *
> - * P.S. note that if early_set_power is not NULL then early fb notifier
> - * would be registered.
> - */
> - int (*early_set_power)(struct lcd_device *, int power);
> - /* revert the effects of the early blank event. */
> - int (*r_early_set_power)(struct lcd_device *, int power);
> /* Enable or disable power to the LCD (0: on; 4: off, see FB_BLANK_XXX) */
> int (*set_power)(struct lcd_device *, int power);
> /* Get the current contrast setting (0-max_contrast) */
> --
> 2.20.1
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
2019-07-25 14:39 ` Daniel Vetter
@ 2019-07-25 15:06 ` Lee Jones
-1 siblings, 0 replies; 22+ messages in thread
From: Lee Jones @ 2019-07-25 15:06 UTC (permalink / raw)
To: Daniel Vetter
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Sam Ravnborg, Peter Rosin
On Thu, 25 Jul 2019, Daniel Vetter wrote:
> On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > There was no users left - so drop the code to support EARLY_FB_BLANK.
> > This patch removes the support in backlight,
> > and drop the notifier in fbmem.
> >
> > That EARLY_FB_BLANK is not used can be verified that no driver set any of:
> >
> > lcd_ops.early_set_power()
> > lcd_ops.r_early_set_power()
> >
> > Noticed while browsing backlight code for other reasons.
>
> Ah I didn't grep hard enough, I didn't realize that no one sets the
> lcd_ops->(r_)early_set_power hooks. Nice find!
>
> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
>
> > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > Cc: Lee Jones <lee.jones@linaro.org>
> > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > Cc: Jingoo Han <jingoohan1@gmail.com>
> > Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > Cc: Sam Ravnborg <sam@ravnborg.org>
> > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> > Cc: Peter Rosin <peda@axentia.se>
> > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > Cc: dri-devel@lists.freedesktop.org
> > Cc: linux-fbdev@vger.kernel.org
> > ---
> >
> > Build tested with various architectures, configs.
> >
> > Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> > maintained today?
>
> backlight is separate from fbdev in Lee's own tree, not in drm-misc. I
> think at least.
That's correct. We'll sort that once we have all the Acks.
> > drivers/video/backlight/lcd.c | 8 --------
> > drivers/video/fbdev/core/fbmem.c | 12 +-----------
> > include/linux/fb.h | 4 ----
> > include/linux/lcd.h | 10 ----------
> > 4 files changed, 1 insertion(+), 33 deletions(-)
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
@ 2019-07-25 15:06 ` Lee Jones
0 siblings, 0 replies; 22+ messages in thread
From: Lee Jones @ 2019-07-25 15:06 UTC (permalink / raw)
To: Daniel Vetter
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Sam Ravnborg, Peter Rosin
On Thu, 25 Jul 2019, Daniel Vetter wrote:
> On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > There was no users left - so drop the code to support EARLY_FB_BLANK.
> > This patch removes the support in backlight,
> > and drop the notifier in fbmem.
> >
> > That EARLY_FB_BLANK is not used can be verified that no driver set any of:
> >
> > lcd_ops.early_set_power()
> > lcd_ops.r_early_set_power()
> >
> > Noticed while browsing backlight code for other reasons.
>
> Ah I didn't grep hard enough, I didn't realize that no one sets the
> lcd_ops->(r_)early_set_power hooks. Nice find!
>
> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
>
> > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > Cc: Lee Jones <lee.jones@linaro.org>
> > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > Cc: Jingoo Han <jingoohan1@gmail.com>
> > Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > Cc: Sam Ravnborg <sam@ravnborg.org>
> > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> > Cc: Peter Rosin <peda@axentia.se>
> > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > Cc: dri-devel@lists.freedesktop.org
> > Cc: linux-fbdev@vger.kernel.org
> > ---
> >
> > Build tested with various architectures, configs.
> >
> > Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> > maintained today?
>
> backlight is separate from fbdev in Lee's own tree, not in drm-misc. I
> think at least.
That's correct. We'll sort that once we have all the Acks.
> > drivers/video/backlight/lcd.c | 8 --------
> > drivers/video/fbdev/core/fbmem.c | 12 +-----------
> > include/linux/fb.h | 4 ----
> > include/linux/lcd.h | 10 ----------
> > 4 files changed, 1 insertion(+), 33 deletions(-)
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
2019-07-25 15:06 ` Lee Jones
@ 2019-07-26 16:09 ` Sam Ravnborg
-1 siblings, 0 replies; 22+ messages in thread
From: Sam Ravnborg @ 2019-07-26 16:09 UTC (permalink / raw)
To: Lee Jones
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Peter Rosin
Hi Lee.
On Thu, Jul 25, 2019 at 04:06:29PM +0100, Lee Jones wrote:
> On Thu, 25 Jul 2019, Daniel Vetter wrote:
>
> > On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > > There was no users left - so drop the code to support EARLY_FB_BLANK.
> > > This patch removes the support in backlight,
> > > and drop the notifier in fbmem.
> > >
> > > That EARLY_FB_BLANK is not used can be verified that no driver set any of:
> > >
> > > lcd_ops.early_set_power()
> > > lcd_ops.r_early_set_power()
> > >
> > > Noticed while browsing backlight code for other reasons.
> >
> > Ah I didn't grep hard enough, I didn't realize that no one sets the
> > lcd_ops->(r_)early_set_power hooks. Nice find!
> >
> > Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> >
> > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > > Cc: Lee Jones <lee.jones@linaro.org>
> > > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > > Cc: Jingoo Han <jingoohan1@gmail.com>
> > > Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > > Cc: Sam Ravnborg <sam@ravnborg.org>
> > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > > Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> > > Cc: Peter Rosin <peda@axentia.se>
> > > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > > Cc: dri-devel@lists.freedesktop.org
> > > Cc: linux-fbdev@vger.kernel.org
> > > ---
> > >
> > > Build tested with various architectures, configs.
> > >
> > > Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> > > maintained today?
> >
> > backlight is separate from fbdev in Lee's own tree, not in drm-misc. I
> > think at least.
>
> That's correct. We'll sort that once we have all the Acks.
We have acks all around now.
OK that I commit this to drm-misc-next?
This is where we maintain fbdev these days. Or you could apply it to
your backlight tree.
Both solutins would be fine as the risk of introducing merge conflicts
in these code paths are minimal.
Sam
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
@ 2019-07-26 16:09 ` Sam Ravnborg
0 siblings, 0 replies; 22+ messages in thread
From: Sam Ravnborg @ 2019-07-26 16:09 UTC (permalink / raw)
To: Lee Jones
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Peter Rosin
Hi Lee.
On Thu, Jul 25, 2019 at 04:06:29PM +0100, Lee Jones wrote:
> On Thu, 25 Jul 2019, Daniel Vetter wrote:
>
> > On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > > There was no users left - so drop the code to support EARLY_FB_BLANK.
> > > This patch removes the support in backlight,
> > > and drop the notifier in fbmem.
> > >
> > > That EARLY_FB_BLANK is not used can be verified that no driver set any of:
> > >
> > > lcd_ops.early_set_power()
> > > lcd_ops.r_early_set_power()
> > >
> > > Noticed while browsing backlight code for other reasons.
> >
> > Ah I didn't grep hard enough, I didn't realize that no one sets the
> > lcd_ops->(r_)early_set_power hooks. Nice find!
> >
> > Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> >
> > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > > Cc: Lee Jones <lee.jones@linaro.org>
> > > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > > Cc: Jingoo Han <jingoohan1@gmail.com>
> > > Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > > Cc: Sam Ravnborg <sam@ravnborg.org>
> > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > > Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> > > Cc: Peter Rosin <peda@axentia.se>
> > > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > > Cc: dri-devel@lists.freedesktop.org
> > > Cc: linux-fbdev@vger.kernel.org
> > > ---
> > >
> > > Build tested with various architectures, configs.
> > >
> > > Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> > > maintained today?
> >
> > backlight is separate from fbdev in Lee's own tree, not in drm-misc. I
> > think at least.
>
> That's correct. We'll sort that once we have all the Acks.
We have acks all around now.
OK that I commit this to drm-misc-next?
This is where we maintain fbdev these days. Or you could apply it to
your backlight tree.
Both solutins would be fine as the risk of introducing merge conflicts
in these code paths are minimal.
Sam
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
2019-07-26 16:09 ` Sam Ravnborg
@ 2019-08-05 10:35 ` Lee Jones
-1 siblings, 0 replies; 22+ messages in thread
From: Lee Jones @ 2019-08-05 10:35 UTC (permalink / raw)
To: Sam Ravnborg
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Peter Rosin
On Fri, 26 Jul 2019, Sam Ravnborg wrote:
> Hi Lee.
>
> On Thu, Jul 25, 2019 at 04:06:29PM +0100, Lee Jones wrote:
> > On Thu, 25 Jul 2019, Daniel Vetter wrote:
> >
> > > On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > > > There was no users left - so drop the code to support EARLY_FB_BLANK.
> > > > This patch removes the support in backlight,
> > > > and drop the notifier in fbmem.
> > > >
> > > > That EARLY_FB_BLANK is not used can be verified that no driver set any of:
> > > >
> > > > lcd_ops.early_set_power()
> > > > lcd_ops.r_early_set_power()
> > > >
> > > > Noticed while browsing backlight code for other reasons.
> > >
> > > Ah I didn't grep hard enough, I didn't realize that no one sets the
> > > lcd_ops->(r_)early_set_power hooks. Nice find!
> > >
> > > Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> > >
> > > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > > > Cc: Lee Jones <lee.jones@linaro.org>
> > > > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > > > Cc: Jingoo Han <jingoohan1@gmail.com>
> > > > Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > > > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > > > Cc: Sam Ravnborg <sam@ravnborg.org>
> > > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > > > Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> > > > Cc: Peter Rosin <peda@axentia.se>
> > > > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > > > Cc: dri-devel@lists.freedesktop.org
> > > > Cc: linux-fbdev@vger.kernel.org
> > > > ---
> > > >
> > > > Build tested with various architectures, configs.
> > > >
> > > > Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> > > > maintained today?
> > >
> > > backlight is separate from fbdev in Lee's own tree, not in drm-misc. I
> > > think at least.
> >
> > That's correct. We'll sort that once we have all the Acks.
>
> We have acks all around now.
I am still missing Daniel T's Ack.
> OK that I commit this to drm-misc-next?
>
> This is where we maintain fbdev these days. Or you could apply it to
> your backlight tree.
>
> Both solutins would be fine as the risk of introducing merge conflicts
> in these code paths are minimal.
If that's the case, then I'd be happy to apply it to the Backlight
tree.
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
@ 2019-08-05 10:35 ` Lee Jones
0 siblings, 0 replies; 22+ messages in thread
From: Lee Jones @ 2019-08-05 10:35 UTC (permalink / raw)
To: Sam Ravnborg
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Peter Rosin
On Fri, 26 Jul 2019, Sam Ravnborg wrote:
> Hi Lee.
>
> On Thu, Jul 25, 2019 at 04:06:29PM +0100, Lee Jones wrote:
> > On Thu, 25 Jul 2019, Daniel Vetter wrote:
> >
> > > On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > > > There was no users left - so drop the code to support EARLY_FB_BLANK.
> > > > This patch removes the support in backlight,
> > > > and drop the notifier in fbmem.
> > > >
> > > > That EARLY_FB_BLANK is not used can be verified that no driver set any of:
> > > >
> > > > lcd_ops.early_set_power()
> > > > lcd_ops.r_early_set_power()
> > > >
> > > > Noticed while browsing backlight code for other reasons.
> > >
> > > Ah I didn't grep hard enough, I didn't realize that no one sets the
> > > lcd_ops->(r_)early_set_power hooks. Nice find!
> > >
> > > Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> > >
> > > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > > > Cc: Lee Jones <lee.jones@linaro.org>
> > > > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > > > Cc: Jingoo Han <jingoohan1@gmail.com>
> > > > Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > > > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > > > Cc: Sam Ravnborg <sam@ravnborg.org>
> > > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > > > Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> > > > Cc: Peter Rosin <peda@axentia.se>
> > > > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > > > Cc: dri-devel@lists.freedesktop.org
> > > > Cc: linux-fbdev@vger.kernel.org
> > > > ---
> > > >
> > > > Build tested with various architectures, configs.
> > > >
> > > > Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> > > > maintained today?
> > >
> > > backlight is separate from fbdev in Lee's own tree, not in drm-misc. I
> > > think at least.
> >
> > That's correct. We'll sort that once we have all the Acks.
>
> We have acks all around now.
I am still missing Daniel T's Ack.
> OK that I commit this to drm-misc-next?
>
> This is where we maintain fbdev these days. Or you could apply it to
> your backlight tree.
>
> Both solutins would be fine as the risk of introducing merge conflicts
> in these code paths are minimal.
If that's the case, then I'd be happy to apply it to the Backlight
tree.
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
2019-07-25 14:32 ` Sam Ravnborg
@ 2019-07-26 9:50 ` Daniel Thompson
-1 siblings, 0 replies; 22+ messages in thread
From: Daniel Thompson @ 2019-07-26 9:50 UTC (permalink / raw)
To: Sam Ravnborg
Cc: linux-fbdev, Bartlomiej Zolnierkiewicz, Daniel Vetter, dri-devel,
Michał Mirosław, Gerd Hoffmann, Jingoo Han, Lee Jones,
Peter Rosin
On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> There was no users left - so drop the code to support EARLY_FB_BLANK.
Why are we using a different noun for the subject and description?
> This patch removes the support in backlight,
> and drop the notifier in fbmem.
>
> That EARLY_FB_BLANK is not used can be verified that no driver set any of:
>
> lcd_ops.early_set_power()
> lcd_ops.r_early_set_power()
>
> Noticed while browsing backlight code for other reasons.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> Cc: Peter Rosin <peda@axentia.se>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-fbdev@vger.kernel.org
Other than the quibble about the description...
Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
> ---
>
> Build tested with various architectures, configs.
>
> Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> maintained today?
>
> Patch needs ack from Bartlomiej first of course.
>
> Sam
>
> drivers/video/backlight/lcd.c | 8 --------
> drivers/video/fbdev/core/fbmem.c | 12 +-----------
> include/linux/fb.h | 4 ----
> include/linux/lcd.h | 10 ----------
> 4 files changed, 1 insertion(+), 33 deletions(-)
>
> diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c
> index d6b653aa4ee9..78b033358625 100644
> --- a/drivers/video/backlight/lcd.c
> +++ b/drivers/video/backlight/lcd.c
> @@ -39,14 +39,6 @@ static int fb_notifier_callback(struct notifier_block *self,
> if (event = FB_EVENT_BLANK) {
> if (ld->ops->set_power)
> ld->ops->set_power(ld, *(int *)evdata->data);
> - } else if (event = FB_EARLY_EVENT_BLANK) {
> - if (ld->ops->early_set_power)
> - ld->ops->early_set_power(ld,
> - *(int *)evdata->data);
> - } else if (event = FB_R_EARLY_EVENT_BLANK) {
> - if (ld->ops->r_early_set_power)
> - ld->ops->r_early_set_power(ld,
> - *(int *)evdata->data);
> } else {
> if (ld->ops->set_mode)
> ld->ops->set_mode(ld, evdata->data);
> diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c
> index 00fe0efeaee9..e6a1c805064f 100644
> --- a/drivers/video/fbdev/core/fbmem.c
> +++ b/drivers/video/fbdev/core/fbmem.c
> @@ -1058,7 +1058,7 @@ int
> fb_blank(struct fb_info *info, int blank)
> {
> struct fb_event event;
> - int ret = -EINVAL, early_ret;
> + int ret = -EINVAL;
>
> if (blank > FB_BLANK_POWERDOWN)
> blank = FB_BLANK_POWERDOWN;
> @@ -1066,21 +1066,11 @@ fb_blank(struct fb_info *info, int blank)
> event.info = info;
> event.data = ␣
>
> - early_ret = fb_notifier_call_chain(FB_EARLY_EVENT_BLANK, &event);
> -
> if (info->fbops->fb_blank)
> ret = info->fbops->fb_blank(blank, info);
>
> if (!ret)
> fb_notifier_call_chain(FB_EVENT_BLANK, &event);
> - else {
> - /*
> - * if fb_blank is failed then revert effects of
> - * the early blank event.
> - */
> - if (!early_ret)
> - fb_notifier_call_chain(FB_R_EARLY_EVENT_BLANK, &event);
> - }
>
> return ret;
> }
> diff --git a/include/linux/fb.h b/include/linux/fb.h
> index 50948e519897..756706b666a1 100644
> --- a/include/linux/fb.h
> +++ b/include/linux/fb.h
> @@ -135,10 +135,6 @@ struct fb_cursor_user {
>
> /* A display blank is requested */
> #define FB_EVENT_BLANK 0x09
> -/* A hardware display blank early change occurred */
> -#define FB_EARLY_EVENT_BLANK 0x10
> -/* A hardware display blank revert early change occurred */
> -#define FB_R_EARLY_EVENT_BLANK 0x11
>
> struct fb_event {
> struct fb_info *info;
> diff --git a/include/linux/lcd.h b/include/linux/lcd.h
> index 851eee8fff25..238fb1dfed98 100644
> --- a/include/linux/lcd.h
> +++ b/include/linux/lcd.h
> @@ -41,16 +41,6 @@ struct lcd_ops {
> /* Get the LCD panel power status (0: full on, 1..3: controller
> power on, flat panel power off, 4: full off), see FB_BLANK_XXX */
> int (*get_power)(struct lcd_device *);
> - /*
> - * Enable or disable power to the LCD(0: on; 4: off, see FB_BLANK_XXX)
> - * and this callback would be called proir to fb driver's callback.
> - *
> - * P.S. note that if early_set_power is not NULL then early fb notifier
> - * would be registered.
> - */
> - int (*early_set_power)(struct lcd_device *, int power);
> - /* revert the effects of the early blank event. */
> - int (*r_early_set_power)(struct lcd_device *, int power);
> /* Enable or disable power to the LCD (0: on; 4: off, see FB_BLANK_XXX) */
> int (*set_power)(struct lcd_device *, int power);
> /* Get the current contrast setting (0-max_contrast) */
> --
> 2.20.1
>
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
@ 2019-07-26 9:50 ` Daniel Thompson
0 siblings, 0 replies; 22+ messages in thread
From: Daniel Thompson @ 2019-07-26 9:50 UTC (permalink / raw)
To: Sam Ravnborg
Cc: linux-fbdev, Bartlomiej Zolnierkiewicz, Daniel Vetter, dri-devel,
Michał Mirosław, Gerd Hoffmann, Jingoo Han, Lee Jones,
Peter Rosin
On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> There was no users left - so drop the code to support EARLY_FB_BLANK.
Why are we using a different noun for the subject and description?
> This patch removes the support in backlight,
> and drop the notifier in fbmem.
>
> That EARLY_FB_BLANK is not used can be verified that no driver set any of:
>
> lcd_ops.early_set_power()
> lcd_ops.r_early_set_power()
>
> Noticed while browsing backlight code for other reasons.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> Cc: Peter Rosin <peda@axentia.se>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-fbdev@vger.kernel.org
Other than the quibble about the description...
Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
> ---
>
> Build tested with various architectures, configs.
>
> Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> maintained today?
>
> Patch needs ack from Bartlomiej first of course.
>
> Sam
>
> drivers/video/backlight/lcd.c | 8 --------
> drivers/video/fbdev/core/fbmem.c | 12 +-----------
> include/linux/fb.h | 4 ----
> include/linux/lcd.h | 10 ----------
> 4 files changed, 1 insertion(+), 33 deletions(-)
>
> diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c
> index d6b653aa4ee9..78b033358625 100644
> --- a/drivers/video/backlight/lcd.c
> +++ b/drivers/video/backlight/lcd.c
> @@ -39,14 +39,6 @@ static int fb_notifier_callback(struct notifier_block *self,
> if (event == FB_EVENT_BLANK) {
> if (ld->ops->set_power)
> ld->ops->set_power(ld, *(int *)evdata->data);
> - } else if (event == FB_EARLY_EVENT_BLANK) {
> - if (ld->ops->early_set_power)
> - ld->ops->early_set_power(ld,
> - *(int *)evdata->data);
> - } else if (event == FB_R_EARLY_EVENT_BLANK) {
> - if (ld->ops->r_early_set_power)
> - ld->ops->r_early_set_power(ld,
> - *(int *)evdata->data);
> } else {
> if (ld->ops->set_mode)
> ld->ops->set_mode(ld, evdata->data);
> diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c
> index 00fe0efeaee9..e6a1c805064f 100644
> --- a/drivers/video/fbdev/core/fbmem.c
> +++ b/drivers/video/fbdev/core/fbmem.c
> @@ -1058,7 +1058,7 @@ int
> fb_blank(struct fb_info *info, int blank)
> {
> struct fb_event event;
> - int ret = -EINVAL, early_ret;
> + int ret = -EINVAL;
>
> if (blank > FB_BLANK_POWERDOWN)
> blank = FB_BLANK_POWERDOWN;
> @@ -1066,21 +1066,11 @@ fb_blank(struct fb_info *info, int blank)
> event.info = info;
> event.data = ␣
>
> - early_ret = fb_notifier_call_chain(FB_EARLY_EVENT_BLANK, &event);
> -
> if (info->fbops->fb_blank)
> ret = info->fbops->fb_blank(blank, info);
>
> if (!ret)
> fb_notifier_call_chain(FB_EVENT_BLANK, &event);
> - else {
> - /*
> - * if fb_blank is failed then revert effects of
> - * the early blank event.
> - */
> - if (!early_ret)
> - fb_notifier_call_chain(FB_R_EARLY_EVENT_BLANK, &event);
> - }
>
> return ret;
> }
> diff --git a/include/linux/fb.h b/include/linux/fb.h
> index 50948e519897..756706b666a1 100644
> --- a/include/linux/fb.h
> +++ b/include/linux/fb.h
> @@ -135,10 +135,6 @@ struct fb_cursor_user {
>
> /* A display blank is requested */
> #define FB_EVENT_BLANK 0x09
> -/* A hardware display blank early change occurred */
> -#define FB_EARLY_EVENT_BLANK 0x10
> -/* A hardware display blank revert early change occurred */
> -#define FB_R_EARLY_EVENT_BLANK 0x11
>
> struct fb_event {
> struct fb_info *info;
> diff --git a/include/linux/lcd.h b/include/linux/lcd.h
> index 851eee8fff25..238fb1dfed98 100644
> --- a/include/linux/lcd.h
> +++ b/include/linux/lcd.h
> @@ -41,16 +41,6 @@ struct lcd_ops {
> /* Get the LCD panel power status (0: full on, 1..3: controller
> power on, flat panel power off, 4: full off), see FB_BLANK_XXX */
> int (*get_power)(struct lcd_device *);
> - /*
> - * Enable or disable power to the LCD(0: on; 4: off, see FB_BLANK_XXX)
> - * and this callback would be called proir to fb driver's callback.
> - *
> - * P.S. note that if early_set_power is not NULL then early fb notifier
> - * would be registered.
> - */
> - int (*early_set_power)(struct lcd_device *, int power);
> - /* revert the effects of the early blank event. */
> - int (*r_early_set_power)(struct lcd_device *, int power);
> /* Enable or disable power to the LCD (0: on; 4: off, see FB_BLANK_XXX) */
> int (*set_power)(struct lcd_device *, int power);
> /* Get the current contrast setting (0-max_contrast) */
> --
> 2.20.1
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
2019-07-26 9:50 ` Daniel Thompson
@ 2019-07-26 11:27 ` Sam Ravnborg
-1 siblings, 0 replies; 22+ messages in thread
From: Sam Ravnborg @ 2019-07-26 11:27 UTC (permalink / raw)
To: Daniel Thompson
Cc: linux-fbdev, Bartlomiej Zolnierkiewicz, Daniel Vetter, dri-devel,
Michał Mirosław, Gerd Hoffmann, Jingoo Han, Lee Jones,
Peter Rosin
Hi Daniel.
On Fri, Jul 26, 2019 at 10:50:16AM +0100, Daniel Thompson wrote:
> On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > There was no users left - so drop the code to support EARLY_FB_BLANK.
>
> Why are we using a different noun for the subject and description?
I fat-fingered the description.
Will fix when I apply - or send out a v2 if requested.
...
> >
> > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > Cc: Lee Jones <lee.jones@linaro.org>
> > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > Cc: Jingoo Han <jingoohan1@gmail.com>
> > Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > Cc: Sam Ravnborg <sam@ravnborg.org>
> > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> > Cc: Peter Rosin <peda@axentia.se>
> > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > Cc: dri-devel@lists.freedesktop.org
> > Cc: linux-fbdev@vger.kernel.org
>
> Other than the quibble about the description...
>
> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
Thanks,
Sam
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
@ 2019-07-26 11:27 ` Sam Ravnborg
0 siblings, 0 replies; 22+ messages in thread
From: Sam Ravnborg @ 2019-07-26 11:27 UTC (permalink / raw)
To: Daniel Thompson
Cc: linux-fbdev, Bartlomiej Zolnierkiewicz, Daniel Vetter, dri-devel,
Michał Mirosław, Gerd Hoffmann, Jingoo Han, Lee Jones,
Peter Rosin
Hi Daniel.
On Fri, Jul 26, 2019 at 10:50:16AM +0100, Daniel Thompson wrote:
> On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > There was no users left - so drop the code to support EARLY_FB_BLANK.
>
> Why are we using a different noun for the subject and description?
I fat-fingered the description.
Will fix when I apply - or send out a v2 if requested.
...
> >
> > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > Cc: Lee Jones <lee.jones@linaro.org>
> > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > Cc: Jingoo Han <jingoohan1@gmail.com>
> > Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > Cc: Sam Ravnborg <sam@ravnborg.org>
> > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> > Cc: Peter Rosin <peda@axentia.se>
> > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > Cc: dri-devel@lists.freedesktop.org
> > Cc: linux-fbdev@vger.kernel.org
>
> Other than the quibble about the description...
>
> Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
Thanks,
Sam
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
2019-07-26 11:27 ` Sam Ravnborg
@ 2019-07-26 13:05 ` Daniel Thompson
-1 siblings, 0 replies; 22+ messages in thread
From: Daniel Thompson @ 2019-07-26 13:05 UTC (permalink / raw)
To: Sam Ravnborg
Cc: linux-fbdev, Bartlomiej Zolnierkiewicz, Daniel Vetter, dri-devel,
Michał Mirosław, Gerd Hoffmann, Jingoo Han, Lee Jones,
Peter Rosin
On Fri, Jul 26, 2019 at 01:27:25PM +0200, Sam Ravnborg wrote:
> Hi Daniel.
>
> On Fri, Jul 26, 2019 at 10:50:16AM +0100, Daniel Thompson wrote:
> > On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > > There was no users left - so drop the code to support EARLY_FB_BLANK.
> >
> > Why are we using a different noun for the subject and description?
> I fat-fingered the description.
> Will fix when I apply - or send out a v2 if requested.
No need to me to see a v2 if that's the only change.
Daniel.
>
> ...
>
> > >
> > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > > Cc: Lee Jones <lee.jones@linaro.org>
> > > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > > Cc: Jingoo Han <jingoohan1@gmail.com>
> > > Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > > Cc: Sam Ravnborg <sam@ravnborg.org>
> > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > > Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> > > Cc: Peter Rosin <peda@axentia.se>
> > > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > > Cc: dri-devel@lists.freedesktop.org
> > > Cc: linux-fbdev@vger.kernel.org
> >
> > Other than the quibble about the description...
> >
> > Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
>
> Thanks,
>
> Sam
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
@ 2019-07-26 13:05 ` Daniel Thompson
0 siblings, 0 replies; 22+ messages in thread
From: Daniel Thompson @ 2019-07-26 13:05 UTC (permalink / raw)
To: Sam Ravnborg
Cc: linux-fbdev, Bartlomiej Zolnierkiewicz, Daniel Vetter, dri-devel,
Michał Mirosław, Gerd Hoffmann, Jingoo Han, Lee Jones,
Peter Rosin
On Fri, Jul 26, 2019 at 01:27:25PM +0200, Sam Ravnborg wrote:
> Hi Daniel.
>
> On Fri, Jul 26, 2019 at 10:50:16AM +0100, Daniel Thompson wrote:
> > On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > > There was no users left - so drop the code to support EARLY_FB_BLANK.
> >
> > Why are we using a different noun for the subject and description?
> I fat-fingered the description.
> Will fix when I apply - or send out a v2 if requested.
No need to me to see a v2 if that's the only change.
Daniel.
>
> ...
>
> > >
> > > Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> > > Cc: Lee Jones <lee.jones@linaro.org>
> > > Cc: Daniel Thompson <daniel.thompson@linaro.org>
> > > Cc: Jingoo Han <jingoohan1@gmail.com>
> > > Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> > > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > > Cc: Sam Ravnborg <sam@ravnborg.org>
> > > Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> > > Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> > > Cc: Peter Rosin <peda@axentia.se>
> > > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > > Cc: dri-devel@lists.freedesktop.org
> > > Cc: linux-fbdev@vger.kernel.org
> >
> > Other than the quibble about the description...
> >
> > Acked-by: Daniel Thompson <daniel.thompson@linaro.org>
>
> Thanks,
>
> Sam
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
2019-07-26 11:27 ` Sam Ravnborg
@ 2019-08-05 10:36 ` Lee Jones
-1 siblings, 0 replies; 22+ messages in thread
From: Lee Jones @ 2019-08-05 10:36 UTC (permalink / raw)
To: Sam Ravnborg
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Peter Rosin
On Fri, 26 Jul 2019, Sam Ravnborg wrote:
> Hi Daniel.
>
> On Fri, Jul 26, 2019 at 10:50:16AM +0100, Daniel Thompson wrote:
> > On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > > There was no users left - so drop the code to support EARLY_FB_BLANK.
> >
> > Why are we using a different noun for the subject and description?
> I fat-fingered the description.
> Will fix when I apply - or send out a v2 if requested.
Okay, just apply it then:
Acked-by: Lee Jones <lee.jones@linaro.org>
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
@ 2019-08-05 10:36 ` Lee Jones
0 siblings, 0 replies; 22+ messages in thread
From: Lee Jones @ 2019-08-05 10:36 UTC (permalink / raw)
To: Sam Ravnborg
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Peter Rosin
On Fri, 26 Jul 2019, Sam Ravnborg wrote:
> Hi Daniel.
>
> On Fri, Jul 26, 2019 at 10:50:16AM +0100, Daniel Thompson wrote:
> > On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > > There was no users left - so drop the code to support EARLY_FB_BLANK.
> >
> > Why are we using a different noun for the subject and description?
> I fat-fingered the description.
> Will fix when I apply - or send out a v2 if requested.
Okay, just apply it then:
Acked-by: Lee Jones <lee.jones@linaro.org>
--
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
2019-08-05 10:36 ` Lee Jones
@ 2019-08-05 20:29 ` Sam Ravnborg
-1 siblings, 0 replies; 22+ messages in thread
From: Sam Ravnborg @ 2019-08-05 20:29 UTC (permalink / raw)
To: Lee Jones
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Peter Rosin
Hi Lee.
On Mon, Aug 05, 2019 at 11:36:00AM +0100, Lee Jones wrote:
> On Fri, 26 Jul 2019, Sam Ravnborg wrote:
>
> > Hi Daniel.
> >
> > On Fri, Jul 26, 2019 at 10:50:16AM +0100, Daniel Thompson wrote:
> > > On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > > > There was no users left - so drop the code to support EARLY_FB_BLANK.
> > >
> > > Why are we using a different noun for the subject and description?
> > I fat-fingered the description.
>
> > Will fix when I apply - or send out a v2 if requested.
>
> Okay, just apply it then:
>
> Acked-by: Lee Jones <lee.jones@linaro.org>
Fixed changelog and applied to drm-misc-next.
Sam
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
@ 2019-08-05 20:29 ` Sam Ravnborg
0 siblings, 0 replies; 22+ messages in thread
From: Sam Ravnborg @ 2019-08-05 20:29 UTC (permalink / raw)
To: Lee Jones
Cc: Daniel Thompson, Bartlomiej Zolnierkiewicz, Daniel Vetter,
linux-fbdev, dri-devel, Michał Mirosław, Gerd Hoffmann,
Jingoo Han, Peter Rosin
Hi Lee.
On Mon, Aug 05, 2019 at 11:36:00AM +0100, Lee Jones wrote:
> On Fri, 26 Jul 2019, Sam Ravnborg wrote:
>
> > Hi Daniel.
> >
> > On Fri, Jul 26, 2019 at 10:50:16AM +0100, Daniel Thompson wrote:
> > > On Thu, Jul 25, 2019 at 04:32:24PM +0200, Sam Ravnborg wrote:
> > > > There was no users left - so drop the code to support EARLY_FB_BLANK.
> > >
> > > Why are we using a different noun for the subject and description?
> > I fat-fingered the description.
>
> > Will fix when I apply - or send out a v2 if requested.
>
> Okay, just apply it then:
>
> Acked-by: Lee Jones <lee.jones@linaro.org>
Fixed changelog and applied to drm-misc-next.
Sam
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
2019-07-25 14:32 ` Sam Ravnborg
@ 2019-07-26 16:05 ` Bartlomiej Zolnierkiewicz
-1 siblings, 0 replies; 22+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2019-07-26 16:05 UTC (permalink / raw)
To: Sam Ravnborg
Cc: Daniel Thompson, Jingoo Han, linux-fbdev, dri-devel,
Michał Mirosław, Gerd Hoffmann, Daniel Vetter,
Lee Jones, Peter Rosin
On 7/25/19 4:32 PM, Sam Ravnborg wrote:
> There was no users left - so drop the code to support EARLY_FB_BLANK.
> This patch removes the support in backlight,
> and drop the notifier in fbmem.
>
> That EARLY_FB_BLANK is not used can be verified that no driver set any of:
>
> lcd_ops.early_set_power()
> lcd_ops.r_early_set_power()
>
> Noticed while browsing backlight code for other reasons.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> Cc: Peter Rosin <peda@axentia.se>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-fbdev@vger.kernel.org
> ---
>
> Build tested with various architectures, configs.
>
> Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> maintained today?
>
> Patch needs ack from Bartlomiej first of course.
Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> drivers/video/backlight/lcd.c | 8 --------
> drivers/video/fbdev/core/fbmem.c | 12 +-----------
> include/linux/fb.h | 4 ----
> include/linux/lcd.h | 10 ----------
> 4 files changed, 1 insertion(+), 33 deletions(-)
>
> diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c
> index d6b653aa4ee9..78b033358625 100644
> --- a/drivers/video/backlight/lcd.c
> +++ b/drivers/video/backlight/lcd.c
> @@ -39,14 +39,6 @@ static int fb_notifier_callback(struct notifier_block *self,
> if (event = FB_EVENT_BLANK) {
> if (ld->ops->set_power)
> ld->ops->set_power(ld, *(int *)evdata->data);
> - } else if (event = FB_EARLY_EVENT_BLANK) {
> - if (ld->ops->early_set_power)
> - ld->ops->early_set_power(ld,
> - *(int *)evdata->data);
> - } else if (event = FB_R_EARLY_EVENT_BLANK) {
> - if (ld->ops->r_early_set_power)
> - ld->ops->r_early_set_power(ld,
> - *(int *)evdata->data);
> } else {
> if (ld->ops->set_mode)
> ld->ops->set_mode(ld, evdata->data);
> diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c
> index 00fe0efeaee9..e6a1c805064f 100644
> --- a/drivers/video/fbdev/core/fbmem.c
> +++ b/drivers/video/fbdev/core/fbmem.c
> @@ -1058,7 +1058,7 @@ int
> fb_blank(struct fb_info *info, int blank)
> {
> struct fb_event event;
> - int ret = -EINVAL, early_ret;
> + int ret = -EINVAL;
>
> if (blank > FB_BLANK_POWERDOWN)
> blank = FB_BLANK_POWERDOWN;
> @@ -1066,21 +1066,11 @@ fb_blank(struct fb_info *info, int blank)
> event.info = info;
> event.data = ␣
>
> - early_ret = fb_notifier_call_chain(FB_EARLY_EVENT_BLANK, &event);
> -
> if (info->fbops->fb_blank)
> ret = info->fbops->fb_blank(blank, info);
>
> if (!ret)
> fb_notifier_call_chain(FB_EVENT_BLANK, &event);
> - else {
> - /*
> - * if fb_blank is failed then revert effects of
> - * the early blank event.
> - */
> - if (!early_ret)
> - fb_notifier_call_chain(FB_R_EARLY_EVENT_BLANK, &event);
> - }
>
> return ret;
> }
> diff --git a/include/linux/fb.h b/include/linux/fb.h
> index 50948e519897..756706b666a1 100644
> --- a/include/linux/fb.h
> +++ b/include/linux/fb.h
> @@ -135,10 +135,6 @@ struct fb_cursor_user {
>
> /* A display blank is requested */
> #define FB_EVENT_BLANK 0x09
> -/* A hardware display blank early change occurred */
> -#define FB_EARLY_EVENT_BLANK 0x10
> -/* A hardware display blank revert early change occurred */
> -#define FB_R_EARLY_EVENT_BLANK 0x11
>
> struct fb_event {
> struct fb_info *info;
> diff --git a/include/linux/lcd.h b/include/linux/lcd.h
> index 851eee8fff25..238fb1dfed98 100644
> --- a/include/linux/lcd.h
> +++ b/include/linux/lcd.h
> @@ -41,16 +41,6 @@ struct lcd_ops {
> /* Get the LCD panel power status (0: full on, 1..3: controller
> power on, flat panel power off, 4: full off), see FB_BLANK_XXX */
> int (*get_power)(struct lcd_device *);
> - /*
> - * Enable or disable power to the LCD(0: on; 4: off, see FB_BLANK_XXX)
> - * and this callback would be called proir to fb driver's callback.
> - *
> - * P.S. note that if early_set_power is not NULL then early fb notifier
> - * would be registered.
> - */
> - int (*early_set_power)(struct lcd_device *, int power);
> - /* revert the effects of the early blank event. */
> - int (*r_early_set_power)(struct lcd_device *, int power);
> /* Enable or disable power to the LCD (0: on; 4: off, see FB_BLANK_XXX) */
> int (*set_power)(struct lcd_device *, int power);
> /* Get the current contrast setting (0-max_contrast) */
Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v1 1/1] backlight: drop EARLY_EVENT_BLANK support
@ 2019-07-26 16:05 ` Bartlomiej Zolnierkiewicz
0 siblings, 0 replies; 22+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2019-07-26 16:05 UTC (permalink / raw)
To: Sam Ravnborg
Cc: Daniel Thompson, Jingoo Han, linux-fbdev, dri-devel,
Michał Mirosław, Gerd Hoffmann, Daniel Vetter,
Lee Jones, Peter Rosin
On 7/25/19 4:32 PM, Sam Ravnborg wrote:
> There was no users left - so drop the code to support EARLY_FB_BLANK.
> This patch removes the support in backlight,
> and drop the notifier in fbmem.
>
> That EARLY_FB_BLANK is not used can be verified that no driver set any of:
>
> lcd_ops.early_set_power()
> lcd_ops.r_early_set_power()
>
> Noticed while browsing backlight code for other reasons.
>
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Daniel Thompson <daniel.thompson@linaro.org>
> Cc: Jingoo Han <jingoohan1@gmail.com>
> Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Sam Ravnborg <sam@ravnborg.org>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: "Michał Mirosław" <mirq-linux@rere.qmqm.pl>
> Cc: Peter Rosin <peda@axentia.se>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: dri-devel@lists.freedesktop.org
> Cc: linux-fbdev@vger.kernel.org
> ---
>
> Build tested with various architectures, configs.
>
> Lee, Daniel - OK to commit to drm-misc-next where fbdev stuff is
> maintained today?
>
> Patch needs ack from Bartlomiej first of course.
Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
> drivers/video/backlight/lcd.c | 8 --------
> drivers/video/fbdev/core/fbmem.c | 12 +-----------
> include/linux/fb.h | 4 ----
> include/linux/lcd.h | 10 ----------
> 4 files changed, 1 insertion(+), 33 deletions(-)
>
> diff --git a/drivers/video/backlight/lcd.c b/drivers/video/backlight/lcd.c
> index d6b653aa4ee9..78b033358625 100644
> --- a/drivers/video/backlight/lcd.c
> +++ b/drivers/video/backlight/lcd.c
> @@ -39,14 +39,6 @@ static int fb_notifier_callback(struct notifier_block *self,
> if (event == FB_EVENT_BLANK) {
> if (ld->ops->set_power)
> ld->ops->set_power(ld, *(int *)evdata->data);
> - } else if (event == FB_EARLY_EVENT_BLANK) {
> - if (ld->ops->early_set_power)
> - ld->ops->early_set_power(ld,
> - *(int *)evdata->data);
> - } else if (event == FB_R_EARLY_EVENT_BLANK) {
> - if (ld->ops->r_early_set_power)
> - ld->ops->r_early_set_power(ld,
> - *(int *)evdata->data);
> } else {
> if (ld->ops->set_mode)
> ld->ops->set_mode(ld, evdata->data);
> diff --git a/drivers/video/fbdev/core/fbmem.c b/drivers/video/fbdev/core/fbmem.c
> index 00fe0efeaee9..e6a1c805064f 100644
> --- a/drivers/video/fbdev/core/fbmem.c
> +++ b/drivers/video/fbdev/core/fbmem.c
> @@ -1058,7 +1058,7 @@ int
> fb_blank(struct fb_info *info, int blank)
> {
> struct fb_event event;
> - int ret = -EINVAL, early_ret;
> + int ret = -EINVAL;
>
> if (blank > FB_BLANK_POWERDOWN)
> blank = FB_BLANK_POWERDOWN;
> @@ -1066,21 +1066,11 @@ fb_blank(struct fb_info *info, int blank)
> event.info = info;
> event.data = ␣
>
> - early_ret = fb_notifier_call_chain(FB_EARLY_EVENT_BLANK, &event);
> -
> if (info->fbops->fb_blank)
> ret = info->fbops->fb_blank(blank, info);
>
> if (!ret)
> fb_notifier_call_chain(FB_EVENT_BLANK, &event);
> - else {
> - /*
> - * if fb_blank is failed then revert effects of
> - * the early blank event.
> - */
> - if (!early_ret)
> - fb_notifier_call_chain(FB_R_EARLY_EVENT_BLANK, &event);
> - }
>
> return ret;
> }
> diff --git a/include/linux/fb.h b/include/linux/fb.h
> index 50948e519897..756706b666a1 100644
> --- a/include/linux/fb.h
> +++ b/include/linux/fb.h
> @@ -135,10 +135,6 @@ struct fb_cursor_user {
>
> /* A display blank is requested */
> #define FB_EVENT_BLANK 0x09
> -/* A hardware display blank early change occurred */
> -#define FB_EARLY_EVENT_BLANK 0x10
> -/* A hardware display blank revert early change occurred */
> -#define FB_R_EARLY_EVENT_BLANK 0x11
>
> struct fb_event {
> struct fb_info *info;
> diff --git a/include/linux/lcd.h b/include/linux/lcd.h
> index 851eee8fff25..238fb1dfed98 100644
> --- a/include/linux/lcd.h
> +++ b/include/linux/lcd.h
> @@ -41,16 +41,6 @@ struct lcd_ops {
> /* Get the LCD panel power status (0: full on, 1..3: controller
> power on, flat panel power off, 4: full off), see FB_BLANK_XXX */
> int (*get_power)(struct lcd_device *);
> - /*
> - * Enable or disable power to the LCD(0: on; 4: off, see FB_BLANK_XXX)
> - * and this callback would be called proir to fb driver's callback.
> - *
> - * P.S. note that if early_set_power is not NULL then early fb notifier
> - * would be registered.
> - */
> - int (*early_set_power)(struct lcd_device *, int power);
> - /* revert the effects of the early blank event. */
> - int (*r_early_set_power)(struct lcd_device *, int power);
> /* Enable or disable power to the LCD (0: on; 4: off, see FB_BLANK_XXX) */
> int (*set_power)(struct lcd_device *, int power);
> /* Get the current contrast setting (0-max_contrast) */
Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 22+ messages in thread