Re: Re: [PATCH] PM / devfreq: tegra: Set freq in rate callback

Re: Re: [PATCH] PM / devfreq: tegra: Set freq in rate callback

[MyungJoo Ham]

> Hi Thierry,
> 
> On 21 January 2016 at 02:54, Thierry Reding <thierry.reding@gmail.com> wrote:
> > On Thu, Jan 21, 2016 at 08:52:26AM +0100, Tomeu Vizoso wrote:
[]
> >> Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
> >> Reported-by: Tyler Baker <tyler.baker@linaro.org>
[]
> >>
> >
> > I had noticed it recently but hadn't got around to look into it, thanks
> > for fixing it.
> >
> > Tested-by: Thierry Reding <treding@nvidia.com>
> > Acked-by: Thierry Reding <treding@nvidia.com>
> 
> I don't see this patch in -next yet, so I'm guess it hasn't been
> applied. Any chance you could pick it up? This issue is now
> manifesting in mainline, and causing ton of noise in the boot reports.
> 
> Cheers,
> 
> Tyler


Hey, the patch appeared at the mailing list a week ago.
Linux 4.5-rc1 appeared 3 days ago.
The fastest possible for mainline inclusion is 4.5-rc2 or 3.

You cannot simply expect the patch to be included in the maiinline that fast.

Re: Re: [PATCH] PM / devfreq: tegra: Set freq in rate callback

[Tyler Baker]

On 27 January 2016 at 20:14, MyungJoo Ham <myungjoo.ham-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> wrote:
>> Hi Thierry,
>>
>> On 21 January 2016 at 02:54, Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>> > On Thu, Jan 21, 2016 at 08:52:26AM +0100, Tomeu Vizoso wrote:
> []
>> >> Signed-off-by: Tomeu Vizoso <tomeu.vizoso-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>
>> >> Reported-by: Tyler Baker <tyler.baker-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> []
>> >>
>> >
>> > I had noticed it recently but hadn't got around to look into it, thanks
>> > for fixing it.
>> >
>> > Tested-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>> > Acked-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>
>> I don't see this patch in -next yet, so I'm guess it hasn't been
>> applied. Any chance you could pick it up? This issue is now
>> manifesting in mainline, and causing ton of noise in the boot reports.
>>
>> Cheers,
>>
>> Tyler
>
>
> Hey, the patch appeared at the mailing list a week ago.
> Linux 4.5-rc1 appeared 3 days ago.
> The fastest possible for mainline inclusion is 4.5-rc2 or 3.
>
> You cannot simply expect the patch to be included in the maiinline that fast.

I agree, and I was not asking for this patch to be included into
mainline yet, just that it would be applied to maintainers tree so it
could be included in linux-next. I want to ensure this issue is not
masking other issues in linux-next.

Cheers,

Tyler

Re: Re: [PATCH] PM / devfreq: tegra: Set freq in rate callback

[Tomeu Vizoso]

On 28 January 2016 at 05:14, MyungJoo Ham <myungjoo.ham@samsung.com> wrote:
>> Hi Thierry,
>>
>> On 21 January 2016 at 02:54, Thierry Reding <thierry.reding@gmail.com> wrote:
>> > On Thu, Jan 21, 2016 at 08:52:26AM +0100, Tomeu Vizoso wrote:
> []
>> >> Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
>> >> Reported-by: Tyler Baker <tyler.baker@linaro.org>
> []
>> >>
>> >
>> > I had noticed it recently but hadn't got around to look into it, thanks
>> > for fixing it.
>> >
>> > Tested-by: Thierry Reding <treding@nvidia.com>
>> > Acked-by: Thierry Reding <treding@nvidia.com>
>>
>> I don't see this patch in -next yet, so I'm guess it hasn't been
>> applied. Any chance you could pick it up? This issue is now
>> manifesting in mainline, and causing ton of noise in the boot reports.
>>
>> Cheers,
>>
>> Tyler
>
>
> Hey, the patch appeared at the mailing list a week ago.
> Linux 4.5-rc1 appeared 3 days ago.
> The fastest possible for mainline inclusion is 4.5-rc2 or 3.
>
> You cannot simply expect the patch to be included in the maiinline that fast.

Hi MyungJoo Ham,

the problem is that linux-next has been showing that error in
multi_v7_defconfig builds since your changes got there. This is so
annoying that people have been putting their tegra124 boards offline
in their CI labs until this gets fixed in linux-next.

Regards,

Tomeu