From patchwork Fri Apr 23 10:29:51 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tomi Valkeinen X-Patchwork-Id: 73501 Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from ) id 1lZt4Z-003r5h-UN; Fri, 23 Apr 2021 10:30:08 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229961AbhDWKal (ORCPT + 1 other); Fri, 23 Apr 2021 06:30:41 -0400 Received: from perceval.ideasonboard.com ([213.167.242.64]:35954 "EHLO perceval.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229772AbhDWKak (ORCPT ); Fri, 23 Apr 2021 06:30:40 -0400 Received: from deskari.lan (91-157-208-71.elisa-laajakaista.fi [91.157.208.71]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 85FC2332; Fri, 23 Apr 2021 12:30:02 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1619173803; bh=HnB0Z8PkQFOAewJm05DotevU7SY3hBGxLYttCNC7Vto=; h=From:To:Cc:Subject:Date:From; b=vkFj3YwEiCsS5yFZ6y/pgFJ96CwAdx219w9lHUQ8TQhXCSKWFTaYokqIRTPFxmyx3 DQZcZ9lZS3mJ3gxqrm/O5OkMRDP2fLOnUN/F58BwFzdUCG9GAqqN6Dy8tUnqhxZJzw 11Tv66+VwSbUCOBv46cZhWFnkmUFMKjmEi8IjS7Y= From: Tomi Valkeinen To: Mauro Carvalho Chehab , Hans Verkuil , linux-media@vger.kernel.org, Laurent Pinchart , Sakari Ailus Cc: Tomi Valkeinen Subject: [PATCH v2 0/1] RFC: media: v4l2-subdev: add subdev-wide config struct Date: Fri, 23 Apr 2021 13:29:51 +0300 Message-Id: <20210423102952.137638-1-tomi.valkeinen@ideasonboard.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.5 (--) X-LSpam-Report: No, score=-2.5 required=5.0 tests=BAYES_00=-1.9,DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1 autolearn=ham autolearn_force=no Hi, Second version of the RFC. The previous one is here: https://lore.kernel.org/linux-media/20210409133659.389544-1-tomi.valkeinen@ideasonboard.com/ In v2: - Rename v4l2_subdev_config -> v4l2_subdev_state - Rename pad_configs -> pads - Allocate the v4l2_subdev_state instead of embedding it I didn't: - Rename v4l2_subdev_pad_ops->init_cfg to init_state. There will be a huge amount of changes to drivers, and I'd like to keep any extra changes out of this series. For the same reason, I intend to keep the v4l2_subdev_state variable names as 'cfg' instead of trying to rename them to 'state'. - Add number of pads or a pointer to subdev to v4l2_subdev_state. It can be added easily if someone has a need for it. I also dropped the few driver changes I made. They are obvious trivial changes and don't really give anything to the discussion. Tomi Tomi Valkeinen (1): RFC: media: v4l2-subdev: add subdev-wide state struct drivers/media/v4l2-core/v4l2-subdev.c | 141 +++++++++++++++----------- include/media/v4l2-subdev.h | 72 +++++++------ 2 files changed, 120 insertions(+), 93 deletions(-)