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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 28B28C4167B for ; Fri, 1 Dec 2023 06:12:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:CC: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=isZfnC9+ymAUdBsGateQJ2tdUanvpxMM3UV981ANGtU=; b=zXWZZ/ED89YpbP tg5TteTruptoms47E2txYrBwMYlivSnlHfGEDlIUbpP53iSpQYNSnzLkD2Wxh5e88Q0Pwl+mlEVc3 wAiKtZTrTdG0vtsbPX3bxoUDzj/ThAKA/yWlLUTHgXMobGIEwqATM+l/lg5uCCRXSLOAkGfAclV5x hqHnXGILiTNn9etk3l5KIJbZctn0XW2HBlaUwJ5Pi0t1N7NRTo1OphRr4QZOVaFaQewB6ofjf0wZT D0z18j7GfKyHWiW1CokGrc3gEUAL8/R6I2DLfA+WNTtN9gCoMSUvCqwu4KrWKXf8mqby0SnszDgfk Yr2F2ICvtY1UpVSJIgKw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1r8wkN-00Ckag-0p; Fri, 01 Dec 2023 06:11:31 +0000 Received: from fllv0016.ext.ti.com ([198.47.19.142]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1r8wkJ-00CkYv-27 for linux-arm-kernel@lists.infradead.org; Fri, 01 Dec 2023 06:11:30 +0000 Received: from lelv0266.itg.ti.com ([10.180.67.225]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 3B16BDfs038786; Fri, 1 Dec 2023 00:11:13 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1701411073; bh=epgSri49vYEanu4EViex8BzQiAMqKTxnJBmbQuxifM0=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=t6PJjZxaE/YtFzz/B70/3TNeztm9LWWkw4Mfud1GkBiyFotsioiKmtNipJ2Ot7c0E uix24CTz1p4MZTgBCORTXNCg7kJpaoYYyKxR6yq0V7880YozN2BGPYFiKVL4IWu2E1 cD/WcgQl1E+nfecTyEi17z6l25t5vSxKzxq/fSdQ= Received: from DLEE115.ent.ti.com (dlee115.ent.ti.com [157.170.170.26]) by lelv0266.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 3B16BCN2059564 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Fri, 1 Dec 2023 00:11:12 -0600 Received: from DLEE114.ent.ti.com (157.170.170.25) by DLEE115.ent.ti.com (157.170.170.26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Fri, 1 Dec 2023 00:11:12 -0600 Received: from fllv0039.itg.ti.com (10.64.41.19) by DLEE114.ent.ti.com (157.170.170.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Fri, 1 Dec 2023 00:11:12 -0600 Received: from localhost (ileaxei01-snat.itg.ti.com [10.180.69.5]) by fllv0039.itg.ti.com (8.15.2/8.15.2) with ESMTP id 3B16BCeA082440; Fri, 1 Dec 2023 00:11:12 -0600 Date: Fri, 1 Dec 2023 00:11:12 -0600 From: Nishanth Menon To: Sebastian Fricke CC: Geert Uytterhoeven , Nas Chung , Jackson Lee , Mauro Carvalho Chehab , Robert Beckett , Hans Verkuil , Nicolas Dufresne , Vignesh Raghavendra , Tero Kristo , , , Subject: Re: [PATCH -next] media: chips-media: VIDEO_WAVE_VPU should depend on ARCH_K3 Message-ID: <20231201061112.jvera4kirvoqseqz@reseal> References: <20231130100647.pbgk7oqgynx3pzrw@basti-XPS-13-9310> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20231130100647.pbgk7oqgynx3pzrw@basti-XPS-13-9310> X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231130_221127_784292_740ACB17 X-CRM114-Status: GOOD ( 24.55 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 11:07-20231130, Sebastian Fricke wrote: > Hey Geert, > > Thanks for the patch! > > Could you please adjust the subject line to: > media: chips-media: wave5: VIDEO_WAVE_VPU should depend on ARCH_K3 > ^^^^^^ > > On 28.11.2023 19:26, Geert Uytterhoeven wrote: > > The Chips&Media Wave 5 Series multi-standard codec IP is currently only > > supported on Texas Instruments K3 J721S2 SoC. Hence add a dependency on > > While it is true that is currently only tested on the K3 architecture , > it is not only supported by that exact SoC, as you can see here: > https://lore.kernel.org/all/20231127223718.2651185-4-b-brnich@ti.com/T/ > > So, maybe this commit is worded better with: > > is currently only supported on the Texas Instruments K3 architecture. > > The change however is obviously correct. > > Greetings, > Sebastian > > > ARCH_K3, to prevent asking the user about this driver when configuring a > > kernel without Texas Instruments K3 Multicore SoC support. > > > > Fixes: 9707a6254a8a6b97 ("media: chips-media: wave5: Add the v4l2 layer") Thank you Sebastian, in addition, a nitpick: checkpatch.pl complains: Please use correct Fixes: style 'Fixes: <12 chars of sha1> ("")' https://docs.kernel.org/process/submitting-patches.html Quote: If your patch fixes a bug in a specific commit, e.g. you found an issue using git bisect, please use the 'Fixes:' tag with the first 12 characters of the SHA-1 ID, and the one line summary. Do not split the tag across multiple lines, tags are exempt from the "wrap at 75 columns" rule in order to simplify parsing scripts with the above fixed, please feel free to add: Reviewed-by: Nishanth Menon <nm@ti.com> > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > --- > > drivers/media/platform/chips-media/wave5/Kconfig | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/media/platform/chips-media/wave5/Kconfig b/drivers/media/platform/chips-media/wave5/Kconfig > > index 77e7ae5c8f35f454..9ccc1f7e32f3874f 100644 > > --- a/drivers/media/platform/chips-media/wave5/Kconfig > > +++ b/drivers/media/platform/chips-media/wave5/Kconfig > > @@ -3,6 +3,7 @@ config VIDEO_WAVE_VPU > > tristate "Chips&Media Wave Codec Driver" > > depends on V4L_MEM2MEM_DRIVERS > > depends on VIDEO_DEV && OF > > + depends on ARCH_K3 || COMPILE_TEST > > select VIDEOBUF2_DMA_CONTIG > > select VIDEOBUF2_VMALLOC > > select V4L2_MEM2MEM_DEV -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel