All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2 0/9] Reorganize R8A7779/Marzen USB code
Date: Tue, 09 Apr 2013 12:44:17 +0000	[thread overview]
Message-ID: <51640D21.3090800@cogentembedded.com> (raw)
In-Reply-To: <87zjx89rfb.wl%kuninori.morimoto.gx@renesas.com>

Hello.

On 09-04-2013 9:58, Kuninori Morimoto wrote:

>>     Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo,
>> 'renesas-next-20130405v2' tag.  It was created to fix the shortcomings in the
>> R8A7779/Marzen USB platform code and R8A7779 USB common PHY driver, and so
>> spans both arch/arm/mach-shmobile/ and drivers/usb/ subtrees (some patches have
>> to touch both subtrees). The patches were conceived with the complete
>> bisectability goal in mind. The patches have been tested on the Marzen board.

> Please add this "tested on xxx" comment on each patch's log area, not only on [0/x].
> We need it on "git log"

    I'm going to post R8A7778/BOCK-W series following this one, and all the 
patches in 1st series should additionally be tested on BOCK-W. Well, I 
probably can hold up posting version 3 until I have the second series verified.
    BTW, about R8A7778/BOCK-W, R-Car M1A user manual talks about a ferrite 
bead in 49.4.1 (3) Setting USB-PHY. Do you know for sure if it's used or not
on BOCK-W board? PHY initialization seems to work with either settings...

>> [1/9] ARM: shmobile: Marzen: move USB EHCI, OHCI, and PHY devices to R8A7779 code
>> [2/9] ehci-platform: add pre_setup() method to platform data
>> [3/9] ARM: shmobile: R8A7779: setup EHCI internal buffer
>> [4/9] rcar-phy: remove EHCI internal buffer setup
>> [5/9] ARM: shmobile: R8A7779: remove USB PHY 2nd memory resource
>> [6/9] rcar-phy: correct base address
>> [7/9] rcar-phy: add platform data
>> [8/9] ARM: shmobile: Marzen: pass platform data to USB PHY device
>> [9/9] rcar-phy: handle platform data

> I didn't get [7/9][9/9] patch somehow

    Here they are in the archive:

http://marc.info/?l=linux-usb&m\x136545680506170
http://marc.info/?l=linux-usb&m\x136545698906257

WBR, Sergei


WARNING: multiple messages have this Message-ID (diff)
From: sergei.shtylyov@cogentembedded.com (Sergei Shtylyov)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 0/9] Reorganize R8A7779/Marzen USB code
Date: Tue, 09 Apr 2013 16:44:17 +0400	[thread overview]
Message-ID: <51640D21.3090800@cogentembedded.com> (raw)
In-Reply-To: <87zjx89rfb.wl%kuninori.morimoto.gx@renesas.com>

Hello.

On 09-04-2013 9:58, Kuninori Morimoto wrote:

>>     Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo,
>> 'renesas-next-20130405v2' tag.  It was created to fix the shortcomings in the
>> R8A7779/Marzen USB platform code and R8A7779 USB common PHY driver, and so
>> spans both arch/arm/mach-shmobile/ and drivers/usb/ subtrees (some patches have
>> to touch both subtrees). The patches were conceived with the complete
>> bisectability goal in mind. The patches have been tested on the Marzen board.

> Please add this "tested on xxx" comment on each patch's log area, not only on [0/x].
> We need it on "git log"

    I'm going to post R8A7778/BOCK-W series following this one, and all the 
patches in 1st series should additionally be tested on BOCK-W. Well, I 
probably can hold up posting version 3 until I have the second series verified.
    BTW, about R8A7778/BOCK-W, R-Car M1A user manual talks about a ferrite 
bead in 49.4.1 (3) Setting USB-PHY. Do you know for sure if it's used or not
on BOCK-W board? PHY initialization seems to work with either settings...

>> [1/9] ARM: shmobile: Marzen: move USB EHCI, OHCI, and PHY devices to R8A7779 code
>> [2/9] ehci-platform: add pre_setup() method to platform data
>> [3/9] ARM: shmobile: R8A7779: setup EHCI internal buffer
>> [4/9] rcar-phy: remove EHCI internal buffer setup
>> [5/9] ARM: shmobile: R8A7779: remove USB PHY 2nd memory resource
>> [6/9] rcar-phy: correct base address
>> [7/9] rcar-phy: add platform data
>> [8/9] ARM: shmobile: Marzen: pass platform data to USB PHY device
>> [9/9] rcar-phy: handle platform data

> I didn't get [7/9][9/9] patch somehow

    Here they are in the archive:

http://marc.info/?l=linux-usb&m=136545680506170
http://marc.info/?l=linux-usb&m=136545698906257

WBR, Sergei

  reply	other threads:[~2013-04-09 12:44 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-08 21:10 [PATCH v2 0/9] Reorganize R8A7779/Marzen USB code Sergei Shtylyov
2013-04-08 21:10 ` Sergei Shtylyov
2013-04-09  5:58 ` Kuninori Morimoto
2013-04-09  5:58   ` Kuninori Morimoto
2013-04-09 12:44   ` Sergei Shtylyov [this message]
2013-04-09 12:44     ` Sergei Shtylyov
2013-04-10  1:31     ` Kuninori Morimoto
2013-04-10  1:31       ` Kuninori Morimoto
2013-04-12 23:07       ` Sergei Shtylyov
2013-04-12 23:07         ` Sergei Shtylyov
2013-04-12 23:11         ` Sergei Shtylyov
2013-04-12 23:11           ` Sergei Shtylyov
2013-04-17  1:14           ` Kuninori Morimoto
2013-04-17  1:14             ` Kuninori Morimoto
2013-04-19  0:27             ` Kuninori Morimoto
2013-04-19  0:27               ` Kuninori Morimoto
2013-04-19 17:54               ` Sergei Shtylyov
2013-04-19 17:54                 ` Sergei Shtylyov
2013-04-09  8:32 ` Simon Horman
2013-04-09  8:32   ` Simon Horman

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=51640D21.3090800@cogentembedded.com \
    --to=sergei.shtylyov@cogentembedded.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.