2015-02-10 23:10:27

by Paul Walmsley

[permalink] [raw]
Subject: Re: [PATCH v3] ARM: AM43xx: hwmod: add VPFE hwmod entries

On Wed, 28 Jan 2015, Benoit Parrot wrote:

> Suspend/resume is functional with this patch.
>
> Tested-by: Benoit Parrot <[email protected]>

Thanks folks, queued for v3.21.


- Paul


2015-04-10 22:34:57

by Lad, Prabhakar

[permalink] [raw]
Subject: Re: [PATCH v3] ARM: AM43xx: hwmod: add VPFE hwmod entries

Hi Paul,

On Tue, Feb 10, 2015 at 11:10 PM, Paul Walmsley <[email protected]> wrote:
> On Wed, 28 Jan 2015, Benoit Parrot wrote:
>
>> Suspend/resume is functional with this patch.
>>
>> Tested-by: Benoit Parrot <[email protected]>
>
> Thanks folks, queued for v3.21.
>
>
I see that this patch is not into linux-next yet.

Cheers,
--Prabhakar Lad

2015-04-10 22:51:36

by Paul Walmsley

[permalink] [raw]
Subject: Re: [PATCH v3] ARM: AM43xx: hwmod: add VPFE hwmod entries

Hi Prabhakar

On Fri, 10 Apr 2015, Lad, Prabhakar wrote:

> Hi Paul,
>
> On Tue, Feb 10, 2015 at 11:10 PM, Paul Walmsley <[email protected]> wrote:
> > On Wed, 28 Jan 2015, Benoit Parrot wrote:
> >
> >> Suspend/resume is functional with this patch.
> >>
> >> Tested-by: Benoit Parrot <[email protected]>
> >
> > Thanks folks, queued for v3.21.
> >
> >
> I see that this patch is not into linux-next yet.

