Re: [GIT FIXES FOR v6.3] Venus fixes

[Stanimir Varbanov <stanimir.k.varbanov@gmail.com>]

Hi,

On 13.03.23 г. 16:28 ч., Thorsten Leemhuis wrote:
> On 02.03.23 09:27, Linux regression tracking (Thorsten Leemhuis) wrote:
>> On 02.03.23 07:04, Stanimir Varbanov wrote:
>>>
>>> This pull request includes a fix for regression in venus hardware reset.
>>> The reverted commit has been merged in v6.2.
>>>
>>> Please pull.
> 
> Mauro: any reason why this is still not pulled? It fixes a regression
> that made it into 6.2 and the revert was now posted 34 days ago, hence
> it would be good if this could finally go to Linus, so that it can be
> backported to stable.

I guess the best I can do is to include the revert in the next pull 
request for v6.4.

> 
> BTW, Stan: What...
> 
>>> The following changes since commit 3e62aba8284de0994a669d07983299242e68fe72:
>>>
>>>    media: imx-mipi-csis: Check csis_fmt validity before use (2023-02-26 11:21:33 +0100)
>>>
>>> are available in the Git repository at:
>>>
>>>    git://linuxtv.org/svarbanov/media_tree.git tags/tag-venus-fixes-for-v6.3
>>>
>>> for you to fetch changes up to 1440cfcf24db8c50d929d3c35ab6f87f868fa628:
>>>
>>>    Revert "venus: firmware: Correct non-pix start and end addresses" (2023-03-02 07:52:10 +0200)
>>>
>>> ----------------------------------------------------------------
>>> Venus fixes for v6.3
>>>
>>> ----------------------------------------------------------------
>>> Javier Martinez Canillas (1):
>>>        Revert "venus: firmware: Correct non-pix start and end addresses"
>>
>> Good to see that this finally is heading towards mainline, thx.
>>
>> What about the other venus regression[1] Javier provided this patch for:
>>
>> https://patchwork.kernel.org/project/linux-media/patch/20230210081835.2054482-1-javierm@redhat.com/
> 
> ...about this revert from Feb, 10th? There was some discussion recently,
> but it would to get this finally resolved, too.

I still have doubts about test coverage on db410c and RB3, because the 
firmware there behaves very differently and we can introduce another 
regression. If that happens we have to fix the fix in this patch. I'm 
working on the testing on those two devices.

> 
> Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker' hat)
> --
> Everything you wanna know about Linux kernel regression tracking:
> https://linux-regtracking.leemhuis.info/about/#tldr
> If I did something stupid, please tell me, as explained on that page.
> 
> #regzbot poke
> #regzbot ignore-activity

-- 
regards,
Stan