From nobody Tue Dec 2 02:18:55 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 616D2337115 for ; Wed, 19 Nov 2025 09:19:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763543995; cv=none; b=lADemaT99yjbvkv1OgTWkA490XO60h+nf3xVxPJ+WZ67UqVvQM1K8xwOELdsl+MCXHr+zB4DmWytUyKzqrf51G1gbgwQI86MDEMr2pqWLOh/sK/0Y9zOT/Gdfvq4vZ56u04AHANTuN0eXyy3ZFMVFS7K8UkArfuG5bvkKhbLevY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763543995; c=relaxed/simple; bh=+twEGSWVkU24dJT6HixJJ1t7rYB4ynDjp6p5U/hZqH4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=UEnCojxm0ed0XnH7R96uhQW0NwI8BCvA+iUWIbCopmBVp+OfSzw9Hy2YeIBnzCaFzL49W/RnQ8ATieAU61oQDr0SgCQeSm5iQrywZndRpVHm7tA0QlssFaerZDPxxGDCjhtpCx8FA99gbEm2D5WypjRfchsdkeFcSiYydgXbRPI= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=U0KNHSTu; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="U0KNHSTu" Received: by smtp.kernel.org (Postfix) with ESMTPSA id AF17DC2BCB7; Wed, 19 Nov 2025 09:19:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1763543995; bh=+twEGSWVkU24dJT6HixJJ1t7rYB4ynDjp6p5U/hZqH4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=U0KNHSTuMIHApax5my+qC6fTC2BId+urIca4CKMs2dSw/xe2/sbUm0Yg65gkJElUM 3nj8Yytvndf8MuzgrbFUMzzBREZZFW1CGGRKA6F+2TWuDpfotq1na2PqVWitjvP/wr wyCwv6EV7HsA9fG6oD18dRV/+P3NzG1oJf2Q+pnWsEiQ1wjT8vEc9SstrwsmEiHBB9 6MZ21h+pZ8ZP3WhykkBoxn4ruxijaIL3921bu3owuDJtT28KmAMW5VxrIfqML04p4c 1RAO2xNmJTuso75dEpdhi0FYHuJNdgLzakq4FeeFP3FinOnQ5GL1RYXyi3Qh0QvWl3 SEujPuIYjp+Rg== From: "Jiri Slaby (SUSE)" To: gregkh@linuxfoundation.org Cc: arnd@arndb.de, linux-kernel@vger.kernel.org, "Jiri Slaby (SUSE)" Subject: [PATCH 1/6] char/mwave: remove dead code Date: Wed, 19 Nov 2025 10:19:44 +0100 Message-ID: <20251119091949.825958-2-jirislaby@kernel.org> X-Mailer: git-send-email 2.51.1 In-Reply-To: <20251119091949.825958-1-jirislaby@kernel.org> References: <20251119091949.825958-1-jirislaby@kernel.org> 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" In mwave, there is a lot of commented code for a long time. Drop it. Signed-off-by: Jiri Slaby (SUSE) Acked-by: Arnd Bergmann --- drivers/char/mwave/3780i.c | 29 ---------------- drivers/char/mwave/mwavedd.c | 67 ------------------------------------ drivers/char/mwave/smapi.c | 22 ------------ 3 files changed, 118 deletions(-) diff --git a/drivers/char/mwave/3780i.c b/drivers/char/mwave/3780i.c index 4a8937f80570..321dbd03d007 100644 --- a/drivers/char/mwave/3780i.c +++ b/drivers/char/mwave/3780i.c @@ -140,35 +140,6 @@ static void dsp3780I_WriteGenCfg(unsigned short usDspB= aseIO, unsigned uIndex, =20 } =20 -#if 0 -unsigned char dsp3780I_ReadGenCfg(unsigned short usDspBaseIO, - unsigned uIndex) -{ - DSP_ISA_SLAVE_CONTROL rSlaveControl; - DSP_ISA_SLAVE_CONTROL rSlaveControl_Save; - unsigned char ucValue; - - - PRINTK_3(TRACE_3780I, - "3780i::dsp3780i_ReadGenCfg entry usDspBaseIO %x uIndex %x\n", - usDspBaseIO, uIndex); - - MKBYTE(rSlaveControl) =3D InByteDsp(DSP_IsaSlaveControl); - rSlaveControl_Save =3D rSlaveControl; - rSlaveControl.ConfigMode =3D true; - OutByteDsp(DSP_IsaSlaveControl, MKBYTE(rSlaveControl)); - OutByteDsp(DSP_ConfigAddress, (unsigned char) uIndex); - ucValue =3D InByteDsp(DSP_ConfigData); - OutByteDsp(DSP_IsaSlaveControl, MKBYTE(rSlaveControl_Save)); - - PRINTK_2(TRACE_3780I, - "3780i::dsp3780i_ReadGenCfg exit ucValue %x\n", ucValue); - - - return ucValue; -} -#endif /* 0 */ - int dsp3780I_EnableDSP(DSP_3780I_CONFIG_SETTINGS * pSettings, unsigned short *pIrqMap, unsigned short *pDmaMap) diff --git a/drivers/char/mwave/mwavedd.c b/drivers/char/mwave/mwavedd.c index 11272d605ecd..86e33c28beac 100644 --- a/drivers/char/mwave/mwavedd.c +++ b/drivers/char/mwave/mwavedd.c @@ -492,42 +492,6 @@ static const struct file_operations mwave_fops =3D { =20 static struct miscdevice mwave_misc_dev =3D { MWAVE_MINOR, "mwave", &mwave= _fops }; =20 -#if 0 /* totally b0rked */ -/* - * sysfs support - */ - -struct device mwave_device; - -/* Prevent code redundancy, create a macro for mwave_show_* functions. */ -#define mwave_show_function(attr_name, format_string, field) \ -static ssize_t mwave_show_##attr_name(struct device *dev, struct device_at= tribute *attr, char *buf) \ -{ \ - DSP_3780I_CONFIG_SETTINGS *pSettings =3D \ - &mwave_s_mdd.rBDData.rDspSettings; \ - return sprintf(buf, format_string, pSettings->field); \ -} - -/* All of our attributes are read attributes. */ -#define mwave_dev_rd_attr(attr_name, format_string, field) \ - mwave_show_function(attr_name, format_string, field) \ -static DEVICE_ATTR(attr_name, S_IRUGO, mwave_show_##attr_name, NULL) - -mwave_dev_rd_attr (3780i_dma, "%i\n", usDspDma); -mwave_dev_rd_attr (3780i_irq, "%i\n", usDspIrq); -mwave_dev_rd_attr (3780i_io, "%#.4x\n", usDspBaseIO); -mwave_dev_rd_attr (uart_irq, "%i\n", usUartIrq); -mwave_dev_rd_attr (uart_io, "%#.4x\n", usUartBaseIO); - -static struct device_attribute * const mwave_dev_attrs[] =3D { - &dev_attr_3780i_dma, - &dev_attr_3780i_irq, - &dev_attr_3780i_io, - &dev_attr_uart_irq, - &dev_attr_uart_io, -}; -#endif - /* * mwave_init is called on module load * @@ -540,17 +504,6 @@ static void mwave_exit(void) =20 PRINTK_1(TRACE_MWAVE, "mwavedd::mwave_exit entry\n"); =20 -#if 0 - for (i =3D 0; i < pDrvData->nr_registered_attrs; i++) - device_remove_file(&mwave_device, mwave_dev_attrs[i]); - pDrvData->nr_registered_attrs =3D 0; - - if (pDrvData->device_registered) { - device_unregister(&mwave_device); - pDrvData->device_registered =3D false; - } -#endif - if ( pDrvData->sLine >=3D 0 ) { serial8250_unregister_port(pDrvData->sLine); } @@ -667,26 +620,6 @@ static int __init mwave_init(void) } /* uart is registered */ =20 -#if 0 - /* sysfs */ - memset(&mwave_device, 0, sizeof (struct device)); - dev_set_name(&mwave_device, "mwave"); - - if (device_register(&mwave_device)) - goto cleanup_error; - pDrvData->device_registered =3D true; - for (i =3D 0; i < ARRAY_SIZE(mwave_dev_attrs); i++) { - if(device_create_file(&mwave_device, mwave_dev_attrs[i])) { - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd:mwave_init: Error:" - " Failed to create sysfs file %s\n", - mwave_dev_attrs[i]->attr.name); - goto cleanup_error; - } - pDrvData->nr_registered_attrs++; - } -#endif - /* SUCCESS! */ return 0; =20 diff --git a/drivers/char/mwave/smapi.c b/drivers/char/mwave/smapi.c index f8d79d393b69..f586752ec463 100644 --- a/drivers/char/mwave/smapi.c +++ b/drivers/char/mwave/smapi.c @@ -513,28 +513,6 @@ int smapi_set_DSP_power_state(bool bOn) return bRC; } =20 -#if 0 -static int SmapiQuerySystemID(void) -{ - int bRC =3D -EIO; - unsigned short usAX =3D 0xffff, usBX =3D 0xffff, usCX =3D 0xffff, - usDX =3D 0xffff, usDI =3D 0xffff, usSI =3D 0xffff; - - printk("smapi::SmapiQUerySystemID entry\n"); - bRC =3D smapi_request(0x0000, 0, 0, 0, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - - if (bRC =3D=3D 0) { - printk("AX=3D%x, BX=3D%x, CX=3D%x, DX=3D%x, DI=3D%x, SI=3D%x\n", - usAX, usBX, usCX, usDX, usDI, usSI); - } else { - printk("smapi::SmapiQuerySystemID smapi_request error\n"); - } - - return bRC; -} -#endif /* 0 */ - int smapi_init(void) { int retval =3D -EIO; --=20 2.51.1 From nobody Tue Dec 2 02:18:55 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 DD20533C1AB for ; Wed, 19 Nov 2025 09:19:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763543997; cv=none; b=U5UvxraHcyF6gNT0hLLgS287L5qKUuKVrBUZOuiNZ/kOFQppRyeAy+djNDCmL9xsJ+7O0JqvnyJ/yBGtgLpZW6UyyhIru5EjAzPDqiiqOXP5UjornnB63QSfTokS3OVgywG7n5DvZPs3MERRoLjmlcvzcXWiXakewR9KK9rO2qo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763543997; c=relaxed/simple; bh=cpODIUIWg1AgyCmRKkLcGbkD30nqerqSzEX7YlYnpAQ=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Qyp1zHIcDRuMTL2QiIFpiRpbHqnGkrfEJKvl8Jfl8yKS9+46q950KTE+W+URjJQkgXu0Yx2UJ86iYCllmJz5U031hM3dViBgFi+RFk/iIQVOwDFaKfRGkSGC+jepJwp3Gj/BvY51SegkAng652WJ55IkR+9YVTQi0MMMEbMIQgY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=UO5aRuge; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="UO5aRuge" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 88198C19422; Wed, 19 Nov 2025 09:19:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1763543996; bh=cpODIUIWg1AgyCmRKkLcGbkD30nqerqSzEX7YlYnpAQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=UO5aRugefN4AzoBE4dMaxX+LMdaJyYGKh5IyTd7PO311obykDJE3sRc3riJv+QTOl G2U2Ff1k3N5kz6PuATGrY617POOIE0TXHv8EPS1P8tT6t+fUnOEIskH+7AETUPD7vj 05mjz9JLCJP+iRkIDkCWaYqIQ1vO66cad7hU/S5pn9eWS5uX2F9oVQilv+RauhrfF1 4EhaCMcqLeoo5riax59I2v8ou0uCd5ndbCkwALfTQIu4yieBaHbcgtqlbIfFcWJaV+ +zdL7l+QrFVNzE1LQQgG7OY1XfhsVoEoufHbI3kbAUFQbr+me1OSkcXVxu7DYSfbvS tvKXjWihwATAA== From: "Jiri Slaby (SUSE)" To: gregkh@linuxfoundation.org Cc: arnd@arndb.de, linux-kernel@vger.kernel.org, "Jiri Slaby (SUSE)" Subject: [PATCH 2/6] char/mwave: remove MWAVE_FUTZ_WITH_OTHER_DEVICES ifdeffery Date: Wed, 19 Nov 2025 10:19:45 +0100 Message-ID: <20251119091949.825958-3-jirislaby@kernel.org> X-Mailer: git-send-email 2.51.1 In-Reply-To: <20251119091949.825958-1-jirislaby@kernel.org> References: <20251119091949.825958-1-jirislaby@kernel.org> 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" In mwave, a lot of code depends on the MWAVE_FUTZ_WITH_OTHER_DEVICES macro. That can be defined in Makefile to compile this in. 1) The code is completely unreadable. 2) Recompiling the kernel to have this untested code compiled in is not a good idea. Drop all this. Signed-off-by: Jiri Slaby (SUSE) Acked-by: Arnd Bergmann --- drivers/char/mwave/Makefile | 3 - drivers/char/mwave/smapi.c | 108 ------------------------------------ 2 files changed, 111 deletions(-) diff --git a/drivers/char/mwave/Makefile b/drivers/char/mwave/Makefile index a24fe96e3c96..836bfa25c541 100644 --- a/drivers/char/mwave/Makefile +++ b/drivers/char/mwave/Makefile @@ -9,8 +9,5 @@ obj-$(CONFIG_MWAVE) +=3D mwave.o =20 mwave-y :=3D mwavedd.o smapi.o tp3780i.o 3780i.o =20 -# To have the mwave driver disable other uarts if necessary -# ccflags-y :=3D -DMWAVE_FUTZ_WITH_OTHER_DEVICES - # To compile in lots (~20 KiB) of run-time enablable printk()s for debuggi= ng: ccflags-y +=3D -DMW_TRACE diff --git a/drivers/char/mwave/smapi.c b/drivers/char/mwave/smapi.c index f586752ec463..5e2fe3235714 100644 --- a/drivers/char/mwave/smapi.c +++ b/drivers/char/mwave/smapi.c @@ -279,46 +279,14 @@ int smapi_set_DSP_cfg(void) if (usBX & 0x0100) { /* serial port A is present */ if (usCX & 1) { /* serial port is enabled */ if ((usSI & 0xFF) =3D=3D mwave_uart_irq) { -#ifndef MWAVE_FUTZ_WITH_OTHER_DEVICES PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_set_DSP_cfg: Serial port A irq %x conflicts with mwave= _uart_irq %x\n", usSI & 0xFF, mwave_uart_irq); -#else - PRINTK_3(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg: Serial port A irq %x conflicts with mwave= _uart_irq %x\n", usSI & 0xFF, mwave_uart_irq); -#endif -#ifdef MWAVE_FUTZ_WITH_OTHER_DEVICES - PRINTK_1(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg Disabling conflicting serial port\n"); - bRC =3D smapi_request(0x1403, 0x0100, 0, usSI, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; - bRC =3D smapi_request(0x1402, 0x0000, 0, 0, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; -#else goto exit_conflict; -#endif } else { if ((usSI >> 8) =3D=3D uartio_index) { -#ifndef MWAVE_FUTZ_WITH_OTHER_DEVICES PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_set_DSP_cfg: Serial port A base I/O address %x confli= cts with mwave uart I/O %x\n", ausUartBases[usSI >> 8], ausUartBases[uartio= _index]); -#else - PRINTK_3(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg: Serial port A base I/O address %x confli= cts with mwave uart I/O %x\n", ausUartBases[usSI >> 8], ausUartBases[uartio= _index]); -#endif -#ifdef MWAVE_FUTZ_WITH_OTHER_DEVICES - PRINTK_1(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg Disabling conflicting serial port A\n"); - bRC =3D smapi_request (0x1403, 0x0100, 0, usSI, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; - bRC =3D smapi_request (0x1402, 0x0000, 0, 0, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; -#else goto exit_conflict; -#endif } } } @@ -332,46 +300,14 @@ int smapi_set_DSP_cfg(void) if (usBX & 0x0100) { /* serial port B is present */ if (usCX & 1) { /* serial port is enabled */ if ((usSI & 0xFF) =3D=3D mwave_uart_irq) { -#ifndef MWAVE_FUTZ_WITH_OTHER_DEVICES PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_set_DSP_cfg: Serial port B irq %x conflicts with mwave= _uart_irq %x\n", usSI & 0xFF, mwave_uart_irq); -#else - PRINTK_3(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg: Serial port B irq %x conflicts with mwave= _uart_irq %x\n", usSI & 0xFF, mwave_uart_irq); -#endif -#ifdef MWAVE_FUTZ_WITH_OTHER_DEVICES - PRINTK_1(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg Disabling conflicting serial port B\n"); - bRC =3D smapi_request(0x1405, 0x0100, 0, usSI, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; - bRC =3D smapi_request(0x1404, 0x0000, 0, 0, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; -#else goto exit_conflict; -#endif } else { if ((usSI >> 8) =3D=3D uartio_index) { -#ifndef MWAVE_FUTZ_WITH_OTHER_DEVICES PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_set_DSP_cfg: Serial port B base I/O address %x confli= cts with mwave uart I/O %x\n", ausUartBases[usSI >> 8], ausUartBases[uartio= _index]); -#else - PRINTK_3(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg: Serial port B base I/O address %x confli= cts with mwave uart I/O %x\n", ausUartBases[usSI >> 8], ausUartBases[uartio= _index]); -#endif -#ifdef MWAVE_FUTZ_WITH_OTHER_DEVICES - PRINTK_1 (TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg Disabling conflicting serial port B\n"= ); - bRC =3D smapi_request (0x1405, 0x0100, 0, usSI, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; - bRC =3D smapi_request (0x1404, 0x0000, 0, 0, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; -#else goto exit_conflict; -#endif } } } @@ -387,58 +323,14 @@ int smapi_set_DSP_cfg(void) /* bRC =3D=3D 0 */ if ((usCX & 0xff) !=3D 0xff) { /* IR port not disabled */ if ((usCX & 0xff) =3D=3D mwave_uart_irq) { -#ifndef MWAVE_FUTZ_WITH_OTHER_DEVICES PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_set_DSP_cfg: IR port irq %x conflicts with mwave_uart_i= rq %x\n", usCX & 0xff, mwave_uart_irq); -#else - PRINTK_3(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg: IR port irq %x conflicts with mwave_uart_i= rq %x\n", usCX & 0xff, mwave_uart_irq); -#endif -#ifdef MWAVE_FUTZ_WITH_OTHER_DEVICES - PRINTK_1(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg Disabling conflicting IR port\n"); - bRC =3D smapi_request(0x1701, 0x0100, 0, 0, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; - bRC =3D smapi_request(0x1700, 0, 0, 0, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; - bRC =3D smapi_request(0x1705, 0x01ff, 0, usSI, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; - bRC =3D smapi_request(0x1704, 0x0000, 0, 0, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; -#else goto exit_conflict; -#endif } else { if ((usSI & 0xff) =3D=3D uartio_index) { -#ifndef MWAVE_FUTZ_WITH_OTHER_DEVICES PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_set_DSP_cfg: IR port base I/O address %x conflicts wit= h mwave uart I/O %x\n", ausUartBases[usSI & 0xff], ausUartBases[uartio_inde= x]); -#else - PRINTK_3(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg: IR port base I/O address %x conflicts wit= h mwave uart I/O %x\n", ausUartBases[usSI & 0xff], ausUartBases[uartio_inde= x]); -#endif -#ifdef MWAVE_FUTZ_WITH_OTHER_DEVICES - PRINTK_1(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg Disabling conflicting IR port\n"); - bRC =3D smapi_request(0x1701, 0x0100, 0, 0, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; - bRC =3D smapi_request(0x1700, 0, 0, 0, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; - bRC =3D smapi_request(0x1705, 0x01ff, 0, usSI, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; - bRC =3D smapi_request(0x1704, 0x0000, 0, 0, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - if (bRC) goto exit_smapi_request_error; -#else goto exit_conflict; -#endif } } } --=20 2.51.1 From nobody Tue Dec 2 02:18:55 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 8970B33C529 for ; Wed, 19 Nov 2025 09:19:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763543999; cv=none; b=iVto3HiUwzWH+g+lV5yXUcPVLrV0I5gXS74LpR4sYTf1Y9ossyRLEoe50BhiLw87gltW3E/ryxdLb2mrzFd8I62zCGPFV9rngbb6yu03l4wAWaidbS3/G3wxw6reUbsC+JYgXOzdZphA8HmTu8h0SRitOKJuiwDOR2f/zpQK2JY= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763543999; c=relaxed/simple; bh=4Xmf8Cb7RjdD1kwGJIipCL2/6UgIAC6pEb6CYdI7LB8=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=QLeu/WkpmNsuNN+xrV5ioxFzuboRo7ZL1mxopTgzeUSFlLr0AqAquZCGIRTybuD+ifwfxTzFTlWQqHHCtECBJ1CblMY/2FMgFnYBKdC1SM2epUZWSBPe5h110RZnpESbZXUTDyUGPTkuuzcVjfqp33slbA9F5cRytopf4CTAx40= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=TZ2GLMyD; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="TZ2GLMyD" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7B6ABC4AF0C; Wed, 19 Nov 2025 09:19:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1763543998; bh=4Xmf8Cb7RjdD1kwGJIipCL2/6UgIAC6pEb6CYdI7LB8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=TZ2GLMyDvcrM0QpJ03DhOsncuiJob1ZHVxd5HHDze0Be2wM/QKIoTjsEIRsrUA7yQ ep2eoDLbZqrD6S2QNinm6bkCXiTlZ87pDPdQXNRNy3AxJlketyNRxen4e5J9TWL+7W pfKKKSWPhDlSYPVeljIvsR9i6xmRVVuW1lZYevmdEG91XWovOvEKe1TaPbbLaIHyed muHQJo7qakOHUKD70wpd8JW0Rde18FKjkV2o2UZ9vIq7rAXYquPURi2wyWYuvsVa5X Z2bRDgRJZRUHE4KCpEKKCHbIO0zBceoN+mc62ct2ZXnyH5vCtgntHWNoD9NOOQ7NRu i2m0b9OhOiOOA== From: "Jiri Slaby (SUSE)" To: gregkh@linuxfoundation.org Cc: arnd@arndb.de, linux-kernel@vger.kernel.org, "Jiri Slaby (SUSE)" Subject: [PATCH 3/6] char/mwave: remove unneeded fops Date: Wed, 19 Nov 2025 10:19:46 +0100 Message-ID: <20251119091949.825958-4-jirislaby@kernel.org> X-Mailer: git-send-email 2.51.1 In-Reply-To: <20251119091949.825958-1-jirislaby@kernel.org> References: <20251119091949.825958-1-jirislaby@kernel.org> 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" file_operations::{read/write/open/release} need not be defined. The core code return proper values already (the same as the being removed ones). So there is no need to preserve these just for tracing via printk. Signed-off-by: Jiri Slaby (SUSE) Acked-by: Arnd Bergmann --- drivers/char/mwave/mwavedd.c | 62 ------------------------------------ 1 file changed, 62 deletions(-) diff --git a/drivers/char/mwave/mwavedd.c b/drivers/char/mwave/mwavedd.c index 86e33c28beac..b6d7a8b04183 100644 --- a/drivers/char/mwave/mwavedd.c +++ b/drivers/char/mwave/mwavedd.c @@ -86,40 +86,8 @@ module_param_hw(mwave_3780i_io, int, ioport, 0); module_param_hw(mwave_uart_irq, int, irq, 0); module_param_hw(mwave_uart_io, int, ioport, 0); =20 -static int mwave_open(struct inode *inode, struct file *file); -static int mwave_close(struct inode *inode, struct file *file); -static long mwave_ioctl(struct file *filp, unsigned int iocmd, - unsigned long ioarg); - MWAVE_DEVICE_DATA mwave_s_mdd; =20 -static int mwave_open(struct inode *inode, struct file *file) -{ - unsigned int retval =3D 0; - - PRINTK_3(TRACE_MWAVE, - "mwavedd::mwave_open, entry inode %p file %p\n", - inode, file); - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_open, exit return retval %x\n", retval); - - return retval; -} - -static int mwave_close(struct inode *inode, struct file *file) -{ - unsigned int retval =3D 0; - - PRINTK_3(TRACE_MWAVE, - "mwavedd::mwave_close, entry inode %p file %p\n", - inode, file); - - PRINTK_2(TRACE_MWAVE, "mwavedd::mwave_close, exit retval %x\n", - retval); - - return retval; -} - static long mwave_ioctl(struct file *file, unsigned int iocmd, unsigned long ioarg) { @@ -410,30 +378,6 @@ static long mwave_ioctl(struct file *file, unsigned in= t iocmd, return retval; } =20 - -static ssize_t mwave_read(struct file *file, char __user *buf, size_t coun= t, - loff_t * ppos) -{ - PRINTK_5(TRACE_MWAVE, - "mwavedd::mwave_read entry file %p, buf %p, count %zx ppos %p\n", - file, buf, count, ppos); - - return -EINVAL; -} - - -static ssize_t mwave_write(struct file *file, const char __user *buf, - size_t count, loff_t * ppos) -{ - PRINTK_5(TRACE_MWAVE, - "mwavedd::mwave_write entry file %p, buf %p," - " count %zx ppos %p\n", - file, buf, count, ppos); - - return -EINVAL; -} - - static int register_serial_portandirq(unsigned int port, int irq) { struct uart_8250_port uart; @@ -478,18 +422,12 @@ static int register_serial_portandirq(unsigned int po= rt, int irq) return serial8250_register_8250_port(&uart); } =20 - static const struct file_operations mwave_fops =3D { .owner =3D THIS_MODULE, - .read =3D mwave_read, - .write =3D mwave_write, .unlocked_ioctl =3D mwave_ioctl, - .open =3D mwave_open, - .release =3D mwave_close, .llseek =3D default_llseek, }; =20 - static struct miscdevice mwave_misc_dev =3D { MWAVE_MINOR, "mwave", &mwave= _fops }; =20 /* --=20 2.51.1 From nobody Tue Dec 2 02:18:55 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 9B39533D6E3 for ; Wed, 19 Nov 2025 09:20:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763544002; cv=none; b=ne+tTJFPce2XzJhP/m4sTk2FONv0dE0ct4p3U4R8O/w/B3hQCNjb6GiX9Fy7rVUBp1NBBJLZ8cphEvPwHJbxj3SseEg1jeIN8gGV6HPzeCA2hiGpYbpVpWEv4VDF25CqWb3YcYDYk+flcHeTihlfyM89TL9tjI8au0/JNYR7Mhw= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763544002; c=relaxed/simple; bh=PAlYIkEMdmlluPcivpK3LI1g+3MhXpd/wmyjZDUBV44=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=YBR16M0cYSMn0H8l3ukuvGKo/groKLBBTBpKIa3loJ/dv54pP5NqyQ70P4G7gD/3jw3txMF3h+c7uFr3viiT5ncKRErs2vO8gA1FLrh/OqqZHxb7fqlRLPTEWJNpPEE995Ng9HXuBZSr+P49EiKE6VuKz1xqBftgwIG/VLdZN+Q= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=AIlIEBlF; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="AIlIEBlF" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 667B8C19423; Wed, 19 Nov 2025 09:19:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1763544001; bh=PAlYIkEMdmlluPcivpK3LI1g+3MhXpd/wmyjZDUBV44=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=AIlIEBlF90llAHrMZ+TqLPQUJ0umGCfWxh3LIIU4AD2BRSuv8kgUBQ4WiGJ8Fre6W Z15jOhomltEdwULsxYehM+ulnBDGEm9++scw0MSkx+ivHBOcRVDVIfpkKr1oFaUMC1 Mgg8gLbBm3EBnVvioYP8WXl9ce8HKtgTMmvkEpI6g9yzIYssBXnsGp7g3d5mclzMEZ NjGd6K+I+DzZ6s5sZrnwSbnyAWm+KGXGVqN9osnfREvHY8mspdcafvs+6MEgkBy/k+ FZ1VW6Z9LXBY25dD4F7H8r14zmGWMjaq4f6PaaoIxSbPx66ZcSXWIuCEX4gAdEHxcl syIViPitPUAog== From: "Jiri Slaby (SUSE)" To: gregkh@linuxfoundation.org Cc: arnd@arndb.de, linux-kernel@vger.kernel.org, "Jiri Slaby (SUSE)" Subject: [PATCH 4/6] char/mwave: remove printk tracing Date: Wed, 19 Nov 2025 10:19:47 +0100 Message-ID: <20251119091949.825958-5-jirislaby@kernel.org> X-Mailer: git-send-email 2.51.1 In-Reply-To: <20251119091949.825958-1-jirislaby@kernel.org> References: <20251119091949.825958-1-jirislaby@kernel.org> 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" The printk tracing makes the code hard to follow for no good benefit. Everyone can use dynamic tracing and/or kprobes. Drop this unreadable bloatware too. Signed-off-by: Jiri Slaby (SUSE) Acked-by: Arnd Bergmann --- drivers/char/mwave/3780i.c | 177 +---------------------------------- drivers/char/mwave/Makefile | 3 - drivers/char/mwave/README | 10 -- drivers/char/mwave/mwavedd.c | 116 +---------------------- drivers/char/mwave/mwavedd.h | 59 ------------ drivers/char/mwave/smapi.c | 53 +---------- drivers/char/mwave/tp3780i.c | 120 ++---------------------- 7 files changed, 14 insertions(+), 524 deletions(-) diff --git a/drivers/char/mwave/3780i.c b/drivers/char/mwave/3780i.c index 321dbd03d007..6024cf7d5705 100644 --- a/drivers/char/mwave/3780i.c +++ b/drivers/char/mwave/3780i.c @@ -75,18 +75,12 @@ unsigned short dsp3780I_ReadMsaCfg(unsigned short usDsp= BaseIO, unsigned long flags; unsigned short val; =20 - PRINTK_3(TRACE_3780I, - "3780i::dsp3780I_ReadMsaCfg entry usDspBaseIO %x ulMsaAddr %lx\n", - usDspBaseIO, ulMsaAddr); - spin_lock_irqsave(&dsp_lock, flags); OutWordDsp(DSP_MsaAddrLow, (unsigned short) ulMsaAddr); OutWordDsp(DSP_MsaAddrHigh, (unsigned short) (ulMsaAddr >> 16)); val =3D InWordDsp(DSP_MsaDataDSISHigh); spin_unlock_irqrestore(&dsp_lock, flags); =20 - PRINTK_2(TRACE_3780I, "3780i::dsp3780I_ReadMsaCfg exit val %x\n", val); - return val; } =20 @@ -95,10 +89,6 @@ void dsp3780I_WriteMsaCfg(unsigned short usDspBaseIO, { unsigned long flags; =20 - PRINTK_4(TRACE_3780I, - "3780i::dsp3780i_WriteMsaCfg entry usDspBaseIO %x ulMsaAddr %lx usValue = %x\n", - usDspBaseIO, ulMsaAddr, usValue); - spin_lock_irqsave(&dsp_lock, flags); OutWordDsp(DSP_MsaAddrLow, (unsigned short) ulMsaAddr); OutWordDsp(DSP_MsaAddrHigh, (unsigned short) (ulMsaAddr >> 16)); @@ -112,32 +102,15 @@ static void dsp3780I_WriteGenCfg(unsigned short usDsp= BaseIO, unsigned uIndex, DSP_ISA_SLAVE_CONTROL rSlaveControl; DSP_ISA_SLAVE_CONTROL rSlaveControl_Save; =20 - - PRINTK_4(TRACE_3780I, - "3780i::dsp3780i_WriteGenCfg entry usDspBaseIO %x uIndex %x ucValue %x\n= ", - usDspBaseIO, uIndex, ucValue); - MKBYTE(rSlaveControl) =3D InByteDsp(DSP_IsaSlaveControl); =20 - PRINTK_2(TRACE_3780I, - "3780i::dsp3780i_WriteGenCfg rSlaveControl %x\n", - MKBYTE(rSlaveControl)); - rSlaveControl_Save =3D rSlaveControl; rSlaveControl.ConfigMode =3D true; =20 - PRINTK_2(TRACE_3780I, - "3780i::dsp3780i_WriteGenCfg entry rSlaveControl+ConfigMode %x\n", - MKBYTE(rSlaveControl)); - OutByteDsp(DSP_IsaSlaveControl, MKBYTE(rSlaveControl)); OutByteDsp(DSP_ConfigAddress, (unsigned char) uIndex); OutByteDsp(DSP_ConfigData, ucValue); OutByteDsp(DSP_IsaSlaveControl, MKBYTE(rSlaveControl_Save)); - - PRINTK_1(TRACE_3780I, "3780i::dsp3780i_WriteGenCfg exit\n"); - - } =20 int dsp3780I_EnableDSP(DSP_3780I_CONFIG_SETTINGS * pSettings, @@ -162,25 +135,13 @@ int dsp3780I_EnableDSP(DSP_3780I_CONFIG_SETTINGS * pS= ettings, DSP_CLOCK_CONTROL_2 rClockControl2; DSP_ISA_SLAVE_CONTROL rSlaveControl; DSP_HBRIDGE_CONTROL rHBridgeControl; - unsigned short ChipID =3D 0; unsigned short tval; =20 - - PRINTK_2(TRACE_3780I, - "3780i::dsp3780I_EnableDSP entry pSettings->bDSPEnabled %x\n", - pSettings->bDSPEnabled); - - if (!pSettings->bDSPEnabled) { PRINTK_ERROR( KERN_ERR "3780i::dsp3780I_EnableDSP: Error: DSP not enable= d. Aborting.\n" ); return -EIO; } =20 - - PRINTK_2(TRACE_3780I, - "3780i::dsp3780i_EnableDSP entry pSettings->bModemEnabled %x\n", - pSettings->bModemEnabled); - if (pSettings->bModemEnabled) { rUartCfg1.Reserved =3D rUartCfg2.Reserved =3D 0; rUartCfg1.IrqActiveLow =3D pSettings->bUartIrqActiveLow; @@ -253,23 +214,10 @@ int dsp3780I_EnableDSP(DSP_3780I_CONFIG_SETTINGS * pS= ettings, rSlaveControl.ConfigMode =3D false; rSlaveControl.Reserved =3D 0; =20 - PRINTK_4(TRACE_3780I, - "3780i::dsp3780i_EnableDSP usDspBaseIO %x index %x taddr %x\n", - usDspBaseIO, DSP_IsaSlaveControl, - usDspBaseIO + DSP_IsaSlaveControl); - - PRINTK_2(TRACE_3780I, - "3780i::dsp3780i_EnableDSP rSlaveContrl %x\n", - MKWORD(rSlaveControl)); - spin_lock_irqsave(&dsp_lock, flags); OutWordDsp(DSP_IsaSlaveControl, MKWORD(rSlaveControl)); MKWORD(tval) =3D InWordDsp(DSP_IsaSlaveControl); =20 - PRINTK_2(TRACE_3780I, - "3780i::dsp3780i_EnableDSP rSlaveControl 2 %x\n", tval); - - for (i =3D 0; i < 11; i++) udelay(2000); =20 @@ -278,10 +226,6 @@ int dsp3780I_EnableDSP(DSP_3780I_CONFIG_SETTINGS * pSe= ttings, =20 MKWORD(tval) =3D InWordDsp(DSP_IsaSlaveControl); =20 - PRINTK_2(TRACE_3780I, - "3780i::dsp3780i_EnableDSP rSlaveControl 3 %x\n", tval); - - /* Program our general configuration registers */ WriteGenCfg(DSP_HBridgeCfg1Index, MKBYTE(rHBridgeCfg1)); WriteGenCfg(DSP_HBridgeCfg2Index, MKBYTE(rHBridgeCfg2)); @@ -302,10 +246,6 @@ int dsp3780I_EnableDSP(DSP_3780I_CONFIG_SETTINGS * pSe= ttings, rHBridgeControl.IoAutoInc =3D false; rHBridgeControl.DiagnosticMode =3D false; =20 - PRINTK_3(TRACE_3780I, - "3780i::dsp3780i_EnableDSP DSP_HBridgeControl %x rHBridgeControl %x\n", - DSP_HBridgeControl, MKWORD(rHBridgeControl)); - OutWordDsp(DSP_HBridgeControl, MKWORD(rHBridgeControl)); spin_unlock_irqrestore(&dsp_lock, flags); WriteMsaCfg(DSP_LBusTimeoutDisable, MKWORD(rLBusTimeoutDisable)); @@ -313,11 +253,7 @@ int dsp3780I_EnableDSP(DSP_3780I_CONFIG_SETTINGS * pSe= ttings, WriteMsaCfg(DSP_ClockControl_2, MKWORD(rClockControl2)); WriteMsaCfg(DSP_ChipReset, MKWORD(rChipReset)); =20 - ChipID =3D ReadMsaCfg(DSP_ChipID); - - PRINTK_2(TRACE_3780I, - "3780i::dsp3780I_EnableDSP exiting bRC=3Dtrue, ChipID %x\n", - ChipID); + ReadMsaCfg(DSP_ChipID); =20 return 0; } @@ -328,9 +264,6 @@ int dsp3780I_DisableDSP(DSP_3780I_CONFIG_SETTINGS * pSe= ttings) unsigned short usDspBaseIO =3D pSettings->usDspBaseIO; DSP_ISA_SLAVE_CONTROL rSlaveControl; =20 - - PRINTK_1(TRACE_3780I, "3780i::dsp3780i_DisableDSP entry\n"); - rSlaveControl.ClockControl =3D 0; rSlaveControl.SoftReset =3D true; rSlaveControl.ConfigMode =3D false; @@ -346,9 +279,6 @@ int dsp3780I_DisableDSP(DSP_3780I_CONFIG_SETTINGS * pSe= ttings) =20 udelay(5); =20 - - PRINTK_1(TRACE_3780I, "3780i::dsp3780i_DisableDSP exit\n"); - return 0; } =20 @@ -359,16 +289,10 @@ int dsp3780I_Reset(DSP_3780I_CONFIG_SETTINGS * pSetti= ngs) DSP_BOOT_DOMAIN rBootDomain; DSP_HBRIDGE_CONTROL rHBridgeControl; =20 - - PRINTK_1(TRACE_3780I, "3780i::dsp3780i_Reset entry\n"); - spin_lock_irqsave(&dsp_lock, flags); /* Mask DSP to PC interrupt */ MKWORD(rHBridgeControl) =3D InWordDsp(DSP_HBridgeControl); =20 - PRINTK_2(TRACE_3780I, "3780i::dsp3780i_Reset rHBridgeControl %x\n", - MKWORD(rHBridgeControl)); - rHBridgeControl.EnableDspInt =3D false; OutWordDsp(DSP_HBridgeControl, MKWORD(rHBridgeControl)); spin_unlock_irqrestore(&dsp_lock, flags); @@ -379,9 +303,6 @@ int dsp3780I_Reset(DSP_3780I_CONFIG_SETTINGS * pSetting= s) rBootDomain.NMI =3D true; rBootDomain.Reserved =3D 0; =20 - PRINTK_2(TRACE_3780I, "3780i::dsp3780i_Reset rBootDomain %x\n", - MKWORD(rBootDomain)); - WriteMsaCfg(DSP_MspBootDomain, MKWORD(rBootDomain)); =20 /* Reset all the chiplets and then reactivate them */ @@ -390,9 +311,6 @@ int dsp3780I_Reset(DSP_3780I_CONFIG_SETTINGS * pSetting= s) WriteMsaCfg(DSP_ChipReset, (unsigned short) (~pSettings->usChipletEnable)); =20 - - PRINTK_1(TRACE_3780I, "3780i::dsp3780i_Reset exit bRC=3D0\n"); - return 0; } =20 @@ -404,10 +322,6 @@ int dsp3780I_Run(DSP_3780I_CONFIG_SETTINGS * pSettings) DSP_BOOT_DOMAIN rBootDomain; DSP_HBRIDGE_CONTROL rHBridgeControl; =20 - - PRINTK_1(TRACE_3780I, "3780i::dsp3780i_Run entry\n"); - - /* Transition the core to a running state */ rBootDomain.ResetCore =3D true; rBootDomain.Halt =3D false; @@ -430,15 +344,9 @@ int dsp3780I_Run(DSP_3780I_CONFIG_SETTINGS * pSettings) MKWORD(rHBridgeControl) =3D InWordDsp(DSP_HBridgeControl); rHBridgeControl.EnableDspInt =3D true; =20 - PRINTK_2(TRACE_3780I, "3780i::dsp3780i_Run rHBridgeControl %x\n", - MKWORD(rHBridgeControl)); - OutWordDsp(DSP_HBridgeControl, MKWORD(rHBridgeControl)); spin_unlock_irqrestore(&dsp_lock, flags); =20 - - PRINTK_1(TRACE_3780I, "3780i::dsp3780i_Run exit bRC=3Dtrue\n"); - return 0; } =20 @@ -450,12 +358,6 @@ int dsp3780I_ReadDStore(unsigned short usDspBaseIO, vo= id __user *pvBuffer, unsigned short __user *pusBuffer =3D pvBuffer; unsigned short val; =20 - - PRINTK_5(TRACE_3780I, - "3780i::dsp3780I_ReadDStore entry usDspBaseIO %x, pusBuffer %p, uCount %= x, ulDSPAddr %lx\n", - usDspBaseIO, pusBuffer, uCount, ulDSPAddr); - - /* Set the initial MSA address. No adjustments need to be made to data st= ore addresses */ spin_lock_irqsave(&dsp_lock, flags); OutWordDsp(DSP_MsaAddrLow, (unsigned short) ulDSPAddr); @@ -470,17 +372,9 @@ int dsp3780I_ReadDStore(unsigned short usDspBaseIO, vo= id __user *pvBuffer, if(put_user(val, pusBuffer++)) return -EFAULT; =20 - PRINTK_3(TRACE_3780I, - "3780I::dsp3780I_ReadDStore uCount %x val %x\n", - uCount, val); - PaceMsaAccess(usDspBaseIO); } =20 - - PRINTK_1(TRACE_3780I, - "3780I::dsp3780I_ReadDStore exit bRC=3Dtrue\n"); - return 0; } =20 @@ -492,12 +386,6 @@ int dsp3780I_ReadAndClearDStore(unsigned short usDspBa= seIO, unsigned short __user *pusBuffer =3D pvBuffer; unsigned short val; =20 - - PRINTK_5(TRACE_3780I, - "3780i::dsp3780I_ReadAndDStore entry usDspBaseIO %x, pusBuffer %p, uCoun= t %x, ulDSPAddr %lx\n", - usDspBaseIO, pusBuffer, uCount, ulDSPAddr); - - /* Set the initial MSA address. No adjustments need to be made to data st= ore addresses */ spin_lock_irqsave(&dsp_lock, flags); OutWordDsp(DSP_MsaAddrLow, (unsigned short) ulDSPAddr); @@ -512,17 +400,9 @@ int dsp3780I_ReadAndClearDStore(unsigned short usDspBa= seIO, if(put_user(val, pusBuffer++)) return -EFAULT; =20 - PRINTK_3(TRACE_3780I, - "3780I::dsp3780I_ReadAndCleanDStore uCount %x val %x\n", - uCount, val); - PaceMsaAccess(usDspBaseIO); } =20 - - PRINTK_1(TRACE_3780I, - "3780I::dsp3780I_ReadAndClearDStore exit bRC=3Dtrue\n"); - return 0; } =20 @@ -533,12 +413,6 @@ int dsp3780I_WriteDStore(unsigned short usDspBaseIO, v= oid __user *pvBuffer, unsigned long flags; unsigned short __user *pusBuffer =3D pvBuffer; =20 - - PRINTK_5(TRACE_3780I, - "3780i::dsp3780D_WriteDStore entry usDspBaseIO %x, pusBuffer %p, uCount = %x, ulDSPAddr %lx\n", - usDspBaseIO, pusBuffer, uCount, ulDSPAddr); - - /* Set the initial MSA address. No adjustments need to be made to data st= ore addresses */ spin_lock_irqsave(&dsp_lock, flags); OutWordDsp(DSP_MsaAddrLow, (unsigned short) ulDSPAddr); @@ -554,17 +428,9 @@ int dsp3780I_WriteDStore(unsigned short usDspBaseIO, v= oid __user *pvBuffer, OutWordDsp(DSP_MsaDataDSISHigh, val); spin_unlock_irqrestore(&dsp_lock, flags); =20 - PRINTK_3(TRACE_3780I, - "3780I::dsp3780I_WriteDStore uCount %x val %x\n", - uCount, val); - PaceMsaAccess(usDspBaseIO); } =20 - - PRINTK_1(TRACE_3780I, - "3780I::dsp3780D_WriteDStore exit bRC=3Dtrue\n"); - return 0; } =20 @@ -575,10 +441,6 @@ int dsp3780I_ReadIStore(unsigned short usDspBaseIO, vo= id __user *pvBuffer, unsigned long flags; unsigned short __user *pusBuffer =3D pvBuffer; =20 - PRINTK_5(TRACE_3780I, - "3780i::dsp3780I_ReadIStore entry usDspBaseIO %x, pusBuffer %p, uCount %= x, ulDSPAddr %lx\n", - usDspBaseIO, pusBuffer, uCount, ulDSPAddr); - /* * Set the initial MSA address. To convert from an instruction store * address to an MSA address @@ -602,17 +464,10 @@ int dsp3780I_ReadIStore(unsigned short usDspBaseIO, v= oid __user *pvBuffer, if(put_user(val_hi, pusBuffer++)) return -EFAULT; =20 - PRINTK_4(TRACE_3780I, - "3780I::dsp3780I_ReadIStore uCount %x val_lo %x val_hi %x\n", - uCount, val_lo, val_hi); - PaceMsaAccess(usDspBaseIO); =20 } =20 - PRINTK_1(TRACE_3780I, - "3780I::dsp3780I_ReadIStore exit bRC=3Dtrue\n"); - return 0; } =20 @@ -623,11 +478,6 @@ int dsp3780I_WriteIStore(unsigned short usDspBaseIO, v= oid __user *pvBuffer, unsigned long flags; unsigned short __user *pusBuffer =3D pvBuffer; =20 - PRINTK_5(TRACE_3780I, - "3780i::dsp3780I_WriteIStore entry usDspBaseIO %x, pusBuffer %p, uCount = %x, ulDSPAddr %lx\n", - usDspBaseIO, pusBuffer, uCount, ulDSPAddr); - - /* * Set the initial MSA address. To convert from an instruction store * address to an MSA address @@ -651,17 +501,9 @@ int dsp3780I_WriteIStore(unsigned short usDspBaseIO, v= oid __user *pvBuffer, OutWordDsp(DSP_MsaDataDSISHigh, val_hi); spin_unlock_irqrestore(&dsp_lock, flags); =20 - PRINTK_4(TRACE_3780I, - "3780I::dsp3780I_WriteIStore uCount %x val_lo %x val_hi %x\n", - uCount, val_lo, val_hi); - PaceMsaAccess(usDspBaseIO); - } =20 - PRINTK_1(TRACE_3780I, - "3780I::dsp3780I_WriteIStore exit bRC=3Dtrue\n"); - return 0; } =20 @@ -671,12 +513,6 @@ int dsp3780I_GetIPCSource(unsigned short usDspBaseIO, { unsigned long flags; DSP_HBRIDGE_CONTROL rHBridgeControl; - unsigned short temp; - - - PRINTK_3(TRACE_3780I, - "3780i::dsp3780I_GetIPCSource entry usDspBaseIO %x pusIPCSource %p\n", - usDspBaseIO, pusIPCSource); =20 /* * Disable DSP to PC interrupts, read the interrupt register, @@ -688,22 +524,11 @@ int dsp3780I_GetIPCSource(unsigned short usDspBaseIO, OutWordDsp(DSP_HBridgeControl, MKWORD(rHBridgeControl)); =20 *pusIPCSource =3D InWordDsp(DSP_Interrupt); - temp =3D (unsigned short) ~(*pusIPCSource); - - PRINTK_3(TRACE_3780I, - "3780i::dsp3780I_GetIPCSource, usIPCSource %x ~ %x\n", - *pusIPCSource, temp); - OutWordDsp(DSP_Interrupt, (unsigned short) ~(*pusIPCSource)); =20 rHBridgeControl.EnableDspInt =3D true; OutWordDsp(DSP_HBridgeControl, MKWORD(rHBridgeControl)); spin_unlock_irqrestore(&dsp_lock, flags); =20 - - PRINTK_2(TRACE_3780I, - "3780i::dsp3780I_GetIPCSource exit usIPCSource %x\n", - *pusIPCSource); - return 0; } diff --git a/drivers/char/mwave/Makefile b/drivers/char/mwave/Makefile index 836bfa25c541..e56c1a375535 100644 --- a/drivers/char/mwave/Makefile +++ b/drivers/char/mwave/Makefile @@ -8,6 +8,3 @@ obj-$(CONFIG_MWAVE) +=3D mwave.o =20 mwave-y :=3D mwavedd.o smapi.o tp3780i.o 3780i.o - -# To compile in lots (~20 KiB) of run-time enablable printk()s for debuggi= ng: -ccflags-y +=3D -DMW_TRACE diff --git a/drivers/char/mwave/README b/drivers/char/mwave/README index c2a58f428bc8..6224aa814c62 100644 --- a/drivers/char/mwave/README +++ b/drivers/char/mwave/README @@ -4,16 +4,6 @@ Module options The mwave module takes the following options. Note that these options are not saved by the BIOS and so do not persist after unload and reload. =20 - mwave_debug=3Dvalue, where value is bitwise OR of trace flags: - 0x0001 mwavedd api tracing - 0x0002 smapi api tracing - 0x0004 3780i tracing - 0x0008 tp3780i tracing - - Tracing only occurs if the driver has been compiled with the - MW_TRACE macro #defined (i.e. let ccflags-y :=3D -DMW_TRACE - in the Makefile). - mwave_3780i_irq=3D5/7/10/11/15 If the dsp irq has not been setup and stored in bios by the=20 thinkpad configuration utility then this parameter allows the diff --git a/drivers/char/mwave/mwavedd.c b/drivers/char/mwave/mwavedd.c index b6d7a8b04183..f01c6f7d54a3 100644 --- a/drivers/char/mwave/mwavedd.c +++ b/drivers/char/mwave/mwavedd.c @@ -75,12 +75,10 @@ MODULE_LICENSE("GPL"); * We'll depend on users using the tpctl utility to do that for now */ static DEFINE_MUTEX(mwave_mutex); -int mwave_debug =3D 0; int mwave_3780i_irq =3D 0; int mwave_3780i_io =3D 0; int mwave_uart_irq =3D 0; int mwave_uart_io =3D 0; -module_param(mwave_debug, int, 0); module_param_hw(mwave_3780i_irq, int, irq, 0); module_param_hw(mwave_3780i_io, int, ioport, 0); module_param_hw(mwave_uart_irq, int, irq, 0); @@ -95,62 +93,32 @@ static long mwave_ioctl(struct file *file, unsigned int= iocmd, pMWAVE_DEVICE_DATA pDrvData =3D &mwave_s_mdd; void __user *arg =3D (void __user *)ioarg; =20 - PRINTK_4(TRACE_MWAVE, - "mwavedd::mwave_ioctl, entry file %p cmd %x arg %x\n", - file, iocmd, (int) ioarg); - switch (iocmd) { =20 case IOCTL_MW_RESET: - PRINTK_1(TRACE_MWAVE, - "mwavedd::mwave_ioctl, IOCTL_MW_RESET" - " calling tp3780I_ResetDSP\n"); mutex_lock(&mwave_mutex); retval =3D tp3780I_ResetDSP(&pDrvData->rBDData); mutex_unlock(&mwave_mutex); - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_ioctl, IOCTL_MW_RESET" - " retval %x from tp3780I_ResetDSP\n", - retval); break; =09 case IOCTL_MW_RUN: - PRINTK_1(TRACE_MWAVE, - "mwavedd::mwave_ioctl, IOCTL_MW_RUN" - " calling tp3780I_StartDSP\n"); mutex_lock(&mwave_mutex); retval =3D tp3780I_StartDSP(&pDrvData->rBDData); mutex_unlock(&mwave_mutex); - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_ioctl, IOCTL_MW_RUN" - " retval %x from tp3780I_StartDSP\n", - retval); break; =09 case IOCTL_MW_DSP_ABILITIES: { MW_ABILITIES rAbilities; =09 - PRINTK_1(TRACE_MWAVE, - "mwavedd::mwave_ioctl," - " IOCTL_MW_DSP_ABILITIES calling" - " tp3780I_QueryAbilities\n"); mutex_lock(&mwave_mutex); retval =3D tp3780I_QueryAbilities(&pDrvData->rBDData, &rAbilities); mutex_unlock(&mwave_mutex); - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_ioctl, IOCTL_MW_DSP_ABILITIES" - " retval %x from tp3780I_QueryAbilities\n", - retval); if (retval =3D=3D 0) { if( copy_to_user(arg, &rAbilities, sizeof(MW_ABILITIES)) ) return -EFAULT; } - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_ioctl, IOCTL_MW_DSP_ABILITIES" - " exit retval %x\n", - retval); } break; =09 @@ -164,10 +132,6 @@ static long mwave_ioctl(struct file *file, unsigned in= t iocmd, return -EFAULT; pusBuffer =3D (unsigned short __user *) (rReadData.pBuf); =09 - PRINTK_4(TRACE_MWAVE, - "mwavedd::mwave_ioctl IOCTL_MW_READ_DATA," - " size %lx, ioarg %lx pusBuffer %p\n", - rReadData.ulDataLength, ioarg, pusBuffer); mutex_lock(&mwave_mutex); retval =3D tp3780I_ReadWriteDspDStore(&pDrvData->rBDData, iocmd, @@ -187,11 +151,6 @@ static long mwave_ioctl(struct file *file, unsigned in= t iocmd, return -EFAULT; pusBuffer =3D (unsigned short __user *) (rReadData.pBuf); =09 - PRINTK_4(TRACE_MWAVE, - "mwavedd::mwave_ioctl IOCTL_MW_READ_INST," - " size %lx, ioarg %lx pusBuffer %p\n", - rReadData.ulDataLength / 2, ioarg, - pusBuffer); mutex_lock(&mwave_mutex); retval =3D tp3780I_ReadWriteDspDStore(&pDrvData->rBDData, iocmd, pusBuffer, @@ -210,11 +169,6 @@ static long mwave_ioctl(struct file *file, unsigned in= t iocmd, return -EFAULT; pusBuffer =3D (unsigned short __user *) (rWriteData.pBuf); =09 - PRINTK_4(TRACE_MWAVE, - "mwavedd::mwave_ioctl IOCTL_MW_WRITE_DATA," - " size %lx, ioarg %lx pusBuffer %p\n", - rWriteData.ulDataLength, ioarg, - pusBuffer); mutex_lock(&mwave_mutex); retval =3D tp3780I_ReadWriteDspDStore(&pDrvData->rBDData, iocmd, pusBuffer, @@ -233,11 +187,6 @@ static long mwave_ioctl(struct file *file, unsigned in= t iocmd, return -EFAULT; pusBuffer =3D (unsigned short __user *)(rWriteData.pBuf); =09 - PRINTK_4(TRACE_MWAVE, - "mwavedd::mwave_ioctl IOCTL_MW_WRITE_INST," - " size %lx, ioarg %lx pusBuffer %p\n", - rWriteData.ulDataLength, ioarg, - pusBuffer); mutex_lock(&mwave_mutex); retval =3D tp3780I_ReadWriteDspIStore(&pDrvData->rBDData, iocmd, pusBuffer, @@ -260,21 +209,11 @@ static long mwave_ioctl(struct file *file, unsigned i= nt iocmd, } ipcnum =3D array_index_nospec(ipcnum, ARRAY_SIZE(pDrvData->IPCs)); - PRINTK_3(TRACE_MWAVE, - "mwavedd::mwave_ioctl IOCTL_MW_REGISTER_IPC" - " ipcnum %x entry usIntCount %x\n", - ipcnum, - pDrvData->IPCs[ipcnum].usIntCount); =20 mutex_lock(&mwave_mutex); pDrvData->IPCs[ipcnum].bIsHere =3D false; pDrvData->IPCs[ipcnum].bIsEnabled =3D true; mutex_unlock(&mwave_mutex); -=09 - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_ioctl IOCTL_MW_REGISTER_IPC" - " ipcnum %x exit\n", - ipcnum); } break; =09 @@ -290,20 +229,11 @@ static long mwave_ioctl(struct file *file, unsigned i= nt iocmd, } ipcnum =3D array_index_nospec(ipcnum, ARRAY_SIZE(pDrvData->IPCs)); - PRINTK_3(TRACE_MWAVE, - "mwavedd::mwave_ioctl IOCTL_MW_GET_IPC" - " ipcnum %x, usIntCount %x\n", - ipcnum, - pDrvData->IPCs[ipcnum].usIntCount); -=09 + mutex_lock(&mwave_mutex); if (pDrvData->IPCs[ipcnum].bIsEnabled =3D=3D true) { DECLARE_WAITQUEUE(wait, current); =20 - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_ioctl, thread for" - " ipc %x going to sleep\n", - ipcnum); add_wait_queue(&pDrvData->IPCs[ipcnum].ipc_wait_queue, &wait); pDrvData->IPCs[ipcnum].bIsHere =3D true; set_current_state(TASK_INTERRUPTIBLE); @@ -311,31 +241,15 @@ static long mwave_ioctl(struct file *file, unsigned i= nt iocmd, /* the interrupt handler while we were gone */ if (pDrvData->IPCs[ipcnum].usIntCount =3D=3D 1) { /* first int has occ= urred (race condition) */ pDrvData->IPCs[ipcnum].usIntCount =3D 2; /* first int has been handle= d */ - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_ioctl" - " IOCTL_MW_GET_IPC ipcnum %x" - " handling first int\n", - ipcnum); } else { /* either 1st int has not yet occurred, or we have already ha= ndled the first int */ schedule(); if (pDrvData->IPCs[ipcnum].usIntCount =3D=3D 1) { pDrvData->IPCs[ipcnum].usIntCount =3D 2; } - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_ioctl" - " IOCTL_MW_GET_IPC ipcnum %x" - " woke up and returning to" - " application\n", - ipcnum); } pDrvData->IPCs[ipcnum].bIsHere =3D false; remove_wait_queue(&pDrvData->IPCs[ipcnum].ipc_wait_queue, &wait); set_current_state(TASK_RUNNING); - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_ioctl IOCTL_MW_GET_IPC," - " returning thread for ipc %x" - " processing\n", - ipcnum); } mutex_unlock(&mwave_mutex); } @@ -344,10 +258,6 @@ static long mwave_ioctl(struct file *file, unsigned in= t iocmd, case IOCTL_MW_UNREGISTER_IPC: { unsigned int ipcnum =3D (unsigned int) ioarg; =09 - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_ioctl IOCTL_MW_UNREGISTER_IPC" - " ipcnum %x\n", - ipcnum); if (ipcnum >=3D ARRAY_SIZE(pDrvData->IPCs)) { PRINTK_ERROR(KERN_ERR_MWAVE "mwavedd::mwave_ioctl:" @@ -373,8 +283,6 @@ static long mwave_ioctl(struct file *file, unsigned int= iocmd, return -ENOTTY; } /* switch */ =20 - PRINTK_2(TRACE_MWAVE, "mwavedd::mwave_ioctl, exit retval %x\n", retval); - return retval; } =20 @@ -440,8 +348,6 @@ static void mwave_exit(void) { pMWAVE_DEVICE_DATA pDrvData =3D &mwave_s_mdd; =20 - PRINTK_1(TRACE_MWAVE, "mwavedd::mwave_exit entry\n"); - if ( pDrvData->sLine >=3D 0 ) { serial8250_unregister_port(pDrvData->sLine); } @@ -457,8 +363,6 @@ static void mwave_exit(void) if (pDrvData->bBDInitialized) { tp3780I_Cleanup(&pDrvData->rBDData); } - - PRINTK_1(TRACE_MWAVE, "mwavedd::mwave_exit exit\n"); } =20 module_exit(mwave_exit); @@ -469,8 +373,6 @@ static int __init mwave_init(void) int retval =3D 0; pMWAVE_DEVICE_DATA pDrvData =3D &mwave_s_mdd; =20 - PRINTK_1(TRACE_MWAVE, "mwavedd::mwave_init entry\n"); - memset(&mwave_s_mdd, 0, sizeof(MWAVE_DEVICE_DATA)); =20 pDrvData->bBDInitialized =3D false; @@ -488,10 +390,6 @@ static int __init mwave_init(void) } =20 retval =3D tp3780I_InitializeBoardData(&pDrvData->rBDData); - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_init, return from tp3780I_InitializeBoardData" - " retval %x\n", - retval); if (retval) { PRINTK_ERROR(KERN_ERR_MWAVE "mwavedd::mwave_init: Error:" @@ -501,10 +399,6 @@ static int __init mwave_init(void) pDrvData->bBDInitialized =3D true; =20 retval =3D tp3780I_CalcResources(&pDrvData->rBDData); - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_init, return from tp3780I_CalcResources" - " retval %x\n", - retval); if (retval) { PRINTK_ERROR(KERN_ERR_MWAVE "mwavedd:mwave_init: Error:" @@ -513,10 +407,6 @@ static int __init mwave_init(void) } =20 retval =3D tp3780I_ClaimResources(&pDrvData->rBDData); - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_init, return from tp3780I_ClaimResources" - " retval %x\n", - retval); if (retval) { PRINTK_ERROR(KERN_ERR_MWAVE "mwavedd:mwave_init: Error:" @@ -526,10 +416,6 @@ static int __init mwave_init(void) pDrvData->bResourcesClaimed =3D true; =20 retval =3D tp3780I_EnableDSP(&pDrvData->rBDData); - PRINTK_2(TRACE_MWAVE, - "mwavedd::mwave_init, return from tp3780I_EnableDSP" - " retval %x\n", - retval); if (retval) { PRINTK_ERROR(KERN_ERR_MWAVE "mwavedd:mwave_init: Error:" diff --git a/drivers/char/mwave/mwavedd.h b/drivers/char/mwave/mwavedd.h index 21cb09c7bed7..fdd5dfdd3639 100644 --- a/drivers/char/mwave/mwavedd.h +++ b/drivers/char/mwave/mwavedd.h @@ -56,7 +56,6 @@ #include #include =20 -extern int mwave_debug; extern int mwave_3780i_irq; extern int mwave_3780i_io; extern int mwave_uart_irq; @@ -65,64 +64,6 @@ extern int mwave_uart_io; #define PRINTK_ERROR printk #define KERN_ERR_MWAVE KERN_ERR "mwave: " =20 -#define TRACE_MWAVE 0x0001 -#define TRACE_SMAPI 0x0002 -#define TRACE_3780I 0x0004 -#define TRACE_TP3780I 0x0008 - -#ifdef MW_TRACE -#define PRINTK_1(f,s) \ - if (f & (mwave_debug)) { \ - printk(s); \ - } - -#define PRINTK_2(f,s,v1) \ - if (f & (mwave_debug)) { \ - printk(s,v1); \ - } - -#define PRINTK_3(f,s,v1,v2) \ - if (f & (mwave_debug)) { \ - printk(s,v1,v2); \ - } - -#define PRINTK_4(f,s,v1,v2,v3) \ - if (f & (mwave_debug)) { \ - printk(s,v1,v2,v3); \ - } - -#define PRINTK_5(f,s,v1,v2,v3,v4) \ - if (f & (mwave_debug)) { \ - printk(s,v1,v2,v3,v4); \ - } - -#define PRINTK_6(f,s,v1,v2,v3,v4,v5) \ - if (f & (mwave_debug)) { \ - printk(s,v1,v2,v3,v4,v5); \ - } - -#define PRINTK_7(f,s,v1,v2,v3,v4,v5,v6) \ - if (f & (mwave_debug)) { \ - printk(s,v1,v2,v3,v4,v5,v6); \ - } - -#define PRINTK_8(f,s,v1,v2,v3,v4,v5,v6,v7) \ - if (f & (mwave_debug)) { \ - printk(s,v1,v2,v3,v4,v5,v6,v7); \ - } - -#else -#define PRINTK_1(f,s) -#define PRINTK_2(f,s,v1) -#define PRINTK_3(f,s,v1,v2) -#define PRINTK_4(f,s,v1,v2,v3) -#define PRINTK_5(f,s,v1,v2,v3,v4) -#define PRINTK_6(f,s,v1,v2,v3,v4,v5) -#define PRINTK_7(f,s,v1,v2,v3,v4,v5,v6) -#define PRINTK_8(f,s,v1,v2,v3,v4,v5,v6,v7) -#endif - - typedef struct _MWAVE_IPC { unsigned short usIntCount; /* 0=3Dnone, 1=3Dfirst, 2=3Dgreater than 1st */ bool bIsEnabled; diff --git a/drivers/char/mwave/smapi.c b/drivers/char/mwave/smapi.c index 5e2fe3235714..1efddb34bef0 100644 --- a/drivers/char/mwave/smapi.c +++ b/drivers/char/mwave/smapi.c @@ -69,10 +69,6 @@ static int smapi_request(unsigned short inBX, unsigned s= hort inCX, unsigned short usSmapiOK =3D -EIO, *pusSmapiOK =3D &usSmapiOK; unsigned int inBXCX =3D (inBX << 16) | inCX; unsigned int inDISI =3D (inDI << 16) | inSI; - int retval =3D 0; - - PRINTK_5(TRACE_SMAPI, "inBX %x inCX %x inDI %x inSI %x\n", - inBX, inCX, inDI, inSI); =20 __asm__ __volatile__("movw $0x5380,%%ax\n\t" "movl %7,%%ebx\n\t" @@ -107,10 +103,6 @@ static int smapi_request(unsigned short inBX, unsigned= short inCX, :"%eax", "%ebx", "%ecx", "%edx", "%edi", "%esi"); =20 - PRINTK_8(TRACE_SMAPI, - "myoutAX %x myoutBX %x myoutCX %x myoutDX %x myoutDI %x myoutSI %x usSma= piOK %x\n", - myoutAX, myoutBX, myoutCX, myoutDX, myoutDI, myoutSI, - usSmapiOK); *outAX =3D myoutAX; *outBX =3D myoutBX; *outCX =3D myoutCX; @@ -118,9 +110,7 @@ static int smapi_request(unsigned short inBX, unsigned = short inCX, *outDI =3D myoutDI; *outSI =3D myoutSI; =20 - retval =3D (usSmapiOK =3D=3D 1) ? 0 : -EIO; - PRINTK_2(TRACE_SMAPI, "smapi::smapi_request exit retval %x\n", retval); - return retval; + return usSmapiOK =3D=3D 1 ? 0 : -EIO; } =20 =20 @@ -134,8 +124,6 @@ int smapi_query_DSP_cfg(SMAPI_DSP_SETTINGS * pSettings) static const unsigned short ausUartBases[] =3D { 0x03F8, 0x02F8, 0x03E8, 0x02E8 }; =20 - PRINTK_1(TRACE_SMAPI, "smapi::smapi_query_DSP_cfg entry\n"); - bRC =3D smapi_request(0x1802, 0x0000, 0, 0, &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); if (bRC) { @@ -143,8 +131,6 @@ int smapi_query_DSP_cfg(SMAPI_DSP_SETTINGS * pSettings) return bRC; } =20 - PRINTK_1(TRACE_SMAPI, "smapi::smapi_query_DSP_cfg, smapi_request OK\n"); - pSettings->bDSPPresent =3D ((usBX & 0x0100) !=3D 0); pSettings->bDSPEnabled =3D ((usCX & 0x0001) !=3D 0); pSettings->usDspIRQ =3D usSI & 0x00FF; @@ -154,11 +140,6 @@ int smapi_query_DSP_cfg(SMAPI_DSP_SETTINGS * pSettings) } else { pSettings->usDspBaseIO =3D 0; } - PRINTK_6(TRACE_SMAPI, - "smapi::smapi_query_DSP_cfg get DSP Settings bDSPPresent %x bDSPEnabled = %x usDspIRQ %x usDspDMA %x usDspBaseIO %x\n", - pSettings->bDSPPresent, pSettings->bDSPEnabled, - pSettings->usDspIRQ, pSettings->usDspDMA, - pSettings->usDspBaseIO); =20 /* check for illegal values */ if ( pSettings->usDspBaseIO =3D=3D 0 )=20 @@ -173,8 +154,6 @@ int smapi_query_DSP_cfg(SMAPI_DSP_SETTINGS * pSettings) return bRC; }=20 =20 - PRINTK_1(TRACE_SMAPI, "smapi::smapi_query_DSP_cfg, smapi_request OK\n"); - pSettings->bModemEnabled =3D ((usCX & 0x0001) !=3D 0); pSettings->usUartIRQ =3D usSI & 0x000F; if (((usSI & 0xFF00) >> 8) < ARRAY_SIZE(ausUartBases)) { @@ -183,20 +162,12 @@ int smapi_query_DSP_cfg(SMAPI_DSP_SETTINGS * pSetting= s) pSettings->usUartBaseIO =3D 0; } =20 - PRINTK_4(TRACE_SMAPI, - "smapi::smapi_query_DSP_cfg get DSP modem settings bModemEnabled %x usUa= rtIRQ %x usUartBaseIO %x\n", - pSettings->bModemEnabled, - pSettings->usUartIRQ, - pSettings->usUartBaseIO); - /* check for illegal values */ if ( pSettings->usUartBaseIO =3D=3D 0 )=20 PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_query_DSP_cfg: Worry: UART bas= e I/O address is 0\n"); if ( pSettings->usUartIRQ =3D=3D 0 ) PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_query_DSP_cfg: Worry: UART IRQ= line is 0\n"); =20 - PRINTK_2(TRACE_SMAPI, "smapi::smapi_query_DSP_cfg exit bRC %x\n", bRC); - return bRC; } =20 @@ -218,10 +189,6 @@ int smapi_set_DSP_cfg(void) =20 unsigned short dspio_index =3D 0, uartio_index =3D 0; =20 - PRINTK_5(TRACE_SMAPI, - "smapi::smapi_set_DSP_cfg entry mwave_3780i_irq %x mwave_3780i_io %x mwa= ve_uart_irq %x mwave_uart_io %x\n", - mwave_3780i_irq, mwave_3780i_io, mwave_uart_irq, mwave_uart_io); - if (mwave_3780i_io) { for (i =3D 0; i < ARRAY_SIZE(ausDspBases); i++) { if (mwave_3780i_io =3D=3D ausDspBases[i]) @@ -374,7 +341,6 @@ int smapi_set_DSP_cfg(void) if (bRC) goto exit_smapi_request_error; =20 /* normal exit: */ - PRINTK_1(TRACE_SMAPI, "smapi::smapi_set_DSP_cfg exit\n"); return 0; =20 exit_conflict: @@ -389,20 +355,13 @@ int smapi_set_DSP_cfg(void) =20 int smapi_set_DSP_power_state(bool bOn) { - int bRC; unsigned short usAX, usBX, usCX, usDX, usDI, usSI; unsigned short usPowerFunction; =20 - PRINTK_2(TRACE_SMAPI, "smapi::smapi_set_DSP_power_state entry bOn %x\n", = bOn); - usPowerFunction =3D (bOn) ? 1 : 0; =20 - bRC =3D smapi_request(0x4901, 0x0000, 0, usPowerFunction, - &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); - - PRINTK_2(TRACE_SMAPI, "smapi::smapi_set_DSP_power_state exit bRC %x\n", b= RC); - - return bRC; + return smapi_request(0x4901, 0x0000, 0, usPowerFunction, &usAX, &usBX, &u= sCX, &usDX, &usDI, + &usSI); } =20 int smapi_init(void) @@ -411,13 +370,10 @@ int smapi_init(void) unsigned short usSmapiID =3D 0; unsigned long flags; =20 - PRINTK_1(TRACE_SMAPI, "smapi::smapi_init entry\n"); - spin_lock_irqsave(&rtc_lock, flags); usSmapiID =3D CMOS_READ(0x7C); usSmapiID |=3D (CMOS_READ(0x7D) << 8); spin_unlock_irqrestore(&rtc_lock, flags); - PRINTK_2(TRACE_SMAPI, "smapi::smapi_init usSmapiID %x\n", usSmapiID); =20 if (usSmapiID =3D=3D 0x5349) { spin_lock_irqsave(&rtc_lock, flags); @@ -427,9 +383,6 @@ int smapi_init(void) if (g_usSmapiPort =3D=3D 0) { PRINTK_ERROR("smapi::smapi_init, ERROR unable to read from SMAPI port\n= "); } else { - PRINTK_2(TRACE_SMAPI, - "smapi::smapi_init, exit true g_usSmapiPort %x\n", - g_usSmapiPort); retval =3D 0; //SmapiQuerySystemID(); } diff --git a/drivers/char/mwave/tp3780i.c b/drivers/char/mwave/tp3780i.c index 83eaffeb22c8..b7e95c57a1c4 100644 --- a/drivers/char/mwave/tp3780i.c +++ b/drivers/char/mwave/tp3780i.c @@ -73,8 +73,6 @@ static void EnableSRAM(THINKPAD_BD_DATA * pBDData) DSP_GPIO_DRIVER_ENABLE_15_8 rGpioDriverEnable; DSP_GPIO_MODE_15_8 rGpioMode; =20 - PRINTK_1(TRACE_TP3780I, "tp3780i::EnableSRAM, entry\n"); - MKWORD(rGpioMode) =3D ReadMsaCfg(DSP_GpioModeControl_15_8); rGpioMode.GpioMode10 =3D 0; WriteMsaCfg(DSP_GpioModeControl_15_8, MKWORD(rGpioMode)); @@ -88,15 +86,11 @@ static void EnableSRAM(THINKPAD_BD_DATA * pBDData) rGpioOutputData.Latch10 =3D 0; rGpioOutputData.Mask10 =3D true; WriteMsaCfg(DSP_GpioOutputData_15_8, MKWORD(rGpioOutputData)); - - PRINTK_1(TRACE_TP3780I, "tp3780i::EnableSRAM exit\n"); } =20 =20 static irqreturn_t UartInterrupt(int irq, void *dev_id) { - PRINTK_3(TRACE_TP3780I, - "tp3780i::UartInterrupt entry irq %x dev_id %p\n", irq, dev_id); return IRQ_HANDLED; } =20 @@ -107,35 +101,16 @@ static irqreturn_t DspInterrupt(int irq, void *dev_id) unsigned short usDspBaseIO =3D pSettings->usDspBaseIO; unsigned short usIPCSource =3D 0, usIsolationMask, usPCNum; =20 - PRINTK_3(TRACE_TP3780I, - "tp3780i::DspInterrupt entry irq %x dev_id %p\n", irq, dev_id); - if (dsp3780I_GetIPCSource(usDspBaseIO, &usIPCSource) =3D=3D 0) { - PRINTK_2(TRACE_TP3780I, - "tp3780i::DspInterrupt, return from dsp3780i_GetIPCSource, usIPCSource = %x\n", - usIPCSource); usIsolationMask =3D 1; for (usPCNum =3D 1; usPCNum <=3D 16; usPCNum++) { if (usIPCSource & usIsolationMask) { usIPCSource &=3D ~usIsolationMask; - PRINTK_3(TRACE_TP3780I, - "tp3780i::DspInterrupt usPCNum %x usIPCSource %x\n", - usPCNum, usIPCSource); if (pDrvData->IPCs[usPCNum - 1].usIntCount =3D=3D 0) { pDrvData->IPCs[usPCNum - 1].usIntCount =3D 1; } - PRINTK_2(TRACE_TP3780I, - "tp3780i::DspInterrupt usIntCount %x\n", - pDrvData->IPCs[usPCNum - 1].usIntCount); if (pDrvData->IPCs[usPCNum - 1].bIsEnabled =3D=3D true) { - PRINTK_2(TRACE_TP3780I, - "tp3780i::DspInterrupt, waking up usPCNum %x\n", - usPCNum - 1); wake_up_interruptible(&pDrvData->IPCs[usPCNum - 1].ipc_wait_queue); - } else { - PRINTK_2(TRACE_TP3780I, - "tp3780i::DspInterrupt, no one waiting for IPC %x\n", - usPCNum - 1); } } if (usIPCSource =3D=3D 0) @@ -143,11 +118,7 @@ static irqreturn_t DspInterrupt(int irq, void *dev_id) /* try next IPC */ usIsolationMask =3D usIsolationMask << 1; } - } else { - PRINTK_1(TRACE_TP3780I, - "tp3780i::DspInterrupt, return false from dsp3780i_GetIPCSource\n"); } - PRINTK_1(TRACE_TP3780I, "tp3780i::DspInterrupt exit\n"); return IRQ_HANDLED; } =20 @@ -157,9 +128,6 @@ int tp3780I_InitializeBoardData(THINKPAD_BD_DATA * pBDD= ata) int retval =3D 0; DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; =20 - - PRINTK_2(TRACE_TP3780I, "tp3780i::tp3780I_InitializeBoardData entry pBDDa= ta %p\n", pBDData); - pBDData->bDSPEnabled =3D false; pSettings->bInterruptClaimed =3D false; =20 @@ -172,15 +140,11 @@ int tp3780I_InitializeBoardData(THINKPAD_BD_DATA * pB= DData) } } =20 - PRINTK_2(TRACE_TP3780I, "tp3780i::tp3780I_InitializeBoardData exit retval= %x\n", retval); - return retval; } =20 void tp3780I_Cleanup(THINKPAD_BD_DATA *pBDData) { - PRINTK_2(TRACE_TP3780I, - "tp3780i::tp3780I_Cleanup entry and exit pBDData %p\n", pBDData); } =20 int tp3780I_CalcResources(THINKPAD_BD_DATA * pBDData) @@ -188,9 +152,6 @@ int tp3780I_CalcResources(THINKPAD_BD_DATA * pBDData) SMAPI_DSP_SETTINGS rSmapiInfo; DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; =20 - PRINTK_2(TRACE_TP3780I, - "tp3780i::tp3780I_CalcResources entry pBDData %p\n", pBDData); - if (smapi_query_DSP_cfg(&rSmapiInfo)) { PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_CalcResources: Error: Coul= d not query DSP config. Aborting.\n"); return -EIO; @@ -225,8 +186,6 @@ int tp3780I_CalcResources(THINKPAD_BD_DATA * pBDData) pBDData->bShareDspIrq =3D pBDData->bShareUartIrq =3D 0; } =20 - PRINTK_1(TRACE_TP3780I, "tp3780i::tp3780I_CalcResources exit\n"); - return 0; } =20 @@ -237,30 +196,21 @@ int tp3780I_ClaimResources(THINKPAD_BD_DATA * pBDData) DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; struct resource *pres; =20 - PRINTK_2(TRACE_TP3780I, - "tp3780i::tp3780I_ClaimResources entry pBDData %p\n", pBDData); - pres =3D request_region(pSettings->usDspBaseIO, 16, "mwave_3780i"); if ( pres =3D=3D NULL ) retval =3D -EIO; =20 if (retval) { PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_ClaimResources: Error: Cou= ld not claim I/O region starting at %x\n", pSettings->usDspBaseIO); - retval =3D -EIO; + return -EIO; } =20 - PRINTK_2(TRACE_TP3780I, "tp3780i::tp3780I_ClaimResources exit retval %x\n= ", retval); - return retval; } =20 int tp3780I_ReleaseResources(THINKPAD_BD_DATA * pBDData) { - int retval =3D 0; DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; =20 - PRINTK_2(TRACE_TP3780I, - "tp3780i::tp3780I_ReleaseResources entry pBDData %p\n", pBDData); - release_region(pSettings->usDspBaseIO & (~3), 16); =20 if (pSettings->bInterruptClaimed) { @@ -268,10 +218,7 @@ int tp3780I_ReleaseResources(THINKPAD_BD_DATA * pBDDat= a) pSettings->bInterruptClaimed =3D false; } =20 - PRINTK_2(TRACE_TP3780I, - "tp3780i::tp3780I_ReleaseResources exit retval %x\n", retval); - - return retval; + return 0; } =20 =20 @@ -281,8 +228,6 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; bool bDSPPoweredUp =3D false, bInterruptAllocated =3D false; =20 - PRINTK_2(TRACE_TP3780I, "tp3780i::tp3780I_EnableDSP entry pBDData %p\n", = pBDData); - if (pBDData->bDSPEnabled) { PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_EnableDSP: Error: DSP alre= ady enabled!\n"); goto exit_cleanup; @@ -366,9 +311,6 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) PRINTK_ERROR("tp3780i::tp3780I_EnableDSP: Error: Could not get 3780i IRQ= %x\n", pSettings->usDspIrq); goto exit_cleanup; } else { - PRINTK_3(TRACE_TP3780I, - "tp3780i::tp3780I_EnableDSP, got interrupt %x bShareDspIrq %x\n", - pSettings->usDspIrq, pBDData->bShareDspIrq); bInterruptAllocated =3D true; pSettings->bInterruptClaimed =3D true; } @@ -390,8 +332,6 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) =20 pBDData->bDSPEnabled =3D true; =20 - PRINTK_1(TRACE_TP3780I, "tp3780i::tp3780I_EnableDSP exit\n"); - return 0; =20 exit_cleanup: @@ -408,11 +348,8 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) =20 int tp3780I_DisableDSP(THINKPAD_BD_DATA * pBDData) { - int retval =3D 0; DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; =20 - PRINTK_2(TRACE_TP3780I, "tp3780i::tp3780I_DisableDSP entry pBDData %p\n",= pBDData); - if (pBDData->bDSPEnabled) { dsp3780I_DisableDSP(&pBDData->rDspSettings); if (pSettings->bInterruptClaimed) { @@ -423,56 +360,38 @@ int tp3780I_DisableDSP(THINKPAD_BD_DATA * pBDData) pBDData->bDSPEnabled =3D false; } =20 - PRINTK_2(TRACE_TP3780I, "tp3780i::tp3780I_DisableDSP exit retval %x\n", r= etval); - - return retval; + return 0; } =20 =20 int tp3780I_ResetDSP(THINKPAD_BD_DATA * pBDData) { - int retval =3D 0; DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; =20 - PRINTK_2(TRACE_TP3780I, "tp3780i::tp3780I_ResetDSP entry pBDData %p\n", - pBDData); - if (dsp3780I_Reset(pSettings) =3D=3D 0) { EnableSRAM(pBDData); - } else { - retval =3D -EIO; + return 0; } - - PRINTK_2(TRACE_TP3780I, "tp3780i::tp3780I_ResetDSP exit retval %x\n", ret= val); - - return retval; + return -EIO; } =20 =20 int tp3780I_StartDSP(THINKPAD_BD_DATA * pBDData) { - int retval =3D 0; DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; =20 - PRINTK_2(TRACE_TP3780I, "tp3780i::tp3780I_StartDSP entry pBDData %p\n", p= BDData); - if (dsp3780I_Run(pSettings) =3D=3D 0) { // @BUG @TBD EnableSRAM(pBDData); } else { - retval =3D -EIO; + return -EIO; } =20 - PRINTK_2(TRACE_TP3780I, "tp3780i::tp3780I_StartDSP exit retval %x\n", ret= val); - - return retval; + return 0; } =20 =20 int tp3780I_QueryAbilities(THINKPAD_BD_DATA * pBDData, MW_ABILITIES * pAbi= lities) { - PRINTK_2(TRACE_TP3780I, - "tp3780i::tp3780I_QueryAbilities entry pBDData %p\n", pBDData); - memset(pAbilities, 0, sizeof(*pAbilities)); /* fill out standard constant fields */ pAbilities->instr_per_sec =3D pBDData->rDspSettings.uIps; @@ -497,9 +416,6 @@ int tp3780I_QueryAbilities(THINKPAD_BD_DATA * pBDData, = MW_ABILITIES * pAbilities memcpy(pAbilities->bios_task_name, TP_ABILITIES_BIOSTASK_NAME, sizeof(TP_ABILITIES_BIOSTASK_NAME)); =20 - PRINTK_1(TRACE_TP3780I, - "tp3780i::tp3780I_QueryAbilities exit retval=3DSUCCESSFUL\n"); - return 0; } =20 @@ -507,15 +423,10 @@ int tp3780I_ReadWriteDspDStore(THINKPAD_BD_DATA * pBD= Data, unsigned int uOpcode, void __user *pvBuffer, unsigned int uCount, unsigned long ulDSPAddr) { - int retval =3D 0; DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; unsigned short usDspBaseIO =3D pSettings->usDspBaseIO; bool bRC =3D 0; =20 - PRINTK_6(TRACE_TP3780I, - "tp3780i::tp3780I_ReadWriteDspDStore entry pBDData %p, uOpcode %x, pvBuf= fer %p, uCount %x, ulDSPAddr %lx\n", - pBDData, uOpcode, pvBuffer, uCount, ulDSPAddr); - if (pBDData->bDSPEnabled) { switch (uOpcode) { case IOCTL_MW_READ_DATA: @@ -532,10 +443,7 @@ int tp3780I_ReadWriteDspDStore(THINKPAD_BD_DATA * pBDD= ata, unsigned int uOpcode, } } =20 - retval =3D (bRC) ? -EIO : 0; - PRINTK_2(TRACE_TP3780I, "tp3780i::tp3780I_ReadWriteDspDStore exit retval = %x\n", retval); - - return retval; + return bRC ? -EIO : 0; } =20 =20 @@ -543,15 +451,10 @@ int tp3780I_ReadWriteDspIStore(THINKPAD_BD_DATA * pBD= Data, unsigned int uOpcode, void __user *pvBuffer, unsigned int uCount, unsigned long ulDSPAddr) { - int retval =3D 0; DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; unsigned short usDspBaseIO =3D pSettings->usDspBaseIO; bool bRC =3D 0; =20 - PRINTK_6(TRACE_TP3780I, - "tp3780i::tp3780I_ReadWriteDspIStore entry pBDData %p, uOpcode %x, pvBuf= fer %p, uCount %x, ulDSPAddr %lx\n", - pBDData, uOpcode, pvBuffer, uCount, ulDSPAddr); - if (pBDData->bDSPEnabled) { switch (uOpcode) { case IOCTL_MW_READ_INST: @@ -564,11 +467,6 @@ int tp3780I_ReadWriteDspIStore(THINKPAD_BD_DATA * pBDD= ata, unsigned int uOpcode, } } =20 - retval =3D (bRC) ? -EIO : 0; - - PRINTK_2(TRACE_TP3780I, - "tp3780i::tp3780I_ReadWriteDspIStore exit retval %x\n", retval); - - return retval; + return bRC ? -EIO : 0; } =20 --=20 2.51.1 From nobody Tue Dec 2 02:18:55 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 3D11333A018 for ; Wed, 19 Nov 2025 09:20:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763544004; cv=none; b=Ccr5ygPYpslVDXPHjnkeAKGTi0aOv+q4lkcKXFN/RVDJ7b29wXV7ap4VuRPIpYJ8w7dSTQv91s+bf4uq/He/qEdQpyI2Q/YZs4i2zuQo9KR8DRHctv4ZE45mAAuCVKZyXK+L7SplKE8QPB2PkiLQGyeQGdzR+hHTXar/c3xmRZg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763544004; c=relaxed/simple; bh=umVcvpZch6lHgvMwSr4NUVIVWHCjRQL8dZsocsznTuM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=YK9q6DcKvH7Vspjkv6gFADsPx+sMhjS9vA5tWC7RO9pMPbWTKmG19oLwDwTHdoDtrdArvRIBwIGKbMuocswStf0f+WTJmgmXzPZg9pCjhfULdqb/nRN4dfmY4bYTYQyaUv50LRbLWUziOEY3orfX84QeVvxE037P9XUhPZddqS8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=dhRN62b/; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="dhRN62b/" Received: by smtp.kernel.org (Postfix) with ESMTPSA id EFBE2C2BCB1; Wed, 19 Nov 2025 09:20:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1763544003; bh=umVcvpZch6lHgvMwSr4NUVIVWHCjRQL8dZsocsznTuM=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=dhRN62b/Ozp/zX4FHx9Hgxj3Xf041nySu91CGRt6k/75PYgcg0MrNHg+hWv/UkkYy eTfVKGdwCss+laI9qLJF3UJ4d3mq1CJh69c1k8No92hc5HBums58NULJA+mcXvjE8U yAyBSXqYS0HgQaA2wTMNuoooZJgh8oXuUedR5rh8gE9+j/QS5vH78LzKkzx6Sdjo9K 8ur5piURCg7ivlPx+tdqoJd1NbtMDilluC0naCumSUiwwtdRAV6tdzHZ8lUFQy9TDa 7RqcwkbHvs26A93M4SFM4OQVbIbYhNK2UuZfMyCSuUy/LL1NS+P4hsLD7GpNrlzqjd +8ITRtbAvXgcg== From: "Jiri Slaby (SUSE)" To: gregkh@linuxfoundation.org Cc: arnd@arndb.de, linux-kernel@vger.kernel.org, "Jiri Slaby (SUSE)" Subject: [PATCH 5/6] char/mwave: drop printk wrapper Date: Wed, 19 Nov 2025 10:19:48 +0100 Message-ID: <20251119091949.825958-6-jirislaby@kernel.org> X-Mailer: git-send-email 2.51.1 In-Reply-To: <20251119091949.825958-1-jirislaby@kernel.org> References: <20251119091949.825958-1-jirislaby@kernel.org> 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" PRINTK_ERROR() + KERN_ERR_MWAVE are just wrappers around printk() with a prefix. Instead, pr_fmt() can be used. Drop the former and use the latter. Signed-off-by: Jiri Slaby (SUSE) Acked-by: Arnd Bergmann --- drivers/char/mwave/3780i.c | 4 ++- drivers/char/mwave/mwavedd.c | 58 +++++++++++------------------------ drivers/char/mwave/mwavedd.h | 3 -- drivers/char/mwave/smapi.c | 59 +++++++++++++++++++++--------------- drivers/char/mwave/tp3780i.c | 35 ++++++++++++--------- 5 files changed, 74 insertions(+), 85 deletions(-) diff --git a/drivers/char/mwave/3780i.c b/drivers/char/mwave/3780i.c index 6024cf7d5705..a1bd9dd36d21 100644 --- a/drivers/char/mwave/3780i.c +++ b/drivers/char/mwave/3780i.c @@ -46,6 +46,8 @@ * First release to the public */ =20 +#define pr_fmt(fmt) "3780i: " fmt + #include #include #include @@ -138,7 +140,7 @@ int dsp3780I_EnableDSP(DSP_3780I_CONFIG_SETTINGS * pSet= tings, unsigned short tval; =20 if (!pSettings->bDSPEnabled) { - PRINTK_ERROR( KERN_ERR "3780i::dsp3780I_EnableDSP: Error: DSP not enable= d. Aborting.\n" ); + pr_err("%s: Error: DSP not enabled. Aborting.\n", __func__); return -EIO; } =20 diff --git a/drivers/char/mwave/mwavedd.c b/drivers/char/mwave/mwavedd.c index f01c6f7d54a3..6ab355cfe43e 100644 --- a/drivers/char/mwave/mwavedd.c +++ b/drivers/char/mwave/mwavedd.c @@ -46,6 +46,8 @@ * First release to the public */ =20 +#define pr_fmt(fmt) "mwavedd: " fmt + #include #include #include @@ -200,11 +202,8 @@ static long mwave_ioctl(struct file *file, unsigned in= t iocmd, unsigned int ipcnum =3D (unsigned int) ioarg; =09 if (ipcnum >=3D ARRAY_SIZE(pDrvData->IPCs)) { - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd::mwave_ioctl:" - " IOCTL_MW_REGISTER_IPC:" - " Error: Invalid ipcnum %x\n", - ipcnum); + pr_err("%s: IOCTL_MW_REGISTER_IPC: Error: Invalid ipcnum %x\n", + __func__, ipcnum); return -EINVAL; } ipcnum =3D array_index_nospec(ipcnum, @@ -221,10 +220,8 @@ static long mwave_ioctl(struct file *file, unsigned in= t iocmd, unsigned int ipcnum =3D (unsigned int) ioarg; =09 if (ipcnum >=3D ARRAY_SIZE(pDrvData->IPCs)) { - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd::mwave_ioctl:" - " IOCTL_MW_GET_IPC: Error:" - " Invalid ipcnum %x\n", ipcnum); + pr_err("%s: IOCTL_MW_GET_IPC: Error: Invalid ipcnum %x\n", __func__, + ipcnum); return -EINVAL; } ipcnum =3D array_index_nospec(ipcnum, @@ -259,11 +256,8 @@ static long mwave_ioctl(struct file *file, unsigned in= t iocmd, unsigned int ipcnum =3D (unsigned int) ioarg; =09 if (ipcnum >=3D ARRAY_SIZE(pDrvData->IPCs)) { - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd::mwave_ioctl:" - " IOCTL_MW_UNREGISTER_IPC:" - " Error: Invalid ipcnum %x\n", - ipcnum); + pr_err("%s: IOCTL_MW_UNREGISTER_IPC: Error: Invalid ipcnum %x\n", + __func__, ipcnum); return -EINVAL; } ipcnum =3D array_index_nospec(ipcnum, @@ -298,9 +292,7 @@ static int register_serial_portandirq(unsigned int port= , int irq) /* OK */ break; default: - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd::register_serial_portandirq:" - " Error: Illegal port %x\n", port ); + pr_err("%s: Error: Illegal port %x\n", __func__, port); return -1; } /* switch */ /* port is okay */ @@ -313,9 +305,7 @@ static int register_serial_portandirq(unsigned int port= , int irq) /* OK */ break; default: - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd::register_serial_portandirq:" - " Error: Illegal irq %x\n", irq ); + pr_err("%s: Error: Illegal irq %x\n", __func__, irq); return -1; } /* switch */ /* irq is okay */ @@ -391,43 +381,33 @@ static int __init mwave_init(void) =20 retval =3D tp3780I_InitializeBoardData(&pDrvData->rBDData); if (retval) { - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd::mwave_init: Error:" - " Failed to initialize board data\n"); + pr_err("%s: Error: Failed to initialize board data\n", __func__); goto cleanup_error; } pDrvData->bBDInitialized =3D true; =20 retval =3D tp3780I_CalcResources(&pDrvData->rBDData); if (retval) { - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd:mwave_init: Error:" - " Failed to calculate resources\n"); + pr_err("%s: Error: Failed to calculate resources\n", __func__); goto cleanup_error; } =20 retval =3D tp3780I_ClaimResources(&pDrvData->rBDData); if (retval) { - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd:mwave_init: Error:" - " Failed to claim resources\n"); + pr_err("%s: Error: Failed to claim resources\n", __func__); goto cleanup_error; } pDrvData->bResourcesClaimed =3D true; =20 retval =3D tp3780I_EnableDSP(&pDrvData->rBDData); if (retval) { - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd:mwave_init: Error:" - " Failed to enable DSP\n"); + pr_err("%s: Error: Failed to enable DSP\n", __func__); goto cleanup_error; } pDrvData->bDSPEnabled =3D true; =20 if (misc_register(&mwave_misc_dev) < 0) { - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd:mwave_init: Error:" - " Failed to register misc device\n"); + pr_err("%s: Error: Failed to register misc device\n", __func__); goto cleanup_error; } pDrvData->bMwaveDevRegistered =3D true; @@ -437,9 +417,7 @@ static int __init mwave_init(void) pDrvData->rBDData.rDspSettings.usUartIrq ); if (pDrvData->sLine < 0) { - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd:mwave_init: Error:" - " Failed to register serial driver\n"); + pr_err("%s: Error: Failed to register serial driver\n", __func__); goto cleanup_error; } /* uart is registered */ @@ -448,9 +426,7 @@ static int __init mwave_init(void) return 0; =20 cleanup_error: - PRINTK_ERROR(KERN_ERR_MWAVE - "mwavedd::mwave_init: Error:" - " Failed to initialize\n"); + pr_err("%s: Error: Failed to initialize\n", __func__); mwave_exit(); /* clean up */ =20 return -EIO; diff --git a/drivers/char/mwave/mwavedd.h b/drivers/char/mwave/mwavedd.h index fdd5dfdd3639..453305494d12 100644 --- a/drivers/char/mwave/mwavedd.h +++ b/drivers/char/mwave/mwavedd.h @@ -61,9 +61,6 @@ extern int mwave_3780i_io; extern int mwave_uart_irq; extern int mwave_uart_io; =20 -#define PRINTK_ERROR printk -#define KERN_ERR_MWAVE KERN_ERR "mwave: " - typedef struct _MWAVE_IPC { unsigned short usIntCount; /* 0=3Dnone, 1=3Dfirst, 2=3Dgreater than 1st */ bool bIsEnabled; diff --git a/drivers/char/mwave/smapi.c b/drivers/char/mwave/smapi.c index 1efddb34bef0..107a2cb9c31c 100644 --- a/drivers/char/mwave/smapi.c +++ b/drivers/char/mwave/smapi.c @@ -46,6 +46,8 @@ * First release to the public */ =20 +#define pr_fmt(fmt) "smapi: " fmt + #include #include /* CMOS defines */ #include "smapi.h" @@ -127,7 +129,7 @@ int smapi_query_DSP_cfg(SMAPI_DSP_SETTINGS * pSettings) bRC =3D smapi_request(0x1802, 0x0000, 0, 0, &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); if (bRC) { - PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_query_DSP_cfg: Error: Could no= t get DSP Settings. Aborting.\n"); + pr_err("%s: Error: Could not get DSP Settings. Aborting.\n", __func__); return bRC; } =20 @@ -143,14 +145,14 @@ int smapi_query_DSP_cfg(SMAPI_DSP_SETTINGS * pSetting= s) =20 /* check for illegal values */ if ( pSettings->usDspBaseIO =3D=3D 0 )=20 - PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_query_DSP_cfg: Worry: DSP base= I/O address is 0\n"); + pr_err("%s: Worry: DSP base I/O address is 0\n", __func__); if ( pSettings->usDspIRQ =3D=3D 0 ) - PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_query_DSP_cfg: Worry: DSP IRQ = line is 0\n"); + pr_err("%s: Worry: DSP IRQ line is 0\n", __func__); =20 bRC =3D smapi_request(0x1804, 0x0000, 0, 0, &usAX, &usBX, &usCX, &usDX, &usDI, &usSI); if (bRC) { - PRINTK_ERROR("smapi::smapi_query_DSP_cfg: Error: Could not get DSP modem= settings. Aborting.\n"); + pr_err("%s: Error: Could not get DSP modem settings. Aborting.\n", __fun= c__); return bRC; }=20 =20 @@ -164,9 +166,9 @@ int smapi_query_DSP_cfg(SMAPI_DSP_SETTINGS * pSettings) =20 /* check for illegal values */ if ( pSettings->usUartBaseIO =3D=3D 0 )=20 - PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_query_DSP_cfg: Worry: UART bas= e I/O address is 0\n"); + pr_err("%s: Worry: UART base I/O address is 0\n", __func__); if ( pSettings->usUartIRQ =3D=3D 0 ) - PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_query_DSP_cfg: Worry: UART IRQ= line is 0\n"); + pr_err("%s: Worry: UART IRQ line is 0\n", __func__); =20 return bRC; } @@ -195,7 +197,8 @@ int smapi_set_DSP_cfg(void) break; } if (i =3D=3D ARRAY_SIZE(ausDspBases)) { - PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_set_DSP_cfg: Error: Invalid m= wave_3780i_io address %x. Aborting.\n", mwave_3780i_io); + pr_err("%s: Error: Invalid mwave_3780i_io address %x. Aborting.\n", + __func__, mwave_3780i_io); return bRC; } dspio_index =3D i; @@ -207,7 +210,8 @@ int smapi_set_DSP_cfg(void) break; } if (i =3D=3D ARRAY_SIZE(ausDspIrqs)) { - PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_set_DSP_cfg: Error: Invalid m= wave_3780i_irq %x. Aborting.\n", mwave_3780i_irq); + pr_err("%s: Error: Invalid mwave_3780i_irq %x. Aborting.\n", __func__, + mwave_3780i_irq); return bRC; } } @@ -218,7 +222,8 @@ int smapi_set_DSP_cfg(void) break; } if (i =3D=3D ARRAY_SIZE(ausUartBases)) { - PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_set_DSP_cfg: Error: Invalid m= wave_uart_io address %x. Aborting.\n", mwave_uart_io); + pr_err("%s: Error: Invalid mwave_uart_io address %x. Aborting.\n", __fu= nc__, + mwave_uart_io); return bRC; } uartio_index =3D i; @@ -231,7 +236,8 @@ int smapi_set_DSP_cfg(void) break; } if (i =3D=3D ARRAY_SIZE(ausUartIrqs)) { - PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_set_DSP_cfg: Error: Invalid m= wave_uart_irq %x. Aborting.\n", mwave_uart_irq); + pr_err("%s: Error: Invalid mwave_uart_irq %x. Aborting.\n", __func__, + mwave_uart_irq); return bRC; } } @@ -246,13 +252,14 @@ int smapi_set_DSP_cfg(void) if (usBX & 0x0100) { /* serial port A is present */ if (usCX & 1) { /* serial port is enabled */ if ((usSI & 0xFF) =3D=3D mwave_uart_irq) { - PRINTK_ERROR(KERN_ERR_MWAVE - "smapi::smapi_set_DSP_cfg: Serial port A irq %x conflicts with mwave= _uart_irq %x\n", usSI & 0xFF, mwave_uart_irq); + pr_err("%s: Serial port A irq %x conflicts with mwave_uart_irq %x\n", + __func__, usSI & 0xFF, mwave_uart_irq); goto exit_conflict; } else { if ((usSI >> 8) =3D=3D uartio_index) { - PRINTK_ERROR(KERN_ERR_MWAVE - "smapi::smapi_set_DSP_cfg: Serial port A base I/O address %x confli= cts with mwave uart I/O %x\n", ausUartBases[usSI >> 8], ausUartBases[uartio= _index]); + pr_err("%s: Serial port A base I/O address %x conflicts with mwave u= art I/O %x\n", + __func__, ausUartBases[usSI >> 8], + ausUartBases[uartio_index]); goto exit_conflict; } } @@ -267,13 +274,14 @@ int smapi_set_DSP_cfg(void) if (usBX & 0x0100) { /* serial port B is present */ if (usCX & 1) { /* serial port is enabled */ if ((usSI & 0xFF) =3D=3D mwave_uart_irq) { - PRINTK_ERROR(KERN_ERR_MWAVE - "smapi::smapi_set_DSP_cfg: Serial port B irq %x conflicts with mwave= _uart_irq %x\n", usSI & 0xFF, mwave_uart_irq); + pr_err("%s: Serial port B irq %x conflicts with mwave_uart_irq %x\n", + __func__, usSI & 0xFF, mwave_uart_irq); goto exit_conflict; } else { if ((usSI >> 8) =3D=3D uartio_index) { - PRINTK_ERROR(KERN_ERR_MWAVE - "smapi::smapi_set_DSP_cfg: Serial port B base I/O address %x confli= cts with mwave uart I/O %x\n", ausUartBases[usSI >> 8], ausUartBases[uartio= _index]); + pr_err("%s: Serial port B base I/O address %x conflicts with mwave u= art I/O %x\n", + __func__, ausUartBases[usSI >> 8], + ausUartBases[uartio_index]); goto exit_conflict; } } @@ -290,13 +298,14 @@ int smapi_set_DSP_cfg(void) /* bRC =3D=3D 0 */ if ((usCX & 0xff) !=3D 0xff) { /* IR port not disabled */ if ((usCX & 0xff) =3D=3D mwave_uart_irq) { - PRINTK_ERROR(KERN_ERR_MWAVE - "smapi::smapi_set_DSP_cfg: IR port irq %x conflicts with mwave_uart_i= rq %x\n", usCX & 0xff, mwave_uart_irq); + pr_err("%s: IR port irq %x conflicts with mwave_uart_irq %x\n", + __func__, usCX & 0xff, mwave_uart_irq); goto exit_conflict; } else { if ((usSI & 0xff) =3D=3D uartio_index) { - PRINTK_ERROR(KERN_ERR_MWAVE - "smapi::smapi_set_DSP_cfg: IR port base I/O address %x conflicts wit= h mwave uart I/O %x\n", ausUartBases[usSI & 0xff], ausUartBases[uartio_inde= x]); + pr_err("%s: IR port base I/O address %x conflicts with mwave uart I/O= %x\n", + __func__, ausUartBases[usSI & 0xff], + ausUartBases[uartio_index]); goto exit_conflict; } } @@ -348,7 +357,7 @@ int smapi_set_DSP_cfg(void) return -EIO; =20 exit_smapi_request_error: - PRINTK_ERROR(KERN_ERR_MWAVE "smapi::smapi_set_DSP_cfg exit on smapi_reque= st error bRC %x\n", bRC); + pr_err("%s: exit on smapi_request error bRC %x\n", __func__, bRC); return bRC; } =20 @@ -381,13 +390,13 @@ int smapi_init(void) g_usSmapiPort |=3D (CMOS_READ(0x7F) << 8); spin_unlock_irqrestore(&rtc_lock, flags); if (g_usSmapiPort =3D=3D 0) { - PRINTK_ERROR("smapi::smapi_init, ERROR unable to read from SMAPI port\n= "); + pr_err("%s: ERROR unable to read from SMAPI port\n", __func__); } else { retval =3D 0; //SmapiQuerySystemID(); } } else { - PRINTK_ERROR("smapi::smapi_init, ERROR invalid usSmapiID\n"); + pr_err("%s: ERROR invalid usSmapiID\n", __func__); retval =3D -ENXIO; } =20 diff --git a/drivers/char/mwave/tp3780i.c b/drivers/char/mwave/tp3780i.c index b7e95c57a1c4..6a924e1bc678 100644 --- a/drivers/char/mwave/tp3780i.c +++ b/drivers/char/mwave/tp3780i.c @@ -46,6 +46,8 @@ * First release to the public */ =20 +#define pr_fmt(fmt) "tp3780i: " fmt + #include #include #include @@ -133,7 +135,7 @@ int tp3780I_InitializeBoardData(THINKPAD_BD_DATA * pBDD= ata) =20 retval =3D smapi_init(); if (retval) { - PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_InitializeBoardData: Error= : SMAPI is not available on this machine\n"); + pr_err("%s: Error: SMAPI is not available on this machine\n", __func__); } else { if (mwave_3780i_irq || mwave_3780i_io || mwave_uart_irq || mwave_uart_io= ) { retval =3D smapi_set_DSP_cfg(); @@ -153,7 +155,7 @@ int tp3780I_CalcResources(THINKPAD_BD_DATA * pBDData) DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; =20 if (smapi_query_DSP_cfg(&rSmapiInfo)) { - PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_CalcResources: Error: Coul= d not query DSP config. Aborting.\n"); + pr_err("%s: Error: Could not query DSP config. Aborting.\n", __func__); return -EIO; } =20 @@ -164,7 +166,7 @@ int tp3780I_CalcResources(THINKPAD_BD_DATA * pBDData) || ( rSmapiInfo.usUartIRQ =3D=3D 0 ) || ( rSmapiInfo.usUartBaseIO =3D=3D 0 ) ) { - PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_CalcResources: Error: Ille= gal resource setting. Aborting.\n"); + pr_err("%s: Error: Illegal resource setting. Aborting.\n", __func__); return -EIO; } =20 @@ -200,7 +202,8 @@ int tp3780I_ClaimResources(THINKPAD_BD_DATA * pBDData) if ( pres =3D=3D NULL ) retval =3D -EIO; =20 if (retval) { - PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_ClaimResources: Error: Cou= ld not claim I/O region starting at %x\n", pSettings->usDspBaseIO); + pr_err("%s: Error: Could not claim I/O region starting at %x\n", __func_= _, + pSettings->usDspBaseIO); return -EIO; } =20 @@ -229,12 +232,12 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) bool bDSPPoweredUp =3D false, bInterruptAllocated =3D false; =20 if (pBDData->bDSPEnabled) { - PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_EnableDSP: Error: DSP alre= ady enabled!\n"); + pr_err("%s: Error: DSP already enabled!\n", __func__); goto exit_cleanup; } =20 if (!pSettings->bDSPEnabled) { - PRINTK_ERROR(KERN_ERR_MWAVE "tp3780::tp3780I_EnableDSP: Error: pSettings= ->bDSPEnabled not set\n"); + pr_err("%s: Error: pSettings->bDSPEnabled not set\n", __func__); goto exit_cleanup; } =20 @@ -244,7 +247,7 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) || (s_ausThinkpadIrqToField[pSettings->usDspIrq] =3D=3D 0xFFFF) || (s_ausThinkpadDmaToField[pSettings->usDspDma] =3D=3D 0xFFFF) ) { - PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_EnableDSP: Error: invalid = irq %x\n", pSettings->usDspIrq); + pr_err("%s: Error: invalid irq %x\n", __func__, pSettings->usDspIrq); goto exit_cleanup; } =20 @@ -252,7 +255,8 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) ((pSettings->usDspBaseIO & 0xF00F) !=3D 0) || (pSettings->usDspBaseIO & 0x0FF0) =3D=3D 0 ) { - PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_EnableDSP: Error: Invalid = DSP base I/O address %x\n", pSettings->usDspBaseIO); + pr_err("%s: Error: Invalid DSP base I/O address %x\n", __func__, + pSettings->usDspBaseIO); goto exit_cleanup; } =20 @@ -261,7 +265,7 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) pSettings->usUartIrq >=3D s_numIrqs || s_ausThinkpadIrqToField[pSettings->usUartIrq] =3D=3D 0xFFFF ) { - PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_EnableDSP: Error: Invalid= UART IRQ %x\n", pSettings->usUartIrq); + pr_err("%s: Error: Invalid UART IRQ %x\n", __func__, pSettings->usUartI= rq); goto exit_cleanup; } switch (pSettings->usUartBaseIO) { @@ -272,7 +276,8 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) break; =20 default: - PRINTK_ERROR("tp3780i::tp3780I_EnableDSP: Error: Invalid UART base I/O= address %x\n", pSettings->usUartBaseIO); + pr_err("%s: Error: Invalid UART base I/O address %x\n", __func__, + pSettings->usUartBaseIO); goto exit_cleanup; } } @@ -301,14 +306,14 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) pSettings->usChipletEnable =3D TP_CFG_ChipletEnable; =20 if (request_irq(pSettings->usUartIrq, &UartInterrupt, 0, "mwave_uart", NU= LL)) { - PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_EnableDSP: Error: Could no= t get UART IRQ %x\n", pSettings->usUartIrq); + pr_err("%s: Error: Could not get UART IRQ %x\n", __func__, pSettings->us= UartIrq); goto exit_cleanup; } else { /* no conflict just release */ free_irq(pSettings->usUartIrq, NULL); } =20 if (request_irq(pSettings->usDspIrq, &DspInterrupt, 0, "mwave_3780i", NUL= L)) { - PRINTK_ERROR("tp3780i::tp3780I_EnableDSP: Error: Could not get 3780i IRQ= %x\n", pSettings->usDspIrq); + pr_err("%s: Error: Could not get 3780i IRQ %x\n", __func__, pSettings->u= sDspIrq); goto exit_cleanup; } else { bInterruptAllocated =3D true; @@ -317,14 +322,14 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) =20 smapi_set_DSP_power_state(false); if (smapi_set_DSP_power_state(true)) { - PRINTK_ERROR(KERN_ERR_MWAVE "tp3780i::tp3780I_EnableDSP: Error: smapi_se= t_DSP_power_state(true) failed\n"); + pr_err("%s: Error: smapi_set_DSP_power_state(true) failed\n", __func__); goto exit_cleanup; } else { bDSPPoweredUp =3D true; } =20 if (dsp3780I_EnableDSP(pSettings, s_ausThinkpadIrqToField, s_ausThinkpadD= maToField)) { - PRINTK_ERROR("tp3780i::tp3780I_EnableDSP: Error: dsp7880I_EnableDSP() fa= iled\n"); + pr_err("%s: Error: dsp7880I_EnableDSP() failed\n", __func__); goto exit_cleanup; } =20 @@ -335,7 +340,7 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) return 0; =20 exit_cleanup: - PRINTK_ERROR("tp3780i::tp3780I_EnableDSP: Cleaning up\n"); + pr_err("%s: Cleaning up\n", __func__); if (bDSPPoweredUp) smapi_set_DSP_power_state(false); if (bInterruptAllocated) { --=20 2.51.1 From nobody Tue Dec 2 02:18:55 2025 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 4A44833E342 for ; Wed, 19 Nov 2025 09:20:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763544006; cv=none; b=aSvjGZeWJIxmUgEYFONX2q11nfrXVqSDdvwhCxXwh4lk/fEP7blpBLl8il75RVan8JEUdPhdKagi24ef0kRPNmbKTJ4VWxg2xwgOGPrEub941syo30KQjkaYSKmk4UxMqJczOaHE4PtAbVzuQV8DPTKInDo7kBnwkM5qLUiLxwI= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763544006; c=relaxed/simple; bh=4lMjN0zNrKJpnvTH2NUrcF9t57Agd4w1OmRfepH1XgI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ETyGiqsrE+RBrQsQ+Mev+CoDVSLCOxxivkQU9wgbd/hraoa0CO9Fu/koD1w5lb1TotuB9oznY8SLvHtyLFTourhRau00InyPPkMaG53zjsdpne5ZmMPoDrF5uB7lJ9Jivrv6c95tqqBJCjpkymgyVCDfm0vDX6NOUqYr3bFwwk4= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=ih0ftPam; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="ih0ftPam" Received: by smtp.kernel.org (Postfix) with ESMTPSA id E9B0DC2BCB4; Wed, 19 Nov 2025 09:20:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1763544005; bh=4lMjN0zNrKJpnvTH2NUrcF9t57Agd4w1OmRfepH1XgI=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=ih0ftPamsNPzR2Q+VgwIiJMBPjulPtiDXD3WKl7bLBqdYYtjwH8A5h1molD82PYTb LSElZ1nRCCG5yIiH+QSMl+jNQLIoG2foYSA2RXdisnah4tV3Lduv+6rFisP27FKLdE LwNPWsYD4YDI6tq/FO3ZsfGIMShswncpH4GHybH0S1w1MJFMECBD1/Yi6o2CRz17yE pQ/ZYAJNlVXI2pnF1/y+oitc3NC0tWnYse2QXqAj69uFIZjnbIbFr4+QKhm4SvRuur 1a+ZgYS4+uUlr489TpYXbyjtReNCpw0HQ0aPDvu3ymHTYZN0uYiEIQTJ+ud5C8kOLd g93VjEeCakecQ== From: "Jiri Slaby (SUSE)" To: gregkh@linuxfoundation.org Cc: arnd@arndb.de, linux-kernel@vger.kernel.org, "Jiri Slaby (SUSE)" Subject: [PATCH 6/6] char/mwave: drop typedefs Date: Wed, 19 Nov 2025 10:19:49 +0100 Message-ID: <20251119091949.825958-7-jirislaby@kernel.org> X-Mailer: git-send-email 2.51.1 In-Reply-To: <20251119091949.825958-1-jirislaby@kernel.org> References: <20251119091949.825958-1-jirislaby@kernel.org> 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" typedefs are unnecessary here. They rather obfuscate the code than help. So drop them and use the types directly. Signed-off-by: Jiri Slaby (SUSE) Acked-by: Arnd Bergmann --- drivers/char/mwave/3780i.c | 8 +++--- drivers/char/mwave/3780i.h | 12 ++++---- drivers/char/mwave/mwavedd.c | 34 ++++++++++------------ drivers/char/mwave/mwavedd.h | 14 ++++----- drivers/char/mwave/mwavepub.h | 22 +++++++------- drivers/char/mwave/smapi.c | 2 +- drivers/char/mwave/smapi.h | 6 ++-- drivers/char/mwave/tp3780i.c | 54 +++++++++++++++++------------------ drivers/char/mwave/tp3780i.h | 30 +++++++++---------- 9 files changed, 89 insertions(+), 93 deletions(-) diff --git a/drivers/char/mwave/3780i.c b/drivers/char/mwave/3780i.c index a1bd9dd36d21..90f93cefb21c 100644 --- a/drivers/char/mwave/3780i.c +++ b/drivers/char/mwave/3780i.c @@ -115,7 +115,7 @@ static void dsp3780I_WriteGenCfg(unsigned short usDspBa= seIO, unsigned uIndex, OutByteDsp(DSP_IsaSlaveControl, MKBYTE(rSlaveControl_Save)); } =20 -int dsp3780I_EnableDSP(DSP_3780I_CONFIG_SETTINGS * pSettings, +int dsp3780I_EnableDSP(struct dsp_3780i_config_settings *pSettings, unsigned short *pIrqMap, unsigned short *pDmaMap) { @@ -260,7 +260,7 @@ int dsp3780I_EnableDSP(DSP_3780I_CONFIG_SETTINGS * pSet= tings, return 0; } =20 -int dsp3780I_DisableDSP(DSP_3780I_CONFIG_SETTINGS * pSettings) +int dsp3780I_DisableDSP(struct dsp_3780i_config_settings *pSettings) { unsigned long flags; unsigned short usDspBaseIO =3D pSettings->usDspBaseIO; @@ -284,7 +284,7 @@ int dsp3780I_DisableDSP(DSP_3780I_CONFIG_SETTINGS * pSe= ttings) return 0; } =20 -int dsp3780I_Reset(DSP_3780I_CONFIG_SETTINGS * pSettings) +int dsp3780I_Reset(struct dsp_3780i_config_settings *pSettings) { unsigned long flags; unsigned short usDspBaseIO =3D pSettings->usDspBaseIO; @@ -317,7 +317,7 @@ int dsp3780I_Reset(DSP_3780I_CONFIG_SETTINGS * pSetting= s) } =20 =20 -int dsp3780I_Run(DSP_3780I_CONFIG_SETTINGS * pSettings) +int dsp3780I_Run(struct dsp_3780i_config_settings *pSettings) { unsigned long flags; unsigned short usDspBaseIO =3D pSettings->usDspBaseIO; diff --git a/drivers/char/mwave/3780i.h b/drivers/char/mwave/3780i.h index 95164246afd1..53dafceb20e0 100644 --- a/drivers/char/mwave/3780i.h +++ b/drivers/char/mwave/3780i.h @@ -261,7 +261,7 @@ typedef struct { * the only values maintained by the 3780i support layer are the saved UART * registers. */ -typedef struct _DSP_3780I_CONFIG_SETTINGS { +struct dsp_3780i_config_settings { =20 /* Location of base configuration register */ unsigned short usBaseConfigIO; @@ -313,16 +313,16 @@ typedef struct _DSP_3780I_CONFIG_SETTINGS { unsigned char ucSCR; /* Scratch register */ unsigned char ucDLL; /* Divisor latch, low byte */ unsigned char ucDLM; /* Divisor latch, high byte */ -} DSP_3780I_CONFIG_SETTINGS; +}; =20 =20 /* 3780i support functions */ -int dsp3780I_EnableDSP(DSP_3780I_CONFIG_SETTINGS * pSettings, +int dsp3780I_EnableDSP(struct dsp_3780i_config_settings *pSettings, unsigned short *pIrqMap, unsigned short *pDmaMap); -int dsp3780I_DisableDSP(DSP_3780I_CONFIG_SETTINGS * pSettings); -int dsp3780I_Reset(DSP_3780I_CONFIG_SETTINGS * pSettings); -int dsp3780I_Run(DSP_3780I_CONFIG_SETTINGS * pSettings); +int dsp3780I_DisableDSP(struct dsp_3780i_config_settings *pSettings); +int dsp3780I_Reset(struct dsp_3780i_config_settings *pSettings); +int dsp3780I_Run(struct dsp_3780i_config_settings *pSettings); int dsp3780I_ReadDStore(unsigned short usDspBaseIO, void __user *pvBuffer, unsigned uCount, unsigned long ulDSPAddr); int dsp3780I_ReadAndClearDStore(unsigned short usDspBaseIO, diff --git a/drivers/char/mwave/mwavedd.c b/drivers/char/mwave/mwavedd.c index 6ab355cfe43e..640a9cb0dd8d 100644 --- a/drivers/char/mwave/mwavedd.c +++ b/drivers/char/mwave/mwavedd.c @@ -86,13 +86,13 @@ module_param_hw(mwave_3780i_io, int, ioport, 0); module_param_hw(mwave_uart_irq, int, irq, 0); module_param_hw(mwave_uart_io, int, ioport, 0); =20 -MWAVE_DEVICE_DATA mwave_s_mdd; +struct mwave_device_data mwave_s_mdd; =20 static long mwave_ioctl(struct file *file, unsigned int iocmd, unsigned long ioarg) { unsigned int retval =3D 0; - pMWAVE_DEVICE_DATA pDrvData =3D &mwave_s_mdd; + struct mwave_device_data *pDrvData =3D &mwave_s_mdd; void __user *arg =3D (void __user *)ioarg; =20 switch (iocmd) { @@ -110,15 +110,14 @@ static long mwave_ioctl(struct file *file, unsigned i= nt iocmd, break; =09 case IOCTL_MW_DSP_ABILITIES: { - MW_ABILITIES rAbilities; + struct mw_abilities rAbilities; =09 mutex_lock(&mwave_mutex); retval =3D tp3780I_QueryAbilities(&pDrvData->rBDData, &rAbilities); mutex_unlock(&mwave_mutex); if (retval =3D=3D 0) { - if( copy_to_user(arg, &rAbilities, - sizeof(MW_ABILITIES)) ) + if (copy_to_user(arg, &rAbilities, sizeof(rAbilities))) return -EFAULT; } } @@ -126,11 +125,11 @@ static long mwave_ioctl(struct file *file, unsigned i= nt iocmd, =09 case IOCTL_MW_READ_DATA: case IOCTL_MW_READCLEAR_DATA: { - MW_READWRITE rReadData; + struct mw_readwrite rReadData; unsigned short __user *pusBuffer =3D NULL; =09 if( copy_from_user(&rReadData, arg, - sizeof(MW_READWRITE)) ) + sizeof(struct mw_readwrite)) ) return -EFAULT; pusBuffer =3D (unsigned short __user *) (rReadData.pBuf); =09 @@ -145,11 +144,10 @@ static long mwave_ioctl(struct file *file, unsigned i= nt iocmd, break; =09 case IOCTL_MW_READ_INST: { - MW_READWRITE rReadData; + struct mw_readwrite rReadData; unsigned short __user *pusBuffer =3D NULL; =09 - if( copy_from_user(&rReadData, arg, - sizeof(MW_READWRITE)) ) + if (copy_from_user(&rReadData, arg, sizeof(rReadData))) return -EFAULT; pusBuffer =3D (unsigned short __user *) (rReadData.pBuf); =09 @@ -163,11 +161,10 @@ static long mwave_ioctl(struct file *file, unsigned i= nt iocmd, break; =09 case IOCTL_MW_WRITE_DATA: { - MW_READWRITE rWriteData; + struct mw_readwrite rWriteData; unsigned short __user *pusBuffer =3D NULL; =09 - if( copy_from_user(&rWriteData, arg, - sizeof(MW_READWRITE)) ) + if (copy_from_user(&rWriteData, arg, sizeof(rWriteData))) return -EFAULT; pusBuffer =3D (unsigned short __user *) (rWriteData.pBuf); =09 @@ -181,11 +178,10 @@ static long mwave_ioctl(struct file *file, unsigned i= nt iocmd, break; =09 case IOCTL_MW_WRITE_INST: { - MW_READWRITE rWriteData; + struct mw_readwrite rWriteData; unsigned short __user *pusBuffer =3D NULL; =09 - if( copy_from_user(&rWriteData, arg, - sizeof(MW_READWRITE)) ) + if (copy_from_user(&rWriteData, arg, sizeof(rWriteData))) return -EFAULT; pusBuffer =3D (unsigned short __user *)(rWriteData.pBuf); =09 @@ -336,7 +332,7 @@ static struct miscdevice mwave_misc_dev =3D { MWAVE_MIN= OR, "mwave", &mwave_fops }; */ static void mwave_exit(void) { - pMWAVE_DEVICE_DATA pDrvData =3D &mwave_s_mdd; + struct mwave_device_data *pDrvData =3D &mwave_s_mdd; =20 if ( pDrvData->sLine >=3D 0 ) { serial8250_unregister_port(pDrvData->sLine); @@ -361,9 +357,9 @@ static int __init mwave_init(void) { int i; int retval =3D 0; - pMWAVE_DEVICE_DATA pDrvData =3D &mwave_s_mdd; + struct mwave_device_data *pDrvData =3D &mwave_s_mdd; =20 - memset(&mwave_s_mdd, 0, sizeof(MWAVE_DEVICE_DATA)); + memset(&mwave_s_mdd, 0, sizeof(mwave_s_mdd)); =20 pDrvData->bBDInitialized =3D false; pDrvData->bResourcesClaimed =3D false; diff --git a/drivers/char/mwave/mwavedd.h b/drivers/char/mwave/mwavedd.h index 453305494d12..e1da1493eec5 100644 --- a/drivers/char/mwave/mwavedd.h +++ b/drivers/char/mwave/mwavedd.h @@ -61,30 +61,30 @@ extern int mwave_3780i_io; extern int mwave_uart_irq; extern int mwave_uart_io; =20 -typedef struct _MWAVE_IPC { +struct mwave_ipc { unsigned short usIntCount; /* 0=3Dnone, 1=3Dfirst, 2=3Dgreater than 1st */ bool bIsEnabled; bool bIsHere; /* entry spin lock */ wait_queue_head_t ipc_wait_queue; -} MWAVE_IPC; +}; =20 -typedef struct _MWAVE_DEVICE_DATA { - THINKPAD_BD_DATA rBDData; /* board driver's data area */ +struct mwave_device_data { + struct thinkpad_bd_data rBDData; /* board driver's data area */ unsigned long ulIPCSource_ISR; /* IPC source bits for recently processed = intr, set during ISR processing */ unsigned long ulIPCSource_DPC; /* IPC source bits for recently processed = intr, set during DPC processing */ bool bBDInitialized; bool bResourcesClaimed; bool bDSPEnabled; bool bDSPReset; - MWAVE_IPC IPCs[16]; + struct mwave_ipc IPCs[16]; bool bMwaveDevRegistered; short sLine; int nr_registered_attrs; int device_registered; =20 -} MWAVE_DEVICE_DATA, *pMWAVE_DEVICE_DATA; +}; =20 -extern MWAVE_DEVICE_DATA mwave_s_mdd; +extern struct mwave_device_data mwave_s_mdd; =20 #endif diff --git a/drivers/char/mwave/mwavepub.h b/drivers/char/mwave/mwavepub.h index 60c961ae23b4..280327bdaa38 100644 --- a/drivers/char/mwave/mwavepub.h +++ b/drivers/char/mwave/mwavepub.h @@ -53,7 +53,7 @@ #include =20 =20 -typedef struct _MW_ABILITIES { +struct mw_abilities { unsigned long instr_per_sec; unsigned long data_size; unsigned long inst_size; @@ -63,27 +63,27 @@ typedef struct _MW_ABILITIES { unsigned long component_list[7]; char mwave_os_name[16]; char bios_task_name[16]; -} MW_ABILITIES, *pMW_ABILITIES; +}; =20 =20 -typedef struct _MW_READWRITE { +struct mw_readwrite { unsigned short usDspAddress; /* The dsp address */ unsigned long ulDataLength; /* The size in bytes of the data or user buff= er */ void __user *pBuf; /* Input:variable sized buffer */ -} MW_READWRITE, *pMW_READWRITE; +}; =20 #define IOCTL_MW_RESET _IO(MWAVE_MINOR,1) #define IOCTL_MW_RUN _IO(MWAVE_MINOR,2) -#define IOCTL_MW_DSP_ABILITIES _IOR(MWAVE_MINOR,3,MW_ABILITIES) -#define IOCTL_MW_READ_DATA _IOR(MWAVE_MINOR,4,MW_READWRITE) -#define IOCTL_MW_READCLEAR_DATA _IOR(MWAVE_MINOR,5,MW_READWRITE) -#define IOCTL_MW_READ_INST _IOR(MWAVE_MINOR,6,MW_READWRITE) -#define IOCTL_MW_WRITE_DATA _IOW(MWAVE_MINOR,7,MW_READWRITE) -#define IOCTL_MW_WRITE_INST _IOW(MWAVE_MINOR,8,MW_READWRITE) +#define IOCTL_MW_DSP_ABILITIES _IOR(MWAVE_MINOR,3,struct mw_abilities) +#define IOCTL_MW_READ_DATA _IOR(MWAVE_MINOR,4,struct mw_readwrite) +#define IOCTL_MW_READCLEAR_DATA _IOR(MWAVE_MINOR,5,struct mw_readwrite) +#define IOCTL_MW_READ_INST _IOR(MWAVE_MINOR,6,struct mw_readwrite) +#define IOCTL_MW_WRITE_DATA _IOW(MWAVE_MINOR,7,struct mw_readwrite) +#define IOCTL_MW_WRITE_INST _IOW(MWAVE_MINOR,8,struct mw_readwrite) #define IOCTL_MW_REGISTER_IPC _IOW(MWAVE_MINOR,9,int) #define IOCTL_MW_UNREGISTER_IPC _IOW(MWAVE_MINOR,10,int) #define IOCTL_MW_GET_IPC _IOW(MWAVE_MINOR,11,int) -#define IOCTL_MW_TRACE _IOR(MWAVE_MINOR,12,MW_READWRITE) +#define IOCTL_MW_TRACE _IOR(MWAVE_MINOR,12,struct mw_readwrite) =20 =20 #endif diff --git a/drivers/char/mwave/smapi.c b/drivers/char/mwave/smapi.c index 107a2cb9c31c..df6354b24339 100644 --- a/drivers/char/mwave/smapi.c +++ b/drivers/char/mwave/smapi.c @@ -116,7 +116,7 @@ static int smapi_request(unsigned short inBX, unsigned = short inCX, } =20 =20 -int smapi_query_DSP_cfg(SMAPI_DSP_SETTINGS * pSettings) +int smapi_query_DSP_cfg(struct smapi_dsp_settings *pSettings) { int bRC; unsigned short usAX, usBX, usCX, usDX, usDI, usSI; diff --git a/drivers/char/mwave/smapi.h b/drivers/char/mwave/smapi.h index ebc206b000b9..e605b16ed23c 100644 --- a/drivers/char/mwave/smapi.h +++ b/drivers/char/mwave/smapi.h @@ -49,7 +49,7 @@ #ifndef _LINUX_SMAPI_H #define _LINUX_SMAPI_H =20 -typedef struct { +struct smapi_dsp_settings { int bDSPPresent; int bDSPEnabled; int bModemEnabled; @@ -65,10 +65,10 @@ typedef struct { unsigned short usSndblstIRQ; unsigned short usSndblstDMA; unsigned short usSndblstBaseIO; -} SMAPI_DSP_SETTINGS; +}; =20 int smapi_init(void); -int smapi_query_DSP_cfg(SMAPI_DSP_SETTINGS * pSettings); +int smapi_query_DSP_cfg(struct smapi_dsp_settings *pSettings); int smapi_set_DSP_cfg(void); int smapi_set_DSP_power_state(bool bOn); =20 diff --git a/drivers/char/mwave/tp3780i.c b/drivers/char/mwave/tp3780i.c index 6a924e1bc678..7363b0f764e0 100644 --- a/drivers/char/mwave/tp3780i.c +++ b/drivers/char/mwave/tp3780i.c @@ -67,9 +67,9 @@ static unsigned short s_ausThinkpadDmaToField[8] =3D static unsigned short s_numIrqs =3D 16, s_numDmas =3D 8; =20 =20 -static void EnableSRAM(THINKPAD_BD_DATA * pBDData) +static void EnableSRAM(struct thinkpad_bd_data *pBDData) { - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; + struct dsp_3780i_config_settings *pSettings =3D &pBDData->rDspSettings; unsigned short usDspBaseIO =3D pSettings->usDspBaseIO; DSP_GPIO_OUTPUT_DATA_15_8 rGpioOutputData; DSP_GPIO_DRIVER_ENABLE_15_8 rGpioDriverEnable; @@ -98,8 +98,8 @@ static irqreturn_t UartInterrupt(int irq, void *dev_id) =20 static irqreturn_t DspInterrupt(int irq, void *dev_id) { - pMWAVE_DEVICE_DATA pDrvData =3D &mwave_s_mdd; - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pDrvData->rBDData.rDspSettings; + struct mwave_device_data *pDrvData =3D &mwave_s_mdd; + struct dsp_3780i_config_settings *pSettings =3D &pDrvData->rBDData.rDspSe= ttings; unsigned short usDspBaseIO =3D pSettings->usDspBaseIO; unsigned short usIPCSource =3D 0, usIsolationMask, usPCNum; =20 @@ -125,10 +125,10 @@ static irqreturn_t DspInterrupt(int irq, void *dev_id) } =20 =20 -int tp3780I_InitializeBoardData(THINKPAD_BD_DATA * pBDData) +int tp3780I_InitializeBoardData(struct thinkpad_bd_data *pBDData) { int retval =3D 0; - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; + struct dsp_3780i_config_settings *pSettings =3D &pBDData->rDspSettings; =20 pBDData->bDSPEnabled =3D false; pSettings->bInterruptClaimed =3D false; @@ -145,14 +145,14 @@ int tp3780I_InitializeBoardData(THINKPAD_BD_DATA * pB= DData) return retval; } =20 -void tp3780I_Cleanup(THINKPAD_BD_DATA *pBDData) +void tp3780I_Cleanup(struct thinkpad_bd_data *pBDData) { } =20 -int tp3780I_CalcResources(THINKPAD_BD_DATA * pBDData) +int tp3780I_CalcResources(struct thinkpad_bd_data *pBDData) { - SMAPI_DSP_SETTINGS rSmapiInfo; - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; + struct smapi_dsp_settings rSmapiInfo; + struct dsp_3780i_config_settings *pSettings =3D &pBDData->rDspSettings; =20 if (smapi_query_DSP_cfg(&rSmapiInfo)) { pr_err("%s: Error: Could not query DSP config. Aborting.\n", __func__); @@ -192,10 +192,10 @@ int tp3780I_CalcResources(THINKPAD_BD_DATA * pBDData) } =20 =20 -int tp3780I_ClaimResources(THINKPAD_BD_DATA * pBDData) +int tp3780I_ClaimResources(struct thinkpad_bd_data *pBDData) { int retval =3D 0; - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; + struct dsp_3780i_config_settings *pSettings =3D &pBDData->rDspSettings; struct resource *pres; =20 pres =3D request_region(pSettings->usDspBaseIO, 16, "mwave_3780i"); @@ -210,9 +210,9 @@ int tp3780I_ClaimResources(THINKPAD_BD_DATA * pBDData) return retval; } =20 -int tp3780I_ReleaseResources(THINKPAD_BD_DATA * pBDData) +int tp3780I_ReleaseResources(struct thinkpad_bd_data *pBDData) { - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; + struct dsp_3780i_config_settings *pSettings =3D &pBDData->rDspSettings; =20 release_region(pSettings->usDspBaseIO & (~3), 16); =20 @@ -226,9 +226,9 @@ int tp3780I_ReleaseResources(THINKPAD_BD_DATA * pBDData) =20 =20 =20 -int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) +int tp3780I_EnableDSP(struct thinkpad_bd_data *pBDData) { - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; + struct dsp_3780i_config_settings *pSettings =3D &pBDData->rDspSettings; bool bDSPPoweredUp =3D false, bInterruptAllocated =3D false; =20 if (pBDData->bDSPEnabled) { @@ -351,9 +351,9 @@ int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData) } =20 =20 -int tp3780I_DisableDSP(THINKPAD_BD_DATA * pBDData) +int tp3780I_DisableDSP(struct thinkpad_bd_data *pBDData) { - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; + struct dsp_3780i_config_settings *pSettings =3D &pBDData->rDspSettings; =20 if (pBDData->bDSPEnabled) { dsp3780I_DisableDSP(&pBDData->rDspSettings); @@ -369,9 +369,9 @@ int tp3780I_DisableDSP(THINKPAD_BD_DATA * pBDData) } =20 =20 -int tp3780I_ResetDSP(THINKPAD_BD_DATA * pBDData) +int tp3780I_ResetDSP(struct thinkpad_bd_data *pBDData) { - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; + struct dsp_3780i_config_settings *pSettings =3D &pBDData->rDspSettings; =20 if (dsp3780I_Reset(pSettings) =3D=3D 0) { EnableSRAM(pBDData); @@ -381,9 +381,9 @@ int tp3780I_ResetDSP(THINKPAD_BD_DATA * pBDData) } =20 =20 -int tp3780I_StartDSP(THINKPAD_BD_DATA * pBDData) +int tp3780I_StartDSP(struct thinkpad_bd_data *pBDData) { - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; + struct dsp_3780i_config_settings *pSettings =3D &pBDData->rDspSettings; =20 if (dsp3780I_Run(pSettings) =3D=3D 0) { // @BUG @TBD EnableSRAM(pBDData); @@ -395,7 +395,7 @@ int tp3780I_StartDSP(THINKPAD_BD_DATA * pBDData) } =20 =20 -int tp3780I_QueryAbilities(THINKPAD_BD_DATA * pBDData, MW_ABILITIES * pAbi= lities) +int tp3780I_QueryAbilities(struct thinkpad_bd_data *pBDData, struct mw_abi= lities *pAbilities) { memset(pAbilities, 0, sizeof(*pAbilities)); /* fill out standard constant fields */ @@ -424,11 +424,11 @@ int tp3780I_QueryAbilities(THINKPAD_BD_DATA * pBDData= , MW_ABILITIES * pAbilities return 0; } =20 -int tp3780I_ReadWriteDspDStore(THINKPAD_BD_DATA * pBDData, unsigned int uO= pcode, +int tp3780I_ReadWriteDspDStore(struct thinkpad_bd_data *pBDData, unsigned = int uOpcode, void __user *pvBuffer, unsigned int uCount, unsigned long ulDSPAddr) { - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; + struct dsp_3780i_config_settings *pSettings =3D &pBDData->rDspSettings; unsigned short usDspBaseIO =3D pSettings->usDspBaseIO; bool bRC =3D 0; =20 @@ -452,11 +452,11 @@ int tp3780I_ReadWriteDspDStore(THINKPAD_BD_DATA * pBD= Data, unsigned int uOpcode, } =20 =20 -int tp3780I_ReadWriteDspIStore(THINKPAD_BD_DATA * pBDData, unsigned int uO= pcode, +int tp3780I_ReadWriteDspIStore(struct thinkpad_bd_data *pBDData, unsigned = int uOpcode, void __user *pvBuffer, unsigned int uCount, unsigned long ulDSPAddr) { - DSP_3780I_CONFIG_SETTINGS *pSettings =3D &pBDData->rDspSettings; + struct dsp_3780i_config_settings *pSettings =3D &pBDData->rDspSettings; unsigned short usDspBaseIO =3D pSettings->usDspBaseIO; bool bRC =3D 0; =20 diff --git a/drivers/char/mwave/tp3780i.h b/drivers/char/mwave/tp3780i.h index 8bd976d42fae..c0001a344741 100644 --- a/drivers/char/mwave/tp3780i.h +++ b/drivers/char/mwave/tp3780i.h @@ -75,27 +75,27 @@ #define TP_CFG_PllBypass 0 /* don't bypass */ #define TP_CFG_ChipletEnable 0xFFFF /* Enable all chiplets */ =20 -typedef struct { +struct thinkpad_bd_data { int bDSPEnabled; int bShareDspIrq; int bShareUartIrq; - DSP_3780I_CONFIG_SETTINGS rDspSettings; -} THINKPAD_BD_DATA; + struct dsp_3780i_config_settings rDspSettings; +}; =20 -int tp3780I_InitializeBoardData(THINKPAD_BD_DATA * pBDData); -int tp3780I_CalcResources(THINKPAD_BD_DATA * pBDData); -int tp3780I_ClaimResources(THINKPAD_BD_DATA * pBDData); -int tp3780I_ReleaseResources(THINKPAD_BD_DATA * pBDData); -int tp3780I_EnableDSP(THINKPAD_BD_DATA * pBDData); -int tp3780I_DisableDSP(THINKPAD_BD_DATA * pBDData); -int tp3780I_ResetDSP(THINKPAD_BD_DATA * pBDData); -int tp3780I_StartDSP(THINKPAD_BD_DATA * pBDData); -int tp3780I_QueryAbilities(THINKPAD_BD_DATA * pBDData, MW_ABILITIES * pAbi= lities); -void tp3780I_Cleanup(THINKPAD_BD_DATA *pBDData); -int tp3780I_ReadWriteDspDStore(THINKPAD_BD_DATA * pBDData, unsigned int uO= pcode, +int tp3780I_InitializeBoardData(struct thinkpad_bd_data *pBDData); +int tp3780I_CalcResources(struct thinkpad_bd_data *pBDData); +int tp3780I_ClaimResources(struct thinkpad_bd_data *pBDData); +int tp3780I_ReleaseResources(struct thinkpad_bd_data *pBDData); +int tp3780I_EnableDSP(struct thinkpad_bd_data *pBDData); +int tp3780I_DisableDSP(struct thinkpad_bd_data *pBDData); +int tp3780I_ResetDSP(struct thinkpad_bd_data *pBDData); +int tp3780I_StartDSP(struct thinkpad_bd_data *pBDData); +int tp3780I_QueryAbilities(struct thinkpad_bd_data *pBDData, struct mw_abi= lities *pAbilities); +void tp3780I_Cleanup(struct thinkpad_bd_data *pBDData); +int tp3780I_ReadWriteDspDStore(struct thinkpad_bd_data *pBDData, unsigned = int uOpcode, void __user *pvBuffer, unsigned int uCount, unsigned long ulDSPAddr); -int tp3780I_ReadWriteDspIStore(THINKPAD_BD_DATA * pBDData, unsigned int uO= pcode, +int tp3780I_ReadWriteDspIStore(struct thinkpad_bd_data *pBDData, unsigned = int uOpcode, void __user *pvBuffer, unsigned int uCount, unsigned long ulDSPAddr); =20 --=20 2.51.1