From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH v2 1/1] spi: dw: Make debugfs name unique between instances Date: Wed, 4 Jan 2017 01:02:04 +0200 Message-ID: References: <1483419249-88956-1-git-send-email-preid@electromag.com.au> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: Mark Brown , kernel-NoMTM8u0nx3R7s880joybQ@public.gmane.org, linux-spi To: Phil Reid Return-path: In-Reply-To: <1483419249-88956-1-git-send-email-preid-qgqNFa1JUf/o2iN0hyhwsIdd74u8MsAO@public.gmane.org> Sender: linux-spi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: On Tue, Jan 3, 2017 at 6:54 AM, Phil Reid wrote: > Some system have multiple dw devices. Currently the driver uses a > fixed name for the debugfs dir. Append bus_num to the debugfs dir > name to make it unique. > Reviewed-by: Andy Shevchenko > Signed-off-by: Phil Reid > --- > > Notes: > Changes from v1: > - Reduce buffer size to 32. > - Use bus_num instead of device name. > > drivers/spi/spi-dw.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/spi/spi-dw.c b/drivers/spi/spi-dw.c > index 27960e4..77439ac 100644 > --- a/drivers/spi/spi-dw.c > +++ b/drivers/spi/spi-dw.c > @@ -107,7 +107,10 @@ static ssize_t dw_spi_show_regs(struct file *file, char __user *user_buf, > > static int dw_spi_debugfs_init(struct dw_spi *dws) > { > - dws->debugfs = debugfs_create_dir("dw_spi", NULL); > + char name[32]; > + > + snprintf(name, 32, "dw_spi%d", dws->master->bus_num); > + dws->debugfs = debugfs_create_dir(name, NULL); > if (!dws->debugfs) > return -ENOMEM; > > -- > 1.8.3.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-spi" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html