linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Javier Martinez Canillas <javierm@redhat.com>
To: Dikshita Agarwal <quic_dikshita@quicinc.com>,
	Vikash Garodia <vgarodia@qti.qualcomm.com>,
	Linux regressions mailing list <regressions@lists.linux.dev>,
	Stanimir Varbanov <stanimir.k.varbanov@gmail.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"mka@chromium.org" <mka@chromium.org>,
	Albert Esteve <aesteve@redhat.com>,
	"stanimir.varbanov@linaro.org" <stanimir.varbanov@linaro.org>,
	Enric Balletbo i Serra <eballetb@redhat.com>,
	Andy Gross <agross@kernel.org>,
	Bjorn Andersson <andersson@kernel.org>,
	Konrad Dybcio <konrad.dybcio@linaro.org>,
	"Vikash Garodia (QUIC)" <quic_vgarodia@quicinc.com>,
	"linux-arm-msm@vger.kernel.org" <linux-arm-msm@vger.kernel.org>,
	"linux-media@vger.kernel.org" <linux-media@vger.kernel.org>,
	Fritz Koenig <frkoenig@google.com>,
	"Rajeshwar Kurapaty (QUIC)" <quic_rkurapat@quicinc.com>,
	Mauro Carvalho Chehab <mchehab@kernel.org>
Subject: Re: [PATCH] Revert "venus: firmware: Correct non-pix start and end addresses"
Date: Mon, 06 Mar 2023 11:43:58 +0100	[thread overview]
Message-ID: <87edq2dus1.fsf@minerva.mail-host-address-is-not-set> (raw)
In-Reply-To: <0c84724d-08d4-ddcb-5f71-4eb8261240c6@quicinc.com>

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


  parent reply	other threads:[~2023-03-06 10:44 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-07 10:22 [PATCH] Revert "venus: firmware: Correct non-pix start and end addresses" Javier Martinez Canillas
2023-02-07 16:40 ` Vikash Garodia
2023-02-07 17:50   ` mka
2023-02-07 22:39     ` mka
2023-02-08  9:06   ` Javier Martinez Canillas
2023-02-10  8:15     ` Javier Martinez Canillas
2023-02-10  9:22       ` Vikash Garodia
2023-02-10 10:07         ` Javier Martinez Canillas
2023-02-11 14:27           ` Linux regression tracking (Thorsten Leemhuis)
2023-02-15 10:53             ` Linux regression tracking (Thorsten Leemhuis)
2023-02-15 10:57               ` Javier Martinez Canillas
2023-02-15 13:18                 ` Linux regression tracking (Thorsten Leemhuis)
2023-02-21 15:03                   ` Thorsten Leemhuis
2023-02-23  5:45                     ` Vikash Garodia
2023-02-23  8:05                       ` Javier Martinez Canillas
2023-02-28 16:03                         ` Javier Martinez Canillas
     [not found]                           ` <d18fac76-6b77-a446-5fe0-7236556e9187@quicinc.com>
     [not found]                             ` <0c84724d-08d4-ddcb-5f71-4eb8261240c6@quicinc.com>
2023-03-06 10:43                               ` Javier Martinez Canillas [this message]
2023-04-01 20:53                               ` Leonard Lausen
2023-04-02  5:02                                 ` Linux regression tracking (Thorsten Leemhuis)
2023-04-03  6:32                                   ` Thorsten Leemhuis
2023-04-03  0:27                                 ` Leonard Lausen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87edq2dus1.fsf@minerva.mail-host-address-is-not-set \
    --to=javierm@redhat.com \
    --cc=aesteve@redhat.com \
    --cc=agross@kernel.org \
    --cc=andersson@kernel.org \
    --cc=eballetb@redhat.com \
    --cc=frkoenig@google.com \
    --cc=konrad.dybcio@linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@kernel.org \
    --cc=mka@chromium.org \
    --cc=quic_dikshita@quicinc.com \
    --cc=quic_rkurapat@quicinc.com \
    --cc=quic_vgarodia@quicinc.com \
    --cc=regressions@lists.linux.dev \
    --cc=stanimir.k.varbanov@gmail.com \
    --cc=stanimir.varbanov@linaro.org \
    --cc=vgarodia@qti.qualcomm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).