thanks for the ping. This slipped through the cracks here due to the
kernel version number change from 3.21 to 4.1 :-( Sorry about that; I
will requeue for either 4.1-rc or 4.2.

Unfortunately I don't have an AM43xx board. Is suspend/resume broken
without this patch? If so, then v4.1-rc seems like the appropriate
target.


- Paul

2015-04-10 22:58:01

by Lad, Prabhakar

[permalink] [raw]
Subject: Re: [PATCH v3] ARM: AM43xx: hwmod: add VPFE hwmod entries

Hi Paul,

On Fri, Apr 10, 2015 at 11:51 PM, Paul Walmsley <[email protected]> wrote:
> Hi Prabhakar
>
> On Fri, 10 Apr 2015, Lad, Prabhakar wrote:
>
>> Hi Paul,
>>
>> On Tue, Feb 10, 2015 at 11:10 PM, Paul Walmsley <[email protected]> wrote:
>> > On Wed, 28 Jan 2015, Benoit Parrot wrote:
>> >
>> >> Suspend/resume is functional with this patch.
>> >>
>> >> Tested-by: Benoit Parrot <[email protected]>
>> >
>> > Thanks folks, queued for v3.21.
>> >
>> >
>> I see that this patch is not into linux-next yet.
>
> thanks for the ping. This slipped through the cracks here due to the
> kernel version number change from 3.21 to 4.1 :-( Sorry about that; I
> will requeue for either 4.1-rc or 4.2.
>
> Unfortunately I don't have an AM43xx board. Is suspend/resume broken
> without this patch? If so, then v4.1-rc seems like the appropriate
> target.
>
there is kernel soft crashes without this patch, so this needs to go
in for v4.1-rc.

Cheers,
--Prabhakar Lad

2015-04-10 23:03:11

by Paul Walmsley

[permalink] [raw]
Subject: Re: [PATCH v3] ARM: AM43xx: hwmod: add VPFE hwmod entries

On Fri, 10 Apr 2015, Lad, Prabhakar wrote:

> On Fri, Apr 10, 2015 at 11:51 PM, Paul Walmsley <[email protected]> wrote:
> > Hi Prabhakar
> >
> > On Fri, 10 Apr 2015, Lad, Prabhakar wrote:
> >
> >> Hi Paul,
> >>
> >> On Tue, Feb 10, 2015 at 11:10 PM, Paul Walmsley <[email protected]> wrote:
> >> > On Wed, 28 Jan 2015, Benoit Parrot wrote:
> >> >
> >> >> Suspend/resume is functional with this patch.
> >> >>
> >> >> Tested-by: Benoit Parrot <[email protected]>
> >> >
> >> > Thanks folks, queued for v3.21.
> >> >
> >> >
> >> I see that this patch is not into linux-next yet.
> >
> > thanks for the ping. This slipped through the cracks here due to the
> > kernel version number change from 3.21 to 4.1 :-( Sorry about that; I
> > will requeue for either 4.1-rc or 4.2.
> >
> > Unfortunately I don't have an AM43xx board. Is suspend/resume broken
> > without this patch? If so, then v4.1-rc seems like the appropriate
> > target.
> >
> there is kernel soft crashes without this patch, so this needs to go
> in for v4.1-rc.

Could you provide some further detail? Does it crash during boot, or
during suspend, or ... ? Also could you describe what you mean by "soft
crash" ?


- Paul

2015-04-11 09:19:59

by Lad, Prabhakar

[permalink] [raw]
Subject: Re: [PATCH v3] ARM: AM43xx: hwmod: add VPFE hwmod entries

On Sat, Apr 11, 2015 at 12:03 AM, Paul Walmsley <[email protected]> wrote:
> On Fri, 10 Apr 2015, Lad, Prabhakar wrote:
>
>> On Fri, Apr 10, 2015 at 11:51 PM, Paul Walmsley <[email protected]> wrote:
>> > Hi Prabhakar
>> >
>> > On Fri, 10 Apr 2015, Lad, Prabhakar wrote:
>> >
>> >> Hi Paul,
>> >>
>> >> On Tue, Feb 10, 2015 at 11:10 PM, Paul Walmsley <[email protected]> wrote:
>> >> > On Wed, 28 Jan 2015, Benoit Parrot wrote:
>> >> >
>> >> >> Suspend/resume is functional with this patch.
>> >> >>
>> >> >> Tested-by: Benoit Parrot <[email protected]>
>> >> >
>> >> > Thanks folks, queued for v3.21.
>> >> >
>> >> >
>> >> I see that this patch is not into linux-next yet.
>> >
>> > thanks for the ping. This slipped through the cracks here due to the
>> > kernel version number change from 3.21 to 4.1 :-( Sorry about that; I
>> > will requeue for either 4.1-rc or 4.2.
>> >
>> > Unfortunately I don't have an AM43xx board. Is suspend/resume broken
>> > without this patch? If so, then v4.1-rc seems like the appropriate
>> > target.
>> >
>> there is kernel soft crashes without this patch, so this needs to go
>> in for v4.1-rc.
>
> Could you provide some further detail? Does it crash during boot, or
> during suspend, or ... ? Also could you describe what you mean by "soft
> crash" ?
>
with patch [1] applied and VPFE being enabled and this patch missing
(ARM: AM43xx: hwmod: add VPFE hwmod entries) I have attached the
boot log.
By soft crash I meant it doesn't get hung :)

With the above patch (ARM: AM43xx: hwmod: add VPFE hwmod entries)
applied all goes well.


[1] https://lkml.org/lkml/2015/3/12/1001

Cheers,
--Prabhakar Lad


Attachments:
boot-log.txt (330.33 kB)

2015-04-16 14:44:50

by Lad, Prabhakar

[permalink] [raw]
Subject: Re: [PATCH v3] ARM: AM43xx: hwmod: add VPFE hwmod entries

Hi Paul,

On Sat, Apr 11, 2015 at 10:19 AM, Lad, Prabhakar
<[email protected]> wrote:
> On Sat, Apr 11, 2015 at 12:03 AM, Paul Walmsley <[email protected]> wrote:
>> On Fri, 10 Apr 2015, Lad, Prabhakar wrote:
>>
>>> On Fri, Apr 10, 2015 at 11:51 PM, Paul Walmsley <[email protected]> wrote:
>>> > Hi Prabhakar
>>> >
>>> > On Fri, 10 Apr 2015, Lad, Prabhakar wrote:
>>> >
>>> >> Hi Paul,
>>> >>
>>> >> On Tue, Feb 10, 2015 at 11:10 PM, Paul Walmsley <[email protected]> wrote:
>>> >> > On Wed, 28 Jan 2015, Benoit Parrot wrote:
>>> >> >
>>> >> >> Suspend/resume is functional with this patch.
>>> >> >>
>>> >> >> Tested-by: Benoit Parrot <[email protected]>
>>> >> >
>>> >> > Thanks folks, queued for v3.21.
>>> >> >
>>> >> >
>>> >> I see that this patch is not into linux-next yet.
>>> >
>>> > thanks for the ping. This slipped through the cracks here due to the
>>> > kernel version number change from 3.21 to 4.1 :-( Sorry about that; I
>>> > will requeue for either 4.1-rc or 4.2.
>>> >
>>> > Unfortunately I don't have an AM43xx board. Is suspend/resume broken
>>> > without this patch? If so, then v4.1-rc seems like the appropriate
>>> > target.
>>> >
>>> there is kernel soft crashes without this patch, so this needs to go
>>> in for v4.1-rc.
>>
>> Could you provide some further detail? Does it crash during boot, or
>> during suspend, or ... ? Also could you describe what you mean by "soft
>> crash" ?
>>
> with patch [1] applied and VPFE being enabled and this patch missing
> (ARM: AM43xx: hwmod: add VPFE hwmod entries) I have attached the
> boot log.
> By soft crash I meant it doesn't get hung :)
>
> With the above patch (ARM: AM43xx: hwmod: add VPFE hwmod entries)
> applied all goes well.
>
>
> [1] https://lkml.org/lkml/2015/3/12/1001
>

Any update on this ?

Cheers,
--Prabhakar Lad

2015-05-01 09:01:33

by Lad, Prabhakar

[permalink] [raw]
Subject: Re: [PATCH v3] ARM: AM43xx: hwmod: add VPFE hwmod entries

On Fri, Apr 10, 2015 at 11:51 PM, Paul Walmsley <[email protected]> wrote:
> Hi Prabhakar
>
> On Fri, 10 Apr 2015, Lad, Prabhakar wrote:
>
>> Hi Paul,
>>
>> On Tue, Feb 10, 2015 at 11:10 PM, Paul Walmsley <[email protected]> wrote:
>> > On Wed, 28 Jan 2015, Benoit Parrot wrote:
>> >
>> >> Suspend/resume is functional with this patch.
>> >>
>> >> Tested-by: Benoit Parrot <[email protected]>
>> >
>> > Thanks folks, queued for v3.21.
>> >
>> >
>> I see that this patch is not into linux-next yet.
>
> thanks for the ping. This slipped through the cracks here due to the
> kernel version number change from 3.21 to 4.1 :-( Sorry about that; I
> will requeue for either 4.1-rc or 4.2.
>
> Unfortunately I don't have an AM43xx board. Is suspend/resume broken
> without this patch? If so, then v4.1-rc seems like the appropriate
> target.
>
>
Ping..

Cheers,
--Prabhakar Lad