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=-12.3 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=unavailable 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 8CC05C433B4 for ; Wed, 7 Apr 2021 07:31:03 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 4BD376139B for ; Wed, 7 Apr 2021 07:31:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234951AbhDGHbH (ORCPT ); Wed, 7 Apr 2021 03:31:07 -0400 Received: from lb3-smtp-cloud9.xs4all.net ([194.109.24.30]:55041 "EHLO lb3-smtp-cloud9.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233299AbhDGHbG (ORCPT ); Wed, 7 Apr 2021 03:31:06 -0400 Received: from cust-b5b5937f ([IPv6:fc0c:c16d:66b8:757f:c639:739b:9d66:799d]) by smtp-cloud9.xs4all.net with ESMTPA id U2eJlABJQ43ycU2eMlPXXc; Wed, 07 Apr 2021 09:30:55 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xs4all.nl; s=s2; t=1617780655; bh=HiovhSw7IvJpTXadCmtti0e+cPIsSYCvJmRC1aHG4BU=; h=Subject:To:From:Message-ID:Date:MIME-Version:Content-Type:From: Subject; b=fv84menCqp+5MkN5qAY4MYfN1Cv/4fe1ALAajMZnniVw+3dqp3hJl7U46wa8uW7C5 vHEepJ4mlO8is1QzV7uZ9hIEXU1TMBOnYbxOaf21kRvkvYDw7cDA0Rn9GzdFM1CmVY qPr4FbYp3jN8mrkBtfq4iM506blk+exVJkq2Nkl8YObcjgfkFb5vgIDC2OnHIZiZWe rUSZBYIyS6PiRRLDqD+qAb+M0PpZAsjwmJW0HPUMqD1Mn/qgv+nQ9AT/tDPb6Qt091 G2Ny6YMVgjIe5WyGYc4yAyuAs8TcdM8trclAxK9Hl/LpP88zF4NzgiNTG4JPMxxlqo Nsfb7gysvn+LQ== Subject: Re: [PATCH v2 00/30] media: atmel: atmel-isc: add support for xisc To: Eugen Hristev , devicetree@vger.kernel.org, linux-media@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20210405155105.162529-1-eugen.hristev@microchip.com> From: Hans Verkuil Message-ID: Date: Wed, 7 Apr 2021 09:30:51 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Firefox/78.0 Thunderbird/78.9.0 MIME-Version: 1.0 In-Reply-To: <20210405155105.162529-1-eugen.hristev@microchip.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-CMAE-Envelope: MS4xfNsM/RAXgd5azz7ICrRqZGw+dEssfgliUD4LKscmZIFveWs8gE561/RwIHnDZfe3QhWzVjo8jd8WYnd8w6pwJdLSj/4DC3Ne7l1fZT2nwS8SA+t8wgbb nPOJinBongpzzjrn0bLo2FDH4XSP7GDDNELzMzM+oQmhR90CSxORv6zp7NKLmyys1pxKBiebufuIutTvCqlAxc4q3TWxG6DmnZoBP5jPn32nk+MmhO6uV48m nh4VLJfjoqn7E36ABmx5CiTvRg33q+EoTYybW21uprSvAiqby3VQnEHqWvrA5Re7Qh9IwAufls9x+AOEJ4w729PHAVjV0TW0hXGHiWuyfo8QbvB1F0vbCwE5 3QryamxEiLYXGYwPcYVQJl0wQS+ikA== Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Eugen, On 05/04/2021 17:50, Eugen Hristev wrote: > Hello, > > This series adds support for a variant of the ISC named XISC. > This block is present in the product named sama7g5. > > I started by moving code around, the code which was specialized for sama5d2 > type of ISC, to have it inside the dedicated sama5d2 file. > > I added several new pipeline elements to the code base, which would be common > to sama5d2 and the new sama7g5, but only used by the new style pipeline. > > I separated the input and output formats on a per-product separate array. > > I added the new sama7g5 compatible driver for the xisc, which is similar with > the sama5d2, but with differences in terms of DT, clocks and callbacks to > specific operations. > > I added the binding for the xisc by copying and modifying the existing > isc one. I know that it has to be converted to yaml, and I will do that if > it looks good. This series looks good to me. One thing that can be improved in a v3 is a short explanation of the various abbreviations: CBV, RLP, HIS, DPC, etc. Regards, Hans > > Feedback is appreciated. > Thanks, > Eugen > > Changes in v2: > - Fixed krobot warnings with W=1 regarding functions with no prototype > - Fixed new sama7g5 driver to use the new subdev fwnode API in kernel 5.12. my driver was > based on old 5.10 style API. > > Eugen Hristev (30): > media: atmel: atmel-isc: specialize gamma table into product specific > media: atmel: atmel-isc: specialize driver name constant > media: atmel: atmel-isc: add checks for limiting frame sizes > media: atmel: atmel-isc: specialize max width and max height > media: atmel: atmel-isc: specialize dma cfg > media: atmel: atmel-isc: extract CSC submodule config into separate > function > media: atmel: atmel-isc-base: add id to clock debug message > media: atmel: atmel-isc: create register offsets struct > media: atmel: atmel-isc: extract CBC submodule config into separate > function > media: atmel: atmel-isc: add CBC to the reg offsets struct > media: atmel: atmel-isc: add SUB422 and SUB420 to register offsets > media: atmel: atmel-isc: add RLP to register offsets > media: atmel: atmel-isc: add HIS to register offsets > media: atmel: atmel-isc: add DMA to register offsets > media: atmel: atmel-isc: add support for version register > media: atmel: atmel-isc: add his_entry to register offsets > media: atmel: atmel-isc: add register description for additional > modules > media: atmel: atmel-isc: extend pipeline with extra modules > media: atmel: atmel-isc: add CC initialization function > media: atmel: atmel-isc: create product specific v4l2 controls config > media: atmel: atmel-isc: create callback for DPC submodule product > specific > media: atmel: atmel-isc: create callback for GAM submodule product > specific > media: atmel: atmel-isc: create callback for RLP submodule product > specific > media: atmel: atmel-isc: move the formats list into product specific > code > media: atmel: atmel-isc: create an adapt pipeline callback for product > specific > media: atmel: atmel-isc-regs: add additional fields for sama7g5 type > pipeline > media: atmel: atmel-isc-base: add support for more formats and > additional pipeline modules > dt-bindings: media: atmel: add microchip-xisc binding > media: atmel: atmel-isc-sama5d2: remove duplicate define > media: atmel: atmel-isc: add microchip-xisc driver > > .../bindings/media/microchip-xisc.txt | 64 ++ > drivers/media/platform/Makefile | 1 + > drivers/media/platform/atmel/Kconfig | 11 + > drivers/media/platform/atmel/Makefile | 2 + > drivers/media/platform/atmel/atmel-isc-base.c | 381 ++++------- > drivers/media/platform/atmel/atmel-isc-regs.h | 133 +++- > drivers/media/platform/atmel/atmel-isc.h | 122 +++- > .../media/platform/atmel/atmel-sama5d2-isc.c | 311 ++++++++- > .../media/platform/atmel/atmel-sama7g5-isc.c | 643 ++++++++++++++++++ > 9 files changed, 1391 insertions(+), 277 deletions(-) > create mode 100644 Documentation/devicetree/bindings/media/microchip-xisc.txt > create mode 100644 drivers/media/platform/atmel/atmel-sama7g5-isc.c > 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=-10.3 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=unavailable 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 1E0FBC433ED for ; Wed, 7 Apr 2021 07:33:06 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (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 5F39061245 for ; Wed, 7 Apr 2021 07:33:05 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5F39061245 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=xs4all.nl 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=desiato.20200630; h=Sender:Content-Transfer-Encoding :Content-Type:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Cc:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=36u6CMnk+aZ6rPnAwZdOQkB3YS11lw1o6rI4znEMSTs=; b=FtfKiG6FH0Hu4j6yl7l0YifgW XXSfP6XI6ukwE2IdTMuRDklGTcCpqbO2v/vieJTkau31H/oLsoPo5YSddSx3DxmJWq0jF3SsXSSLP HQLqZW+yIEGtzrkM2wxCJRH3oag5+oUpnUm1PgMLg8HL524f/nQRAyiNMWNtTtWQ2tjTZYsfjYTIb xC7Iy9W8zTh7xPpM4ZMji7SsS5OAUbvbDttvoeTdIOTFZ1gQeryOqXX7NfEL4pCmNkZputTwkq8pn 1W6jRj0FsusTW26ql5uKnwxM+/KPZEBAG309iAIxsDN7HL8aPmaIFrmieseDzLl5s8Vsz+prn499d hWZ0j7OyA==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lU2es-004ML3-4P; Wed, 07 Apr 2021 07:31:26 +0000 Received: from lb1-smtp-cloud9.xs4all.net ([194.109.24.22]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lU2eT-004MBV-UM for linux-arm-kernel@lists.infradead.org; Wed, 07 Apr 2021 07:31:06 +0000 Received: from cust-b5b5937f ([IPv6:fc0c:c16d:66b8:757f:c639:739b:9d66:799d]) by smtp-cloud9.xs4all.net with ESMTPA id U2eJlABJQ43ycU2eMlPXXc; Wed, 07 Apr 2021 09:30:55 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xs4all.nl; s=s2; t=1617780655; bh=HiovhSw7IvJpTXadCmtti0e+cPIsSYCvJmRC1aHG4BU=; h=Subject:To:From:Message-ID:Date:MIME-Version:Content-Type:From: Subject; b=fv84menCqp+5MkN5qAY4MYfN1Cv/4fe1ALAajMZnniVw+3dqp3hJl7U46wa8uW7C5 vHEepJ4mlO8is1QzV7uZ9hIEXU1TMBOnYbxOaf21kRvkvYDw7cDA0Rn9GzdFM1CmVY qPr4FbYp3jN8mrkBtfq4iM506blk+exVJkq2Nkl8YObcjgfkFb5vgIDC2OnHIZiZWe rUSZBYIyS6PiRRLDqD+qAb+M0PpZAsjwmJW0HPUMqD1Mn/qgv+nQ9AT/tDPb6Qt091 G2Ny6YMVgjIe5WyGYc4yAyuAs8TcdM8trclAxK9Hl/LpP88zF4NzgiNTG4JPMxxlqo Nsfb7gysvn+LQ== Subject: Re: [PATCH v2 00/30] media: atmel: atmel-isc: add support for xisc To: Eugen Hristev , devicetree@vger.kernel.org, linux-media@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org References: <20210405155105.162529-1-eugen.hristev@microchip.com> From: Hans Verkuil Message-ID: Date: Wed, 7 Apr 2021 09:30:51 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Firefox/78.0 Thunderbird/78.9.0 MIME-Version: 1.0 In-Reply-To: <20210405155105.162529-1-eugen.hristev@microchip.com> Content-Language: en-US X-CMAE-Envelope: MS4xfNsM/RAXgd5azz7ICrRqZGw+dEssfgliUD4LKscmZIFveWs8gE561/RwIHnDZfe3QhWzVjo8jd8WYnd8w6pwJdLSj/4DC3Ne7l1fZT2nwS8SA+t8wgbb nPOJinBongpzzjrn0bLo2FDH4XSP7GDDNELzMzM+oQmhR90CSxORv6zp7NKLmyys1pxKBiebufuIutTvCqlAxc4q3TWxG6DmnZoBP5jPn32nk+MmhO6uV48m nh4VLJfjoqn7E36ABmx5CiTvRg33q+EoTYybW21uprSvAiqby3VQnEHqWvrA5Re7Qh9IwAufls9x+AOEJ4w729PHAVjV0TW0hXGHiWuyfo8QbvB1F0vbCwE5 3QryamxEiLYXGYwPcYVQJl0wQS+ikA== X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210407_083103_979211_093C2235 X-CRM114-Status: GOOD ( 28.28 ) 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 Hi Eugen, On 05/04/2021 17:50, Eugen Hristev wrote: > Hello, > > This series adds support for a variant of the ISC named XISC. > This block is present in the product named sama7g5. > > I started by moving code around, the code which was specialized for sama5d2 > type of ISC, to have it inside the dedicated sama5d2 file. > > I added several new pipeline elements to the code base, which would be common > to sama5d2 and the new sama7g5, but only used by the new style pipeline. > > I separated the input and output formats on a per-product separate array. > > I added the new sama7g5 compatible driver for the xisc, which is similar with > the sama5d2, but with differences in terms of DT, clocks and callbacks to > specific operations. > > I added the binding for the xisc by copying and modifying the existing > isc one. I know that it has to be converted to yaml, and I will do that if > it looks good. This series looks good to me. One thing that can be improved in a v3 is a short explanation of the various abbreviations: CBV, RLP, HIS, DPC, etc. Regards, Hans > > Feedback is appreciated. > Thanks, > Eugen > > Changes in v2: > - Fixed krobot warnings with W=1 regarding functions with no prototype > - Fixed new sama7g5 driver to use the new subdev fwnode API in kernel 5.12. my driver was > based on old 5.10 style API. > > Eugen Hristev (30): > media: atmel: atmel-isc: specialize gamma table into product specific > media: atmel: atmel-isc: specialize driver name constant > media: atmel: atmel-isc: add checks for limiting frame sizes > media: atmel: atmel-isc: specialize max width and max height > media: atmel: atmel-isc: specialize dma cfg > media: atmel: atmel-isc: extract CSC submodule config into separate > function > media: atmel: atmel-isc-base: add id to clock debug message > media: atmel: atmel-isc: create register offsets struct > media: atmel: atmel-isc: extract CBC submodule config into separate > function > media: atmel: atmel-isc: add CBC to the reg offsets struct > media: atmel: atmel-isc: add SUB422 and SUB420 to register offsets > media: atmel: atmel-isc: add RLP to register offsets > media: atmel: atmel-isc: add HIS to register offsets > media: atmel: atmel-isc: add DMA to register offsets > media: atmel: atmel-isc: add support for version register > media: atmel: atmel-isc: add his_entry to register offsets > media: atmel: atmel-isc: add register description for additional > modules > media: atmel: atmel-isc: extend pipeline with extra modules > media: atmel: atmel-isc: add CC initialization function > media: atmel: atmel-isc: create product specific v4l2 controls config > media: atmel: atmel-isc: create callback for DPC submodule product > specific > media: atmel: atmel-isc: create callback for GAM submodule product > specific > media: atmel: atmel-isc: create callback for RLP submodule product > specific > media: atmel: atmel-isc: move the formats list into product specific > code > media: atmel: atmel-isc: create an adapt pipeline callback for product > specific > media: atmel: atmel-isc-regs: add additional fields for sama7g5 type > pipeline > media: atmel: atmel-isc-base: add support for more formats and > additional pipeline modules > dt-bindings: media: atmel: add microchip-xisc binding > media: atmel: atmel-isc-sama5d2: remove duplicate define > media: atmel: atmel-isc: add microchip-xisc driver > > .../bindings/media/microchip-xisc.txt | 64 ++ > drivers/media/platform/Makefile | 1 + > drivers/media/platform/atmel/Kconfig | 11 + > drivers/media/platform/atmel/Makefile | 2 + > drivers/media/platform/atmel/atmel-isc-base.c | 381 ++++------- > drivers/media/platform/atmel/atmel-isc-regs.h | 133 +++- > drivers/media/platform/atmel/atmel-isc.h | 122 +++- > .../media/platform/atmel/atmel-sama5d2-isc.c | 311 ++++++++- > .../media/platform/atmel/atmel-sama7g5-isc.c | 643 ++++++++++++++++++ > 9 files changed, 1391 insertions(+), 277 deletions(-) > create mode 100644 Documentation/devicetree/bindings/media/microchip-xisc.txt > create mode 100644 drivers/media/platform/atmel/atmel-sama7g5-isc.c > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel