From nobody Thu Apr 2 15:36:26 2026 Received: from outbound.ci.icloud.com (ci-2004b-snip4-6.eps.apple.com [57.103.89.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 044D2375F87 for ; Fri, 27 Mar 2026 17:15:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=57.103.89.19 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774631748; cv=none; b=kTmrLij0sV54cHpr8lmWDPO12aiXtHsncTlDLgZ2k2K+Sa5ozs4tQ5ekGEXq0xG647HqyOkMKkaIsd0Ng904cjeEZ4BUKv1av1W42acNhECLGK8c6Hbu58y1Nr/SA08YpXCy3VxKF8Fqi5aY3ZTpqrJNWdANZNuPgsQu2joT62E= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774631748; c=relaxed/simple; bh=eE+Lv3q1cTH3MliDRISoypOC4fAS09x205TL76Oa2MA=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=a/GRSxmgl3jFC02dKjLUAfizQ4kIGHN4mLiNEj7bHU3k6oO9uqY73CNZhOHNA93wsGt+HdJ5NPoXvj5TlwSmpwbPay8kUAaO92QTHB6ME2s47ncWZmovE3K3I0Os1hfyPgU/HY9XTOhMwBT1Pb0Ru62LSxnptMH1O4dzfcKpkXo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=icloud.com; spf=pass smtp.mailfrom=icloud.com; dkim=pass (2048-bit key) header.d=icloud.com header.i=@icloud.com header.b=jV3Nd3kP; arc=none smtp.client-ip=57.103.89.19 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=icloud.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=icloud.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=icloud.com header.i=@icloud.com header.b="jV3Nd3kP" Received: from outbound.ci.icloud.com (unknown [127.0.0.2]) by p00-icloudmta-asmtp-us-central-1k-60-percent-10 (Postfix) with ESMTPS id 5580E1800210; Fri, 27 Mar 2026 17:15:41 +0000 (UTC) Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=icloud.com; s=1a1hai; t=1774631744; x=1777223744; bh=qi1zMVm1pj6mdkzckXQNDQWwTNbwY3kuvKsAh6bdd1c=; h=From:To:Subject:Date:Message-ID:MIME-Version:Content-Type:x-icloud-hme; b=jV3Nd3kPJHy6N6qpN4goz7W3A5TQRF4L06tqKq1LjiRGckF8lvWdn/Ero7soCxx02TvF58vaqFbgSz0BFrb4zKInP2NaMFm4SVojcBm5HtJkCWoVY2QXQaZpJ+KHaOyp0oBVO+0+c4PA1ldO8kK5I/QIoX7XsUQqSXaD2XIDOuV7bv2+goKRo+bKy+IULpMh+h0H6pF96NqHZwUOi5kevfZDLTn+gEP2TcwCfgX3e82uiyotaVN+9xUvveokRWEe+HW5g3oyZhxjx5GBzyT0SUtaMJ+AwEtaMoO81aIBOhfWgzxeenkWVsvQxSjVSEvO/OLsPL5n90n6hX5CO5FEIw== Received: from nixos-thinkpad.fritz.box (unknown [17.57.156.36]) by p00-icloudmta-asmtp-us-central-1k-60-percent-10 (Postfix) with ESMTPSA id A61E6180009F; Fri, 27 Mar 2026 17:15:38 +0000 (UTC) From: Hadi Chokr To: sudipm.mukherjee@gmail.com, teddy.wang@siliconmotion.com Cc: gregkh@linuxfoundation.org, linux-fbdev@vger.kernel.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Hadi Chokr Subject: [PATCH v2] staging: sm750fb: fix naming conventions and remove dead code Date: Fri, 27 Mar 2026 18:08:47 +0100 Message-ID: <20260327170846.11077-2-hadichokr@icloud.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260327091402.55366-1-hadichokr@icloud.com> References: <20260327091402.55366-1-hadichokr@icloud.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-Proofpoint-GUID: kTmlP841OpejwNo6WFEgEU3j7Fu3_RQK X-Authority-Info-Out: v=2.4 cv=K6Qv3iWI c=1 sm=1 tr=0 ts=69c6bb3e cx=c_apl:c_pps:t_out a=2G65uMN5HjSv0sBfM2Yj2w==:117 a=2G65uMN5HjSv0sBfM2Yj2w==:17 a=IkcTkHD0fZMA:10 a=Yq5XynenixoA:10 a=x7bEGLp0ZPQA:10 a=r945xf9LNZYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=v3ZZPjhaAAAA:8 a=zx_4TM15JYspshava_gA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwMzI3MDEyMCBTYWx0ZWRfX9Y5JHMUBUKqM P01fqgzwqAcjgm6y3+uwZ3vDS6YHs70WBuC2gRuSAJNoNtf02EXMiWhv0IurfNa9jHUuMVNMgde 7/KxJuJc7lCysNOpt6JPKU1gMoDNU0ATTENa3bpjHtDWkUSI/B5fmWziBzvFmbo1XEYxtv2NCSJ EQPjyC9Z4c9S+5FJECH4vIck69ub6IcfOMJF7AlK/xYm9Jr781KZ3SAfM4yfgZ13I0qCbNqZHY8 KLvBRXm+AS4DpDRJkCdD8PFQN3PDk3SVmHz+iSFLZNSg5KTEN6mChIkeMrsZbKP4AIhgdqB8X30 sjqytUmIELOryY0TH2Ds6AMlK/u2l6iFlFIkz1NiUPfVX4Rwwbps5++PsUvW98= X-Proofpoint-ORIG-GUID: kTmlP841OpejwNo6WFEgEU3j7Fu3_RQK X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.51,FMLib:17.12.100.49 definitions=2026-03-27_01,2026-03-26_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 adultscore=0 mlxscore=0 suspectscore=0 spamscore=0 lowpriorityscore=0 malwarescore=0 phishscore=0 mlxlogscore=999 clxscore=1015 bulkscore=0 classifier=spam authscore=0 adjust=0 reason=mlx scancount=1 engine=8.22.0-2601150000 definitions=main-2603270120 - Convert camelCase to snake_case to fix all checkpatch warnings. - Remove unused function sm750_hw_cursor_set_data2() and its declaration. - checkpatch warned that g_fbmode should be 'static const char * const'; this revealed that lynxfb_set_fbinfo() was writing back into g_fbmode unnecessarily. Refactor to resolve the mode string into a local const char * instead. Also drop the redundant NULL initializer since static storage is zero-initialized. - Remove manual write to pdev->dev.power.power_state.event in lynxfb_resume(); fb_set_suspend() already handles the resume state correctly. Changes in v2: - Further rename Hungarian-encoded or mixed-case variables etc. to descript= ive names: pv_mem =E2=86=92 fb_mem, pv_reg =E2=86=92 reg_base, bpp =E2=86=92 bits_pe= r_pixel, src_delta =E2=86=92 src_pitch, f_color =E2=86=92 fg_color etc. Signed-off-by: Hadi Chokr --- drivers/staging/sm750fb/readme | 6 +-- drivers/staging/sm750fb/sm750.c | 71 +++++++++++++------------- drivers/staging/sm750fb/sm750.h | 26 +++++----- drivers/staging/sm750fb/sm750_accel.c | 32 ++++++------ drivers/staging/sm750fb/sm750_accel.h | 44 ++++++++-------- drivers/staging/sm750fb/sm750_cursor.c | 43 ---------------- drivers/staging/sm750fb/sm750_cursor.h | 2 - drivers/staging/sm750fb/sm750_hw.c | 22 ++++---- 8 files changed, 100 insertions(+), 146 deletions(-) diff --git a/drivers/staging/sm750fb/readme b/drivers/staging/sm750fb/readme index cfa45958b..95b7d2539 100644 --- a/drivers/staging/sm750fb/readme +++ b/drivers/staging/sm750fb/readme @@ -7,7 +7,7 @@ Introduction: =20 About the kernel module parameter of driver: =20 - Use 1280,8bpp index color and 60 hz mode: + Use 1280,8 bits_per_pixel index color and 60 hz mode: insmod ./sm750fb.ko g_option=3D"1280x1024-8@60" =20 Disable MTRR,Disable 2d acceleration,Disable hardware cursor, @@ -29,8 +29,8 @@ About the kernel module parameter of driver: it equal to modular method with below command: insmod ./sm750fb.ko g_option=3D"noaccel:1280x1024@60:otherparm:etc..." =20 - 2) if you put 800x600 into the parameter without bpp and - refresh rate, kernel driver will defaulty use 16bpp and 60hz + 2) if you put 800x600 into the parameter without bits_per_pixel and + refresh rate, kernel driver will defaulty use 16 bits_per_pixel and 60hz =20 Important: if you have vesafb enabled in your config then /dev/fb0 will be created b= y vesafb diff --git a/drivers/staging/sm750fb/sm750.c b/drivers/staging/sm750fb/sm75= 0.c index 9a42a08c8..103258819 100644 --- a/drivers/staging/sm750fb/sm750.c +++ b/drivers/staging/sm750fb/sm750.c @@ -33,7 +33,7 @@ static int g_hwcursor =3D 1; static int g_noaccel; static int g_nomtrr; -static const char *g_fbmode[] =3D {NULL, NULL}; +static const char *g_fbmode[2]; static const char *g_def_fbmode =3D "1024x768-32@60"; static char *g_settings; static int g_dualview; @@ -160,7 +160,7 @@ static void lynxfb_ops_fillrect(struct fb_info *info, { struct lynxfb_par *par; struct sm750_dev *sm750_dev; - unsigned int base, pitch, bpp, rop; + unsigned int base, pitch, bits_per_pixel, rop; u32 color; =20 if (info->state !=3D FBINFO_STATE_RUNNING) @@ -175,9 +175,9 @@ static void lynxfb_ops_fillrect(struct fb_info *info, */ base =3D par->crtc.o_screen; pitch =3D info->fix.line_length; - bpp =3D info->var.bits_per_pixel >> 3; + bits_per_pixel =3D info->var.bits_per_pixel >> 3; =20 - color =3D (bpp =3D=3D 1) ? region->color : + color =3D (bits_per_pixel =3D=3D 1) ? region->color : ((u32 *)info->pseudo_palette)[region->color]; rop =3D (region->rop !=3D ROP_COPY) ? HW_ROP2_XOR : HW_ROP2_COPY; =20 @@ -190,7 +190,7 @@ static void lynxfb_ops_fillrect(struct fb_info *info, spin_lock(&sm750_dev->slock); =20 sm750_dev->accel.de_fillrect(&sm750_dev->accel, - base, pitch, bpp, + base, pitch, bits_per_pixel, region->dx, region->dy, region->width, region->height, color, rop); @@ -202,7 +202,7 @@ static void lynxfb_ops_copyarea(struct fb_info *info, { struct lynxfb_par *par; struct sm750_dev *sm750_dev; - unsigned int base, pitch, bpp; + unsigned int base, pitch, bits_per_pixel; =20 par =3D info->par; sm750_dev =3D par->dev; @@ -213,7 +213,7 @@ static void lynxfb_ops_copyarea(struct fb_info *info, */ base =3D par->crtc.o_screen; pitch =3D info->fix.line_length; - bpp =3D info->var.bits_per_pixel >> 3; + bits_per_pixel =3D info->var.bits_per_pixel >> 3; =20 /* * If not use spin_lock, system will die if user load driver @@ -225,7 +225,7 @@ static void lynxfb_ops_copyarea(struct fb_info *info, =20 sm750_dev->accel.de_copyarea(&sm750_dev->accel, base, pitch, region->sx, region->sy, - base, pitch, bpp, region->dx, region->dy, + base, pitch, bits_per_pixel, region->dx, region->dy, region->width, region->height, HW_ROP2_COPY); spin_unlock(&sm750_dev->slock); @@ -234,7 +234,7 @@ static void lynxfb_ops_copyarea(struct fb_info *info, static void lynxfb_ops_imageblit(struct fb_info *info, const struct fb_image *image) { - unsigned int base, pitch, bpp; + unsigned int base, pitch, bits_per_pixel; unsigned int fgcol, bgcol; struct lynxfb_par *par; struct sm750_dev *sm750_dev; @@ -247,7 +247,7 @@ static void lynxfb_ops_imageblit(struct fb_info *info, */ base =3D par->crtc.o_screen; pitch =3D info->fix.line_length; - bpp =3D info->var.bits_per_pixel >> 3; + bits_per_pixel =3D info->var.bits_per_pixel >> 3; =20 /* TODO: Implement hardware acceleration for image->depth > 1 */ if (image->depth !=3D 1) { @@ -274,7 +274,7 @@ static void lynxfb_ops_imageblit(struct fb_info *info, =20 sm750_dev->accel.de_imageblit(&sm750_dev->accel, image->data, image->width >> 3, 0, - base, pitch, bpp, + base, pitch, bits_per_pixel, image->dx, image->dy, image->width, image->height, fgcol, bgcol, HW_ROP2_COPY); @@ -388,7 +388,7 @@ static int lynxfb_ops_set_par(struct fb_info *info) var->accel_flags =3D 0;/*FB_ACCELF_TEXT;*/ =20 if (ret) { - dev_err(info->device, "bpp %d not supported\n", + dev_err(info->device, "bits_per_pixel %d not supported\n", var->bits_per_pixel); return ret; } @@ -467,8 +467,6 @@ static int __maybe_unused lynxfb_resume(struct device *= dev) fb_set_suspend(info, 0); } =20 - pdev->dev.power.power_state.event =3D PM_EVENT_RESUME; - console_unlock(); return 0; } @@ -488,7 +486,7 @@ static int lynxfb_ops_check_var(struct fb_var_screeninf= o *var, ret =3D lynxfb_set_color_offsets(info); =20 if (ret) { - dev_err(info->device, "bpp %d not supported\n", + dev_err(info->device, "bits_per_pixel %d not supported\n", var->bits_per_pixel); return ret; } @@ -619,26 +617,26 @@ static int sm750fb_set_drv(struct lynxfb_par *par) output->paths =3D sm750_pnc; crtc->channel =3D sm750_primary; crtc->o_screen =3D 0; - crtc->v_screen =3D sm750_dev->pvMem; + crtc->v_screen =3D sm750_dev->fb_mem; break; case sm750_simul_sec: output->paths =3D sm750_pnc; crtc->channel =3D sm750_secondary; crtc->o_screen =3D 0; - crtc->v_screen =3D sm750_dev->pvMem; + crtc->v_screen =3D sm750_dev->fb_mem; break; case sm750_dual_normal: if (par->index =3D=3D 0) { output->paths =3D sm750_panel; crtc->channel =3D sm750_primary; crtc->o_screen =3D 0; - crtc->v_screen =3D sm750_dev->pvMem; + crtc->v_screen =3D sm750_dev->fb_mem; } else { output->paths =3D sm750_crt; crtc->channel =3D sm750_secondary; /* not consider of padding stuffs for o_screen,need fix */ crtc->o_screen =3D sm750_dev->vidmem_size >> 1; - crtc->v_screen =3D sm750_dev->pvMem + crtc->o_screen; + crtc->v_screen =3D sm750_dev->fb_mem + crtc->o_screen; } break; case sm750_dual_swap: @@ -646,7 +644,7 @@ static int sm750fb_set_drv(struct lynxfb_par *par) output->paths =3D sm750_panel; crtc->channel =3D sm750_secondary; crtc->o_screen =3D 0; - crtc->v_screen =3D sm750_dev->pvMem; + crtc->v_screen =3D sm750_dev->fb_mem; } else { output->paths =3D sm750_crt; crtc->channel =3D sm750_primary; @@ -654,7 +652,7 @@ static int sm750fb_set_drv(struct lynxfb_par *par) * need fix */ crtc->o_screen =3D sm750_dev->vidmem_size >> 1; - crtc->v_screen =3D sm750_dev->pvMem + crtc->o_screen; + crtc->v_screen =3D sm750_dev->fb_mem + crtc->o_screen; } break; default: @@ -752,13 +750,13 @@ static int lynxfb_set_fbinfo(struct fb_info *info, in= t index) * must be set after crtc member initialized */ crtc->cursor.offset =3D crtc->o_screen + crtc->vidmem_size - 1024; - crtc->cursor.mmio =3D sm750_dev->pvReg + + crtc->cursor.mmio =3D sm750_dev->reg_base + 0x800f0 + (int)crtc->channel * 0x140; =20 crtc->cursor.max_h =3D 64; crtc->cursor.max_w =3D 64; crtc->cursor.size =3D crtc->cursor.max_h * crtc->cursor.max_w * 2 / 8; - crtc->cursor.vstart =3D sm750_dev->pvMem + crtc->cursor.offset; + crtc->cursor.vstart =3D sm750_dev->fb_mem + crtc->cursor.offset; =20 memset_io(crtc->cursor.vstart, 0, crtc->cursor.size); if (!g_hwcursor) @@ -777,15 +775,16 @@ static int lynxfb_set_fbinfo(struct fb_info *info, in= t index) else info->fbops =3D &lynxfb_ops_with_cursor; } + const char *mode =3D g_fbmode[index]; =20 - if (!g_fbmode[index]) { - g_fbmode[index] =3D g_def_fbmode; - if (index) - g_fbmode[index] =3D g_fbmode[0]; + if (!mode) { + mode =3D g_def_fbmode; + if (index && g_fbmode[0]) + mode =3D g_fbmode[0]; } =20 for (i =3D 0; i < 3; i++) { - ret =3D fb_find_mode(var, info, g_fbmode[index], + ret =3D fb_find_mode(var, info, mode, pdb[i], cdb[i], NULL, 8); =20 if (ret =3D=3D 1 || ret =3D=3D 2) @@ -856,9 +855,9 @@ static void sm750fb_setup(struct sm750_dev *sm750_dev, = char *src) sm750_dev->init_parm.chip_clk =3D 0; sm750_dev->init_parm.mem_clk =3D 0; sm750_dev->init_parm.master_clk =3D 0; - sm750_dev->init_parm.powerMode =3D 0; - sm750_dev->init_parm.setAllEngOff =3D 0; - sm750_dev->init_parm.resetMemory =3D 1; + sm750_dev->init_parm.power_mode =3D 0; + sm750_dev->init_parm.set_all_eng_off =3D 0; + sm750_dev->init_parm.reset_memory =3D 1; =20 /* defaultly turn g_hwcursor on for both view */ g_hwcursor =3D 3; @@ -877,9 +876,9 @@ static void sm750fb_setup(struct sm750_dev *sm750_dev, = char *src) } else if (!strncmp(opt, "nocrt", strlen("nocrt"))) { sm750_dev->nocrt =3D 1; } else if (!strncmp(opt, "36bit", strlen("36bit"))) { - sm750_dev->pnltype =3D sm750_doubleTFT; + sm750_dev->pnltype =3D sm750_double_tft; } else if (!strncmp(opt, "18bit", strlen("18bit"))) { - sm750_dev->pnltype =3D sm750_dualTFT; + sm750_dev->pnltype =3D sm750_dual_tft; } else if (!strncmp(opt, "24bit", strlen("24bit"))) { sm750_dev->pnltype =3D sm750_24TFT; } else if (!strncmp(opt, "nohwc0", strlen("nohwc0"))) { @@ -1025,7 +1024,7 @@ static int lynxfb_pci_probe(struct pci_dev *pdev, sm750_dev->mtrr.vram =3D arch_phys_wc_add(sm750_dev->vidmem_start, sm750_dev->vidmem_size); =20 - memset_io(sm750_dev->pvMem, 0, sm750_dev->vidmem_size); + memset_io(sm750_dev->fb_mem, 0, sm750_dev->vidmem_size); =20 pci_set_drvdata(pdev, sm750_dev); =20 @@ -1056,8 +1055,8 @@ static void lynxfb_pci_remove(struct pci_dev *pdev) sm750fb_framebuffer_release(sm750_dev); arch_phys_wc_del(sm750_dev->mtrr.vram); =20 - iounmap(sm750_dev->pvReg); - iounmap(sm750_dev->pvMem); + iounmap(sm750_dev->reg_base); + iounmap(sm750_dev->fb_mem); pci_release_region(pdev, 1); kfree(g_settings); } diff --git a/drivers/staging/sm750fb/sm750.h b/drivers/staging/sm750fb/sm75= 0.h index 67b9bfa23..130ec3770 100644 --- a/drivers/staging/sm750fb/sm750.h +++ b/drivers/staging/sm750fb/sm750.h @@ -14,8 +14,8 @@ =20 enum sm750_pnltype { sm750_24TFT =3D 0, /* 24bit tft */ - sm750_dualTFT =3D 2, /* dual 18 bit tft */ - sm750_doubleTFT =3D 1, /* 36 bit double pixel tft */ + sm750_dual_tft =3D 2, /* dual 18 bit tft */ + sm750_double_tft =3D 1, /* 36 bit double pixel tft */ }; =20 /* vga channel is not concerned */ @@ -39,13 +39,13 @@ enum sm750_path { }; =20 struct init_status { - ushort powerMode; + ushort power_mode; /* below three clocks are in unit of MHZ*/ ushort chip_clk; ushort mem_clk; ushort master_clk; - ushort setAllEngOff; - ushort resetMemory; + ushort set_all_eng_off; + ushort reset_memory; }; =20 struct lynx_accel { @@ -60,22 +60,22 @@ struct lynx_accel { int (*de_wait)(void);/* see if hardware ready to work */ =20 int (*de_fillrect)(struct lynx_accel *accel, - u32 base, u32 pitch, u32 bpp, + u32 base, u32 pitch, u32 bits_per_pixel, u32 x, u32 y, u32 width, u32 height, u32 color, u32 rop); =20 int (*de_copyarea)(struct lynx_accel *accel, u32 s_base, u32 s_pitch, u32 sx, u32 sy, - u32 d_base, u32 d_pitch, - u32 bpp, u32 dx, u32 dy, + u32 dest_base, u32 dest_pitch, + u32 bits_per_pixel, u32 dx, u32 dy, u32 width, u32 height, u32 rop2); =20 - int (*de_imageblit)(struct lynx_accel *accel, const char *p_srcbuf, - u32 src_delta, u32 start_bit, u32 d_base, u32 d_pitch, + int (*de_imageblit)(struct lynx_accel *accel, const char *src_buf, + u32 src_pitch, u32 start_bit, u32 dest_base, u32 dest_pitch, u32 byte_per_pixel, u32 dx, u32 dy, u32 width, - u32 height, u32 f_color, u32 b_color, u32 rop2); + u32 height, u32 fg_color, u32 bg_color, u32 rop2); =20 }; =20 @@ -97,8 +97,8 @@ struct sm750_dev { unsigned long vidreg_start; __u32 vidmem_size; __u32 vidreg_size; - void __iomem *pvReg; - unsigned char __iomem *pvMem; + void __iomem *reg_base; + unsigned char __iomem *fb_mem; /* locks*/ spinlock_t slock; =20 diff --git a/drivers/staging/sm750fb/sm750_accel.c b/drivers/staging/sm750f= b/sm750_accel.c index 0f94d859e..b72d7f78e 100644 --- a/drivers/staging/sm750fb/sm750_accel.c +++ b/drivers/staging/sm750fb/sm750_accel.c @@ -48,7 +48,7 @@ void sm750_hw_de_init(struct lynx_accel *accel) DE_STRETCH_FORMAT_ADDRESSING_MASK | DE_STRETCH_FORMAT_SOURCE_HEIGHT_MASK; =20 - /* DE_STRETCH bpp format need be initialized in setMode routine */ + /* DE_STRETCH bits_per_pixel format need be initialized in setMode routin= e */ write_dpr(accel, DE_STRETCH_FORMAT, (read_dpr(accel, DE_STRETCH_FORMAT) & ~clr) | reg); =20 @@ -76,7 +76,7 @@ void sm750_hw_set2dformat(struct lynx_accel *accel, int f= mt) { u32 reg; =20 - /* fmt=3D0,1,2 for 8,16,32,bpp on sm718/750/502 */ + /* fmt=3D0,1,2 for 8,16,32,bits_per_pixel on sm718/750/502 */ reg =3D read_dpr(accel, DE_STRETCH_FORMAT); reg &=3D ~DE_STRETCH_FORMAT_PIXEL_FORMAT_MASK; reg |=3D ((fmt << DE_STRETCH_FORMAT_PIXEL_FORMAT_SHIFT) & @@ -85,7 +85,7 @@ void sm750_hw_set2dformat(struct lynx_accel *accel, int f= mt) } =20 int sm750_hw_fillrect(struct lynx_accel *accel, - u32 base, u32 pitch, u32 Bpp, + u32 base, u32 pitch, u32 bits_per_pixel, u32 x, u32 y, u32 width, u32 height, u32 color, u32 rop) { @@ -102,14 +102,14 @@ int sm750_hw_fillrect(struct lynx_accel *accel, =20 write_dpr(accel, DE_WINDOW_DESTINATION_BASE, base); /* dpr40 */ write_dpr(accel, DE_PITCH, - ((pitch / Bpp << DE_PITCH_DESTINATION_SHIFT) & + ((pitch / bits_per_pixel << DE_PITCH_DESTINATION_SHIFT) & DE_PITCH_DESTINATION_MASK) | - (pitch / Bpp & DE_PITCH_SOURCE_MASK)); /* dpr10 */ + (pitch / bits_per_pixel & DE_PITCH_SOURCE_MASK)); /* dpr10 */ =20 write_dpr(accel, DE_WINDOW_WIDTH, - ((pitch / Bpp << DE_WINDOW_WIDTH_DST_SHIFT) & + ((pitch / bits_per_pixel << DE_WINDOW_WIDTH_DST_SHIFT) & DE_WINDOW_WIDTH_DST_MASK) | - (pitch / Bpp & DE_WINDOW_WIDTH_SRC_MASK)); /* dpr44 */ + (pitch / bits_per_pixel & DE_WINDOW_WIDTH_SRC_MASK)); /* dpr44 */ =20 write_dpr(accel, DE_FOREGROUND, color); /* DPR14 */ =20 @@ -138,7 +138,7 @@ int sm750_hw_fillrect(struct lynx_accel *accel, * @sy: Starting y coordinate of source surface * @dest_base: Address of destination: offset in frame buffer * @dest_pitch: Pitch value of destination surface in BYTE - * @Bpp: Color depth of destination surface + * @bits_per_pixel: Color depth of destination surface * @dx: Starting x coordinate of destination surface * @dy: Starting y coordinate of destination surface * @width: width of rectangle in pixel value @@ -149,7 +149,7 @@ int sm750_hw_copyarea(struct lynx_accel *accel, unsigned int source_base, unsigned int source_pitch, unsigned int sx, unsigned int sy, unsigned int dest_base, unsigned int dest_pitch, - unsigned int Bpp, unsigned int dx, unsigned int dy, + unsigned int bits_per_pixel, unsigned int dx, unsigned int dy, unsigned int width, unsigned int height, unsigned int rop2) { @@ -249,9 +249,9 @@ int sm750_hw_copyarea(struct lynx_accel *accel, * pixel values. Need Byte to pixel conversion. */ write_dpr(accel, DE_PITCH, - ((dest_pitch / Bpp << DE_PITCH_DESTINATION_SHIFT) & + ((dest_pitch / bits_per_pixel << DE_PITCH_DESTINATION_SHIFT) & DE_PITCH_DESTINATION_MASK) | - (source_pitch / Bpp & DE_PITCH_SOURCE_MASK)); /* dpr10 */ + (source_pitch / bits_per_pixel & DE_PITCH_SOURCE_MASK)); /* dpr10 */ =20 /* * Screen Window width in Pixels. @@ -259,9 +259,9 @@ int sm750_hw_copyarea(struct lynx_accel *accel, * for a given point. */ write_dpr(accel, DE_WINDOW_WIDTH, - ((dest_pitch / Bpp << DE_WINDOW_WIDTH_DST_SHIFT) & + ((dest_pitch / bits_per_pixel << DE_WINDOW_WIDTH_DST_SHIFT) & DE_WINDOW_WIDTH_DST_MASK) | - (source_pitch / Bpp & DE_WINDOW_WIDTH_SRC_MASK)); /* dpr3c */ + (source_pitch / bits_per_pixel & DE_WINDOW_WIDTH_SRC_MASK)); /* dpr3c = */ =20 if (accel->de_wait() !=3D 0) return -1; @@ -300,7 +300,7 @@ static unsigned int de_get_transparency(struct lynx_acc= el *accel) * sm750_hw_imageblit * @accel: Acceleration device data * @src_buf: pointer to start of source buffer in system memory - * @src_delta: Pitch value (in bytes) of the source buffer, +ive means top= down + * @src_pitch: Pitch value (in bytes) of the source buffer, +ive means top= down * and -ive mean button up * @start_bit: Mono data can start at any bit in a byte, this value should= be * 0 to 7 @@ -316,7 +316,7 @@ static unsigned int de_get_transparency(struct lynx_acc= el *accel) * @rop2: ROP value */ int sm750_hw_imageblit(struct lynx_accel *accel, const char *src_buf, - u32 src_delta, u32 start_bit, u32 dest_base, u32 dest_pitch, + u32 src_pitch, u32 start_bit, u32 dest_base, u32 dest_pitch, u32 byte_per_pixel, u32 dx, u32 dy, u32 width, u32 height, u32 fg_color, u32 bg_color, u32 rop2) { @@ -405,7 +405,7 @@ int sm750_hw_imageblit(struct lynx_accel *accel, const = char *src_buf, write_dp_port(accel, *(unsigned int *)remain); } =20 - src_buf +=3D src_delta; + src_buf +=3D src_pitch; } =20 return 0; diff --git a/drivers/staging/sm750fb/sm750_accel.h b/drivers/staging/sm750f= b/sm750_accel.h index 2c79cb730..d0b3b653a 100644 --- a/drivers/staging/sm750fb/sm750_accel.h +++ b/drivers/staging/sm750fb/sm750_accel.h @@ -190,19 +190,19 @@ void sm750_hw_set2dformat(struct lynx_accel *accel, i= nt fmt); void sm750_hw_de_init(struct lynx_accel *accel); =20 int sm750_hw_fillrect(struct lynx_accel *accel, - u32 base, u32 pitch, u32 Bpp, + u32 base, u32 pitch, u32 bits_per_pixel, u32 x, u32 y, u32 width, u32 height, u32 color, u32 rop); =20 /** - * sm750_hm_copyarea - * @sBase: Address of source: offset in frame buffer - * @sPitch: Pitch value of source surface in BYTE + * sm750_hw_copyarea + * @s_base: Address of source: offset in frame buffer + * @s_pitch: Pitch value of source surface in BYTE * @sx: Starting x coordinate of source surface * @sy: Starting y coordinate of source surface - * @dBase: Address of destination: offset in frame buffer - * @dPitch: Pitch value of destination surface in BYTE - * @Bpp: Color depth of destination surface + * @dest_base: Address of destination: offset in frame buffer + * @dest_pitch: Pitch value of destination surface in BYTE + * @bits_per_pixel: Color depth of destination surface * @dx: Starting x coordinate of destination surface * @dy: Starting y coordinate of destination surface * @width: width of rectangle in pixel value @@ -210,34 +210,34 @@ int sm750_hw_fillrect(struct lynx_accel *accel, * @rop2: ROP value */ int sm750_hw_copyarea(struct lynx_accel *accel, - unsigned int sBase, unsigned int sPitch, + unsigned int s_base, unsigned int s_pitch, unsigned int sx, unsigned int sy, - unsigned int dBase, unsigned int dPitch, - unsigned int Bpp, unsigned int dx, unsigned int dy, + unsigned int dest_base, unsigned int dest_pitch, + unsigned int bits_per_pixel, unsigned int dx, unsigned int dy, unsigned int width, unsigned int height, unsigned int rop2); =20 /** * sm750_hw_imageblit - * @pSrcbuf: pointer to start of source buffer in system memory - * @srcDelta: Pitch value (in bytes) of the source buffer, +ive means top = down + * @src_buf: pointer to start of source buffer in system memory + * @src_pitch: Pitch value (in bytes) of the source buffer, +ive means top= down *>----- and -ive mean button up - * @startBit: Mono data can start at any bit in a byte, this value should = be + * @start_bit: Mono data can start at any bit in a byte, this value should= be *>----- 0 to 7 - * @dBase: Address of destination: offset in frame buffer - * @dPitch: Pitch value of destination surface in BYTE - * @bytePerPixel: Color depth of destination surface + * @dest_base: Address of destination: offset in frame buffer + * @dest_pitch: Pitch value of destination surface in BYTE + * @byte_per_pixel: Color depth of destination surface * @dx: Starting x coordinate of destination surface * @dy: Starting y coordinate of destination surface * @width: width of rectangle in pixel value * @height: height of rectangle in pixel value - * @fColor: Foreground color (corresponding to a 1 in the monochrome data - * @bColor: Background color (corresponding to a 0 in the monochrome data + * @fg_color: Foreground color (corresponding to a 1 in the monochrome data + * @bg_color: Background color (corresponding to a 0 in the monochrome data * @rop2: ROP value */ -int sm750_hw_imageblit(struct lynx_accel *accel, const char *pSrcbuf, - u32 srcDelta, u32 startBit, u32 dBase, u32 dPitch, - u32 bytePerPixel, u32 dx, u32 dy, u32 width, - u32 height, u32 fColor, u32 bColor, u32 rop2); +int sm750_hw_imageblit(struct lynx_accel *accel, const char *src_buf, + u32 src_pitch, u32 start_bit, u32 dest_base, u32 dest_pitch, + u32 byte_per_pixel, u32 dx, u32 dy, u32 width, + u32 height, u32 fg_color, u32 bg_color, u32 rop2); =20 #endif diff --git a/drivers/staging/sm750fb/sm750_cursor.c b/drivers/staging/sm750= fb/sm750_cursor.c index 7ede14490..f0338e6e7 100644 --- a/drivers/staging/sm750fb/sm750_cursor.c +++ b/drivers/staging/sm750fb/sm750_cursor.c @@ -130,46 +130,3 @@ void sm750_hw_cursor_set_data(struct lynx_cursor *curs= or, u16 rop, } } } - -void sm750_hw_cursor_set_data2(struct lynx_cursor *cursor, u16 rop, - const u8 *pcol, const u8 *pmsk) -{ - int i, j, count, pitch, offset; - u8 color, mask; - u16 data; - void __iomem *pbuffer, *pstart; - - /* in byte*/ - pitch =3D cursor->w >> 3; - - /* in byte */ - count =3D pitch * cursor->h; - - /* in byte */ - offset =3D cursor->max_w * 2 / 8; - - data =3D 0; - pstart =3D cursor->vstart; - pbuffer =3D pstart; - - for (i =3D 0; i < count; i++) { - color =3D *pcol++; - mask =3D *pmsk++; - data =3D 0; - - for (j =3D 0; j < 8; j++) { - if (mask & (1 << j)) - data |=3D ((color & (1 << j)) ? 1 : 2) << (j * 2); - } - iowrite16(data, pbuffer); - - /* assume pitch is 1,2,4,8,...*/ - if (!(i & (pitch - 1))) { - /* need a return */ - pstart +=3D offset; - pbuffer =3D pstart; - } else { - pbuffer +=3D sizeof(u16); - } - } -} diff --git a/drivers/staging/sm750fb/sm750_cursor.h b/drivers/staging/sm750= fb/sm750_cursor.h index 88fa02f63..51ba0da02 100644 --- a/drivers/staging/sm750fb/sm750_cursor.h +++ b/drivers/staging/sm750fb/sm750_cursor.h @@ -10,6 +10,4 @@ void sm750_hw_cursor_set_pos(struct lynx_cursor *cursor, = int x, int y); void sm750_hw_cursor_set_color(struct lynx_cursor *cursor, u32 fg, u32 bg); void sm750_hw_cursor_set_data(struct lynx_cursor *cursor, u16 rop, const u8 *data, const u8 *mask); -void sm750_hw_cursor_set_data2(struct lynx_cursor *cursor, u16 rop, - const u8 *data, const u8 *mask); #endif diff --git a/drivers/staging/sm750fb/sm750_hw.c b/drivers/staging/sm750fb/s= m750_hw.c index a2798d428..b1045b088 100644 --- a/drivers/staging/sm750fb/sm750_hw.c +++ b/drivers/staging/sm750fb/sm750_hw.c @@ -42,18 +42,18 @@ int hw_sm750_map(struct sm750_dev *sm750_dev, struct pc= i_dev *pdev) } =20 /* now map mmio and vidmem */ - sm750_dev->pvReg =3D + sm750_dev->reg_base =3D ioremap(sm750_dev->vidreg_start, sm750_dev->vidreg_size); - if (!sm750_dev->pvReg) { + if (!sm750_dev->reg_base) { dev_err(&pdev->dev, "mmio failed\n"); ret =3D -EFAULT; goto err_release_region; } =20 - sm750_dev->accel.dpr_base =3D sm750_dev->pvReg + DE_BASE_ADDR_TYPE1; - sm750_dev->accel.dp_port_base =3D sm750_dev->pvReg + DE_PORT_ADDR_TYPE1; + sm750_dev->accel.dpr_base =3D sm750_dev->reg_base + DE_BASE_ADDR_TYPE1; + sm750_dev->accel.dp_port_base =3D sm750_dev->reg_base + DE_PORT_ADDR_TYPE= 1; =20 - mmio750 =3D sm750_dev->pvReg; + mmio750 =3D sm750_dev->reg_base; sm750_set_chip_type(sm750_dev->devid, sm750_dev->revid); =20 sm750_dev->vidmem_start =3D pci_resource_start(pdev, 0); @@ -66,9 +66,9 @@ int hw_sm750_map(struct sm750_dev *sm750_dev, struct pci_= dev *pdev) sm750_dev->vidmem_size =3D ddk750_get_vm_size(); =20 /* reserve the vidmem space of smi adaptor */ - sm750_dev->pvMem =3D + sm750_dev->fb_mem =3D ioremap_wc(sm750_dev->vidmem_start, sm750_dev->vidmem_size); - if (!sm750_dev->pvMem) { + if (!sm750_dev->fb_mem) { dev_err(&pdev->dev, "Map video memory failed\n"); ret =3D -EFAULT; goto err_unmap_reg; @@ -77,7 +77,7 @@ int hw_sm750_map(struct sm750_dev *sm750_dev, struct pci_= dev *pdev) return 0; =20 err_unmap_reg: - iounmap(sm750_dev->pvReg); + iounmap(sm750_dev->reg_base); err_release_region: pci_release_region(pdev, 1); return ret; @@ -130,10 +130,10 @@ int hw_sm750_inithw(struct sm750_dev *sm750_dev, stru= ct pci_dev *pdev) switch (sm750_dev->pnltype) { case sm750_24TFT: break; - case sm750_doubleTFT: + case sm750_double_tft: val |=3D PANEL_DISPLAY_CTRL_DOUBLE_PIXEL; break; - case sm750_dualTFT: + case sm750_dual_tft: val |=3D PANEL_DISPLAY_CTRL_DUAL_DISPLAY; break; } @@ -248,7 +248,7 @@ int hw_sm750_crtc_set_mode(struct lynxfb_crtc *crtc, sm750_dev =3D par->dev; =20 if (!sm750_dev->accel_off) { - /* set 2d engine pixel format according to mode bpp */ + /* set 2d engine pixel format according to mode bits_per_pixel */ switch (var->bits_per_pixel) { case 8: fmt =3D 0; --=20 2.53.0