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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id EB873C433EF for ; Mon, 15 Nov 2021 14:44:44 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D00CE61BE1 for ; Mon, 15 Nov 2021 14:44:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234216AbhKOOrg (ORCPT ); Mon, 15 Nov 2021 09:47:36 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59202 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232125AbhKOOqy (ORCPT ); Mon, 15 Nov 2021 09:46:54 -0500 Received: from mail-il1-x12c.google.com (mail-il1-x12c.google.com [IPv6:2607:f8b0:4864:20::12c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5B58CC06120B for ; Mon, 15 Nov 2021 06:43:43 -0800 (PST) Received: by mail-il1-x12c.google.com with SMTP id i12so16887833ila.12 for ; Mon, 15 Nov 2021 06:43:43 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ndufresne-ca.20210112.gappssmtp.com; s=20210112; h=message-id:subject:from:to:cc:date:in-reply-to:references :user-agent:mime-version:content-transfer-encoding; bh=JfI3JoXz0Ca1OgsWEHyo6RS3UZraM1AMQfqSQejQjIo=; b=EBIcyDEqfb8Y3brMg/s64Zs7YM6DOuNsOA0EX1xqAl6dxUcFFBBLbUpWS/d8vjHk+a z1svOa2SM8nlola2pw6iU6vEkk8oi16VJAZ2GZX6KodNhW5XkLYx9cAQRipwuVoG4HLI OPDlJkfElrYm1BJo0jadYYXreXomivz2vs0ZvICu+FrXYcQ8dRQoczdnS9/ozcdNQ14D e2ooYa2baQbWot0HL/bZh/MqYuZeYgjGRoECYsZDMwgDORUZC5zPNdyJztt2su9dZvMP dyb1zu427uumWkeW4COKrCQEtrwsvREwSnY0U48YQwC9d2M/I9CTvs1fg1QPdDBw5fcp 31zg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to :references:user-agent:mime-version:content-transfer-encoding; bh=JfI3JoXz0Ca1OgsWEHyo6RS3UZraM1AMQfqSQejQjIo=; b=q51aR58Wr4GexivIrfthi6c5Q+it5xKB8+d/ypWTgqA/IU9YAeIIQ5KlpRhCcSZfxc XXqVqimsN/Pjmd3G7L1V04A9JY5jEhDrxOeBSZLOyASfBPT6yaidkqJQyHttk+WQWbDK cknt8Yi8/XsXkgjLW1pofJJIOmSMMdjwqz42rzbQXl4VniQzvrqyOSUSd0iS/SbzTe6c pD8lFk8xsakEEgVZeIrgJZKasIzo3gmc59iv5QQM/hI4DXY6525tdPCG/Pr0F8S23rLq +82wFLp5A359S7EAxwHbJ15rCHEWJzEKm+yBLJ7mlL7IYzJtY3n8pRFSP46ipOLi/3E0 c2+w== X-Gm-Message-State: AOAM532V4L8kaasHKCN4MkeG9JimeilolbctSNIc0o4ozY8gKVip2jtP 9Ue9Brl7Wf4zT/nfWd5edlM98Q== X-Google-Smtp-Source: ABdhPJz2yCZ0EINqTWjSs3tzPzMvPN86GdXRv8KNX4/5f+hxt1DRdnl/BVl1YvBWMPNlUjsZEE/hQw== X-Received: by 2002:a05:6e02:178a:: with SMTP id y10mr21823246ilu.257.1636987422853; Mon, 15 Nov 2021 06:43:42 -0800 (PST) Received: from nicolas-tpx395.localdomain (mtl.collabora.ca. [66.171.169.34]) by smtp.gmail.com with ESMTPSA id o10sm8997947ilc.56.2021.11.15.06.43.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 15 Nov 2021 06:43:42 -0800 (PST) Message-ID: <1411d7a3913677fee58e103f7610e122686e87bf.camel@ndufresne.ca> Subject: Re: [PATCH v2 0/3] Add NVIDIA Tegra114 support to video decoder driver From: Nicolas Dufresne To: Dmitry Osipenko , Thierry Reding , Jonathan Hunter , Mauro Carvalho Chehab , Greg Kroah-Hartman , Anton Bambura , Hans Verkuil Cc: linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org Date: Mon, 15 Nov 2021 09:43:40 -0500 In-Reply-To: <20211114224732.11550-1-digetx@gmail.com> References: <20211114224732.11550-1-digetx@gmail.com> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.42.1 (3.42.1-1.fc35) MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Dmitry, Le lundi 15 novembre 2021 à 01:47 +0300, Dmitry Osipenko a écrit : > Video decoder of Tegra114/124 SoCs uses additional memory buffer required > for decoding of protected content. We won't support that content, but it > is impossible to disable access to the buffer, hence a stub buffer needs > to be provided. This series enables decoder driver only for Tegra114 > because Tegra124 support requires more non-trivial changes on both kernel > and userspace sides. I believe the stateless API is quite in place now, but I only see maintenance on this staging driver. I don't believe it really make sense to keep maintaining a staging driver without any step forward de-staging it. I believe it gives the wrong message on the Kernel staging purpose. I'm not criticizing your effort, I believe you are doing nice work for you community, but would prefer to see this driver be ported to the official kernel APIs rather then being maintain as staging till the end of time. regards, Nicolas > > Changelog: > > v2: - Changed tegra_vde_alloc_bo() to return errno and fix unassigned > error code in tegra_vde_probe(). > > Dmitry Osipenko (1): > media: staging: tegra-vde: Reorder misc device registration > > Thierry Reding (2): > media: staging: tegra-vde: Support reference picture marking > media: staging: tegra-vde: Properly mark invalid entries > > drivers/staging/media/tegra-vde/vde.c | 147 +++++++++++++++++++++++--- > drivers/staging/media/tegra-vde/vde.h | 18 ++++ > 2 files changed, 152 insertions(+), 13 deletions(-) >