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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id A4766C77B70 for ; Mon, 17 Apr 2023 06:12:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230092AbjDQGMn (ORCPT ); Mon, 17 Apr 2023 02:12:43 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49134 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229519AbjDQGMl (ORCPT ); Mon, 17 Apr 2023 02:12:41 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 633692116; Sun, 16 Apr 2023 23:12:40 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id F2C97610A0; Mon, 17 Apr 2023 06:12:39 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id ED16CC433EF; Mon, 17 Apr 2023 06:12:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1681711959; bh=9+bae1oGpumyWoxfJVr4FvUHUo2hifL7pYORTIPZ5lY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=XHFNHE3w23C46xvz4Ln3qBh5fKn6g/XVa7NKO8wmO6GQ3NGFZmyNNYkDO7cF40s3c ZVwbDeA9umPcoAzPaYQkoPZbY7XgGfI7gzQuetbN0Mg0OdznWW5Tymhi8KcEzwQiKC Y7jInNrxzIaSEZZJTUVlb3U7rhmNVvgQIKmDtE9g= Date: Mon, 17 Apr 2023 08:12:36 +0200 From: Greg KH To: Rajat Khandelwal Cc: heikki.krogerus@linux.intel.com, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] usb: typec: intel_pmc_mux: Expose IOM port status to debugfs Message-ID: References: <20230414081910.1336405-1-rajat.khandelwal@linux.intel.com> <2c960f0c-5cbb-4c2d-07cb-dafd94d22414@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <2c960f0c-5cbb-4c2d-07cb-dafd94d22414@linux.intel.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Apr 17, 2023 at 11:28:18AM +0530, Rajat Khandelwal wrote: > Hi, > > On 4/15/2023 11:01 AM, Greg KH wrote: > > On Fri, Apr 14, 2023 at 01:49:10PM +0530, Rajat Khandelwal wrote: > > > IOM status has a crucial role during debugging to check the > > > current state of the type-C port. > > > There are ways to fetch the status, but all those require the > > > IOM port status offset, which could change with platform. > > > > > > Make a debugfs directory for intel_pmc_mux and expose the status > > > under it per port basis. > > > > > > Signed-off-by: Rajat Khandelwal > > > --- > > > > > > v2: > > > 1. Remove static declaration of the debugfs root for 'intel_pmc_mux' > > > 2. Remove explicitly defined one-liner functions > > > > > > drivers/usb/typec/mux/intel_pmc_mux.c | 34 +++++++++++++++++++++++++++ > > > 1 file changed, 34 insertions(+) > > > > > > diff --git a/drivers/usb/typec/mux/intel_pmc_mux.c b/drivers/usb/typec/mux/intel_pmc_mux.c > > > index 34e4188a40ff..1d43b111781e 100644 > > > --- a/drivers/usb/typec/mux/intel_pmc_mux.c > > > +++ b/drivers/usb/typec/mux/intel_pmc_mux.c > > > @@ -15,6 +15,7 @@ > > > #include > > > #include > > > #include > > > +#include > > > #include > > > @@ -639,9 +640,34 @@ static int pmc_usb_probe_iom(struct pmc_usb *pmc) > > > return 0; > > > } > > > +static int port_iom_status_show(struct seq_file *s, void *unused) > > > +{ > > > + struct pmc_usb_port *port = s->private; > > > + > > > + update_port_status(port); > > > + seq_printf(s, "0x%08x\n", port->iom_status); > > > + > > > + return 0; > > > +} > > > +DEFINE_SHOW_ATTRIBUTE(port_iom_status); > > > + > > > +static void pmc_mux_port_debugfs_init(struct pmc_usb_port *port, > > > + struct dentry *pmc_mux_debugfs_root) > > > +{ > > > + struct dentry *debugfs_dir; > > > + char name[6]; > > > + > > > + snprintf(name, sizeof(name), "port%d", port->usb3_port - 1); > > > + > > > + debugfs_dir = debugfs_create_dir(name, pmc_mux_debugfs_root); > > > + debugfs_create_file("iom_status", 0400, debugfs_dir, port, > > > + &port_iom_status_fops); > > > +} > > > + > > > static int pmc_usb_probe(struct platform_device *pdev) > > > { > > > struct fwnode_handle *fwnode = NULL; > > > + struct dentry *pmc_mux_debugfs_root; > > > struct pmc_usb *pmc; > > > int i = 0; > > > int ret; > > > @@ -674,6 +700,8 @@ static int pmc_usb_probe(struct platform_device *pdev) > > > if (ret) > > > return ret; > > > + pmc_mux_debugfs_root = debugfs_create_dir("intel_pmc_mux", NULL); > > What happens when you have more than one device in the system at the > > same time? > > I'm sorry I didn't understand the question. We would have separate directories > for all the ports which would contain the 'iom_status' file, thus representing > status for all the ports individually. > Can you rephrase the question since I guess you had something else in mind? Can you please show the output of the directory /sys/kernel/debug/intel_pmc_mux/ with multiple pmc devices in the system at the same time? This code seems to want to create that directory for every platform device that matches this signature, so that implies you can not have more than one of them at a time, which is not good and an artificial restriction you are placing on the driver. thanks, greg k-h