From nobody Tue Dec 2 02:29:49 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) --- 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