From: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
To: Chunfeng Yun <chunfeng.yun@mediatek.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Felipe Balbi <balbi@kernel.org>
Cc: Peter Chen <Peter.Chen@nxp.com>,
Alexandre Belloni <alexandre.belloni@bootlin.com>,
"Gustavo A. R. Silva" <gustavo@embeddedor.com>,
Yangtao Li <tiny.windzz@gmail.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Minas Harutyunyan <hminas@synopsys.com>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Fabio Estevam <festevam@gmail.com>,
Florian Fainelli <f.fainelli@gmail.com>,
Robert Jarzmik <robert.jarzmik@free.fr>,
Kevin Cernekee <cernekee@gmail.com>,
Ludovic Desroches <ludovic.desroches@microchip.com>,
"bcm-kernel-feedback-list@broadcom.com"
<bcm-kernel-feedback-list@broadcom.com>,
NXP Linux Team <linux-imx@nxp.com>,
Cristian Birsan <cristian.birsan@microchip.com>,
"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>,
Fabrizio Castro <fabrizio.castro@bp.renesas.com>,
Sascha Hauer <s.hauer@pengutronix.de>,
Haojian Zhuang <haojian.zhuang@gmail.com>,
Stephen Boyd <swboyd@chromium.org>,
"linux-mediatek@lists.infradead.org"
<linux-mediatek@lists.infradead.org>,
Matthias Brugger <matthias.bgg@gmail.com>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Bin Liu <b-liu@ti.com>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Biju Das <biju.das@bp.renesas.com>,
"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
Nicolas Ferre <nicolas.ferre@microchip.com>,
Pengutronix Kernel Team <kernel@pengutronix.de>,
Colin Ian King <colin.king@canonical.com>,
Shawn Guo <shawnguo@kernel.org>, Daniel Mack <daniel@zonque.org>
Subject: RE: [PATCH v3 10/13] usb: gadget: udc: renesas_usb3: create debugfs directory under usb root
Date: Fri, 15 Nov 2019 05:33:24 +0000 [thread overview]
Message-ID: <TYAPR01MB454479F91B948904C213B94CD8700@TYAPR01MB4544.jpnprd01.prod.outlook.com> (raw)
In-Reply-To: <1573728085-29016-10-git-send-email-chunfeng.yun@mediatek.com>
Hi Chunfeng-san,
> From: Chunfeng Yun, Sent: Thursday, November 14, 2019 7:41 PM
>
> Now the USB gadget subsystem can use the USB debugfs root directory,
> so move it's directory from the root of the debugfs filesystem into
> the root of usb
>
> Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com>
Thank you for the patch! When I apply this patch on the Greg's usb.git /
usb-next branch, the following warning happens:
$ patch -p1 < ~/Downloads/v3-10-13-usb-gadget-udc-renesas_usb3-create-debugfs-directory-under-usb-root.patch
patching file drivers/usb/gadget/udc/renesas_usb3.c
Hunk #1 succeeded at 2536 (offset 1 line).
I'm not sure we should fix it strictly though, this patch seems good to me. So,
Reviewed-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
And, I tested this patch on my environment and confirmed that
the debug file moved from /sys/kernel/debug to /sys/kernel/debug/usb. So,
Tested-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Best regards,
Yoshihiro Shimoda
> ---
> v3: no changes
>
> v2:
> 1. abandon new API usb_debugfs_create_dir(), and use usb_debug_root
> ---
> drivers/usb/gadget/udc/renesas_usb3.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/usb/gadget/udc/renesas_usb3.c b/drivers/usb/gadget/udc/renesas_usb3.c
> index e098f16c01cb..4dde69e7f6b1 100644
> --- a/drivers/usb/gadget/udc/renesas_usb3.c
> +++ b/drivers/usb/gadget/udc/renesas_usb3.c
> @@ -2535,7 +2535,7 @@ static const struct file_operations renesas_usb3_b_device_fops = {
> static void renesas_usb3_debugfs_init(struct renesas_usb3 *usb3,
> struct device *dev)
> {
> - usb3->dentry = debugfs_create_dir(dev_name(dev), NULL);
> + usb3->dentry = debugfs_create_dir(dev_name(dev), usb_debug_root);
>
> debugfs_create_file("b_device", 0644, usb3->dentry, usb3,
> &renesas_usb3_b_device_fops);
> --
> 2.23.0
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
next prev parent reply other threads:[~2019-11-15 5:33 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-14 10:41 [PATCH v3 01/13] media: uvcvideo: drop error check of debugfs_create_dir() Chunfeng Yun
2019-11-14 10:41 ` [PATCH v3 02/13] usb: chipidea: debug: create debugfs directory under usb root Chunfeng Yun
2019-11-14 10:41 ` [PATCH v3 03/13] usb: host: imx21: " Chunfeng Yun
2019-11-14 10:41 ` [PATCH v3 04/13] usb: dwc3: " Chunfeng Yun
2019-11-14 10:41 ` [PATCH v3 05/13] usb: musb: " Chunfeng Yun
2019-11-14 10:41 ` [PATCH v3 06/13] usb: musb: dsps: " Chunfeng Yun
2019-11-14 10:41 ` [PATCH v3 07/13] usb: dwc2: " Chunfeng Yun
2019-11-14 10:41 ` [PATCH v3 08/13] usb: gadget: bcm63xx_udc: " Chunfeng Yun
2019-11-14 10:41 ` [PATCH v3 09/13] usb: gadget: atmel: " Chunfeng Yun
2019-11-14 10:41 ` [PATCH v3 10/13] usb: gadget: udc: renesas_usb3: " Chunfeng Yun
2019-11-15 5:33 ` Yoshihiro Shimoda [this message]
2019-11-14 10:41 ` [PATCH v3 11/13] usb: gadget: pxa27x: " Chunfeng Yun
2019-11-14 10:41 ` [PATCH v3 12/13] usb: gadget: udc: gr_udc: " Chunfeng Yun
2019-11-14 10:41 ` [PATCH v3 13/13] usb: gadget: udc: s3c2410_udc: " Chunfeng Yun
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=TYAPR01MB454479F91B948904C213B94CD8700@TYAPR01MB4544.jpnprd01.prod.outlook.com \
--to=yoshihiro.shimoda.uh@renesas.com \
--cc=Peter.Chen@nxp.com \
--cc=alexandre.belloni@bootlin.com \
--cc=b-liu@ti.com \
--cc=balbi@kernel.org \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=biju.das@bp.renesas.com \
--cc=cernekee@gmail.com \
--cc=chunfeng.yun@mediatek.com \
--cc=colin.king@canonical.com \
--cc=cristian.birsan@microchip.com \
--cc=daniel@zonque.org \
--cc=f.fainelli@gmail.com \
--cc=fabrizio.castro@bp.renesas.com \
--cc=festevam@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=gustavo@embeddedor.com \
--cc=haojian.zhuang@gmail.com \
--cc=hminas@synopsys.com \
--cc=kernel@pengutronix.de \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-usb@vger.kernel.org \
--cc=ludovic.desroches@microchip.com \
--cc=matthias.bgg@gmail.com \
--cc=mchehab@kernel.org \
--cc=nicolas.ferre@microchip.com \
--cc=robert.jarzmik@free.fr \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
--cc=swboyd@chromium.org \
--cc=tiny.windzz@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).