Message ID | 1582034720-5249-18-git-send-email-marian-cristian.rotariu.rb@bp.renesas.com (mailing list archive) |
---|---|
State | Rejected, archived |
Delegated to: | Pavel Machek |
Headers | show |
Series | Renesas RZ/G2E USB Type-C Backport | expand |
On Tue 2020-02-18 14:05:14, Marian-Cristian Rotariu wrote: > From: Mao Wenan <maowenan@huawei.com> > > commit da4b5d18dd949abdda7c8ea76c9483b5edd49616 upstream. > > If CONFIG_TYPEC_HD3SS3220=y, CONFIG_USB_ROLE_SWITCH=m, below errors > can be found: > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_remove': > hd3ss3220.c:(.text+0x64): undefined reference to `usb_role_switch_put' > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_dr_set': > hd3ss3220.c:(.text+0x154): undefined reference to `usb_role_switch_set_role' > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_set_role': > hd3ss3220.c:(.text+0x294): undefined reference to `usb_role_switch_set_role' > hd3ss3220.c:(.text+0x2f4): undefined reference to `usb_role_switch_set_role' > hd3ss3220.c:(.text+0x348): undefined reference to `usb_role_switch_set_role' > hd3ss3220.c:(.text+0x390): undefined reference to `usb_role_switch_set_role' > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_probe': > hd3ss3220.c:(.text+0x5e8): undefined reference to `fwnode_usb_role_switch_get' > hd3ss3220.c:(.text+0x8a4): undefined reference to `usb_role_switch_put' > make: *** [vmlinux] Error 1 I guess it is okay here, but for more serious bugs it would be nice to merge the buggy patch with a fix -- not to break bisect. Best regards, Pavel
Hi! > -----Original Message----- > From: Pavel Machek <pavel@denx.de> > Sent: 19 February 2020 08:16 > To: Marian-Cristian Rotariu <marian-cristian.rotariu.rb@bp.renesas.com> > Cc: cip-dev@lists.cip-project.org > Subject: Re: [cip-dev] [PATCH 4.19.y-cip 17/23] usb: typec: add dependency > for TYPEC_HD3SS3220 > > On Tue 2020-02-18 14:05:14, Marian-Cristian Rotariu wrote: > > From: Mao Wenan <maowenan@huawei.com> > > > > commit da4b5d18dd949abdda7c8ea76c9483b5edd49616 upstream. > > > > If CONFIG_TYPEC_HD3SS3220=y, CONFIG_USB_ROLE_SWITCH=m, below > errors > > can be found: > > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_remove': > > hd3ss3220.c:(.text+0x64): undefined reference to `usb_role_switch_put' > > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_dr_set': > > hd3ss3220.c:(.text+0x154): undefined reference to > `usb_role_switch_set_role' > > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_set_role': > > hd3ss3220.c:(.text+0x294): undefined reference to > `usb_role_switch_set_role' > > hd3ss3220.c:(.text+0x2f4): undefined reference to > `usb_role_switch_set_role' > > hd3ss3220.c:(.text+0x348): undefined reference to > `usb_role_switch_set_role' > > hd3ss3220.c:(.text+0x390): undefined reference to > `usb_role_switch_set_role' > > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_probe': > > hd3ss3220.c:(.text+0x5e8): undefined reference to > `fwnode_usb_role_switch_get' > > hd3ss3220.c:(.text+0x8a4): undefined reference to `usb_role_switch_put' > > make: *** [vmlinux] Error 1 > > I guess it is okay here, but for more serious bugs it would be nice to merge > the buggy patch with a fix -- not to break bisect. I kind of forgot about bisect. I was mostly focused on keeping the original patches from upstream (in this case, it is more about the commit message as the fix is trivial). I will send this patch in v2 also, since you mostly agreed with it. Please let me know if otherwise. Cheers, Marian
Hi! > > On Tue 2020-02-18 14:05:14, Marian-Cristian Rotariu wrote: > > > From: Mao Wenan <maowenan@huawei.com> > > > > > > commit da4b5d18dd949abdda7c8ea76c9483b5edd49616 upstream. > > > > > > If CONFIG_TYPEC_HD3SS3220=y, CONFIG_USB_ROLE_SWITCH=m, below > > errors > > > can be found: > > > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_remove': > > > hd3ss3220.c:(.text+0x64): undefined reference to `usb_role_switch_put' > > > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_dr_set': > > > hd3ss3220.c:(.text+0x154): undefined reference to > > `usb_role_switch_set_role' > > > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_set_role': > > > hd3ss3220.c:(.text+0x294): undefined reference to > > `usb_role_switch_set_role' > > > hd3ss3220.c:(.text+0x2f4): undefined reference to > > `usb_role_switch_set_role' > > > hd3ss3220.c:(.text+0x348): undefined reference to > > `usb_role_switch_set_role' > > > hd3ss3220.c:(.text+0x390): undefined reference to > > `usb_role_switch_set_role' > > > drivers/usb/typec/hd3ss3220.o: In function `hd3ss3220_probe': > > > hd3ss3220.c:(.text+0x5e8): undefined reference to > > `fwnode_usb_role_switch_get' > > > hd3ss3220.c:(.text+0x8a4): undefined reference to `usb_role_switch_put' > > > make: *** [vmlinux] Error 1 > > > > I guess it is okay here, but for more serious bugs it would be nice to merge > > the buggy patch with a fix -- not to break bisect. > > I kind of forgot about bisect. I was mostly focused on keeping the original patches > from upstream (in this case, it is more about the commit message as the fix is trivial). > I will send this patch in v2 also, since you mostly agreed with it. Please let me know > if otherwise. This patch is okay. Best regards, Pavel
diff --git a/drivers/usb/typec/Kconfig b/drivers/usb/typec/Kconfig index fde51c5..a4bdb5c 100644 --- a/drivers/usb/typec/Kconfig +++ b/drivers/usb/typec/Kconfig @@ -95,6 +95,7 @@ source "drivers/usb/typec/ucsi/Kconfig" config TYPEC_HD3SS3220 tristate "TI HD3SS3220 Type-C DRP Port controller driver" depends on I2C + depends on USB_ROLE_SWITCH help Say Y or M here if your system has TI HD3SS3220 Type-C DRP Port controller driver.