Messages in this thread | | | From | Javier Martinez Canillas <> | Subject | Re: [PATCH] Revert "venus: firmware: Correct non-pix start and end addresses" | Date | Mon, 06 Mar 2023 11:43:58 +0100 |
| |
Dikshita Agarwal <quic_dikshita@quicinc.com> writes:
Hello Dikshita,
> On 3/1/2023 3:15 PM, Dikshita Agarwal wrote: >> >> >> On 2/28/2023 9:33 PM, Javier Martinez Canillas wrote: >>> Javier Martinez Canillas<javierm@redhat.com> writes: >>> >>>> Vikash Garodia<vgarodia@qti.qualcomm.com> writes: >>>> >>>> Hello Vikash, >>>> >>>>> Hi All, >>>>> >>>> [...] >>>> >>>>>> No reply from Mauro and Linus chose to not apply the revert I pointed him to. >>>>>> That at this point leads to the question: >>>>>> >>>>>> Vikash, did you or somebody else make any progress to fix this properly? >>>>> We tried with different settings for the registers and arrive at a conclusion that >>>>> the original configuration was proper. There is no need to explicitly configure >>>>> the secure non-pixel region when there is no support for the usecase. So, in summary, >>>>> we are good to have the revert. >>>>> >>>> Perfect. Thanks a lot for looking at this. >>>> >>>>> Stan, could you please help with the revert and a pull request having this revert >>>>> alongwith other pending changes ? >>>>> >>>> Other fix posted is "media: venus: dec: Fix capture formats enumeration order": >>>> >>>> https://patchwork.kernel.org/project/linux-media/patch/20230210081835.2054482-1-javierm@redhat.com/ > > Hi Javier, > > Thanks for this patch "media: venus: dec: Fix capture formats > enumeration order". > > Somehow I can't find it in my mailbox to be able to reply there. > > Could you please explain what is the regression you see here? >
You can find the thread and explanation of the issue here:
https://lore.kernel.org/lkml/Y+KPW18o%2FDa+N8UI@google.com/T/
But Stanimir already picked it and sent a PR for v6.3 including it.
-- Best regards,
Javier Martinez Canillas Core Platforms Red Hat
| |