* Re: [PATCH 0/2] net/cadence/macb: add support for dt phy definition
[not found] <1377186980-21902-1-git-send-email-b.brezillon@overkiz.com>
@ 2013-08-26 20:04 ` David Miller
2013-08-27 7:42 ` boris brezillon
0 siblings, 1 reply; 8+ messages in thread
From: David Miller @ 2013-08-26 20:04 UTC (permalink / raw)
To: b.brezillon
Cc: nicolas.ferre, rob.herring, pawel.moll, mark.rutland, swarren,
ian.campbell, linux, f.fainelli, netdev, linux-kernel,
linux-arm-kernel, devicetree
From: Boris BREZILLON <b.brezillon@overkiz.com>
Date: Thu, 22 Aug 2013 17:56:20 +0200
> This patch series adds support for ethernet phy definition using device
> tree.
>
> This may help in moving some at91 boards to dt (some of them define an
> interrupt pin).
>
> Tested on samad31ek.
Series applied to net-next, thanks.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 0/2] net/cadence/macb: add support for dt phy definition
2013-08-26 20:04 ` [PATCH 0/2] net/cadence/macb: add support for dt phy definition David Miller
@ 2013-08-27 7:42 ` boris brezillon
2013-08-27 9:07 ` Florian Fainelli
2013-08-27 16:20 ` David Miller
0 siblings, 2 replies; 8+ messages in thread
From: boris brezillon @ 2013-08-27 7:42 UTC (permalink / raw)
To: David Miller
Cc: nicolas.ferre, rob.herring, pawel.moll, mark.rutland, swarren,
ian.campbell, linux, f.fainelli, netdev, linux-kernel,
linux-arm-kernel, devicetree
Hello Dave,
On 26/08/2013 22:04, David Miller wrote:
> From: Boris BREZILLON <b.brezillon@overkiz.com>
> Date: Thu, 22 Aug 2013 17:56:20 +0200
>
>> This patch series adds support for ethernet phy definition using device
>> tree.
>>
>> This may help in moving some at91 boards to dt (some of them define an
>> interrupt pin).
>>
>> Tested on samad31ek.
> Series applied to net-next, thanks.
Could you apply, the 3rd version of this series instead ?
It fixes one bug when no phy is discovered and use the appropriate address
for the phy dt node.
Sorry for the inconvenience.
Best Regards,
Boris
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 0/2] net/cadence/macb: add support for dt phy definition
2013-08-27 7:42 ` boris brezillon
@ 2013-08-27 9:07 ` Florian Fainelli
2013-08-27 9:13 ` boris brezillon
2013-08-27 16:20 ` David Miller
1 sibling, 1 reply; 8+ messages in thread
From: Florian Fainelli @ 2013-08-27 9:07 UTC (permalink / raw)
To: boris brezillon
Cc: David Miller, nicolas.ferre, Rob Herring, Pawel Moll,
Mark Rutland, Stephen Warren, Ian Campbell, Russell King, netdev,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org
Hello Boris,
2013/8/27 boris brezillon <b.brezillon@overkiz.com>:
> Hello Dave,
>
>
> On 26/08/2013 22:04, David Miller wrote:
>>
>> From: Boris BREZILLON <b.brezillon@overkiz.com>
>> Date: Thu, 22 Aug 2013 17:56:20 +0200
>>
>>> This patch series adds support for ethernet phy definition using device
>>> tree.
>>>
>>> This may help in moving some at91 boards to dt (some of them define an
>>> interrupt pin).
>>>
>>> Tested on samad31ek.
>>
>> Series applied to net-next, thanks.
>
> Could you apply, the 3rd version of this series instead ?
>
> It fixes one bug when no phy is discovered and use the appropriate address
> for the phy dt node.
>
> Sorry for the inconvenience.
You will probably have to resubmit an incremental patch, I have never
seen David pick up another version of a patch once it has been pushed
out:
http://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git/commit/?id=8c038e7e14b1c5f156745e3c4df0a3aa46173dd9
http://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git/commit/?id=148cbb53ace32f584d208764c7f7e6aa8edb970c
--
Florian
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 0/2] net/cadence/macb: add support for dt phy definition
2013-08-27 9:07 ` Florian Fainelli
@ 2013-08-27 9:13 ` boris brezillon
2013-08-27 9:16 ` Nicolas Ferre
0 siblings, 1 reply; 8+ messages in thread
From: boris brezillon @ 2013-08-27 9:13 UTC (permalink / raw)
To: Florian Fainelli
Cc: David Miller, nicolas.ferre, Rob Herring, Pawel Moll,
Mark Rutland, Stephen Warren, Ian Campbell, Russell King, netdev,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org
On 27/08/2013 11:07, Florian Fainelli wrote:
> Hello Boris,
>
> 2013/8/27 boris brezillon <b.brezillon@overkiz.com>:
>> Hello Dave,
>>
>>
>> On 26/08/2013 22:04, David Miller wrote:
>>> From: Boris BREZILLON <b.brezillon@overkiz.com>
>>> Date: Thu, 22 Aug 2013 17:56:20 +0200
>>>
>>>> This patch series adds support for ethernet phy definition using device
>>>> tree.
>>>>
>>>> This may help in moving some at91 boards to dt (some of them define an
>>>> interrupt pin).
>>>>
>>>> Tested on samad31ek.
>>> Series applied to net-next, thanks.
>> Could you apply, the 3rd version of this series instead ?
>>
>> It fixes one bug when no phy is discovered and use the appropriate address
>> for the phy dt node.
>>
>> Sorry for the inconvenience.
> You will probably have to resubmit an incremental patch, I have never
> seen David pick up another version of a patch once it has been pushed
> out:
>
> http://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git/commit/?id=8c038e7e14b1c5f156745e3c4df0a3aa46173dd9
> http://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git/commit/?id=148cbb53ace32f584d208764c7f7e6aa8edb970c
Okay, I will submit patches (based on net-next branch) to fix those bugs.
Thanks.
Best Regards,
Boris
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 0/2] net/cadence/macb: add support for dt phy definition
2013-08-27 9:13 ` boris brezillon
@ 2013-08-27 9:16 ` Nicolas Ferre
0 siblings, 0 replies; 8+ messages in thread
From: Nicolas Ferre @ 2013-08-27 9:16 UTC (permalink / raw)
To: boris brezillon
Cc: Florian Fainelli, David Miller, Rob Herring, Pawel Moll,
Mark Rutland, Stephen Warren, Ian Campbell, Russell King, netdev,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org
On 27/08/2013 11:13, boris brezillon :
> On 27/08/2013 11:07, Florian Fainelli wrote:
>> Hello Boris,
>>
>> 2013/8/27 boris brezillon <b.brezillon@overkiz.com>:
>>> Hello Dave,
>>>
>>>
>>> On 26/08/2013 22:04, David Miller wrote:
>>>> From: Boris BREZILLON <b.brezillon@overkiz.com>
>>>> Date: Thu, 22 Aug 2013 17:56:20 +0200
>>>>
>>>>> This patch series adds support for ethernet phy definition using device
>>>>> tree.
>>>>>
>>>>> This may help in moving some at91 boards to dt (some of them define an
>>>>> interrupt pin).
>>>>>
>>>>> Tested on samad31ek.
>>>> Series applied to net-next, thanks.
>>> Could you apply, the 3rd version of this series instead ?
>>>
>>> It fixes one bug when no phy is discovered and use the appropriate address
>>> for the phy dt node.
>>>
>>> Sorry for the inconvenience.
>> You will probably have to resubmit an incremental patch, I have never
>> seen David pick up another version of a patch once it has been pushed
>> out:
>>
>> http://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git/commit/?id=8c038e7e14b1c5f156745e3c4df0a3aa46173dd9
>> http://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git/commit/?id=148cbb53ace32f584d208764c7f7e6aa8edb970c
> Okay, I will submit patches (based on net-next branch) to fix those bugs.
You can add my Acked-by to your fixes...
Bye,
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 0/2] net/cadence/macb: add support for dt phy definition
2013-08-27 7:42 ` boris brezillon
2013-08-27 9:07 ` Florian Fainelli
@ 2013-08-27 16:20 ` David Miller
2013-08-27 16:38 ` boris brezillon
1 sibling, 1 reply; 8+ messages in thread
From: David Miller @ 2013-08-27 16:20 UTC (permalink / raw)
To: b.brezillon
Cc: nicolas.ferre, rob.herring, pawel.moll, mark.rutland, swarren,
ian.campbell, linux, f.fainelli, netdev, linux-kernel,
linux-arm-kernel, devicetree
From: boris brezillon <b.brezillon@overkiz.com>
Date: Tue, 27 Aug 2013 09:42:34 +0200
> Could you apply, the 3rd version of this series instead ?
There can never be an "instead" or reverting patches I've said I've
applied already.
If you want changes, you have to submit follow-on fixes.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 0/2] net/cadence/macb: add support for dt phy definition
2013-08-27 16:20 ` David Miller
@ 2013-08-27 16:38 ` boris brezillon
2013-08-27 16:41 ` David Miller
0 siblings, 1 reply; 8+ messages in thread
From: boris brezillon @ 2013-08-27 16:38 UTC (permalink / raw)
To: David Miller
Cc: nicolas.ferre, rob.herring, pawel.moll, mark.rutland, swarren,
ian.campbell, linux, f.fainelli, netdev, linux-kernel,
linux-arm-kernel, devicetree
On 27/08/2013 18:20, David Miller wrote:
> From: boris brezillon <b.brezillon@overkiz.com>
> Date: Tue, 27 Aug 2013 09:42:34 +0200
>
>> Could you apply, the 3rd version of this series instead ?
> There can never be an "instead" or reverting patches I've said I've
> applied already.
>
> If you want changes, you have to submit follow-on fixes.
Hello David,
I sent the incremental patches based on your net-next branch:
https://lkml.org/lkml/2013/8/27/257
https://lkml.org/lkml/2013/8/27/259
Thanks.
Best Regards,
Boris
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 0/2] net/cadence/macb: add support for dt phy definition
2013-08-27 16:38 ` boris brezillon
@ 2013-08-27 16:41 ` David Miller
0 siblings, 0 replies; 8+ messages in thread
From: David Miller @ 2013-08-27 16:41 UTC (permalink / raw)
To: b.brezillon
Cc: nicolas.ferre, rob.herring, pawel.moll, mark.rutland, swarren,
ian.campbell, linux, f.fainelli, netdev, linux-kernel,
linux-arm-kernel, devicetree
From: boris brezillon <b.brezillon@overkiz.com>
Date: Tue, 27 Aug 2013 18:38:52 +0200
> On 27/08/2013 18:20, David Miller wrote:
>> From: boris brezillon <b.brezillon@overkiz.com>
>> Date: Tue, 27 Aug 2013 09:42:34 +0200
>>
>>> Could you apply, the 3rd version of this series instead ?
>> There can never be an "instead" or reverting patches I've said I've
>> applied already.
>>
>> If you want changes, you have to submit follow-on fixes.
>
> Hello David,
>
> I sent the incremental patches based on your net-next branch:
> https://lkml.org/lkml/2013/8/27/257
> https://lkml.org/lkml/2013/8/27/259
The place to check to see if your patch is queued up is here:
http://patchwork.ozlabs.org/project/netdev/list/
If it's there, it's not necessary to notify me of anything.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2013-08-27 16:41 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1377186980-21902-1-git-send-email-b.brezillon@overkiz.com>
2013-08-26 20:04 ` [PATCH 0/2] net/cadence/macb: add support for dt phy definition David Miller
2013-08-27 7:42 ` boris brezillon
2013-08-27 9:07 ` Florian Fainelli
2013-08-27 9:13 ` boris brezillon
2013-08-27 9:16 ` Nicolas Ferre
2013-08-27 16:20 ` David Miller
2013-08-27 16:38 ` boris brezillon
2013-08-27 16:41 ` David Miller
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).