From patchwork Tue Dec 8 17:17:17 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mauro Carvalho Chehab X-Patchwork-Id: 7800321 Return-Path: X-Original-To: patchwork-linux-media@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 1F1AFBEEE1 for ; Tue, 8 Dec 2015 17:17:26 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 3D225204D6 for ; Tue, 8 Dec 2015 17:17:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 5D3CD204B5 for ; Tue, 8 Dec 2015 17:17:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751102AbbLHRRW (ORCPT ); Tue, 8 Dec 2015 12:17:22 -0500 Received: from lists.s-osg.org ([54.187.51.154]:46727 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750822AbbLHRRV convert rfc822-to-8bit (ORCPT ); Tue, 8 Dec 2015 12:17:21 -0500 Received: from recife.lan (179.186.106.209.dynamic.adsl.gvt.net.br [179.186.106.209]) by lists.s-osg.org (Postfix) with ESMTPSA id 14D75462A3; Tue, 8 Dec 2015 09:17:19 -0800 (PST) Date: Tue, 8 Dec 2015 15:17:17 -0200 From: Mauro Carvalho Chehab To: Laurent Pinchart Cc: Linux Media Mailing List , Mauro Carvalho Chehab , Kyungmin Park , Andrzej Hajda Subject: Re: [PATCH v8 35/55] [media] s5k5baf: fix subdev type Message-ID: <20151208151717.73cf79e2@recife.lan> In-Reply-To: <2154292.e5cNedqy2f@avalon> References: <7ed3721139e459f5dd4cdd05bd1e58f248fc0781.1440902901.git.mchehab@osg.samsung.com> <2154292.e5cNedqy2f@avalon> Organization: Samsung X-Mailer: Claws Mail 3.12.0 (GTK+ 2.24.28; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Em Sun, 06 Dec 2015 03:55:32 +0200 Laurent Pinchart escreveu: > Hi Mauro, > > Thank you for the patch. > > On Sunday 30 August 2015 00:06:46 Mauro Carvalho Chehab wrote: > > X-Patchwork-Delegate: m.chehab@samsung.com > > This sensor driver is abusing MEDIA_ENT_T_V4L2_SUBDEV, creating > > some subdevs with a non-existing type. > > > > As this is a sensor driver, the proper type is likely > > MEDIA_ENT_T_V4L2_SUBDEV_SENSOR. > > That's actually not correct. The driver creates two subdevs, one for the image > sensor pixel array (and the related readout logic) and one for an ISP. The > first subdev already uses the MEDIA_ENT_T_V4L2_SUBDEV_SENSOR type, but the > second subdev isn't a sensor pixel array. OK. Patch replaced by the one below. Thanks, Mauro From b1acc860aa845e9ea84fa597d540ad34047fe0cc Mon Sep 17 00:00:00 2001 From: Mauro Carvalho Chehab Date: Thu, 7 May 2015 22:12:35 -0300 Subject: [media] s5k5baf: fix subdev type Cc: Linux Media Mailing List , Mauro Carvalho Chehab The driver creates two subdevs, one for the image sensor pixel array (and the related readout logic) and one for an ISP. The first subdev already uses the MEDIA_ENT_T_V4L2_SUBDEV_SENSOR type, but the second subdev isn't a sensor pixel array. So, rename the second subdev as MEDIA_ENT_T_V4L2_SUBDEV_UNKNOWN. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/i2c/s5k5baf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/s5k5baf.c b/drivers/media/i2c/s5k5baf.c index d3bff30bcb6f..0513196bd48c 100644 --- a/drivers/media/i2c/s5k5baf.c +++ b/drivers/media/i2c/s5k5baf.c @@ -1919,7 +1919,7 @@ static int s5k5baf_configure_subdevs(struct s5k5baf *state, state->pads[PAD_CIS].flags = MEDIA_PAD_FL_SINK; state->pads[PAD_OUT].flags = MEDIA_PAD_FL_SOURCE; - sd->entity.type = MEDIA_ENT_T_V4L2_SUBDEV; + sd->entity.type = MEDIA_ENT_T_V4L2_SUBDEV_UNKNOWN; ret = media_entity_init(&sd->entity, NUM_ISP_PADS, state->pads); if (!ret)