Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp3562894pxb; Wed, 13 Oct 2021 08:28:17 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzNx/c/f1X7uwA2WuiIlZatO0z8Uskx1sBgCD6gQ2yGpLxkm7cqzE7fya7gecwI2siJQx6f X-Received: by 2002:a62:1c4e:0:b0:44d:76b:de98 with SMTP id c75-20020a621c4e000000b0044d076bde98mr52476pfc.16.1634138897036; Wed, 13 Oct 2021 08:28:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1634138897; cv=none; d=google.com; s=arc-20160816; b=NLY2mZzywKQV/jJe+VHVOQWCQ4xPmnOsY6ENRo/KrjwQrSMaVqLce6rWArDTlI1dW0 6jctd3cctzOv6MRsjZ4zv/Yejk6zPlAWM+IMIQ/zFgQ3zvzJ/SgWghCwExuy8trmBaub J0d8m4p2o0C+FV3VXNpV8Hx8MwlBSGA6iqFBI9DsUSpmDMjr3Dgch4TOoEkB9XS11GOB X3FEi7l5cJzKNJvpMajk0y4xZKZuP+5qn1BFMRI11Ttml2D/mctv4DmUilC78DyIP5MF ZSOF6OXkd+r+xMwOIDwcsprXQh8894qpsE3VshJxbYG5VxOADSUpvU6Q5/d5bAge7O8/ 80gQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dkim-signature; bh=wLKFl9hJovma3HR1JQUfzFV4dSW0MXFKvmPEDZSEdrw=; b=hKgXr8KkwE6kYvdK/q8EApadlZEl/J1qpRDmc+eTeL/SblxXJsEWz00f2UzDa3PoES /8WR7DzTwdDDg8pz1j+9ckD7lve6GrvbnRK9JK+RfiPpu31fW+LQ3YxKUklT+9++UXu+ 4UZDWLBp7+RH4tyDzIrkH9/rOdvYpB0OSV7r0YcL6hX2L+azfpRXt/1Hn3Bova4a1M/X QPkLQ0JkMft9OH+3FRl8Tm1qY8rr3i6EeSG4Fo0yeyzSKirEE2SQG3BiW2vA4hQc9fAY PNlRORMU4Cv2GntP9iTU8uG2Z8IBawKWo4ajfpXv3V5qvoXhN4+l1n5RjUNpydSgGHc+ nSYQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@infradead.org header.s=bombadil.20210309 header.b=ZH2RJkVX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id x14si8058138pjn.23.2021.10.13.08.28.04; Wed, 13 Oct 2021 08:28:17 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@infradead.org header.s=bombadil.20210309 header.b=ZH2RJkVX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233648AbhJMP1r (ORCPT + 99 others); Wed, 13 Oct 2021 11:27:47 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35162 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232692AbhJMP1q (ORCPT ); Wed, 13 Oct 2021 11:27:46 -0400 Received: from bombadil.infradead.org (unknown [IPv6:2607:7c80:54:e::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C7F3CC061570; Wed, 13 Oct 2021 08:25:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Content-Transfer-Encoding: Content-Type:In-Reply-To:MIME-Version:Date:Message-ID:From:References:Cc:To: Subject:Sender:Reply-To:Content-ID:Content-Description; bh=wLKFl9hJovma3HR1JQUfzFV4dSW0MXFKvmPEDZSEdrw=; b=ZH2RJkVXauMhi5VttRQ3IPzAlc G5OAV4VS/VFJbAP0AHqJcJaLBXIdN42F3co5T7Si3Lz84xyCh5m0F86d2/iJ4IKQlQUvqPretTmcB /WV6r6iEMjSkq6+Z9GaoOolYxiTaFivy+6Rd3KSnE/Gshi2e/Elh9QDJCTF0HotaHc1btqRgC05Yw 4IyKUel7pECtsBgKGvjTzeaVFk8YMOzNJiSv2KzNaVBYZVdHOrbSZroTl7i9HjDWRKkQiulOwb//Z gcLDJl9mCOCoGnBy8KwaW40aqY13RhK0rKhN97hjy1pXQqjn0NDWjsRT0Rn6NwwjG94qj31NlDKKj F1zFWdVw==; Received: from [2601:1c0:6280:3f0::aa0b] by bombadil.infradead.org with esmtpsa (Exim 4.94.2 #2 (Red Hat Linux)) id 1mag8M-00HLgm-Vm; Wed, 13 Oct 2021 15:25:35 +0000 Subject: Re: [PATCH v2 3/6] staging: media: wave5: Add the v4l2 layer To: Dafna Hirschfeld , linux-media@vger.kernel.org Cc: Robert Beckett , Mauro Carvalho Chehab , Greg Kroah-Hartman , "open list:STAGING SUBSYSTEM" , open list , laurent.pinchart@ideasonboard.com, hverkuil@xs4all.nl, kernel@collabora.com, dafna3@gmail.com, kiril.bicevski@collabora.com, Nas Chung , lafley.kim@chipsnmedia.com, scott.woo@chipsnmedia.com, olivier.crete@collabora.com References: <20211013105609.21457-1-dafna.hirschfeld@collabora.com> <20211013105609.21457-4-dafna.hirschfeld@collabora.com> From: Randy Dunlap Message-ID: Date: Wed, 13 Oct 2021 08:25:34 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <20211013105609.21457-4-dafna.hirschfeld@collabora.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On 10/13/21 3:56 AM, Dafna Hirschfeld wrote: > diff --git a/drivers/staging/media/wave5/Kconfig b/drivers/staging/media/wave5/Kconfig > new file mode 100644 > index 000000000000..efaacf391e01 > --- /dev/null > +++ b/drivers/staging/media/wave5/Kconfig > @@ -0,0 +1,12 @@ > +# SPDX-License-Identifier: GPL-2.0 > +config VIDEO_WAVE_VPU > + tristate "Chips&Media Wave Codec Driver" > + depends on VIDEO_DEV && VIDEO_V4L2 && OF > + select VIDEOBUF2_DMA_CONTIG > + select VIDEOBUF2_VMALLOC > + select V4L2_MEM2MEM_DEV > + help > + Chips&Media stateful encoder and decoder driver. > + The driver supports HEVC and H264 formats. > + To compile this driver as modules, choose M here: the > + modules will be called wave5. Please follow coding-style for Kconfig files: (from Documentation/process/coding-style.rst, section 10): For all of the Kconfig* configuration files throughout the source tree, the indentation is somewhat different. Lines under a ``config`` definition are indented with one tab, while help text is indented an additional two spaces. Also, depending on OF is usually not necessary since there are stubs for non-CONFIG_OF functions. That may even allow you to add COMPILE_TEST here. thanks. -- ~Randy