Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1577215 > unrolled thread

[PATCH 0/2] net: ethernet: ti: cpsw: fix susp/resume

Started byIvan Khoronzhuk <ivan.khoronzhuk@linaro.org>
First post2017-02-09 01:10 +0100
Last post2017-02-11 02:20 +0100
Articles 6 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH 0/2] net: ethernet: ti: cpsw: fix susp/resume Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> - 2017-02-09 01:10 +0100
    Re: [PATCH 0/2] net: ethernet: ti: cpsw: fix susp/resume David Miller <davem@davemloft.net> - 2017-02-09 23:40 +0100
      Re: [PATCH 0/2] net: ethernet: ti: cpsw: fix susp/resume Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> - 2017-02-10 00:50 +0100
        Re: [PATCH 0/2] net: ethernet: ti: cpsw: fix susp/resume David Miller <davem@davemloft.net> - 2017-02-10 02:50 +0100
          Re: [PATCH 0/2] net: ethernet: ti: cpsw: fix susp/resume Grygorii Strashko <grygorii.strashko@ti.com> - 2017-02-10 19:10 +0100
            Re: [PATCH 0/2] net: ethernet: ti: cpsw: fix susp/resume Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> - 2017-02-11 02:20 +0100

#1577215 — [PATCH 0/2] net: ethernet: ti: cpsw: fix susp/resume

FromIvan Khoronzhuk <ivan.khoronzhuk@linaro.org>
Date2017-02-09 01:10 +0100
Subject[PATCH 0/2] net: ethernet: ti: cpsw: fix susp/resume
Message-ID<t8Ktb-7a8-3@gated-at.bofh.it>
These two patches fix suspend/resume chain.

Ivan Khoronzhuk (2):
  net: ethernet: ti: cpsw: fix cpsw assignment in resume
  net: ethernet: ti: cpsw: fix resume because of usage count

 drivers/net/ethernet/ti/cpsw.c | 44 +++++++++++++-----------------------------
 1 file changed, 13 insertions(+), 31 deletions(-)

-- 
2.7.4

[toc] | [next] | [standalone]

#1578001

FromDavid Miller <davem@davemloft.net>
Date2017-02-09 23:40 +0100
Message-ID<t95xD-3yO-7@gated-at.bofh.it>
In reply to#1577215
From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
Date: Thu,  9 Feb 2017 02:07:34 +0200

> These two patches fix suspend/resume chain.

Patch 2 doesn't apply cleanly to the 'net' tree, please
respin this series.

[toc] | [prev] | [next] | [standalone]

#1578052

FromIvan Khoronzhuk <ivan.khoronzhuk@linaro.org>
Date2017-02-10 00:50 +0100
Message-ID<t96Do-4dg-13@gated-at.bofh.it>
In reply to#1578001
On Thu, Feb 09, 2017 at 05:21:26PM -0500, David Miller wrote:
> From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
> Date: Thu,  9 Feb 2017 02:07:34 +0200
> 
> > These two patches fix suspend/resume chain.
> 
> Patch 2 doesn't apply cleanly to the 'net' tree, please
> respin this series.

Strange, I've just checked it on net-next/master, it was applied w/o any
warnings.

[toc] | [prev] | [next] | [standalone]

#1578100

FromDavid Miller <davem@davemloft.net>
Date2017-02-10 02:50 +0100
Message-ID<t98vw-5lK-13@gated-at.bofh.it>
In reply to#1578052
From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
Date: Fri, 10 Feb 2017 00:54:24 +0200

> On Thu, Feb 09, 2017 at 05:21:26PM -0500, David Miller wrote:
>> From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
>> Date: Thu,  9 Feb 2017 02:07:34 +0200
>> 
>> > These two patches fix suspend/resume chain.
>> 
>> Patch 2 doesn't apply cleanly to the 'net' tree, please
>> respin this series.
> 
> Strange, I've just checked it on net-next/master, it was applied w/o any
> warnings.

It makes no sense to test "net-next" when I am telling you that it is
the "net" tree it doesn't apply to.

This is a bug fix, so it should be targetting the "net" tree.

[toc] | [prev] | [next] | [standalone]

#1578667

FromGrygorii Strashko <grygorii.strashko@ti.com>
Date2017-02-10 19:10 +0100
Message-ID<t9nNU-6VN-37@gated-at.bofh.it>
In reply to#1578100

On 02/09/2017 07:45 PM, David Miller wrote:
> From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
> Date: Fri, 10 Feb 2017 00:54:24 +0200
>
>> On Thu, Feb 09, 2017 at 05:21:26PM -0500, David Miller wrote:
>>> From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
>>> Date: Thu,  9 Feb 2017 02:07:34 +0200
>>>
>>>> These two patches fix suspend/resume chain.
>>>
>>> Patch 2 doesn't apply cleanly to the 'net' tree, please
>>> respin this series.
>>
>> Strange, I've just checked it on net-next/master, it was applied w/o any
>> warnings.
>
> It makes no sense to test "net-next" when I am telling you that it is
> the "net" tree it doesn't apply to.
>
> This is a bug fix, so it should be targetting the "net" tree.
>

Looks like the first fix is for net, but the second one is for net-next
I do not see
03fd01ad0eead23eb79294b6fb4d71dcac493855
"net: ethernet: ti: cpsw: don't duplicate ndev_running"
in net.

-- 
regards,
-grygorii

[toc] | [prev] | [next] | [standalone]

#1578902

FromIvan Khoronzhuk <ivan.khoronzhuk@linaro.org>
Date2017-02-11 02:20 +0100
Message-ID<t9uw1-2C1-1@gated-at.bofh.it>
In reply to#1578667
On Fri, Feb 10, 2017 at 12:05:07PM -0600, Grygorii Strashko wrote:
> 
> 
> On 02/09/2017 07:45 PM, David Miller wrote:
> >From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
> >Date: Fri, 10 Feb 2017 00:54:24 +0200
> >
> >>On Thu, Feb 09, 2017 at 05:21:26PM -0500, David Miller wrote:
> >>>From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
> >>>Date: Thu,  9 Feb 2017 02:07:34 +0200
> >>>
> >>>>These two patches fix suspend/resume chain.
> >>>
> >>>Patch 2 doesn't apply cleanly to the 'net' tree, please
> >>>respin this series.
> >>
> >>Strange, I've just checked it on net-next/master, it was applied w/o any
> >>warnings.
> >
> >It makes no sense to test "net-next" when I am telling you that it is
> >the "net" tree it doesn't apply to.
> >
> >This is a bug fix, so it should be targetting the "net" tree.
> >
> 
> Looks like the first fix is for net, but the second one is for net-next
> I do not see
> 03fd01ad0eead23eb79294b6fb4d71dcac493855
> "net: ethernet: ti: cpsw: don't duplicate ndev_running"
> in net.

There is dependency, both for net-next and only first is for net tree

> 
> -- 
> regards,
> -grygorii

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web