* [PATCH v3] usb: musb: remove duplicated actions
@ 2016-10-26 14:24 Kirill Esipov
2016-10-26 15:37 ` Bin Liu
0 siblings, 1 reply; 2+ messages in thread
From: Kirill Esipov @ 2016-10-26 14:24 UTC (permalink / raw)
To: linux-kernel; +Cc: linux-usb, gregkh, b-liu, sergei.shtylyov, Kirill Esipov
Removing unnecessary duplicated actions that we've got while merging:
19915e623458 ("Merge 4.1-rc7 into usb-next")
Signed-off-by: Kirill Esipov <yesipov@gmail.com>
---
drivers/usb/musb/musb_core.c | 6 ------
1 file changed, 6 deletions(-)
diff --git a/drivers/usb/musb/musb_core.c b/drivers/usb/musb/musb_core.c
index 27dadc0..2b04f07 100644
--- a/drivers/usb/musb/musb_core.c
+++ b/drivers/usb/musb/musb_core.c
@@ -2120,12 +2120,6 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl)
if (musb->ops->ep_select)
musb->io.ep_select = musb->ops->ep_select;
- /* At least tusb6010 has its own offsets */
- if (musb->ops->ep_offset)
- musb->io.ep_offset = musb->ops->ep_offset;
- if (musb->ops->ep_select)
- musb->io.ep_select = musb->ops->ep_select;
-
if (musb->ops->fifo_mode)
fifo_mode = musb->ops->fifo_mode;
else
--
2.7.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v3] usb: musb: remove duplicated actions
2016-10-26 14:24 [PATCH v3] usb: musb: remove duplicated actions Kirill Esipov
@ 2016-10-26 15:37 ` Bin Liu
0 siblings, 0 replies; 2+ messages in thread
From: Bin Liu @ 2016-10-26 15:37 UTC (permalink / raw)
To: Kirill Esipov; +Cc: linux-kernel, linux-usb, gregkh, sergei.shtylyov
Hi,
On Wed, Oct 26, 2016 at 05:24:27PM +0300, Kirill Esipov wrote:
> Removing unnecessary duplicated actions that we've got while merging:
> 19915e623458 ("Merge 4.1-rc7 into usb-next")
>
> Signed-off-by: Kirill Esipov <yesipov@gmail.com>
> ---
> drivers/usb/musb/musb_core.c | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/drivers/usb/musb/musb_core.c b/drivers/usb/musb/musb_core.c
> index 27dadc0..2b04f07 100644
> --- a/drivers/usb/musb/musb_core.c
> +++ b/drivers/usb/musb/musb_core.c
> @@ -2120,12 +2120,6 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl)
> if (musb->ops->ep_select)
> musb->io.ep_select = musb->ops->ep_select;
>
> - /* At least tusb6010 has its own offsets */
I prefer to keep this comment and drop the first one
"/* And override them with platform specific ops if specified. */"
as it doesn't provide any value.
> - if (musb->ops->ep_offset)
> - musb->io.ep_offset = musb->ops->ep_offset;
> - if (musb->ops->ep_select)
> - musb->io.ep_select = musb->ops->ep_select;
> -
> if (musb->ops->fifo_mode)
> fifo_mode = musb->ops->fifo_mode;
> else
> --
> 2.7.4
Thanks,
-Bin.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-10-26 15:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-26 14:24 [PATCH v3] usb: musb: remove duplicated actions Kirill Esipov
2016-10-26 15:37 ` Bin Liu
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).