All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Rui Miguel Silva" <rui.silva@linaro.org>
To: "Tom Rini" <trini@konsulko.com>, <u-boot@lists.denx.de>
Subject: Re: [PATCH v3 0/3] usb: add isp1760 hcd support
Date: Mon, 20 Jun 2022 17:00:56 +0100	[thread overview]
Message-ID: <CKV35CPQT412.3H8ELJO6XMXDO@arch-thunder> (raw)
In-Reply-To: <20220614174458.axnj6jqaozn26uug@arch-thunder>

Hi Tom,
On Tue Jun 14, 2022 at 6:45 PM WEST, Rui Miguel Silva wrote:
> Hi *,
> On Wed, May 25, 2022 at 02:22:48PM +0100, Rui Miguel Silva wrote:
> > Add support for the usb isp1760 host controller family, which
> > for example is present in MPS3 FPGA board from Arm (isp1763).
> > 
> > First we move some helper functions and defines to a more
> > common place to be shared by several urb users. (patch 1/3)
> > 
> > Then add the driver itself, is a ported version of the kernel
> > actual driver, which I am also the maintainer. (patch 2/3)
> > 
> > And last, enable it for the corstone1000 platform that uses
> > that MPS3 board for its implementation (patch 3/3).
> >
>
> Any chance this series get some feedback?

Am I missing something here? to get some comments on this series?

Thanks,
Cheers,
  Rui
>
> Thanks in advance,
>
> Cheers,
>   Rui
>
> > 
> > Cheers,
> >    Rui
> > 
> > v2[3] -> v3:
> >     - when you think you have amend commit and fix stay
> >       uncommitted.
> >       s/[HC_FIELD_MAX] = {};/[HC_FIELD_MAX] = {},/
> > v1[0] -> v2:
> >     - gentle ping
> >     - merge fix from kernel upstream [1]
> > 
> > PS: This should go on top of the corstone1000 platform enable
> > series [2]
> > 
> > 0: https://lore.kernel.org/u-boot/20220512142016.2025129-1-rui.silva@linaro.org/
> > 1: https://lore.kernel.org/linux-usb/20220516091424.391209-1-linus.walleij@linaro.org/
> > 2: https://lore.kernel.org/u-boot/20220511095541.1461937-1-rui.silva@linaro.org/T/#t
> > 3: https://lore.kernel.org/u-boot/20220523090119.1212016-1-rui.silva@linaro.org/
> > 
> > Rui Miguel Silva (3):
> >   usb: common: move urb code to common
> >   usb: add isp1760 family driver
> >   corstone1000: enable isp1763 usb controller and mmc
> > 
> >  Makefile                                      |    1 +
> >  configs/corstone1000_defconfig                |    3 +
> >  drivers/usb/Kconfig                           |    2 +
> >  drivers/usb/common/Makefile                   |    3 +
> >  drivers/usb/common/usb_urb.c                  |  160 ++
> >  drivers/usb/host/r8a66597-hcd.c               |   30 +-
> >  drivers/usb/isp1760/Kconfig                   |   12 +
> >  drivers/usb/isp1760/Makefile                  |    6 +
> >  drivers/usb/isp1760/isp1760-core.c            |  380 +++
> >  drivers/usb/isp1760/isp1760-core.h            |   96 +
> >  drivers/usb/isp1760/isp1760-hcd.c             | 2477 +++++++++++++++++
> >  drivers/usb/isp1760/isp1760-hcd.h             |   81 +
> >  drivers/usb/isp1760/isp1760-if.c              |  125 +
> >  drivers/usb/isp1760/isp1760-regs.h            |  292 ++
> >  drivers/usb/isp1760/isp1760-uboot.c           |   75 +
> >  drivers/usb/isp1760/isp1760-uboot.h           |   27 +
> >  drivers/usb/musb-new/musb_core.c              |    2 +-
> >  drivers/usb/musb-new/musb_host.c              |    2 +-
> >  drivers/usb/musb-new/musb_host.h              |    2 +-
> >  drivers/usb/musb-new/musb_uboot.c             |   38 +-
> >  drivers/usb/musb-new/musb_uboot.h             |    2 +-
> >  include/configs/corstone1000.h                |    6 +
> >  .../linux/usb/usb_urb_compat.h                |   47 +-
> >  include/usb_defs.h                            |   32 +
> >  24 files changed, 3825 insertions(+), 76 deletions(-)
> >  create mode 100644 drivers/usb/common/usb_urb.c
> >  create mode 100644 drivers/usb/isp1760/Kconfig
> >  create mode 100644 drivers/usb/isp1760/Makefile
> >  create mode 100644 drivers/usb/isp1760/isp1760-core.c
> >  create mode 100644 drivers/usb/isp1760/isp1760-core.h
> >  create mode 100644 drivers/usb/isp1760/isp1760-hcd.c
> >  create mode 100644 drivers/usb/isp1760/isp1760-hcd.h
> >  create mode 100644 drivers/usb/isp1760/isp1760-if.c
> >  create mode 100644 drivers/usb/isp1760/isp1760-regs.h
> >  create mode 100644 drivers/usb/isp1760/isp1760-uboot.c
> >  create mode 100644 drivers/usb/isp1760/isp1760-uboot.h
> >  rename drivers/usb/musb-new/usb-compat.h => include/linux/usb/usb_urb_compat.h (59%)
> > 
> > -- 
> > 2.36.1
> > 




  reply	other threads:[~2022-06-20 16:01 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-25 13:22 [PATCH v3 0/3] usb: add isp1760 hcd support Rui Miguel Silva
2022-05-25 13:22 ` [PATCH v3 1/3] usb: common: move urb code to common Rui Miguel Silva
2022-05-25 13:22 ` [PATCH v3 2/3] usb: add isp1760 family driver Rui Miguel Silva
2022-05-25 13:22 ` [PATCH v3 3/3] corstone1000: enable isp1763 usb controller and mmc Rui Miguel Silva
2022-06-07  8:58 ` [PATCH v3 0/3] usb: add isp1760 hcd support Rui Miguel Silva
2022-06-14 17:44 ` Rui Miguel Silva
2022-06-20 16:00   ` Rui Miguel Silva [this message]
2022-06-20 17:44     ` Tom Rini
2022-06-28 14:59       ` Rui Miguel Silva
2022-06-28 15:44         ` Marek Vasut
2022-06-28 17:39           ` Rui Miguel Silva

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=CKV35CPQT412.3H8ELJO6XMXDO@arch-thunder \
    --to=rui.silva@linaro.org \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    /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 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.