From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D1098CF537D for ; Wed, 23 Oct 2024 12:42:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=fZrP0N4hv+TFz+SIF4ni1gagKIfHSezeTfzgo//YLIg=; b=HpLTHdqwRoesx9mmuuj+9IIp1t f/Q7LooqvXTXzhX8pIWRn309K1bwUs45suPUvJkQtlkR4DPX9pjWChXO9maN3YQP4s2uyCmtU8wMX 7pqB4H1CriiqINVbEPkxSAA/++Smc8RmPVNsVEv7X/kZRZK09wAfosM637eRUx/6mDZehqPfyIpk5 rxQCuthfOv5Psl8iDenA+k9BHTGNYH4K1YoJqoMwFR/s3cbEeQhry1CxM9csr5047FNsPIhcFZVe2 ilwTV2gqBNsTeOSJTrF5TM22ClKjCNMW1/TnJwCRWZgz1kgyVqn84ayywFuu+gcGngRb9sF9g6z6n UpJbTqZQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t3agY-0000000EM2R-2Ks7; Wed, 23 Oct 2024 12:41:58 +0000 Received: from nyc.source.kernel.org ([2604:1380:45d1:ec00::3]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t3af0-0000000ELsE-36sH; Wed, 23 Oct 2024 12:40:24 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by nyc.source.kernel.org (Postfix) with ESMTP id 828AFA44E40; Wed, 23 Oct 2024 12:40:12 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 957E3C4CEC6; Wed, 23 Oct 2024 12:40:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1729687221; bh=dzqN+awO8TcOkYgyl6lEgGX9CYpgV1GK+LsfppVV8w4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=G01LHK/vpGe0lTSOwUHtuMKExgS4os+O+zy89jYCbdgWExVshtObrm0N6Icqhkl+H TdoU04T7Yi5eo5p6jdcc7emXRa4zVPv4vMBnvBwKdedNuusHFYl0x5EgLDBZUsMjve I05pBGjEj2ueFiMMKKHrSOY5cJrY4cfp+/tRBCIjpg7NOIYEKqQI9sk9aLLIzov5w7 1zgrlF7BCFFU8ZgKMLKXNwiNZA2MWMs1xZk2WX4ILgR0Olb2gnfo0ladgTk0Idv2sx Xleqrvsv2wXC6dp71tv5sbiEBgtpqy9QhFfybS6J36EmJxQUKTf253zMIuVlmAXXUK KgqSETdRR9clg== Date: Wed, 23 Oct 2024 13:40:14 +0100 From: Simon Horman To: Jinjian Song Cc: chandrashekar.devegowda@intel.com, chiranjeevi.rapolu@linux.intel.com, haijun.liu@mediatek.com, m.chetan.kumar@linux.intel.com, ricardo.martinez@linux.intel.com, loic.poulain@linaro.org, ryazanov.s.a@gmail.com, johannes@sipsolutions.net, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-doc@vger.kernel.org, angelogioacchino.delregno@collabora.com, linux-arm-kernel@lists.infradead.org, matthias.bgg@gmail.com, corbet@lwn.net, linux-mediatek@lists.infradead.org, helgaas@kernel.org, danielwinkler@google.com, korneld@google.com, Jinjian Song , Jiri Pirko Subject: Re: [net-next,RESEND v6 2/2] net: wwan: t7xx: Add debug port Message-ID: <20241023124014.GU402847@kernel.org> References: <20241021121934.16317-1-jinjian.song@fibocom.com> <20241021121934.16317-3-jinjian.song@fibocom.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241021121934.16317-3-jinjian.song@fibocom.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241023_054022_863125_5C846940 X-CRM114-Status: GOOD ( 11.92 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org + Jiri On Mon, Oct 21, 2024 at 08:19:34PM +0800, Jinjian Song wrote: > From: Jinjian Song > > Add support for userspace to switch on the debug port(ADB,MIPC). > - ADB port: /dev/wwan0adb0 > - MIPC port: /dev/wwan0mipc0 > > Application can use ADB (Android Debg Bridge) port to implement > functions (shell, pull, push ...) by ADB protocol commands. > E.g., ADB commands: > - A_OPEN: OPEN(local-id, 0, "destination") > - A_WRTE: WRITE(local-id, remote-id, "data") > - A_OKEY: READY(local-id, remote-id, "") > - A_CLSE: CLOSE(local-id, remote-id, "") > > Link: https://android.googlesource.com/platform/packages/modules/adb/+/refs/heads/main/README.md > > Application can use MIPC (Modem Information Process Center) port > to debug antenna tunner or noise profiling through this MTK modem > diagnostic interface. > > By default, debug ports are not exposed, so using the command > to enable or disable debug ports. > > Switch on debug port: > - debug: 'echo debug > /sys/bus/pci/devices/${bdf}/t7xx_mode > > Switch off debug port: > - normal: 'echo normal > /sys/bus/pci/devices/${bdf}/t7xx_mode Hi, I am somewhat surprised to see vendor-specific sysfs controls being added. And I am wondering if another mechanism was considered. It seems to me that devlink would be appropriate. Jiri (CCed) may have an opinion on that. ... From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 9BD7DCFA456 for ; Thu, 24 Oct 2024 10:11:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version: Content-Transfer-Encoding:Content-Type:References:In-Reply-To:Message-Id:Date :Subject:Cc:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=BEc7dEDgeqoTXaskQEsftjNfq2L5HwDIQUhIchBi/FA=; b=lTHqbpRx7OvmNu/OaVxzAi69v5 v1ZF6YXOIwKmsLjMBkFpdJ10xrcaRiGgE3zpjU9+SQUa7f7+MBVed9wMfjXh6NV8yDKl0SM117J0U mDkLq5IF0LZ0TMptmJesj4+SsdUtrGlDo3BLNIgAwmwggpdneKCFAzbbT9ZIsHEnnWH4rwjsjhmwC 8BKXo+i0CKdtdDyWRZmSU/bYxr9UQ5fsJxrw8R+r4VHbNEYKjfuQ1MjIl9Tj/9EMxdGA8r51Qvgsi 8OOmzEa7L1SWCKN7Ldkrm3rqR9zwgQXwhWzgAFVw9k1HOzflgvxYQgdb+aPVC7hAc/oZmPrYP4GaK LDuqzS2w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t3uoK-0000000017Z-22dz; Thu, 24 Oct 2024 10:11:20 +0000 Received: from mail-psaapc01on20706.outbound.protection.outlook.com ([2a01:111:f403:200e::706] helo=APC01-PSA-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t3ugp-0000000Hb6N-0GWU; Thu, 24 Oct 2024 10:03:45 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=DV8c0fiNlq1x2OFzilV58weIE7TEaSz0AsegkPsV85wmSAcl8Lmt2VZEzt3+PgwUKRuzn5Lb3hLYaiMjPdSWAlrCxA6AqQMEp7ZwUknnJEaJ/W82N3oEFgl1wWGAVNFnfvQOt0xYLZBRYNxjsInt5IeIubB0xvaRHW5p6EXXbBn9BMfPhZoX4fXjUEo4R6Su2IHj9wu0iyRrHpNUcoOwc4PvpqgOIaTLKp5nOfCZ0vbvgz5i3JZGtHY5Tp6Sh+h+0dfhSlwlA9t9PrZxzC3h0kbFe21l8nNe0bcIc0CaDf6/qjbcDXt+UX6SVUjC6Y4wyt4QHJZImY4IQQ4dZlVPEw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=BEc7dEDgeqoTXaskQEsftjNfq2L5HwDIQUhIchBi/FA=; b=VfsVDZPk363nqsIQm9ae1RKgoDpc6ucwJOIB3htLhQP06tHNaGH0U0LJgEFpiW7N4EvF9QEoErKk4aj46UwYyPT8ZqAGEyd3+Kwf6DfoLL7yTYOB5bIzd8e9TXliyhMkavYoWNPI1ItgP8R9pINItCzN4Kev5SKTLkffEdoOub69VZ+2a0PDXDh8TZxqFGA8CBSrYy16X7RmyVn1M03Imx5laWNvxmSwQhP2I7eRpMmdTRnEey6sbVv3RrjPjBQwWCBUS3mdehi9ksxZyH+yJIK4lJnl5p/ddVaK2DB7WSPzOh04jUd8hGKWBID5CdLUZ++iPczHbWTKyQCDpNSY9Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=fibocom.com; dmarc=pass action=none header.from=fibocom.com; dkim=pass header.d=fibocom.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fibocomcorp.onmicrosoft.com; s=selector1-fibocomcorp-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=BEc7dEDgeqoTXaskQEsftjNfq2L5HwDIQUhIchBi/FA=; b=XQI83o4OzKD+qDXUOdq5tPFQ2vRCK/8MGj4jX6KJs/MIN10bo0S8lA3DgPRci5DOD7JnU3Rpwf4JFmheAvEHkWDmBmJlj/Ib19QE1gMMto3f5wAsAjiyn/RK5As9vT1Ohi/heP7WEeDSu7UM0rNx2+Ji42CjNWqIPgJBm4suWMo= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=fibocom.com; Received: from TY0PR02MB5766.apcprd02.prod.outlook.com (2603:1096:400:1b5::6) by TYZPR02MB7411.apcprd02.prod.outlook.com (2603:1096:405:4a::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8093.17; Thu, 24 Oct 2024 10:03:24 +0000 Received: from TY0PR02MB5766.apcprd02.prod.outlook.com ([fe80::f53d:47b:3b04:9a8b]) by TY0PR02MB5766.apcprd02.prod.outlook.com ([fe80::f53d:47b:3b04:9a8b%4]) with mapi id 15.20.8069.027; Thu, 24 Oct 2024 10:03:23 +0000 From: Jinjian Song To: horms@kernel.org Cc: angelogioacchino.delregno@collabora.com, chandrashekar.devegowda@intel.com, chiranjeevi.rapolu@linux.intel.com, corbet@lwn.net, danielwinkler@google.com, davem@davemloft.net, edumazet@google.com, haijun.liu@mediatek.com, helgaas@kernel.org, jinjian.song@fibocom.com, jiri@resnulli.us, johannes@sipsolutions.net, korneld@google.com, kuba@kernel.org, linux-arm-kernel@lists.infradead.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mediatek@lists.infradead.org, loic.poulain@linaro.org, m.chetan.kumar@linux.intel.com, matthias.bgg@gmail.com, netdev@vger.kernel.org, pabeni@redhat.com, ricardo.martinez@linux.intel.com, ryazanov.s.a@gmail.com, songjinjian@hotmail.com Subject: Re: [net-next,RESEND v6 2/2] net: wwan: t7xx: Add debug port Date: Thu, 24 Oct 2024 18:02:56 +0800 Message-ID: <20241023124014.GU402847@kernel.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20241023124014.GU402847@kernel.org> References: <20241021121934.16317-1-jinjian.song@fibocom.com> <20241021121934.16317-3-jinjian.song@fibocom.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: 8bit X-ClientProxiedBy: SG2PR02CA0131.apcprd02.prod.outlook.com (2603:1096:4:188::6) To TY0PR02MB5766.apcprd02.prod.outlook.com (2603:1096:400:1b5::6) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: TY0PR02MB5766:EE_|TYZPR02MB7411:EE_ X-MS-Office365-Filtering-Correlation-Id: 72f22fe9-09f5-4b9c-d46c-08dcf41317d1 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|4022899009|1800799024|52116014|7416014|376014|366016|38350700014; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?enOZJi4ty58b9aR7VyJL9A+N0LIb6b2B4784zQPVkzUwE0lUCXO2rN+6b7sN?= =?us-ascii?Q?BEZjF1lp7Fb0K0UCx6d1gTK3jRAcLAsw81tnBfJxTyAy77cbMaXgxMc49qnc?= =?us-ascii?Q?DGJYSRGLfiMU3O8IBjnBtHfUaRhYbEKO85lcED8wqrvV2QqA3lSPS/pzsscD?= =?us-ascii?Q?eE48+25umLovwdwvdz+37DS1m8BROwc+g4IOB+z6Z8oZ4HE3pAKOvcRdUdyA?= =?us-ascii?Q?LadkJn10YzuPlfWl/0HBR0+70BMJD7reehDbqBDa5s+W1tChvtSqdAHQ7jaL?= =?us-ascii?Q?Wm85gG7moxjblwG/xl23P4nlV285sTofnBaURjMnRAQsj6G1DZxhbuDoEbxT?= =?us-ascii?Q?1kz6cVMmuWkds29HX9EeGGpbY06sMHHWrPNnOC7Q+EAGilXqB44Odviixzi4?= =?us-ascii?Q?jtdLN8KEigT8zhYd2RsVbp1BOoTs/Kugr1G32SgvY6o4u9LhnUjQYscK/Uo0?= =?us-ascii?Q?HOFqKErFibfrCwIz9JCEF/+kdlft3dI7s8ULLeH1N4xcQH5MLb55dr6cWBYv?= =?us-ascii?Q?AH+tTVU2AeGQj3XTSCvRjzcgF9auJW1HVGuI6bo9t4dAPiplJ0CG1uJxbjKZ?= =?us-ascii?Q?0UQW7jyx7EpL3Nu+KE3ff4kZ2HNuLpEAEJw/dwTERH8zb3kYfN2jWfcULF3a?= =?us-ascii?Q?81WnZkVkBDS0Dr5HKPKbSMmZNKgPmmC9N7godOg2cnxrJVrvG+CUJM4fHpmD?= =?us-ascii?Q?0RXTthEDnt+708MVPk2woNM4NhkfajRB6PFvF03g9O5s28Y9nPSQMfqVl6PA?= =?us-ascii?Q?ctYN8HO9rEO5UIp5fDXLxaJh18gxk0cyEe8PKfNGtgbe3Qc0MruhkSnXDklZ?= =?us-ascii?Q?9tEFkBwsmgabUFZ3MmheoaqMkTQqEcKvkAWnKqqBQiI8LVdY/WV3uCejjJc6?= =?us-ascii?Q?JgFBsNdCK13Wcoc2m+K9z5NaV6wt7egbsKNfWnW1ekdhL+nIkWpxoCQvVp9A?= =?us-ascii?Q?wZKj9sJWgyYTOxYt9LSJIitp1U+qpTVBB+9+9QViDjAsssYe84PDUHOo74qJ?= =?us-ascii?Q?6ri9NqxRJ6Et064FF8/wlFAoHXijFEWCeOA6+edzYH6fklsN8RY3FVzmg93a?= =?us-ascii?Q?RTPknld9XKW7695hoCTbffv0YFAAd3Kz+cnGAZqkQ3pOlPvGNnGh1sDNXYRj?= =?us-ascii?Q?achzlmCC6chT8jSIlhqgCchAxfMRZMYchZhnD6wml8lPplj4JwW58SSCs5ZD?= =?us-ascii?Q?cpx/XB2dy7DlxQZpRR3bJULhMbWMxTpqBIycYn4v/PB9erX8GOhqVj3dtckV?= =?us-ascii?Q?fRI1KbUnGPUgt4JX/jiSUv8pgb6vUP5uYtqHGhZMGPRueQKM0egqbPo+qEKb?= =?us-ascii?Q?uqVLALMoCA0Il5WybeH6+mFj?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:TY0PR02MB5766.apcprd02.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(4022899009)(1800799024)(52116014)(7416014)(376014)(366016)(38350700014);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?Rh0mOwRmaUSoXlOTPOSsrsas1j0WZIiF5WdUONZ09UyNBW5cLCw1cuQZt85t?= =?us-ascii?Q?S72N9e8TmbqIjRowc/2BrkDwo0TNJPWr7jzidj0QB/wdUegs7uL552vjtGPH?= =?us-ascii?Q?siVHpnMW/jfQZyvZmb2Jhsqi7hlUhKHMrQsSwu04sSET6lRxxD/6tTQD5dsc?= =?us-ascii?Q?mNGINMFXcsYZh8pyzt8VWEqNpl5/U5KRSbQIa7TcA0uiDdCJ8d3EZc4Wl0ai?= =?us-ascii?Q?tjh3s00yEzuyfsiR8SWpqSxYlXtRGF+AtRxdUovURODWFOWtfKe7gZz8ngtu?= =?us-ascii?Q?yTt4iix5W4MVyoT25QS8hebAWLDBEFB+bU2aV9P9LOeQx/L+OiN3NJlUVIpV?= =?us-ascii?Q?yhbTWtqFl1en21qvM3SN6RcOp8+VUoi4EmHWIAhVlXIm/DPK4blRlxsHOkZE?= =?us-ascii?Q?oxrg1+8tnJkQmeLbn3YofFSJTlwci55hVogoji9fxXEwm9gNMmeDzVxNKeoK?= =?us-ascii?Q?SKm/BH+hVIzWHhnLuRER+0qAsHy/SjiqoEGUo9hh4dUCZtFl+EOHunyyF1LD?= =?us-ascii?Q?Ym7kCYWAOXgQswTiyIYp4yaxn6aZkAnA1rGLvGIUocGWqmm3ogNDYroQMr+C?= =?us-ascii?Q?OOZObxIugJ88U+FjxCT0EfgToYYfbwu919FCc0zLib+gvVin88r+ozpR/Mbx?= =?us-ascii?Q?QLC88U86BfVobksctdI9mhfSM04GmzR7p5RBwgMVXGswYWY8i8IUf0nJejjT?= =?us-ascii?Q?yAqL8jfznkl4AU0P4TteB0gth3kLKVPjs2asU7DoI4fbfTT4ajOj6qEnsw6/?= =?us-ascii?Q?TJw/86mpo94rODi3ktHQSHipVJYcaRxebDKtGFnErdA2p+B8OeR+1oYODlpX?= =?us-ascii?Q?EQQryVYA82tgvV2s5u9M/kGUp6tQ4MuKTpYWD9Q36vOCPqWpllZWgVyDTmbN?= =?us-ascii?Q?oYyI/0koV5jnCvp93CtW0gh9ifUASdYaaDYHohIgJrmREzR0imRFPVORorlf?= =?us-ascii?Q?nGWtEDVmrt/38E+8OPoV0jrB8HMB43ROe/tpFD5pAxaGbFkrw1pUWxXWKuoS?= =?us-ascii?Q?N0iIqqfxHVid4fV/LT/gX7Ze9fiJf/HRyypujIouxMy2zQHiA4HMaMAu6+Jd?= =?us-ascii?Q?tYs4OfR1vOeHmSA9BEZsu3bAUyXbmUEsggOMjESk6iS+umkmrOBb53GRmn+o?= =?us-ascii?Q?DPGd/vMCgQe0pXJSlKv4Vx5zdv4pWgC3e7QPILsMF1muYgKf2ezT/4LGX5fU?= =?us-ascii?Q?22f3Iy00qSww2LrJqGzmvRVw+vICdGuhDU5bNxhAvu5EhO80J8ShRNicVZ63?= =?us-ascii?Q?B7O8qA2KEy9tJOFOOBzihHq31yJUITihkK+Ve/oRrMfdw84Uol6q+/DZJXmv?= =?us-ascii?Q?G/N7hDh2oAV8ypkoPc1rPwG8IaD1BUZE9f97uWuJegp93vGmtp9F3uFTi7YI?= =?us-ascii?Q?KCl+R/RYK7HmWv5siInk+HXULzHJW0/XFD6ljmrbeLceHMc6Hz5WInRqAU35?= =?us-ascii?Q?yX79lTiUcgZP9VF/RLBh1hlIpIxzGcYt9gmX2DM6QCpS24DG8mdQZWj5L5bo?= =?us-ascii?Q?DemYTBTXf98Tq3zqQi06feel1c8uHhlUfLzHQT8KTSVOQuzD+qDyW3YncUbw?= =?us-ascii?Q?Yi+79++AizygTy7J2wlm3VIq0l7yM+9udXK8ZxS1YBqmLqpAd/br/XkH1Yye?= =?us-ascii?Q?QA=3D=3D?= X-OriginatorOrg: fibocom.com X-MS-Exchange-CrossTenant-Network-Message-Id: 72f22fe9-09f5-4b9c-d46c-08dcf41317d1 X-MS-Exchange-CrossTenant-AuthSource: TY0PR02MB5766.apcprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Oct 2024 10:03:23.5542 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 889bfe61-8c21-436b-bc07-3908050c8236 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: hrs/zmMwtUiViCFPPfF1abctidKl47zdZPGrqyjSuMJf+l4NgkmT7hz8fJxttAaYD/xNdOFcSGf83JBlhLOiQ7ewAhYjELgIK/+0RuNIWWk= X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYZPR02MB7411 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241024_030335_154298_F06C59B4 X-CRM114-Status: GOOD ( 12.17 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Message-ID: <20241024100256.lHPxLjkWIbQ7-qf91X148kDW-ztEGYcYkVvgN19QxIY@z> >+ Jiri > >On Mon, Oct 21, 2024 at 08:19:34PM +0800, Jinjian Song wrote: >> From: Jinjian Song >> >> Add support for userspace to switch on the debug port(ADB,MIPC). >> - ADB port: /dev/wwan0adb0 >> - MIPC port: /dev/wwan0mipc0 >> >> Application can use ADB (Android Debg Bridge) port to implement >> functions (shell, pull, push ...) by ADB protocol commands. >> E.g., ADB commands: >> - A_OPEN: OPEN(local-id, 0, "destination") >> - A_WRTE: WRITE(local-id, remote-id, "data") >> - A_OKEY: READY(local-id, remote-id, "") >> - A_CLSE: CLOSE(local-id, remote-id, "") >> >> Link: https://android.googlesource.com/platform/packages/modules/adb/+/refs/heads/main/README.md >> >> Application can use MIPC (Modem Information Process Center) port >> to debug antenna tunner or noise profiling through this MTK modem >> diagnostic interface. >> >> By default, debug ports are not exposed, so using the command >> to enable or disable debug ports. >> >> Switch on debug port: >> - debug: 'echo debug > /sys/bus/pci/devices/${bdf}/t7xx_mode >> >> Switch off debug port: >> - normal: 'echo normal > /sys/bus/pci/devices/${bdf}/t7xx_mode > >Hi, > >I am somewhat surprised to see vendor-specific sysfs controls being added. >And I am wondering if another mechanism was considered. It seems to >me that devlink would be appropriate. Jiri (CCed) may have an opinion on >that. > >... Hi Jiri, T7XX is MTK WWAN device platform, ADB and MIPC channel is common in all MTK platform, the ports used to debug, and we need a way to create the channels to usespace, so use the sysfs to trigger that. The previous plan consider using devlink framework to set param to create ports inside t7xx driver, but later after communicating with Loic and Jiri , it was replaced with the current plan. https://lore.kernel.org/all/CAMZdPi-qZ3JjZmEAtEmJETNzKd+k6UcLnLkM0MZoSZ1hKaOXuA@mail.gmail.com/ Best Regards, Jinjian,