From nobody Sun Feb 8 02:56:45 2026 Received: from mail-pj1-f46.google.com (mail-pj1-f46.google.com [209.85.216.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 159023806DC for ; Wed, 4 Feb 2026 06:41:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.46 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770187289; cv=none; b=dQ23txpur3SzVsE+7F3cGAJhGJgi3vAly3bkc+yXEPwdZBw9eS/jx7iTtGB/aSD4pCDOWC3hcj+ezj4aT3MDDsHhKN0mByKUwFLf2uge0grnOx0BJTwS0H5tD4q3ebrXPF4Zd/w6T0zZLP31mw4d0/Aueta59dxFa0BBh4mPerU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770187289; c=relaxed/simple; bh=c3Ki7YhXJGsrY1EHJdXQahN87wiUTEcPt2dcaioG84k=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=JBJciLWgvb7isaOL8deGdLUaUZFKcIqJwU8XGYuzyY8QJ+7j+JUupQze/nw4eZAajjcIilS5w5dK6bNnRCy8/qGhxJoN//KamDjl6RUMD3LTIf+/8xAL/KwcrppwVsM3CXst9pFMLEBudhXK7uGijng05dgA8c3CpYsqXsVs0HE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=GvzYGzW3; arc=none smtp.client-ip=209.85.216.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="GvzYGzW3" Received: by mail-pj1-f46.google.com with SMTP id 98e67ed59e1d1-34c708702dfso3531363a91.1 for ; Tue, 03 Feb 2026 22:41:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1770187288; x=1770792088; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=bA1AdF9WhHexthfe8YUVwhafTHXgnPEul4eYzpiPpqI=; b=GvzYGzW3x5Nb5Cxq9f7k+zGyOLpKPyUzgmSMF6hI8LNJ7Xl/NYssu7sHuZboBI9LA1 g1il1G2CTNk/CKgphIllZCE0t3Ay2yzDA6xvcGks0HERRhHtCL/jBwYgveo6MKMDELkz F+JaJNDyWoyUqN/nOt3DwCdJDCfyluqKbxwnUa1dNjW5RiPN4a0xPLxfXPHDce5yYDF4 VHUzlxYpckCewbG+FC8kHIRuruokHgFPIcHr5a+DSMUwZlPFVqwUX078/Rtwaw0KuBEk Y4jt6vSTk5qdZ1mpy/W1FM4iKlhFeYItBAeIX/zpF447a2ZvH9dc+mz98xeJ5JVhDc6l GIoA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1770187288; x=1770792088; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=bA1AdF9WhHexthfe8YUVwhafTHXgnPEul4eYzpiPpqI=; b=kHMjwbSjqbKAc4dTky8wjaNKIl+XbYRHofsqCUT+VqGvv8qW+s1OhGRQiUZ7F+OWoK ub6zTGuOMYphJzJPkz38qWieD1faET/dHTeYD2aZsFVQP13DCd4zjc0Lg9wi7G8AxkY0 iUaAeEPsSAUpiPoqbNrvy6MoVY7IGqKaAqOy/cl2cpPjE8cyTOCboBvavxTmWk8POW9K dH3FTHZNXvPjFrn5BmbT/yJriot6LRfPT60s+prEc2adGP5GusCkW2ZBeTTj+3GTF9hm IOWbzFcrPoh5tbj7rJKtFJxlnA6syTjLniWvBfoZexRhaJXMz24FC5omhs9rClFwv1JJ w+FA== X-Forwarded-Encrypted: i=1; AJvYcCUFqLrSgZc/MOo0DbS1h2YWsaHmzdTZdIsLVE3R8s+YId1NtkIa8F3/LXM+fcXM4wADkDZoNPLxhTFHRWE=@vger.kernel.org X-Gm-Message-State: AOJu0YyjuR8y7rAZojYDeDY1CU6rZ7o6xJOSKlrgZEkZsFFTbRXHzaKJ bBlJU4p99WmWPT00oLXoyqxv+l14+2BUFBy5E98LI9ecJfVs5ABqk9/d X-Gm-Gg: AZuq6aK2OJsBLP9xFhXwoiXBJBSBYoMhQ9ZUVByR3waMPTHw3LcTp6nUCOgEhz6DGec jKbQkUfFWIkczG9k0A1Q+4YeWzBu7co7gRwrbJ4LhDivBdySw651873k/zfSe4HbxzaTTtWZfgs IUSqJkm/yKWLbZ+Ea9PtyEvyNiooMoDPpWpOAg1LaxAe/t5q7d5oaKqBLSdkpjzrV4GRkkWSC/r A70gBBBjZs0BKYAldmj9nmHZXy1oSGIiYwAn9pErRr5M0SGMOELvdOTkn4Z4GbEoqcn/KCFvgwY zqLCrjQYMw65vJpAVlPjuvsqTnLWOZ2OG/qwKCaXQi6HPqVvG55qIvIKzbyrgYeWmTAXl8gGaeP RrivOe6w/uK9vBeTUNTuHclc9ToKEOPRDVKJMLcyf+UpvdSd8xzWDB010dFTaHFqruapTAudNy9 wWOHdVD4KxZwLM X-Received: by 2002:a17:90b:3d84:b0:34c:27ad:71e7 with SMTP id 98e67ed59e1d1-354871f826bmr1665256a91.35.1770187288369; Tue, 03 Feb 2026 22:41:28 -0800 (PST) Received: from mint.. ([2401:4900:53f5:9794:567c:c0b1:37aa:12a9]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-35485edfaa0sm1434650a91.13.2026.02.03.22.41.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 03 Feb 2026 22:41:27 -0800 (PST) From: dhyaan19022009-hue X-Google-Original-From: dhyaan19022009-hue To: Sudip Mukherjee , Greg Kroah-Hartman Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, dhyaan19022009-hue Subject: [PATCH] staging: sm750fb: fix naming styles and const declarations Date: Wed, 4 Feb 2026 12:11:12 +0530 Message-ID: <20260204064113.21801-1-dhyaan19022009@gmail.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Signed-off-by: dhyaan19022009-hue --- drivers/staging/sm750fb/sm750.c | 26 +++++++++++++------------- drivers/staging/sm750fb/sm750.h | 14 +++++++------- drivers/staging/sm750fb/sm750_hw.c | 22 +++++++++++----------- 3 files changed, 31 insertions(+), 31 deletions(-) diff --git a/drivers/staging/sm750fb/sm750.c b/drivers/staging/sm750fb/sm75= 0.c index 9bf3c0a37..13d513bff 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 * const g_fbmode[] =3D {NULL, NULL}; static const char *g_def_fbmode =3D "1024x768-32@60"; static char *g_settings; static int g_dualview; @@ -740,7 +740,7 @@ static int lynxfb_set_fbinfo(struct fb_info *info, int = index) "kernel HELPERS prepared vesa_modes", }; =20 - static const char *fixId[2] =3D { + static const char * const fix_id[2] =3D { "sm750_fb1", "sm750_fb2", }; =20 @@ -764,7 +764,7 @@ static int lynxfb_set_fbinfo(struct fb_info *info, int = 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->pv_reg + 0x800f0 + (int)crtc->channel * 0x140; =20 pr_info("crtc->cursor.mmio =3D %p\n", crtc->cursor.mmio); @@ -862,7 +862,7 @@ static int lynxfb_set_fbinfo(struct fb_info *info, int = index) fix->ywrapstep =3D crtc->ywrapstep; fix->accel =3D FB_ACCEL_SMI; =20 - strscpy(fix->id, fixId[index], sizeof(fix->id)); + strscpy(fix->id, fix_id[index], sizeof(fix->id)); =20 fix->smem_start =3D crtc->o_screen + sm750_dev->vidmem_start; pr_info("fix->smem_start =3D %lx\n", fix->smem_start); @@ -918,12 +918,12 @@ static void sm750fb_setup(struct sm750_dev *sm750_dev= , char *src) =20 swap =3D 0; =20 - sm750_dev->initParm.chip_clk =3D 0; - sm750_dev->initParm.mem_clk =3D 0; - sm750_dev->initParm.master_clk =3D 0; - sm750_dev->initParm.powerMode =3D 0; - sm750_dev->initParm.setAllEngOff =3D 0; - sm750_dev->initParm.resetMemory =3D 1; + 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.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; @@ -942,9 +942,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"))) { @@ -1121,7 +1121,7 @@ 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->pv_reg); iounmap(sm750_dev->pv_mem); kfree(g_settings); } diff --git a/drivers/staging/sm750fb/sm750.h b/drivers/staging/sm750fb/sm75= 0.h index 2dc57bbe7..19dbb9120 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 { @@ -97,12 +97,12 @@ struct sm750_dev { unsigned long vidreg_start; __u32 vidmem_size; __u32 vidreg_size; - void __iomem *pvReg; + void __iomem *pv_reg; unsigned char __iomem *pv_mem; /* locks*/ spinlock_t slock; =20 - struct init_status initParm; + struct init_status init_parm; enum sm750_pnltype pnltype; enum sm750_dataflow dataflow; int nocrt; diff --git a/drivers/staging/sm750fb/sm750_hw.c b/drivers/staging/sm750fb/s= m750_hw.c index de1331c56..9e69f3387 100644 --- a/drivers/staging/sm750fb/sm750_hw.c +++ b/drivers/staging/sm750fb/sm750_hw.c @@ -49,19 +49,19 @@ 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->pv_reg =3D ioremap(sm750_dev->vidreg_start, sm750_dev->vidreg_size); - if (!sm750_dev->pvReg) { + if (!sm750_dev->pv_reg) { pr_err("mmio failed\n"); ret =3D -EFAULT; goto exit; } - pr_info("mmio virtual addr =3D %p\n", sm750_dev->pvReg); + pr_info("mmio virtual addr =3D %p\n", sm750_dev->pv_reg); =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->pv_reg + DE_BASE_ADDR_TYPE1; + sm750_dev->accel.dp_port_base =3D sm750_dev->pv_reg + DE_PORT_ADDR_TYPE1; =20 - mmio750 =3D sm750_dev->pvReg; + mmio750 =3D sm750_dev->pv_reg; sm750_set_chip_type(sm750_dev->devid, sm750_dev->revid); =20 sm750_dev->vidmem_start =3D pci_resource_start(pdev, 0); @@ -79,7 +79,7 @@ int hw_sm750_map(struct sm750_dev *sm750_dev, struct pci_= dev *pdev) sm750_dev->pv_mem =3D ioremap_wc(sm750_dev->vidmem_start, sm750_dev->vidmem_size); if (!sm750_dev->pv_mem) { - iounmap(sm750_dev->pvReg); + iounmap(sm750_dev->pv_reg); pr_err("Map video memory failed\n"); ret =3D -EFAULT; goto exit; @@ -93,7 +93,7 @@ int hw_sm750_inithw(struct sm750_dev *sm750_dev, struct p= ci_dev *pdev) { struct init_status *parm; =20 - parm =3D &sm750_dev->initParm; + parm =3D &sm750_dev->init_parm; if (parm->chip_clk =3D=3D 0) parm->chip_clk =3D (sm750_get_chip_type() =3D=3D SM750LE) ? DEFAULT_SM750LE_CHIP_CLOCK : @@ -104,7 +104,7 @@ int hw_sm750_inithw(struct sm750_dev *sm750_dev, struct= pci_dev *pdev) if (parm->master_clk =3D=3D 0) parm->master_clk =3D parm->chip_clk / 3; =20 - ddk750_init_hw((struct initchip_param *)&sm750_dev->initParm); + ddk750_init_hw((struct initchip_param *)&sm750_dev->init_parm); /* for sm718, open pci burst */ if (sm750_dev->devid =3D=3D 0x718) { poke32(SYSTEM_CTRL, @@ -136,10 +136,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; } --=20 2.43.0