* [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code @ 2013-04-09 22:26 Sergei Shtylyov 2013-04-10 4:49 ` Simon Horman 0 siblings, 1 reply; 10+ messages in thread From: Sergei Shtylyov @ 2013-04-09 22:26 UTC (permalink / raw) To: linux-arm-kernel Hello. Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo, 'renesas-next-20130410' 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. [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'm not sure thru which tree this patchset should be merged, however it turns out that it's too late now to push it thru Felipe Balbi's USB tree for 3.10 (which would have been most convenient probably), and it's probably too late to merge thru Simon's tree for 3.10 too. So it now have to be postponed to 3.11, unfortunately... WBR, Sergei ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code 2013-04-09 22:26 [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code Sergei Shtylyov @ 2013-04-10 4:49 ` Simon Horman 2013-04-10 8:50 ` Felipe Balbi 0 siblings, 1 reply; 10+ messages in thread From: Simon Horman @ 2013-04-10 4:49 UTC (permalink / raw) To: linux-arm-kernel On Wed, Apr 10, 2013 at 02:26:08AM +0400, Sergei Shtylyov wrote: > Hello. > > Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo, > 'renesas-next-20130410' 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. > > [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'm not sure thru which tree this patchset should be merged, however it turns > out that it's too late now to push it thru Felipe Balbi's USB tree for 3.10 > (which would have been most convenient probably), and it's probably too late to > merge thru Simon's tree for 3.10 too. So it now have to be postponed to 3.11, > unfortunately... Felipe, how would you prefer to handle this? I'm happy for you to take it or to take it myself. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code 2013-04-10 4:49 ` Simon Horman @ 2013-04-10 8:50 ` Felipe Balbi 2013-04-10 9:05 ` Simon Horman 0 siblings, 1 reply; 10+ messages in thread From: Felipe Balbi @ 2013-04-10 8:50 UTC (permalink / raw) To: linux-arm-kernel Hi, On Wed, Apr 10, 2013 at 01:49:23PM +0900, Simon Horman wrote: > On Wed, Apr 10, 2013 at 02:26:08AM +0400, Sergei Shtylyov wrote: > > Hello. > > > > Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo, > > 'renesas-next-20130410' 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. > > > > [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'm not sure thru which tree this patchset should be merged, however it turns > > out that it's too late now to push it thru Felipe Balbi's USB tree for 3.10 > > (which would have been most convenient probably), and it's probably too late to > > merge thru Simon's tree for 3.10 too. So it now have to be postponed to 3.11, > > unfortunately... > > Felipe, how would you prefer to handle this? > I'm happy for you to take it or to take it myself. please take it yourself, just keep in mind there will be conflicts, I will go over the patches now and give my Acked-by so you can queue them up. cheers -- balbi -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130410/70dc1ec8/attachment.sig> ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code 2013-04-10 8:50 ` Felipe Balbi @ 2013-04-10 9:05 ` Simon Horman 2013-04-11 0:51 ` Simon Horman 0 siblings, 1 reply; 10+ messages in thread From: Simon Horman @ 2013-04-10 9:05 UTC (permalink / raw) To: linux-arm-kernel On Wed, Apr 10, 2013 at 11:50:33AM +0300, Felipe Balbi wrote: > Hi, > > On Wed, Apr 10, 2013 at 01:49:23PM +0900, Simon Horman wrote: > > On Wed, Apr 10, 2013 at 02:26:08AM +0400, Sergei Shtylyov wrote: > > > Hello. > > > > > > Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo, > > > 'renesas-next-20130410' 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. > > > > > > [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'm not sure thru which tree this patchset should be merged, however it turns > > > out that it's too late now to push it thru Felipe Balbi's USB tree for 3.10 > > > (which would have been most convenient probably), and it's probably too late to > > > merge thru Simon's tree for 3.10 too. So it now have to be postponed to 3.11, > > > unfortunately... > > > > Felipe, how would you prefer to handle this? > > I'm happy for you to take it or to take it myself. > > please take it yourself, just keep in mind there will be conflicts, I > will go over the patches now and give my Acked-by so you can queue them > up. Thanks ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code 2013-04-10 9:05 ` Simon Horman @ 2013-04-11 0:51 ` Simon Horman 2013-04-11 12:02 ` Sergei Shtylyov 0 siblings, 1 reply; 10+ messages in thread From: Simon Horman @ 2013-04-11 0:51 UTC (permalink / raw) To: linux-arm-kernel On Wed, Apr 10, 2013 at 06:05:43PM +0900, Simon Horman wrote: > On Wed, Apr 10, 2013 at 11:50:33AM +0300, Felipe Balbi wrote: > > Hi, > > > > On Wed, Apr 10, 2013 at 01:49:23PM +0900, Simon Horman wrote: > > > On Wed, Apr 10, 2013 at 02:26:08AM +0400, Sergei Shtylyov wrote: > > > > Hello. > > > > > > > > Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo, > > > > 'renesas-next-20130410' 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. > > > > > > > > [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'm not sure thru which tree this patchset should be merged, however it turns > > > > out that it's too late now to push it thru Felipe Balbi's USB tree for 3.10 > > > > (which would have been most convenient probably), and it's probably too late to > > > > merge thru Simon's tree for 3.10 too. So it now have to be postponed to 3.11, > > > > unfortunately... > > > > > > Felipe, how would you prefer to handle this? > > > I'm happy for you to take it or to take it myself. > > > > please take it yourself, just keep in mind there will be conflicts, I > > will go over the patches now and give my Acked-by so you can queue them > > up. > > Thanks Sergei, it looks like patches 1-7 are clear. Do you want me to start queueing them up. Or would you rather wait for the issue with patch 6 to be resolved? ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code 2013-04-11 0:51 ` Simon Horman @ 2013-04-11 12:02 ` Sergei Shtylyov 2013-04-11 12:06 ` Felipe Balbi 2013-04-12 0:34 ` Simon Horman 0 siblings, 2 replies; 10+ messages in thread From: Sergei Shtylyov @ 2013-04-11 12:02 UTC (permalink / raw) To: linux-arm-kernel Hello. On 11-04-2013 4:51, Simon Horman wrote: >>>>> Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo, >>>>> 'renesas-next-20130410' 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. >>>>> [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'm not sure thru which tree this patchset should be merged, however it turns >>>>> out that it's too late now to push it thru Felipe Balbi's USB tree for 3.10 >>>>> (which would have been most convenient probably), and it's probably too late to >>>>> merge thru Simon's tree for 3.10 too. So it now have to be postponed to 3.11, >>>>> unfortunately... >>>> Felipe, how would you prefer to handle this? >>>> I'm happy for you to take it or to take it myself. >>> please take it yourself, just keep in mind there will be conflicts, I >>> will go over the patches now and give my Acked-by so you can queue them >>> up. >> Thanks > Sergei, it looks like patches 1-7 are clear. #7 & #9 have been refused by Felipe (hence #8 isn't good too). > Do you want me to start queueing them up. I was going to repost the whole series soon, adding ACK from Alan to the patch #2 and resolving issues with patches #7..#9. It's worth waiting for that to happen I think. > Or would you rather wait for the issue with patch 6 to be resolved? There's no special issue with patch #6 other than the PHY driver files being renamed in Felipe's tree -- but that concerns all patches touching drivers/usb/phy/rcar-phy.c (some of which precede #6). I don't know how the rename conflict will be handled (this is really bad coincidence), perhaps I still should have done this patchset against Felipe's 'next' branch and pushed it thru his tree. If the patches will be queued for 3.11 anyway, maybe it's worth doing that -- I'm still of the opinion that his tree would be the one most fitting for this patchset. WBR, Sergei ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code 2013-04-11 12:02 ` Sergei Shtylyov @ 2013-04-11 12:06 ` Felipe Balbi 2013-04-11 12:20 ` Sergei Shtylyov 2013-04-12 0:34 ` Simon Horman 1 sibling, 1 reply; 10+ messages in thread From: Felipe Balbi @ 2013-04-11 12:06 UTC (permalink / raw) To: linux-arm-kernel Hi, On Thu, Apr 11, 2013 at 04:02:33PM +0400, Sergei Shtylyov wrote: > Hello. > > On 11-04-2013 4:51, Simon Horman wrote: > > >>>>> Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo, > >>>>>'renesas-next-20130410' 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. > > >>>>>[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'm not sure thru which tree this patchset should be merged, however it turns > >>>>>out that it's too late now to push it thru Felipe Balbi's USB tree for 3.10 > >>>>>(which would have been most convenient probably), and it's probably too late to > >>>>>merge thru Simon's tree for 3.10 too. So it now have to be postponed to 3.11, > >>>>>unfortunately... > > >>>>Felipe, how would you prefer to handle this? > >>>>I'm happy for you to take it or to take it myself. > > >>>please take it yourself, just keep in mind there will be conflicts, I > >>>will go over the patches now and give my Acked-by so you can queue them > >>>up. > > >>Thanks > > >Sergei, it looks like patches 1-7 are clear. > > #7 & #9 have been refused by Felipe (hence #8 isn't good too). > > >Do you want me to start queueing them up. > > I was going to repost the whole series soon, adding ACK from Alan > to the patch #2 and resolving issues with patches #7..#9. It's worth > waiting for that to happen I think. > > >Or would you rather wait for the issue with patch 6 to be resolved? > > There's no special issue with patch #6 other than the PHY driver > files being renamed in Felipe's tree -- but that concerns all patches > touching drivers/usb/phy/rcar-phy.c (some of which precede #6). > I don't know how the rename conflict will be handled (this is > really bad coincidence), perhaps I still should have done this > patchset against Felipe's 'next' branch and pushed it thru his tree. > If the patches will be queued for 3.11 anyway, maybe it's worth doing > that -- I'm still of the opinion that his tree would be the one most > fitting for this patchset. git can handle renames well, don't worry. Conflict will be easy to resolve. -- balbi -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130411/0702ce2d/attachment.sig> ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code 2013-04-11 12:06 ` Felipe Balbi @ 2013-04-11 12:20 ` Sergei Shtylyov 0 siblings, 0 replies; 10+ messages in thread From: Sergei Shtylyov @ 2013-04-11 12:20 UTC (permalink / raw) To: linux-arm-kernel Hello. On 11-04-2013 16:06, Felipe Balbi wrote: >>>>>>> Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo, >>>>>>> 'renesas-next-20130410' 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. >>>>>>> [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'm not sure thru which tree this patchset should be merged, however it turns >>>>>>> out that it's too late now to push it thru Felipe Balbi's USB tree for 3.10 >>>>>>> (which would have been most convenient probably), and it's probably too late to >>>>>>> merge thru Simon's tree for 3.10 too. So it now have to be postponed to 3.11, >>>>>>> unfortunately... >>>>>> Felipe, how would you prefer to handle this? >>>>>> I'm happy for you to take it or to take it myself. >>>>> please take it yourself, just keep in mind there will be conflicts, I >>>>> will go over the patches now and give my Acked-by so you can queue them >>>>> up. >>>> Thanks >>> Sergei, it looks like patches 1-7 are clear. >> #7 & #9 have been refused by Felipe (hence #8 isn't good too). >>> Do you want me to start queueing them up. >> I was going to repost the whole series soon, adding ACK from Alan >> to the patch #2 and resolving issues with patches #7..#9. It's worth >> waiting for that to happen I think. >>> Or would you rather wait for the issue with patch 6 to be resolved? >> There's no special issue with patch #6 other than the PHY driver >> files being renamed in Felipe's tree -- but that concerns all patches >> touching drivers/usb/phy/rcar-phy.c (some of which precede #6). >> I don't know how the rename conflict will be handled (this is >> really bad coincidence), perhaps I still should have done this >> patchset against Felipe's 'next' branch and pushed it thru his tree. >> If the patches will be queued for 3.11 anyway, maybe it's worth doing >> that -- I'm still of the opinion that his tree would be the one most >> fitting for this patchset. > git can handle renames well, don't worry. Conflict will be easy to > resolve. I'm adding a header file to include/linux/usb/ which now should probably be renamed too, after the driver, to 'phy-rcar-usb.h'. I prbably should rename it now, in anticipation, or the one who would resolve the conflict, won't figure it out to do it. Or perhaps indeed patches #1..#6 can be merged now, and the rest postponed... WBR, Sergei ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code 2013-04-11 12:02 ` Sergei Shtylyov 2013-04-11 12:06 ` Felipe Balbi @ 2013-04-12 0:34 ` Simon Horman 2013-04-12 18:11 ` Sergei Shtylyov 1 sibling, 1 reply; 10+ messages in thread From: Simon Horman @ 2013-04-12 0:34 UTC (permalink / raw) To: linux-arm-kernel On Thu, Apr 11, 2013 at 04:02:33PM +0400, Sergei Shtylyov wrote: > Hello. > > On 11-04-2013 4:51, Simon Horman wrote: > > >>>>> Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo, > >>>>>'renesas-next-20130410' 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. > > >>>>>[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'm not sure thru which tree this patchset should be merged, however it turns > >>>>>out that it's too late now to push it thru Felipe Balbi's USB tree for 3.10 > >>>>>(which would have been most convenient probably), and it's probably too late to > >>>>>merge thru Simon's tree for 3.10 too. So it now have to be postponed to 3.11, > >>>>>unfortunately... > > >>>>Felipe, how would you prefer to handle this? > >>>>I'm happy for you to take it or to take it myself. > > >>>please take it yourself, just keep in mind there will be conflicts, I > >>>will go over the patches now and give my Acked-by so you can queue them > >>>up. > > >>Thanks > > >Sergei, it looks like patches 1-7 are clear. > > #7 & #9 have been refused by Felipe (hence #8 isn't good too). > > >Do you want me to start queueing them up. > > I was going to repost the whole series soon, adding ACK from Alan > to the patch #2 and resolving issues with patches #7..#9. It's worth > waiting for that to happen I think. I am quite happy to wait. One minor comment - there is no need to repost just to fix this - could you consider using lower-case for the socs for consistency with the common practice in patch subject lines. e.g.: ARM: shmobile: r8a7779: setup EHCI > >Or would you rather wait for the issue with patch 6 to be resolved? > > There's no special issue with patch #6 other than the PHY driver > files being renamed in Felipe's tree -- but that concerns all > patches touching drivers/usb/phy/rcar-phy.c (some of which precede > #6). Sorry, I made an error. I meant #8 and in turn #9. > I don't know how the rename conflict will be handled (this is > really bad coincidence), perhaps I still should have done this > patchset against Felipe's 'next' branch and pushed it thru his tree. > If the patches will be queued for 3.11 anyway, maybe it's worth > doing that -- I'm still of the opinion that his tree would be the > one most fitting for this patchset. It seems that git will handle it :) ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code 2013-04-12 0:34 ` Simon Horman @ 2013-04-12 18:11 ` Sergei Shtylyov 0 siblings, 0 replies; 10+ messages in thread From: Sergei Shtylyov @ 2013-04-12 18:11 UTC (permalink / raw) To: linux-arm-kernel Hello. On 04/12/2013 04:34 AM, Simon Horman wrote: >>>>>>> Here's the set of 9 patches against the Simon Horman's 'renesas.git' repo, >>>>>>> 'renesas-next-20130410' 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. >>>>>>> [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'm not sure thru which tree this patchset should be merged, however it turns >>>>>>> out that it's too late now to push it thru Felipe Balbi's USB tree for 3.10 >>>>>>> (which would have been most convenient probably), and it's probably too late to >>>>>>> merge thru Simon's tree for 3.10 too. So it now have to be postponed to 3.11, >>>>>>> unfortunately... >>>>>> Felipe, how would you prefer to handle this? >>>>>> I'm happy for you to take it or to take it myself. >>>>> please take it yourself, just keep in mind there will be conflicts, I >>>>> will go over the patches now and give my Acked-by so you can queue them >>>>> up. >>>> Thanks >>> Sergei, it looks like patches 1-7 are clear. >> #7 & #9 have been refused by Felipe (hence #8 isn't good too). >> >>> Do you want me to start queueing them up. >> I was going to repost the whole series soon, adding ACK from Alan It's not as soon as I had hoped but I hope I would be able to do it in several hours... >> to the patch #2 and resolving issues with patches #7..#9. It's worth >> waiting for that to happen I think. > I am quite happy to wait. > > One minor comment - there is no need to repost just to fix this - > could you consider using lower-case for the socs for consistency > with the common practice in patch subject lines. > > e.g.: ARM: shmobile: r8a7779: setup EHCI Alright, although I have a kind of idiosyncrasy to lowercasing otherwise uppercase names. > >>> Or would you rather wait for the issue with patch 6 to be resolved? >> There's no special issue with patch #6 other than the PHY driver >> files being renamed in Felipe's tree -- but that concerns all >> patches touching drivers/usb/phy/rcar-phy.c (some of which precede >> #6). > Sorry, I made an error. I meant #8 and in turn #9. The issues start at patch #7 actually, where the platform data represenation seemed inappropriate to Felipe. > >> I don't know how the rename conflict will be handled (this is >> really bad coincidence), perhaps I still should have done this >> patchset against Felipe's 'next' branch and pushed it thru his tree. >> If the patches will be queued for 3.11 anyway, maybe it's worth >> doing that -- I'm still of the opinion that his tree would be the >> one most fitting for this patchset. > It seems that git will handle it :) Well, let's hope. There's still an issue with header file name it wouldn't be able to handle but I'll probably leave it as is, as the other PHY files in include/linux/usb/ didn't get renamed in the big rename. WBR, Sergei ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2013-04-12 18:11 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-04-09 22:26 [PATCH v3 0/9] Reorganize R8A7779/Marzen USB code Sergei Shtylyov 2013-04-10 4:49 ` Simon Horman 2013-04-10 8:50 ` Felipe Balbi 2013-04-10 9:05 ` Simon Horman 2013-04-11 0:51 ` Simon Horman 2013-04-11 12:02 ` Sergei Shtylyov 2013-04-11 12:06 ` Felipe Balbi 2013-04-11 12:20 ` Sergei Shtylyov 2013-04-12 0:34 ` Simon Horman 2013-04-12 18:11 ` Sergei Shtylyov
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).