public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Igor Grinberg <grinberg@compulab.co.il>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v6] ulpi: add generic ULPI functionality
Date: Thu, 01 Dec 2011 13:12:12 +0200	[thread overview]
Message-ID: <4ED7610C.2060401@compulab.co.il> (raw)
In-Reply-To: <1322509429-20976-1-git-send-email-fermata7@gmail.com>

On 11/28/11 21:43, Jana Rapava wrote:
> Add generic functions for reading, writing and setting bits in ULPI registers.
> 
> Signed-off-by: Jana Rapava <fermata7@gmail.com>
> Cc: Remy Bohmer <linux@bohmer.net>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Igor Grinberg <grinberg@compulab.co.il>
> Cc: Wolfgang Grandegger <wg@denx.de>
> Cc: Simon Glass <sjg@chromium.org>
> 
> Acked-by: Igor Grinberg <grinberg@compulab.co.il>
> Acked-by: Simon Glass <sjg@chromium.org>
> ---
> Changes for v2:
>       - make code EHCI-independent
>       - use udelay() in waiting loop
>       - mark static functions as static
>       - naming changes
> Changes for v3:
>        - merge with patch ulpi: add generic ULPI support header file
>        - rewrite ULPI interface in more functionality-oriented way
> Changes for v4:
>        - add error-checking
>        - add waiting for completion into ulpi_reset() function
> Changes for v5:
>         - CodingStyle changes
>         - add comments
>         - simplify implemenation of the ULPI interface functions
> Changes for v6:
> 	- cleanup function ulpi_drive_vbus()

Lately, I've found several flaws in v6.
Those are a "no goes" as for me.
I've got Jana's permission to take over the patch and
I will submit a reworked version soon (hopefully tomorrow).

-- 
Regards,
Igor.

      parent reply	other threads:[~2011-12-01 11:12 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-05 20:50 [U-Boot] [PATCH] ulpi: add generic ULPI functionality Jana Rapava
2011-11-05 21:37 ` Marek Vasut
2011-11-05 23:08   ` Jana Rapava
2011-11-05 23:13     ` Marek Vasut
2011-11-05 23:32       ` Jana Rapava
2011-11-05 23:35         ` Marek Vasut
2011-11-08 11:08         ` Igor Grinberg
2011-11-08 11:33 ` Igor Grinberg
2011-11-12  1:09   ` Jana Rapava
2011-11-14  7:40     ` Igor Grinberg
2011-11-12 17:29 ` [U-Boot] [PATCH v2] " Jana Rapava
2011-11-14  8:13   ` Igor Grinberg
2011-11-24 12:22   ` [U-Boot] [PATCH v3]ulpi: " Jana Rapava
2011-11-24 13:26     ` Igor Grinberg
2011-11-24 14:21       ` Marek Vasut
2011-11-25 18:39       ` Jana Rapava
2011-11-27  7:50         ` Igor Grinberg
2011-11-25 20:05     ` [U-Boot] [PATCH v4] ulpi: " Jana Rapava
2011-11-27  4:00       ` Simon Glass
2011-11-27  8:08         ` Igor Grinberg
2011-11-27 22:37           ` Jana Rapava
2011-11-27 23:34           ` Simon Glass
2011-11-27 22:30         ` Jana Rapava
2011-11-28  0:19       ` [U-Boot] [PATCH v5] " Jana Rapava
2011-11-28  7:39         ` Igor Grinberg
2011-11-28 17:06         ` Simon Glass
2011-11-28 19:43         ` [U-Boot] [PATCH v6] " Jana Rapava
2011-11-28 20:56           ` Simon Glass
2011-12-01 11:12           ` Igor Grinberg [this message]

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=4ED7610C.2060401@compulab.co.il \
    --to=grinberg@compulab.co.il \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox