Message ID | eb27184d182811520de31e59f449ea49e7cc6963.1701195705.git.geert+renesas@glider.be |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:ce62:0:b0:403:3b70:6f57 with SMTP id o2csp4133373vqx; Tue, 28 Nov 2023 10:26:53 -0800 (PST) X-Google-Smtp-Source: AGHT+IEhX6ia+cpfH0LrYmRPzk0JeU2cc5dMArNyBQjA2FPq4ktKz2PH06Zjmx0t4a6Y/CrQ51Dm X-Received: by 2002:a05:6a20:258d:b0:187:29f9:e12e with SMTP id k13-20020a056a20258d00b0018729f9e12emr19960469pzd.29.1701196013681; Tue, 28 Nov 2023 10:26:53 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1701196013; cv=none; d=google.com; s=arc-20160816; b=rexcXljIB9f/+3AEPipN1X2LPdQDd0Uekv899SUx3nIF/tIKguVZT7+zJd4sAkK0MQ e8txnctgCWVVgq321F9thrX5H5XMTxOc0LElMcuJTd9fmrUli0vbApMXioKmXSOJD+Ab d6/G3gy7VuArJsYbWY1o5DMgAQOx5BJWlohKFt5SG+HCftwfUTMRwPWBUn4FGgPt+qFp J+s4/8BI7aUYZZYculK+1CuiQl80kjBipOmZiFDR4v9OnYbQjLWYVzJLOps5UA20neDv hx+veN4XuH7lo223gh/wVaGTAPa6fX9itdXbzXU5OCNqUgv8g/WZJ8rUlBfRgFDlODBH M+NQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=3L/xM+9oxwqLibkGVnZw4LjANjbRuDi5d76sS3AgqFc=; fh=XLmh38AKR1+lb9cFiKdlHeydNTLvI5HgaM1UfD+emnc=; b=DbtaDS+KlhS0d3XXLWnhAn/ngy2c+5TBUAsZBdJDJvjDn3W/X5FOeQdRQop53xqbVY Jz8OKRkU450qpHxcLSnM1h/+mijmBqhOvl7cUWZDsZjtLTAAhUs5R08gJoboXv3Z5chz QHoweCjfI4WJm8M2erW9RNFkExWxFDld18XpcPtTNfNFfxcyZen+VpA2W3L7FMWm1Lar iOEenc2nFmfSRLXlHvlezlSkD09q56udfg7MNKjII/GIqdCeEe+tx7p7+ttQKWIIt+lc +2Fam6IpiewFSQd9jl6jOIPtQGukPvDxuetflIoPgABo+mFup6MGoD/NXxYBYKdtGl2p Sv0Q== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from lipwig.vger.email (lipwig.vger.email. [2620:137:e000::3:3]) by mx.google.com with ESMTPS id fb21-20020a056a002d9500b006cb6fc94847si13074670pfb.172.2023.11.28.10.26.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 Nov 2023 10:26:53 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) client-ip=2620:137:e000::3:3; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id 81E2F809E22E; Tue, 28 Nov 2023 10:26:50 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1344852AbjK1S0l (ORCPT <rfc822;toshivichauhan@gmail.com> + 99 others); Tue, 28 Nov 2023 13:26:41 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36098 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229921AbjK1S0l (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Tue, 28 Nov 2023 13:26:41 -0500 Received: from andre.telenet-ops.be (andre.telenet-ops.be [IPv6:2a02:1800:120:4::f00:15]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BCF5C1A5 for <linux-kernel@vger.kernel.org>; Tue, 28 Nov 2023 10:26:46 -0800 (PST) Received: from ramsan.of.borg ([IPv6:2a02:1810:ac12:ed40:f4d4:eef9:4487:4eb1]) by andre.telenet-ops.be with bizsmtp id FuSe2B00E2jytoH01uSec1; Tue, 28 Nov 2023 19:26:44 +0100 Received: from rox.of.borg ([192.168.97.57]) by ramsan.of.borg with esmtp (Exim 4.95) (envelope-from <geert@linux-m68k.org>) id 1r82ms-00AHWm-F7; Tue, 28 Nov 2023 19:26:38 +0100 Received: from geert by rox.of.borg with local (Exim 4.95) (envelope-from <geert@linux-m68k.org>) id 1r82n8-00B9me-GS; Tue, 28 Nov 2023 19:26:38 +0100 From: Geert Uytterhoeven <geert+renesas@glider.be> To: Nas Chung <nas.chung@chipsnmedia.com>, Jackson Lee <jackson.lee@chipsnmedia.com>, Mauro Carvalho Chehab <mchehab@kernel.org>, Dafna Hirschfeld <dafna.hirschfeld@collabora.com>, Sebastian Fricke <sebastian.fricke@collabora.com>, Robert Beckett <bob.beckett@collabora.com>, Hans Verkuil <hverkuil-cisco@xs4all.nl>, Nicolas Dufresne <nicolas.dufresne@collabora.com> Cc: Nishanth Menon <nm@ti.com>, Vignesh Raghavendra <vigneshr@ti.com>, Tero Kristo <kristo@kernel.org>, linux-media@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Geert Uytterhoeven <geert+renesas@glider.be> Subject: [PATCH -next] media: chips-media: VIDEO_WAVE_VPU should depend on ARCH_K3 Date: Tue, 28 Nov 2023 19:26:36 +0100 Message-Id: <eb27184d182811520de31e59f449ea49e7cc6963.1701195705.git.geert+renesas@glider.be> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-0.8 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Tue, 28 Nov 2023 10:26:50 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1783833311358800892 X-GMAIL-MSGID: 1783833311358800892 |
Series |
[-next] media: chips-media: VIDEO_WAVE_VPU should depend on ARCH_K3
|
|
Commit Message
Geert Uytterhoeven
Nov. 28, 2023, 6:26 p.m. UTC
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
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")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
drivers/media/platform/chips-media/wave5/Kconfig | 1 +
1 file changed, 1 insertion(+)
Comments
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") >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
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> ("<title line>")' 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
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