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 X-Spam-Level: X-Spam-Status: No, score=-5.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A4426C433E0 for ; Wed, 17 Feb 2021 08:37:46 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 67F6C64DFF for ; Wed, 17 Feb 2021 08:37:46 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231622AbhBQIhb (ORCPT ); Wed, 17 Feb 2021 03:37:31 -0500 Received: from mail.kernel.org ([198.145.29.99]:47386 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230303AbhBQIhZ (ORCPT ); Wed, 17 Feb 2021 03:37:25 -0500 Received: by mail.kernel.org (Postfix) with ESMTPSA id 1AF9264E28; Wed, 17 Feb 2021 08:36:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1613551004; bh=i4kNSveyDKj4UzuU1mXSC8PvJCQPboPkNLn+8F+ab1k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=yv8i9+KWelFKyWLh1az5qyANS5nqW/AHNH77/f72B/vWnXtGRFkSTStnMXIzvn9+/ 7Yapooh4A9SgaLexotU/96Kqx2XvJlhLyMEQmHNANmx1G4weIrm/bwd1/Aq4aDsYwO o+5GaOV3bjjAb3IKdFImyUa6w0MOJR8ndzyMTx1U= Date: Wed, 17 Feb 2021 09:36:41 +0100 From: Greg KH To: Benjamin Gaignard Cc: peng.fan@nxp.com, kernel@collabora.com, devel@driverdev.osuosl.org, Anson.Huang@nxp.com, krzk@kernel.org, linux-rockchip@lists.infradead.org, wens@csie.org, linux-imx@nxp.com, linux-media@vger.kernel.org, devicetree@vger.kernel.org, p.zabel@pengutronix.de, s.hauer@pengutronix.de, mripard@kernel.org, robh+dt@kernel.org, mchehab@kernel.org, ezequiel@collabora.com, linux-arm-kernel@lists.infradead.org, aisheng.dong@nxp.com, jernej.skrabec@siol.net, adrian.ratiu@collabora.com, linux-kernel@vger.kernel.org, paul.kocialkowski@bootlin.com, kernel@pengutronix.de, hverkuil-cisco@xs4all.nl, shawnguo@kernel.org, shengjiu.wang@nxp.com Subject: Re: [PATCH v1 00/18] Add HANTRO G2/HEVC decoder support for IMX8MQ Message-ID: References: <20210217080306.157876-1-benjamin.gaignard@collabora.com> <04dfae0b-92e5-e02d-c687-ba4d28b7aaf2@collabora.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <04dfae0b-92e5-e02d-c687-ba4d28b7aaf2@collabora.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Feb 17, 2021 at 09:28:09AM +0100, Benjamin Gaignard wrote: > > Le 17/02/2021 à 09:08, Greg KH a écrit : > > On Wed, Feb 17, 2021 at 09:02:48AM +0100, Benjamin Gaignard wrote: > > > The IMX8MQ got two VPUs but until now only G1 has been enabled. > > > This series aim to add the second VPU (aka G2) and provide basic > > > HEVC decoding support. > > Why are you adding this directly to drivers/staging/media/ and not > > drivers/media/? Why can't this just go to the main location and not > > live in staging? > > G2/HEVC is added inside the already exiting Hantro driver, it is "just" > an other codec from Hantro driver point of view. > In addition of that v4l2-hevc uAPI is still unstable. > One goal of this series is to have one more consumer of this v4l2-hevc > uAPI so maybe we can claim it to be stable enough to move away from staging > and then do the same for Hantro driver. That would be a great achievement ! I know I do not like seeing new additions/features/whatever being added to staging drivers as that encourages people to do new stuff on them without doing the real work needed to get them out of staging. So what is preventing the existing driver from getting out of staging now? And how are you all creating new userspace apis for staging drivers to the v4l layer? What happens when you export something new and then userspace starts to rely on it and then you change it? Anyway, the media staging drivers are on their own, I don't touch them, it just feels odd to me... thanks, greg k-h 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 X-Spam-Level: X-Spam-Status: No, score=-3.5 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AC6EFC433DB for ; Wed, 17 Feb 2021 08:36:48 +0000 (UTC) Received: from fraxinus.osuosl.org (smtp4.osuosl.org [140.211.166.137]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 401F664DFF for ; Wed, 17 Feb 2021 08:36:48 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 401F664DFF Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=driverdev-devel-bounces@linuxdriverproject.org Received: from localhost (localhost [127.0.0.1]) by fraxinus.osuosl.org (Postfix) with ESMTP id E663585B3D; Wed, 17 Feb 2021 08:36:47 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from fraxinus.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 0FE4raU_czh4; Wed, 17 Feb 2021 08:36:46 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by fraxinus.osuosl.org (Postfix) with ESMTP id BCE568531B; Wed, 17 Feb 2021 08:36:46 +0000 (UTC) Received: from hemlock.osuosl.org (smtp2.osuosl.org [140.211.166.133]) by ash.osuosl.org (Postfix) with ESMTP id AD0A01BF59C for ; Wed, 17 Feb 2021 08:36:45 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by hemlock.osuosl.org (Postfix) with ESMTP id A936385E0D for ; Wed, 17 Feb 2021 08:36:45 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from hemlock.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id n9L4gHFE7geD for ; Wed, 17 Feb 2021 08:36:45 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.7.6 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by hemlock.osuosl.org (Postfix) with ESMTPS id 3D265859B6 for ; Wed, 17 Feb 2021 08:36:45 +0000 (UTC) Received: by mail.kernel.org (Postfix) with ESMTPSA id 1AF9264E28; Wed, 17 Feb 2021 08:36:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1613551004; bh=i4kNSveyDKj4UzuU1mXSC8PvJCQPboPkNLn+8F+ab1k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=yv8i9+KWelFKyWLh1az5qyANS5nqW/AHNH77/f72B/vWnXtGRFkSTStnMXIzvn9+/ 7Yapooh4A9SgaLexotU/96Kqx2XvJlhLyMEQmHNANmx1G4weIrm/bwd1/Aq4aDsYwO o+5GaOV3bjjAb3IKdFImyUa6w0MOJR8ndzyMTx1U= Date: Wed, 17 Feb 2021 09:36:41 +0100 From: Greg KH To: Benjamin Gaignard Subject: Re: [PATCH v1 00/18] Add HANTRO G2/HEVC decoder support for IMX8MQ Message-ID: References: <20210217080306.157876-1-benjamin.gaignard@collabora.com> <04dfae0b-92e5-e02d-c687-ba4d28b7aaf2@collabora.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <04dfae0b-92e5-e02d-c687-ba4d28b7aaf2@collabora.com> X-BeenThere: driverdev-devel@linuxdriverproject.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux Driver Project Developer List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: peng.fan@nxp.com, kernel@collabora.com, devel@driverdev.osuosl.org, Anson.Huang@nxp.com, krzk@kernel.org, linux-rockchip@lists.infradead.org, wens@csie.org, linux-imx@nxp.com, linux-media@vger.kernel.org, devicetree@vger.kernel.org, kernel@pengutronix.de, s.hauer@pengutronix.de, mripard@kernel.org, robh+dt@kernel.org, mchehab@kernel.org, ezequiel@collabora.com, linux-arm-kernel@lists.infradead.org, aisheng.dong@nxp.com, jernej.skrabec@siol.net, adrian.ratiu@collabora.com, linux-kernel@vger.kernel.org, paul.kocialkowski@bootlin.com, p.zabel@pengutronix.de, hverkuil-cisco@xs4all.nl, shawnguo@kernel.org, shengjiu.wang@nxp.com Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" On Wed, Feb 17, 2021 at 09:28:09AM +0100, Benjamin Gaignard wrote: > = > Le 17/02/2021 =E0 09:08, Greg KH a =E9crit=A0: > > On Wed, Feb 17, 2021 at 09:02:48AM +0100, Benjamin Gaignard wrote: > > > The IMX8MQ got two VPUs but until now only G1 has been enabled. > > > This series aim to add the second VPU (aka G2) and provide basic > > > HEVC decoding support. > > Why are you adding this directly to drivers/staging/media/ and not > > drivers/media/? Why can't this just go to the main location and not > > live in staging? > = > G2/HEVC is added inside the already exiting Hantro driver, it is "just" > an other codec from Hantro driver point of view. > In addition of that v4l2-hevc uAPI is still unstable. > One goal of this series is to have one more consumer of this v4l2-hevc > uAPI so maybe we can claim it to be stable enough to move away from stagi= ng > and then do the same for Hantro driver. That would be a great achievement= ! I know I do not like seeing new additions/features/whatever being added to staging drivers as that encourages people to do new stuff on them without doing the real work needed to get them out of staging. So what is preventing the existing driver from getting out of staging now? And how are you all creating new userspace apis for staging drivers to the v4l layer? What happens when you export something new and then userspace starts to rely on it and then you change it? Anyway, the media staging drivers are on their own, I don't touch them, it just feels odd to me... thanks, greg k-h _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel 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 X-Spam-Level: X-Spam-Status: No, score=-3.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AFB64C433DB for ; Wed, 17 Feb 2021 08:36:54 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6086164E2E for ; Wed, 17 Feb 2021 08:36:54 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6086164E2E Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=w9LBnYfflDRhdWK0toUUiuqUe9oF7yjBgGHln8ZWX2I=; b=Xt1AnInR4bikbh0sEmx9eJl6y maPL9m45Pqep4d6bDQ3L7B7RL+6cWizJqIds5T6tvraJt0Bf7eDebe1LawpwfTZcQBJI1caOfQdGC B8XwxU6BdJdbFzMlQBaGwBS/7uozE86F9qaN6Iw3aGgdvt2XCw84ZRFjq+LME2z+494ctOk0nKigz 9L9bfoimDYkNHeB2p0Xt2K2TjxookNDXDokS45sAbNA/6X/qX9gXrnaXxZatX7qnEFHhPp/2sIfPt B5La7lh1IIsbu2UyA4S5IY/KhBc69XDDfM6BpL9/uf3c6B14AT/5qWtMs5l450z5YAD4K6gmkVfVM c6d6Xieyg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1lCIKJ-0005gJ-Gv; Wed, 17 Feb 2021 08:36:51 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1lCIKD-0005eY-Mr; Wed, 17 Feb 2021 08:36:47 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 1AF9264E28; Wed, 17 Feb 2021 08:36:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1613551004; bh=i4kNSveyDKj4UzuU1mXSC8PvJCQPboPkNLn+8F+ab1k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=yv8i9+KWelFKyWLh1az5qyANS5nqW/AHNH77/f72B/vWnXtGRFkSTStnMXIzvn9+/ 7Yapooh4A9SgaLexotU/96Kqx2XvJlhLyMEQmHNANmx1G4weIrm/bwd1/Aq4aDsYwO o+5GaOV3bjjAb3IKdFImyUa6w0MOJR8ndzyMTx1U= Date: Wed, 17 Feb 2021 09:36:41 +0100 From: Greg KH To: Benjamin Gaignard Subject: Re: [PATCH v1 00/18] Add HANTRO G2/HEVC decoder support for IMX8MQ Message-ID: References: <20210217080306.157876-1-benjamin.gaignard@collabora.com> <04dfae0b-92e5-e02d-c687-ba4d28b7aaf2@collabora.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <04dfae0b-92e5-e02d-c687-ba4d28b7aaf2@collabora.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210217_033645_907827_2D64712D X-CRM114-Status: GOOD ( 19.01 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: peng.fan@nxp.com, kernel@collabora.com, devel@driverdev.osuosl.org, Anson.Huang@nxp.com, krzk@kernel.org, linux-rockchip@lists.infradead.org, wens@csie.org, linux-imx@nxp.com, linux-media@vger.kernel.org, devicetree@vger.kernel.org, kernel@pengutronix.de, s.hauer@pengutronix.de, mripard@kernel.org, robh+dt@kernel.org, mchehab@kernel.org, ezequiel@collabora.com, linux-arm-kernel@lists.infradead.org, aisheng.dong@nxp.com, jernej.skrabec@siol.net, adrian.ratiu@collabora.com, linux-kernel@vger.kernel.org, paul.kocialkowski@bootlin.com, p.zabel@pengutronix.de, hverkuil-cisco@xs4all.nl, shawnguo@kernel.org, shengjiu.wang@nxp.com Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org On Wed, Feb 17, 2021 at 09:28:09AM +0100, Benjamin Gaignard wrote: > = > Le 17/02/2021 =E0 09:08, Greg KH a =E9crit=A0: > > On Wed, Feb 17, 2021 at 09:02:48AM +0100, Benjamin Gaignard wrote: > > > The IMX8MQ got two VPUs but until now only G1 has been enabled. > > > This series aim to add the second VPU (aka G2) and provide basic > > > HEVC decoding support. > > Why are you adding this directly to drivers/staging/media/ and not > > drivers/media/? Why can't this just go to the main location and not > > live in staging? > = > G2/HEVC is added inside the already exiting Hantro driver, it is "just" > an other codec from Hantro driver point of view. > In addition of that v4l2-hevc uAPI is still unstable. > One goal of this series is to have one more consumer of this v4l2-hevc > uAPI so maybe we can claim it to be stable enough to move away from stagi= ng > and then do the same for Hantro driver. That would be a great achievement= ! I know I do not like seeing new additions/features/whatever being added to staging drivers as that encourages people to do new stuff on them without doing the real work needed to get them out of staging. So what is preventing the existing driver from getting out of staging now? And how are you all creating new userspace apis for staging drivers to the v4l layer? What happens when you export something new and then userspace starts to rely on it and then you change it? Anyway, the media staging drivers are on their own, I don't touch them, it just feels odd to me... thanks, greg k-h _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip 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 X-Spam-Level: X-Spam-Status: No, score=-3.8 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 87AB3C433E0 for ; Wed, 17 Feb 2021 08:39:29 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 293BC64DFF for ; Wed, 17 Feb 2021 08:39:29 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 293BC64DFF Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=/v/Lsc0qJ2SIJWj9lJehw5Pwz3d4AyW2D9/GjeGqooY=; b=bFJz/r4Du+1kvVRxj5C2uDsQY Tmu9FyKT7ouKRRUuv+WONH7L2MagSGs1/oKVukqMo+AtxNtzTxR0EUqCrfN2G030/K5JikxYY+oOT OqD3rzd7Zk3P0cz/1YNjcb0EVTwYtGzM0TmQcaqXU0kqypXzdp0VoY2Wem+Tkm0SFH6qmLLui43c0 AdBaFp2ctc6uFPg8ARxWTWiDW0mhjCV21whAjzvhbYc2wlGQiko3WXKVoFsx3Q7ZuJJWX14DU1kbg bpUQ+s7qPv0EVr6A9Nnkk7/HW7frwpwozWgaiJyUrNHY7TWr7oXm3x1gdlDnb3yJ7nOi399GLSuvj 2Cn81CfWg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1lCIKH-0005fj-A0; Wed, 17 Feb 2021 08:36:49 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1lCIKD-0005eY-Mr; Wed, 17 Feb 2021 08:36:47 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 1AF9264E28; Wed, 17 Feb 2021 08:36:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1613551004; bh=i4kNSveyDKj4UzuU1mXSC8PvJCQPboPkNLn+8F+ab1k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=yv8i9+KWelFKyWLh1az5qyANS5nqW/AHNH77/f72B/vWnXtGRFkSTStnMXIzvn9+/ 7Yapooh4A9SgaLexotU/96Kqx2XvJlhLyMEQmHNANmx1G4weIrm/bwd1/Aq4aDsYwO o+5GaOV3bjjAb3IKdFImyUa6w0MOJR8ndzyMTx1U= Date: Wed, 17 Feb 2021 09:36:41 +0100 From: Greg KH To: Benjamin Gaignard Subject: Re: [PATCH v1 00/18] Add HANTRO G2/HEVC decoder support for IMX8MQ Message-ID: References: <20210217080306.157876-1-benjamin.gaignard@collabora.com> <04dfae0b-92e5-e02d-c687-ba4d28b7aaf2@collabora.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <04dfae0b-92e5-e02d-c687-ba4d28b7aaf2@collabora.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210217_033645_907827_2D64712D X-CRM114-Status: GOOD ( 19.01 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: peng.fan@nxp.com, kernel@collabora.com, devel@driverdev.osuosl.org, Anson.Huang@nxp.com, krzk@kernel.org, linux-rockchip@lists.infradead.org, wens@csie.org, linux-imx@nxp.com, linux-media@vger.kernel.org, devicetree@vger.kernel.org, kernel@pengutronix.de, s.hauer@pengutronix.de, mripard@kernel.org, robh+dt@kernel.org, mchehab@kernel.org, ezequiel@collabora.com, linux-arm-kernel@lists.infradead.org, aisheng.dong@nxp.com, jernej.skrabec@siol.net, adrian.ratiu@collabora.com, linux-kernel@vger.kernel.org, paul.kocialkowski@bootlin.com, p.zabel@pengutronix.de, hverkuil-cisco@xs4all.nl, shawnguo@kernel.org, shengjiu.wang@nxp.com Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Feb 17, 2021 at 09:28:09AM +0100, Benjamin Gaignard wrote: > = > Le 17/02/2021 =E0 09:08, Greg KH a =E9crit=A0: > > On Wed, Feb 17, 2021 at 09:02:48AM +0100, Benjamin Gaignard wrote: > > > The IMX8MQ got two VPUs but until now only G1 has been enabled. > > > This series aim to add the second VPU (aka G2) and provide basic > > > HEVC decoding support. > > Why are you adding this directly to drivers/staging/media/ and not > > drivers/media/? Why can't this just go to the main location and not > > live in staging? > = > G2/HEVC is added inside the already exiting Hantro driver, it is "just" > an other codec from Hantro driver point of view. > In addition of that v4l2-hevc uAPI is still unstable. > One goal of this series is to have one more consumer of this v4l2-hevc > uAPI so maybe we can claim it to be stable enough to move away from stagi= ng > and then do the same for Hantro driver. That would be a great achievement= ! I know I do not like seeing new additions/features/whatever being added to staging drivers as that encourages people to do new stuff on them without doing the real work needed to get them out of staging. So what is preventing the existing driver from getting out of staging now? And how are you all creating new userspace apis for staging drivers to the v4l layer? What happens when you export something new and then userspace starts to rely on it and then you change it? Anyway, the media staging drivers are on their own, I don't touch them, it just feels odd to me... thanks, greg k-h _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel