Commit 34093433 authored by Thomas Zimmermann's avatar Thomas Zimmermann
Browse files

fbdev/bw2: Use initializer macro for struct fb_ops 



Initialize struct fb_ops to the correct default values with the
macro FB_DEFAULT_SBUS_OPS(). Rename the ioctl and mmap callbacks
to use the infix _sbusfb_. This makes them fit the SBUS helpers'
naming pattern. Also make the driver depend on FB_SBUS_HELPERS,
which selects the correct modules.

Signed-off-by: default avatarThomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: default avatarSam Ravnborg <sam@ravnborg.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20230806120926.5368-5-tzimmermann@suse.de
parent cbe606f3

drivers/video/fbdev/Kconfig

+1 −3
Original line number Diff line number Diff line
@@ -527,9 +527,7 @@ config FB_SBUS_HELPERS 
config FB_BW2

	bool "BWtwo support"

	depends on (FB = y) && (SPARC && FB_SBUS)

	select FB_CFB_FILLRECT

	select FB_CFB_COPYAREA

	select FB_CFB_IMAGEBLIT

	select FB_SBUS_HELPERS

	help

	  This is the frame buffer device driver for the BWtwo frame buffer.

drivers/video/fbdev/bw2.c

+5 −12
Original line number Diff line number Diff line
@@ -30,8 +30,8 @@ 


static int bw2_blank(int, struct fb_info *);



static int bw2_mmap(struct fb_info *, struct vm_area_struct *);

static int bw2_ioctl(struct fb_info *, unsigned int, unsigned long);

static int bw2_sbusfb_mmap(struct fb_info *info, struct vm_area_struct *vma);

static int bw2_sbusfb_ioctl(struct fb_info *info, unsigned int cmd, unsigned long arg);



/*

 *  Frame buffer operations
@@ -39,15 +39,8 @@ static int bw2_ioctl(struct fb_info *, unsigned int, unsigned long); 


static const struct fb_ops bw2_ops = {

	.owner			= THIS_MODULE,

	FB_DEFAULT_SBUS_OPS(bw2),

	.fb_blank		= bw2_blank,

	.fb_fillrect		= cfb_fillrect,

	.fb_copyarea		= cfb_copyarea,

	.fb_imageblit		= cfb_imageblit,

	.fb_mmap		= bw2_mmap,

	.fb_ioctl		= bw2_ioctl,

#ifdef CONFIG_COMPAT

	.fb_compat_ioctl	= sbusfb_compat_ioctl,

#endif

};



/* OBio addresses for the bwtwo registers */
@@ -160,7 +153,7 @@ static struct sbus_mmap_map bw2_mmap_map[] = { 
	{ .size = 0 }

};



static int bw2_mmap(struct fb_info *info, struct vm_area_struct *vma)

static int bw2_sbusfb_mmap(struct fb_info *info, struct vm_area_struct *vma)

{

	struct bw2_par *par = (struct bw2_par *)info->par;
@@ -170,7 +163,7 @@ static int bw2_mmap(struct fb_info *info, struct vm_area_struct *vma) 
				  vma);

}



static int bw2_ioctl(struct fb_info *info, unsigned int cmd, unsigned long arg)

static int bw2_sbusfb_ioctl(struct fb_info *info, unsigned int cmd, unsigned long arg)

{

	return sbusfb_ioctl_helper(cmd, arg, info,

				   FBTYPE_SUN2BW, 1, info->fix.smem_len);