fbdev: gxfb: use generic power management
Switch to the new generic framework by updating function signatures and define a "struct dev_pm_ops" variable to bind PM callbacks. This way we can remove the legacy .suspend & .resume bindings from "gxfb_driver". The gxfb_suspend() is designed to function only in the case of Suspend. Thus, the code was kept inside "if (state.event == PM_EVENT_SUSPEND)" container. This is because, in the legacy framework, this callback was invoked even in the event of Freeze and Hibernate. Hence, added the load of unnecessary function-calls. The goal can be achieved by binding the callback with only "gxfb_pm_ops.suspend" in the new framework. This also avoids the step of checking "if (state.event == PM_EVENT_SUSPEND)" every time the callback is invoked. Signed-off-by: Vaibhav Gupta <vaibhavgupta40@gmail.com> Cc: Bjorn Helgaas <helgaas@kernel.org> Cc: Bjorn Helgaas <bhelgaas@google.com> Cc: Bjorn Helgaas <bjorn@helgaas.com> Cc: Vaibhav Gupta <vaibhav.varodek@gmail.com> Cc: Sam Ravnborg <sam@ravnborg.org> Cc: Paul Mackerras <paulus@samba.org> Cc: Russell King <linux@armlinux.org.uk> Cc: Andres Salomon <dilinger@queued.net> CC: Antonino Daplas <adaplas@gmail.com> Cc: Shuah Khan <skhan@linuxfoundation.org> Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> Link: https://patchwork.freedesktop.org/patch/msgid/20200819185654.151170-2-vaibhavgupta40@gmail.com
This commit is contained in:
committed by
Bartlomiej Zolnierkiewicz
parent
ad6f93e9cd
commit
fa41f28774
@@ -21,7 +21,6 @@ struct gxfb_par {
|
||||
void __iomem *dc_regs;
|
||||
void __iomem *vid_regs;
|
||||
void __iomem *gp_regs;
|
||||
#ifdef CONFIG_PM
|
||||
int powered_down;
|
||||
|
||||
/* register state, for power management functionality */
|
||||
@@ -36,7 +35,6 @@ struct gxfb_par {
|
||||
uint64_t fp[FP_REG_COUNT];
|
||||
|
||||
uint32_t pal[DC_PAL_COUNT];
|
||||
#endif
|
||||
};
|
||||
|
||||
unsigned int gx_frame_buffer_size(void);
|
||||
@@ -49,11 +47,8 @@ void gx_set_dclk_frequency(struct fb_info *info);
|
||||
void gx_configure_display(struct fb_info *info);
|
||||
int gx_blank_display(struct fb_info *info, int blank_mode);
|
||||
|
||||
#ifdef CONFIG_PM
|
||||
int gx_powerdown(struct fb_info *info);
|
||||
int gx_powerup(struct fb_info *info);
|
||||
#endif
|
||||
|
||||
|
||||
/* Graphics Processor registers (table 6-23 from the data book) */
|
||||
enum gp_registers {
|
||||
|
||||
Reference in New Issue
Block a user