From patchwork Mon Jul 15 21:06:41 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ezequiel Garcia X-Patchwork-Id: 57544 Received: from vger.kernel.org ([209.132.180.67]) by www.linuxtv.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hn8BZ-0007hs-El; Mon, 15 Jul 2019 21:07:01 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731534AbfGOVG7 (ORCPT + 1 other); Mon, 15 Jul 2019 17:06:59 -0400 Received: from bhuna.collabora.co.uk ([46.235.227.227]:52692 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731381AbfGOVG7 (ORCPT ); Mon, 15 Jul 2019 17:06:59 -0400 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: ezequiel) with ESMTPSA id 84A0A28B67E From: Ezequiel Garcia To: Hans Verkuil Cc: kernel@collabora.com, linux-media@vger.kernel.org, Ezequiel Garcia Subject: [PATCH 0/6] V4L2 core I2C/SPI code cleanup Date: Mon, 15 Jul 2019 18:06:41 -0300 Message-Id: <20190715210649.31681-1-ezequiel@collabora.com> X-Mailer: git-send-email 2.22.0 MIME-Version: 1.0 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Hi Mauro, Hans: While reading v4l2-common.c, I came across a few ifdefs that could be cleaned-up with some minor reorganization. Patch 1 is just cleaning the Makefile, removing ifeq/endif to make it more readable. Patch 2 merges v4l2-common.ko into videodev.ko, which I think it's now possible. Let me know if having these two modules separated serves a purpose I'm missing. The rest of the patches split the I2C and SPI helpers, so they can be conditionally built. There are a few checkpatch.pl issues triggered here, all of them belonging to the already existing code. Let me know if you want me to clean that as well. The entire series should not affect any functionality, but just clean-up the code a bit. Thanks, Eze Ezequiel Garcia (6): media: v4l2-core: Cleanup Makefile media: v4l2-core: Module re-organization media: v4l2-core: move spi helpers out of v4l2-common.c media: v4l2-core: move i2c helpers out of v4l2-common.c media: v4l2-core: introduce a helper to unregister a SPI subdev media: v4l2-core: introduce a helper to unregister a I2C subdev drivers/media/v4l2-core/Kconfig | 5 + drivers/media/v4l2-core/Makefile | 15 +- drivers/media/v4l2-core/v4l2-common.c | 210 -------------------------- drivers/media/v4l2-core/v4l2-device.c | 39 +---- drivers/media/v4l2-core/v4l2-i2c.c | 167 ++++++++++++++++++++ drivers/media/v4l2-core/v4l2-spi.c | 73 +++++++++ include/media/v4l2-common.h | 150 +++++++++++++----- 7 files changed, 366 insertions(+), 293 deletions(-) create mode 100644 drivers/media/v4l2-core/v4l2-i2c.c create mode 100644 drivers/media/v4l2-core/v4l2-spi.c