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=-2.4 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,USER_AGENT_SANE_1 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 EF030C43331 for ; Thu, 26 Mar 2020 19:53:18 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 9F64D20714 for ; Thu, 26 Mar 2020 19:53:18 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="bjyB77aQ" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727541AbgCZTxS (ORCPT ); Thu, 26 Mar 2020 15:53:18 -0400 Received: from perceval.ideasonboard.com ([213.167.242.64]:37412 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726540AbgCZTxR (ORCPT ); Thu, 26 Mar 2020 15:53:17 -0400 Received: from pendragon.ideasonboard.com (81-175-216-236.bb.dnainternet.fi [81.175.216.236]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 1ACD42DC; Thu, 26 Mar 2020 20:53:15 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1585252395; bh=hD0EgZO3CzzXO8pJyOC2hRISGLSR3i/b2Yr3vuYsZAo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=bjyB77aQtPCtcIrfhTX+I3uk2Uh36IhN0PSJwIBl1bgQY82sg+NRVm4j2gFRb6+aZ dRoO21BDGqigL9tnq19e3PKE5Z9G5ZaFEun2e5EYoaq0l29bNC6p3F9btRTE7tRv4M iWfDVBUY/lZStW6V7GU/Qc33pl2P/0VPif7Yetuc= Date: Thu, 26 Mar 2020 21:53:11 +0200 From: Laurent Pinchart To: Steve Longerbeam Cc: linux-media@vger.kernel.org, Philipp Zabel , Rui Miguel Silva Subject: Re: [PATCH 0/8] media: imx: Miscalleanous format-related cleanups Message-ID: <20200326195311.GA20581@pendragon.ideasonboard.com> References: <20200310161845.1588-1-laurent.pinchart@ideasonboard.com> <5b773016-8a71-9739-237d-69c0dc8bd349@gmail.com> <20200312004735.GG1639@pendragon.ideasonboard.com> <1a10d9b2-3e5b-4cbd-d703-5012fa8fa6ad@gmail.com> <20200314223209.GC5320@pendragon.ideasonboard.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Hi Steve, On Thu, Mar 26, 2020 at 12:05:38PM -0700, Steve Longerbeam wrote: > On 3/14/20 3:32 PM, Laurent Pinchart wrote: > > On Sat, Mar 14, 2020 at 03:30:53PM -0700, Steve Longerbeam wrote: > >> On 3/11/20 5:47 PM, Laurent Pinchart wrote: > >>> On Wed, Mar 11, 2020 at 05:16:49PM -0700, Steve Longerbeam wrote: > >>>> Hi Laurent, > >>>> > >>>> I agree that the find/enum format code in imx-utils needs cleanup, it's > >>>> too confusing. I will be ready to give my ack to the imx-utils patches > >>>> once I do some smoke testing on a sabre target when I return from vacation. > >>>> > >>>> Note that Phillip also submitted a fixup patch to the find/enum format > >>>> code, IIRC it did more consolidating of the imx_media_pixfmt tables. I > >>>> can't find it and it has gotten lost, but I tested and provided a > >>>> reviewed-by at the time. > >>> > >>> I've found them in the mail archive. There were 3 patches, Hans said he > >>> would take the first two, but only the first one got merged. I'll take > >>> the two others and build on top of them, fixing the issues pointed out > >>> by the kbuild robot and addressing Hans concerns. > >> > >> I found the thread finally. In fact I did some work on this set already, > >> I fixed the kbuild warnings as well as added some function headers > >> describing all the format search criteria arguments. Also changed a > >> local var in enum_formats() to better document the enum algorithm. > >> > >> Unless you've started on this work already, I will be returning from > >> vaca tomorrow and can pick this up again, merging in your patches, as > >> well as splitting up find|enum_formats() into find|enum_formats() and > >> find|enum_codes(), which is the remaining suggestion from Hans. > > I've done this already, I'll try to send the patches on Monday. > > If you don't mind, I've done this work and tested on an imx6 target. So > I will post a series. No worries. I've been a bit delayed indeed. I'll still post my cleanups, in case they contain anything useful in addition to yours. > >>>> On 3/10/20 9:18 AM, Laurent Pinchart wrote: > >>>>> Hello, > >>>>> > >>>>> This patch series started as an attempt to fix the format get and set > >>>>> subdev operations on the i.MX7 CSI-2 receiver subdev, which it does in > >>>>> patch 1/8. Patch 2/8 further cleans up the format-related code in that > >>>>> subdev. > >>>>> > >>>>> Patches 3/8 to 8/8 pushes the cleanups further as I was attempting to > >>>>> fix the format enumeration on the video node at the end of the pipeline. > >>>>> I realized as part of that effort that there's more work than I > >>>>> anticipated, and I'm currently evaluating the possible options. > >>>>> Nonetheless, I think the cleanups make sense even without what I wanted > >>>>> to build on top of them, so I'm sending them out already. > >>>>> > >>>>> Laurent Pinchart (8): > >>>>> media: imx: imx7-mipi-csis: Cleanup and fix subdev pad format handling > >>>>> media: imx: imx7-mipi-csis: Centralize initialization of pad formats > >>>>> media: imx: utils: Inline init_mbus_colorimetry() in its caller > >>>>> media: imx: utils: Handle Bayer format lookup through a selection flag > >>>>> media: imx: utils: Simplify IPU format lookup and enumeration > >>>>> media: imx: utils: Make imx_media_pixfmt handle variable number of > >>>>> codes > >>>>> media: imx: utils: Remove unneeded argument to (find|enum)_format() > >>>>> media: imx: utils: Rename format lookup and enumeration functions > >>>>> > >>>>> drivers/staging/media/imx/imx-ic-prp.c | 8 +- > >>>>> drivers/staging/media/imx/imx-ic-prpencvf.c | 6 +- > >>>>> drivers/staging/media/imx/imx-media-capture.c | 22 +- > >>>>> .../staging/media/imx/imx-media-csc-scaler.c | 2 +- > >>>>> drivers/staging/media/imx/imx-media-csi.c | 26 +- > >>>>> drivers/staging/media/imx/imx-media-utils.c | 313 ++++++++---------- > >>>>> drivers/staging/media/imx/imx-media-vdic.c | 6 +- > >>>>> drivers/staging/media/imx/imx-media.h | 24 +- > >>>>> drivers/staging/media/imx/imx7-media-csi.c | 15 +- > >>>>> drivers/staging/media/imx/imx7-mipi-csis.c | 138 ++++---- > >>>>> 10 files changed, 271 insertions(+), 289 deletions(-) -- Regards, Laurent Pinchart