All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anand Moon <linux.amoon@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC 2/6] odroid: exynos: usb clean up for U3/X2
Date: Mon, 1 Apr 2019 21:27:41 +0530	[thread overview]
Message-ID: <CANAwSgSrZRFFtbB+jn4sNrrPL6V5VDzaOcaLJcOAJK0Ri29cqg@mail.gmail.com> (raw)
In-Reply-To: <CAJKOXPdZcfxJrYMTFeSA+JvS5DN69wVebzRQavLrNd4Jvu-4fQ@mail.gmail.com>

Hi Krzysztof,

On Mon, 1 Apr 2019 at 18:21, Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On Mon, 1 Apr 2019 at 13:52, Anand Moon <linux.amoon@gmail.com> wrote:
> >
> > Add board_usb_cleanup routine to cleanup after
> > de-registering it usb devices. Also fixed the
> > compilation error for other board.
>
> Fix for build error should be in separate commit. Please also quote
> the error you are fixing... because the code compiles fine in my case.
>

Compilation error happens on other target boards. like Odroid-XU3

> > Signed-off-by: Anand Moon <linux.amoon@gmail.com>
> > ---
> >  board/samsung/common/board.c  | 4 ++--
> >  board/samsung/odroid/odroid.c | 5 +++++
> >  2 files changed, 7 insertions(+), 2 deletions(-)
> >
> > diff --git a/board/samsung/common/board.c b/board/samsung/common/board.c
> > index 9adbd1e2cf..c74aca9b0a 100644
> > --- a/board/samsung/common/board.c
> > +++ b/board/samsung/common/board.c
> > @@ -351,10 +351,10 @@ void reset_misc(void)
> >         }
> >  }
> >
> > +#ifdef CONFIG_USB_DWC3
> >  int board_usb_cleanup(int index, enum usb_init_type init)
> >  {
> > -#ifdef CONFIG_USB_DWC3
> >         dwc3_uboot_exit(index);
> > -#endif
> >         return 0;
> >  }
> > +#endif
> > diff --git a/board/samsung/odroid/odroid.c b/board/samsung/odroid/odroid.c
> > index 79d14ead01..547ae698cf 100644
> > --- a/board/samsung/odroid/odroid.c
> > +++ b/board/samsung/odroid/odroid.c
> > @@ -538,4 +538,9 @@ int board_usb_init(int index, enum usb_init_type init)
> >         debug("USB_udc_probe\n");
> >         return dwc2_udc_probe(&s5pc210_otg_data);
> >  }
> > +
> > +int board_usb_cleanup(int index, enum usb_init_type init)
> > +{
> > +       return s5pc210_phy_control(index);
>
> Why you pass index of USB controller as argument "int on"? Index != on...
>

Generally board_usb_cleanup do cleanup the index of the board_usb_init,
so I use this naturally. I will check and update this if needed.

> Best regards,
> Krzysztof

Best Regards
-Anand

  reply	other threads:[~2019-04-01 15:57 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-01 11:52 [U-Boot] [RFC 0/6] Odroid U3 usb initialization Anand Moon
2019-04-01 11:52 ` [U-Boot] [RFC 1/6] odroid: exynos: USB initialization on the U3/X2 Anand Moon
2019-04-01 12:48   ` Krzysztof Kozlowski
2019-04-01 16:20     ` Anand Moon
2019-04-01 12:57   ` Lukasz Majewski
2019-04-01 13:22     ` Jack Mitchell
2019-04-03 19:58   ` Tobias Jakobi
2019-04-04  3:01     ` Anand Moon
2019-04-01 11:52 ` [U-Boot] [RFC 2/6] odroid: exynos: usb clean up for U3/X2 Anand Moon
2019-04-01 12:51   ` Krzysztof Kozlowski
2019-04-01 15:57     ` Anand Moon [this message]
2019-04-01 16:12       ` Krzysztof Kozlowski
2019-04-01 11:52 ` [U-Boot] [RFC 3/6] usb: exynos: add init_after_reset for usb reset Anand Moon
2019-04-01 12:55   ` Krzysztof Kozlowski
2019-04-01 16:05     ` Anand Moon
2019-04-01 16:14       ` Krzysztof Kozlowski
2019-04-01 11:52 ` [U-Boot] [RFC 4/6] configs: exynos: Add new CONFIG_SYS_ODROID_USB config option Anand Moon
2019-04-01 12:57   ` Krzysztof Kozlowski
2019-04-01 15:50     ` Anand Moon
2019-04-01 16:16       ` Krzysztof Kozlowski
2019-04-01 17:16         ` Anand Moon
2019-04-02  7:02           ` Krzysztof Kozlowski
2019-04-02  7:52             ` Anand Moon
2019-04-02  7:56               ` Krzysztof Kozlowski
2019-04-01 11:52 ` [U-Boot] [RFC 5/6] arm: exynos: odroid: fix the confict scripaddr extra env setting Anand Moon
2019-04-01 13:00   ` Krzysztof Kozlowski
2019-04-01 15:43     ` Anand Moon
2019-04-01 11:52 ` [U-Boot] [RFC 6/6] arm: exynos: add usbnet_devaddr setting to env Anand Moon
2019-04-01 13:04   ` Krzysztof Kozlowski
2019-04-01 16:42     ` Anand Moon

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=CANAwSgSrZRFFtbB+jn4sNrrPL6V5VDzaOcaLJcOAJK0Ri29cqg@mail.gmail.com \
    --to=linux.amoon@gmail.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.