From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from devico.uberspace.de (devico.uberspace.de [185.26.156.185]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 47E128F66 for ; Sat, 1 Apr 2023 21:00:33 +0000 (UTC) Received: (qmail 1969 invoked by uid 990); 1 Apr 2023 20:53:49 -0000 Authentication-Results: devico.uberspace.de; auth=pass (plain) Precedence: bulk X-Mailing-List: regressions@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Date: Sat, 01 Apr 2023 20:53:48 +0000 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable From: "Leonard Lausen" Message-ID: TLS-Required: No Subject: Re: [PATCH] Revert "venus: firmware: Correct non-pix start and end addresses" To: "Javier Martinez Canillas" , "Dikshita Agarwal" , "Vikash Garodia" , "Linux regressions mailing list" , "Stanimir Varbanov" Cc: linux-kernel@vger.kernel.org, mka@chromium.org, "Albert Esteve" , stanimir.varbanov@linaro.org, "Enric Balletbo i Serra" , "Andy Gross" , "Bjorn Andersson" , "Konrad Dybcio" , "Vikash Garodia (QUIC)" , linux-arm-msm@vger.kernel.org, linux-media@vger.kernel.org, "Fritz Koenig" , "Rajeshwar Kurapaty (QUIC)" , "Mauro Carvalho Chehab" In-Reply-To: <87edq2dus1.fsf@minerva.mail-host-address-is-not-set> References: <87edq2dus1.fsf@minerva.mail-host-address-is-not-set> <20230207102254.1446461-1-javierm@redhat.com> <70c01751-1dd7-c4bd-a96e-94dea437aa40@redhat.com> <6f97a117-0d9c-e21b-9adf-50f2233ba9e3@leemhuis.info> <9a0bfef8-0b5d-f4d0-a8a5-4bbcacc5c0fb@leemhuis.info> <87356wn6xf.fsf@minerva.mail-host-address-is-not-set> <87edq9hj4w.fsf@minerva.mail-host-address-is-not-set> <0c84724d-08d4-ddcb-5f71-4eb8261240c6@quicinc.com> X-Rspamd-Bar: - X-Rspamd-Report: MIME_GOOD(-0.1) BAYES_HAM(-2.911118) SUSPICIOUS_RECIPS(1.5) X-Rspamd-Score: -1.511118 Received: from unknown (HELO unkown) (::1) by devico.uberspace.de (Haraka/3.0.1) with ESMTPSA; Sat, 01 Apr 2023 22:53:49 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=lausen.nl; s=uberspace; h=from; bh=aA2msNrvcbbehcgKux4cUf7WB087VIC6mCd6AEtUyEQ=; b=s4XtsGVE1M/EVfLROVnFJKY2m/ZJILlhO+a2J5zOVsO9r/5JzqHXwZOaORDxQewy+/oLt/E13Z HvywSLWvByzGU9FYoycym8dl/qvMUXsLGg76aiYxtRz6nMJTK6ObltrPRW4UXHjh49Nt5/w6snor uZ/vF9mVBfnhYTOKqsh/167Ga0AzjuczYNQVXHHPAxfb022TAWlmScJj1evxBryqU3t/8XGf4OjU Tdqcrynq20Xdv+DjkzxvJ0KElIi6CbYvd4/C9S+wu4TcVGEpjZNGByI6Ht1xE3auxBpkHazadRza KR8P07gnacvy3pxinaTz10ClHrkxtuqMFuqmbMV6eemFAm8+gKf7uuy7FHHbV4g8QSrVqiokM3VD VOCU7qDxTb5ddA+BTgHMOEoeNUfMe2ZxeZVCOv1CC11c9jzKHtGUxu3IGfEB+pqykhVSd1OMmvnu qSt+lruwuDGYb6fnmI0Spko/H8WYomjH8zE3F1O7TiAA06PjkpoZfXlmrvo6x0v/QU7n0vbGgEZq RrgV4YDpxPtJ3yzXK3VxTaxKk7VOA+LNmz/wZ8hvh3BFY/S7poQ+cJzfBkX6BGMtxd+Lgxu/wcr1 IYadB4A2ajHkUHnNaYkufpaNtqKB/5pnw29CNGAcn+DPkO7f5BvSS5HMdBncjiuJyuaXzX/ozbfD c= Hi Javier, Dikshita, Stan, the revert wasn't applied to v6.2 series. Can you please apply it and inc= lude it for v6.2.10? March 6, 2023 at 5:43 AM, "Javier Martinez Canillas" = wrote: >> On 3/1/2023 3:15 PM, Dikshita Agarwal wrote: >>> On 2/28/2023 9:33 PM, Javier Martinez Canillas wrote: >>>> Javier Martinez Canillas writes: >>>>> Vikash Garodia writes: >>>>> >>>>>> Stan, could you please help with the revert and a pull request hav= ing this revert >>>>>> alongwith other pending changes ? >>>>>> >>>>> Other fix posted is "media: venus: dec: Fix capture formats enumera= tion order": >>>>> >>>>> https://patchwork.kernel.org/project/linux-media/patch/202302100818= 35.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. While "media: venus: dec: Fix capture formats enumeration order" may have= been applied to v6.3, this still leaves the regression introduced by "venus: firmware: Correct non-pix start and end addresses". As pointed out by Mat= thias Kaehlcke, the commit prevents SC7180 and sc7280 AOSS from entering sleep = mode during system suspend. This is a serious regression in v6.2 kernel series= . Best regards, Leonard Lausen