From patchwork Thu Mar 7 09:29:52 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hans Verkuil X-Patchwork-Id: 10842461 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 2D064139A for ; Thu, 7 Mar 2019 09:30:09 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 185AF2E9F3 for ; Thu, 7 Mar 2019 09:30:09 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 04E042EA06; Thu, 7 Mar 2019 09:30:09 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.9 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 8FF312E9EA for ; Thu, 7 Mar 2019 09:30:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726185AbfCGJaH (ORCPT ); Thu, 7 Mar 2019 04:30:07 -0500 Received: from lb3-smtp-cloud7.xs4all.net ([194.109.24.31]:40711 "EHLO lb3-smtp-cloud7.xs4all.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726015AbfCGJaH (ORCPT ); Thu, 7 Mar 2019 04:30:07 -0500 Received: from test-no.fritz.box ([212.251.195.8]) by smtp-cloud7.xs4all.net with ESMTPA id 1pLlh7xLdLMwI1pLphxTBW; Thu, 07 Mar 2019 10:30:05 +0100 From: hverkuil-cisco@xs4all.nl To: linux-media@vger.kernel.org Cc: Laurent Pinchart , Helen Koike Subject: [PATCHv3 0/9] Various core and virtual driver fixes Date: Thu, 7 Mar 2019 10:29:52 +0100 Message-Id: <20190307093001.30435-1-hverkuil-cisco@xs4all.nl> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 X-CMAE-Envelope: MS4wfKm51N5Mw31uFM3/Dh5gqOge3zjtDQmtWu7Us6GOy87bn1oMHsfCuShac/90g+lPaKIiuQYhWukQtM2TJ1lJW7LDL7B+52nNS9bh6Iyald56twABDmTR Bnu5jbwA+0hl38Zso7qP8gjo+UYGzpFzYAlnSD/UkIzAWQBltRjjzJczfkrf0EyWlGlQXl9pKjIWvTru3AVtmHqOLZ0i6i1omsUS423FOFHAeP/jVLWYNCXU iWFWZPwb+vs6YFs6o9QLMwboRx1ryMYWY0/GTMVfxvU= Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Hans Verkuil Various fixes for bugs that I found while working on the regression test-media script. The CONFIG_DEBUG_KOBJECT_RELEASE=y option in particular found a bunch of bugs where things were not released in the right order. Note that the first two patches are not bug fixes, but they help debugging kobjects. Without this the object name is just (null), which makes it hard to figure out what the object is. Changes since v2: - Extended release callback documentation in patch 6/9 - Added missing documentation for the new owner field in patch 7/9 - Removed trailing . in sd_int_ops documentation in patch 9/9 for consistency. With these changes I can run the test-media script without errors (except for a known vimc 'do not call blocking ops when !TASK_RUNNING;' error, a patch for this from Helen arrived yesterday). Many thanks to Laurent for reviewing this series! Regards, Hans Hans Verkuil (9): cec: fill in cec chardev kobject to ease debugging media-devnode: fill in media chardev kobject to ease debugging vivid: use vzalloc for dev->bitmap_out media-entity: set ent_enum->bmap to NULL after freeing it vim2m: replace devm_kzalloc by kzalloc v4l2-subdev: add release() internal op v4l2-subdev: handle module refcounting here vimc: free vimc_cap_device when the last user disappears vimc: use new release op drivers/media/cec/cec-core.c | 1 + drivers/media/media-devnode.c | 1 + drivers/media/media-entity.c | 29 +--------------- drivers/media/platform/vim2m.c | 35 ++++++++++++-------- drivers/media/platform/vimc/vimc-capture.c | 13 ++++++-- drivers/media/platform/vimc/vimc-common.c | 2 ++ drivers/media/platform/vimc/vimc-common.h | 2 ++ drivers/media/platform/vimc/vimc-debayer.c | 15 +++++++-- drivers/media/platform/vimc/vimc-scaler.c | 15 +++++++-- drivers/media/platform/vimc/vimc-sensor.c | 19 ++++++++--- drivers/media/platform/vivid/vivid-vid-out.c | 14 +++++--- drivers/media/v4l2-core/v4l2-device.c | 19 ++++++++--- drivers/media/v4l2-core/v4l2-subdev.c | 22 +++++------- include/media/media-entity.h | 24 -------------- include/media/v4l2-subdev.h | 15 ++++++++- 15 files changed, 126 insertions(+), 100 deletions(-)