Message ID | 1603109998-7258-1-git-send-email-chunfeng.yun@mediatek.com |
---|---|
State | Deferred |
Delegated to: | Tom Rini |
Headers | show |
Series | [next] usb: mtu3: fix build error caused by dev_xxx() | expand |
On 10/19/20 2:19 PM, Chunfeng Yun wrote: > Due to the following to patches: > 4a1989c0bc77 ("dm: Don't undefine dev_xxx macros") > 69dae8902b16 ("linux/compat.h: Remove redefinition of dev_xxx macros") > > Need include device_compat.h and no need use __maybe_unused anymore > to fix unused variable warning > > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> > --- > Please squash in: ab703ebb1cc0 ("usb: add MediaTek USB3 DRD driver") > in u-boot-usb/next Applied, thanks. > Sorry for inconvenience Please make use of the CI next time.
On Tue, 2020-10-20 at 00:15 +0200, Marek Vasut wrote: > On 10/19/20 2:19 PM, Chunfeng Yun wrote: > > Due to the following to patches: > > 4a1989c0bc77 ("dm: Don't undefine dev_xxx macros") > > 69dae8902b16 ("linux/compat.h: Remove redefinition of dev_xxx macros") > > > > Need include device_compat.h and no need use __maybe_unused anymore > > to fix unused variable warning > > > > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> > > --- > > Please squash in: ab703ebb1cc0 ("usb: add MediaTek USB3 DRD driver") > > in u-boot-usb/next > > Applied, thanks. Thank you! > > > Sorry for inconvenience > > Please make use of the CI next time. Is there any reference about CI I can see?
Hi Chunfeng, On Tue, Oct 20, 2020 at 10:22 AM Chunfeng Yun <chunfeng.yun@mediatek.com> wrote: > > On Tue, 2020-10-20 at 00:15 +0200, Marek Vasut wrote: > > On 10/19/20 2:19 PM, Chunfeng Yun wrote: > > > Due to the following to patches: > > > 4a1989c0bc77 ("dm: Don't undefine dev_xxx macros") > > > 69dae8902b16 ("linux/compat.h: Remove redefinition of dev_xxx macros") > > > > > > Need include device_compat.h and no need use __maybe_unused anymore > > > to fix unused variable warning > > > > > > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> > > > --- > > > Please squash in: ab703ebb1cc0 ("usb: add MediaTek USB3 DRD driver") > > > in u-boot-usb/next > > > > Applied, thanks. > Thank you! > > > > > > Sorry for inconvenience > > > > Please make use of the CI next time. > Is there any reference about CI I can see? > You can use Microsoft Azure (recommended), or Travis-CI to test your patches. Regards, Bin
On Tue, 2020-10-20 at 14:15 +0800, Bin Meng wrote: > Hi Chunfeng, > > On Tue, Oct 20, 2020 at 10:22 AM Chunfeng Yun <chunfeng.yun@mediatek.com> wrote: > > > > On Tue, 2020-10-20 at 00:15 +0200, Marek Vasut wrote: > > > On 10/19/20 2:19 PM, Chunfeng Yun wrote: > > > > Due to the following to patches: > > > > 4a1989c0bc77 ("dm: Don't undefine dev_xxx macros") > > > > 69dae8902b16 ("linux/compat.h: Remove redefinition of dev_xxx macros") > > > > > > > > Need include device_compat.h and no need use __maybe_unused anymore > > > > to fix unused variable warning > > > > > > > > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> > > > > --- > > > > Please squash in: ab703ebb1cc0 ("usb: add MediaTek USB3 DRD driver") > > > > in u-boot-usb/next > > > > > > Applied, thanks. > > Thank you! > > > > > > > > > Sorry for inconvenience > > > > > > Please make use of the CI next time. > > Is there any reference about CI I can see? > > > > You can use Microsoft Azure (recommended), or Travis-CI to test your patches. Thank you > > Regards, > Bin
diff --git a/drivers/usb/mtu3/mtu3.h b/drivers/usb/mtu3/mtu3.h index a51251c661..8a7ae83ee9 100644 --- a/drivers/usb/mtu3/mtu3.h +++ b/drivers/usb/mtu3/mtu3.h @@ -13,6 +13,7 @@ #include <asm/io.h> #include <clk.h> #include <dm.h> +#include <dm/device_compat.h> #include <dm/devres.h> #include <generic-phy.h> #include <linux/bug.h> diff --git a/drivers/usb/mtu3/mtu3_gadget.c b/drivers/usb/mtu3/mtu3_gadget.c index c981407b29..027b7e6111 100644 --- a/drivers/usb/mtu3/mtu3_gadget.c +++ b/drivers/usb/mtu3/mtu3_gadget.c @@ -306,7 +306,7 @@ static int mtu3_gadget_dequeue(struct usb_ep *ep, struct usb_request *req) struct mtu3_ep *mep = to_mtu3_ep(ep); struct mtu3_request *mreq = to_mtu3_request(req); struct mtu3_request *r; - __maybe_unused struct mtu3 *mtu = mep->mtu; + struct mtu3 *mtu = mep->mtu; unsigned long flags; int ret = 0; @@ -344,7 +344,7 @@ done: static int mtu3_gadget_ep_set_halt(struct usb_ep *ep, int value) { struct mtu3_ep *mep = to_mtu3_ep(ep); - __maybe_unused struct mtu3 *mtu = mep->mtu; + struct mtu3 *mtu = mep->mtu; struct mtu3_request *mreq; unsigned long flags = 0; int ret = 0; diff --git a/drivers/usb/mtu3/mtu3_plat.c b/drivers/usb/mtu3/mtu3_plat.c index 24ffdb360a..3795e695e8 100644 --- a/drivers/usb/mtu3/mtu3_plat.c +++ b/drivers/usb/mtu3/mtu3_plat.c @@ -347,7 +347,8 @@ static int mtu3_glue_bind(struct udevice *parent) ret = device_bind_driver_to_node(parent, driver, name, node, &dev); if (ret) - dev_err("%s: not able to bind usb device mode\n", __func__); + dev_err(parent, "%s: not able to bind usb device mode\n", + __func__); return ret; }
Due to the following to patches: 4a1989c0bc77 ("dm: Don't undefine dev_xxx macros") 69dae8902b16 ("linux/compat.h: Remove redefinition of dev_xxx macros") Need include device_compat.h and no need use __maybe_unused anymore to fix unused variable warning Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> --- Please squash in: ab703ebb1cc0 ("usb: add MediaTek USB3 DRD driver") in u-boot-usb/next Sorry for inconvenience --- drivers/usb/mtu3/mtu3.h | 1 + drivers/usb/mtu3/mtu3_gadget.c | 4 ++-- drivers/usb/mtu3/mtu3_plat.c | 3 ++- 3 files changed, 5 insertions(+), 3 deletions(-)