From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id EA935C6FD1C for ; Wed, 22 Mar 2023 14:14:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231201AbjCVOOW (ORCPT ); Wed, 22 Mar 2023 10:14:22 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42370 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230222AbjCVOOW (ORCPT ); Wed, 22 Mar 2023 10:14:22 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 92C992449E for ; Wed, 22 Mar 2023 07:13:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1679494410; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=gIhIZ9iTRojGJMYiKTfmFBTTGDFbr5/gvROIxAATKTs=; b=HzFk/p6NUia5JN1Ql8il0qnf/ZeKo4AmC6FPEARhsjZCfvc6V0G7OF3Aat55OSnFa/FrVh CAU0WKA3yQ7eFzwgB4A6OaQZaIlyfPCHkqUbFaWm5V+PFXTp3ZSr37mr3YCC5IZT74rZHM omOZuK2zPtTD7RDK/zk4nF757VSu3EU= Received: from mail-wm1-f69.google.com (mail-wm1-f69.google.com [209.85.128.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-614-GTpy4DkfPNqn6tt0uQy5Eg-1; Wed, 22 Mar 2023 10:13:25 -0400 X-MC-Unique: GTpy4DkfPNqn6tt0uQy5Eg-1 Received: by mail-wm1-f69.google.com with SMTP id m27-20020a05600c3b1b00b003ee502f1b16so1605196wms.9 for ; Wed, 22 Mar 2023 07:13:25 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1679494404; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=gIhIZ9iTRojGJMYiKTfmFBTTGDFbr5/gvROIxAATKTs=; b=2b81sEjQ2oBfnRHk6U7RW/mRtf98QILITykW2ZP4/wkdtvIZkjAW127d4D7/A2w636 9VfbqI6aB1bVoMOn6M69iskk9c3u1ofTWQu9iIDW/eP6WWQQVXf2WQoRLrNo/AzcygBB xH9zdiXYlcILFXx8TxrA5IvSkpBTcHvFpXeLxAkfywulxiGBJpx0wRrXqdajDOx5KC0y x+a59cT1Wgj1OqP2+o4wJHJkGE4el3n+LFW9YXB5PAGxNtfs1/fdss1Tn5CssMdQMGYh 3FLPfVESyfXPC7PhdPWsBAFxxqfQRrxiMy7kEpOM6B1xiSZon6qlwuWe2Nsn9CIs1lzi +8Ig== X-Gm-Message-State: AAQBX9d+Zw7AcMUP6fN48C23LjuYZOC9CcpxiBeE8pLyK1p1mjAhrlxV YBFTwIBXyOw4StktsyyQLf371B6dUqQGaK3prSSryW7PZz8OA296riY4jsEp1VYambDIUjFbmWB 5jJakGMeiKCLZc2QsJWpKHHg= X-Received: by 2002:a05:6000:1cb:b0:2d3:f610:8410 with SMTP id t11-20020a05600001cb00b002d3f6108410mr48478wrx.0.1679494404215; Wed, 22 Mar 2023 07:13:24 -0700 (PDT) X-Google-Smtp-Source: AKy350bwPM1I+KdwU+k+LnKMhL8L8AvemWt4BBwZWovdn++jO3C4QynGXhNPsmQm5XFePAlaJEWMPA== X-Received: by 2002:a05:6000:1cb:b0:2d3:f610:8410 with SMTP id t11-20020a05600001cb00b002d3f6108410mr48464wrx.0.1679494403910; Wed, 22 Mar 2023 07:13:23 -0700 (PDT) Received: from localhost (205.pool92-176-231.dynamic.orange.es. [92.176.231.205]) by smtp.gmail.com with ESMTPSA id c15-20020adfef4f000000b002c7066a6f77sm13914075wrp.31.2023.03.22.07.13.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 22 Mar 2023 07:13:23 -0700 (PDT) From: Javier Martinez Canillas To: Jordan Crouse , Enric Balletbo i Serra Cc: Dikshita Agarwal , linux-kernel@vger.kernel.org, Albert Esteve , Stanimir Varbanov , Sergio Lopez , Andy Gross , Bjorn Andersson , Konrad Dybcio , Mauro Carvalho Chehab , Stanimir Varbanov , Vikash Garodia , linux-arm-msm@vger.kernel.org, linux-media@vger.kernel.org Subject: Re: [PATCH] media: venus: dec: Fix capture formats enumeration order In-Reply-To: <87ttyu54wy.fsf@minerva.mail-host-address-is-not-set> References: <20230210081835.2054482-1-javierm@redhat.com> <20230303220918.qr5ydbin3nye3qtz@amazon.com> <87h6uydwel.fsf@minerva.mail-host-address-is-not-set> <3d0315fa-14ca-dc34-81ae-467d9ed5133d@quicinc.com> <87sfeh0yjn.fsf@minerva.mail-host-address-is-not-set> <20230308181245.nbnwkdtdnsldd65l@amazon.com> <87ttyu54wy.fsf@minerva.mail-host-address-is-not-set> Date: Wed, 22 Mar 2023 15:13:22 +0100 Message-ID: <87bkkkoovx.fsf@minerva.mail-host-address-is-not-set> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Javier Martinez Canillas writes: Hello Stanimir and Dikshita, > Jordan Crouse writes: > >> On Tue, Mar 07, 2023 at 05:20:18PM +0100, Enric Balletbo i Serra wrote: > > [...] > >>> > >>> > But regardless, I think that it would be better for a driver to >>> > not change the order of advertised VIDIOC_ENUM_FMT pixel formats. >>> > >>> > Because what happens now is that a decoding that was previously >>> > working by default is not working anymore due a combination of >>> > the default being changed and S_FMT not working as expected. >> >> For my part, I was using the gstreamer v4l2 decoder which for some reason tries >> to verify it can support whatever format it gets with G_FMT *before* >> trying a S_FMT. I can't confirm or deny if S_FMT currently works or not. >> >> That said, I entirely agree with Javier. While it might be more >> bandwidth efficient, QC08C is a obscure format. It is far more likely that the >> average open source user would rather use a well known output format and, as >> has been mentioned, once S_FMT is fixed those in the know can use the other >> formats if they are working with other Qualcomm hardware blocks. >> > > Agreed. The rule is that the kernel shouldn't regress user-space and the > patches that changed the default format certainly did that. So from that > point of view I think that this patch should land. > > There's also Enric's point that NV12 is a more common format and supported > by more user-space programs. That's why think that regardless of the S_FMT > situation, makes sense to revert to the previous default behaviour. > Any news on this patch? It would be great to fix this at least for v6.3. -- Best regards, Javier Martinez Canillas Core Platforms Red Hat