* [PATCH next] usb: mtu3: fix build error caused by dev_xxx()
@ 2020-10-19 12:19 Chunfeng Yun
2020-10-19 22:15 ` Marek Vasut
0 siblings, 1 reply; 5+ messages in thread
From: Chunfeng Yun @ 2020-10-19 12:19 UTC (permalink / raw)
To: u-boot
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(-)
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;
}
--
2.18.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH next] usb: mtu3: fix build error caused by dev_xxx()
2020-10-19 12:19 [PATCH next] usb: mtu3: fix build error caused by dev_xxx() Chunfeng Yun
@ 2020-10-19 22:15 ` Marek Vasut
2020-10-20 2:21 ` Chunfeng Yun
0 siblings, 1 reply; 5+ messages in thread
From: Marek Vasut @ 2020-10-19 22:15 UTC (permalink / raw)
To: u-boot
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.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH next] usb: mtu3: fix build error caused by dev_xxx()
2020-10-19 22:15 ` Marek Vasut
@ 2020-10-20 2:21 ` Chunfeng Yun
2020-10-20 6:15 ` Bin Meng
0 siblings, 1 reply; 5+ messages in thread
From: Chunfeng Yun @ 2020-10-20 2:21 UTC (permalink / raw)
To: u-boot
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?
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH next] usb: mtu3: fix build error caused by dev_xxx()
2020-10-20 2:21 ` Chunfeng Yun
@ 2020-10-20 6:15 ` Bin Meng
2020-10-20 6:46 ` Chunfeng Yun
0 siblings, 1 reply; 5+ messages in thread
From: Bin Meng @ 2020-10-20 6:15 UTC (permalink / raw)
To: u-boot
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH next] usb: mtu3: fix build error caused by dev_xxx()
2020-10-20 6:15 ` Bin Meng
@ 2020-10-20 6:46 ` Chunfeng Yun
0 siblings, 0 replies; 5+ messages in thread
From: Chunfeng Yun @ 2020-10-20 6:46 UTC (permalink / raw)
To: u-boot
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
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-10-20 6:46 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-19 12:19 [PATCH next] usb: mtu3: fix build error caused by dev_xxx() Chunfeng Yun
2020-10-19 22:15 ` Marek Vasut
2020-10-20 2:21 ` Chunfeng Yun
2020-10-20 6:15 ` Bin Meng
2020-10-20 6:46 ` Chunfeng Yun
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.