From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 863D772 for ; Wed, 8 Sep 2021 22:43:30 +0000 (UTC) Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 33B7E883; Thu, 9 Sep 2021 00:37:32 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1631140652; bh=1bGvqSulJnZHe9TfBwp2sU+A8oEVgTU8e0GgVdcU/ws=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=cef2d0qWc8/lv1lki7yGSBS6cZX5fhhyJWDWBDSt4oq6beKouAYBKJzPayLQxOM/9 31dAAad6plu2+pKQWCvzlYuR1nhBraJfy7JRu6F2fAajaiLYHoTaqPbXkjYu5HEMQk hNf/pqe830GoTcdoHx7+bjaYR3JlEeSM0ETYTP8E= Date: Thu, 9 Sep 2021 01:37:11 +0300 From: Laurent Pinchart To: Martin Kepplinger Cc: slongerbeam@gmail.com, p.zabel@pengutronix.de, mchehab@kernel.org, kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com, linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] media: imx: set a media_device bus_info string Message-ID: References: <20210908084746.243359-1-martin.kepplinger@puri.sm> Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20210908084746.243359-1-martin.kepplinger@puri.sm> Hi Martin, Thank you for the patch. On Wed, Sep 08, 2021 at 10:47:46AM +0200, Martin Kepplinger wrote: > some tools like v4l2-compliance let users select a media device based s/some/Some/ I'll fix then when applying the patch to my tree. > on the bus_info string which can be quite convenient. Use a unique > string for that. > > This also fixes the following v4l2-compliance warning: > warn: v4l2-test-media.cpp(52): empty bus_info > > Signed-off-by: Martin Kepplinger Reviewed-by: Laurent Pinchart > --- > drivers/staging/media/imx/imx-media-dev-common.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/staging/media/imx/imx-media-dev-common.c b/drivers/staging/media/imx/imx-media-dev-common.c > index d006e961d8f4..80b69a9a752c 100644 > --- a/drivers/staging/media/imx/imx-media-dev-common.c > +++ b/drivers/staging/media/imx/imx-media-dev-common.c > @@ -367,6 +367,8 @@ struct imx_media_dev *imx_media_dev_init(struct device *dev, > imxmd->v4l2_dev.notify = imx_media_notify; > strscpy(imxmd->v4l2_dev.name, "imx-media", > sizeof(imxmd->v4l2_dev.name)); > + snprintf(imxmd->md.bus_info, sizeof(imxmd->md.bus_info), > + "platform:%s", dev_name(imxmd->md.dev)); > > media_device_init(&imxmd->md); > -- Regards, Laurent Pinchart 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=-14.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham 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 BA627C433F5 for ; Wed, 8 Sep 2021 22:39:36 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 8DA5660F5E for ; Wed, 8 Sep 2021 22:39:36 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 8DA5660F5E Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=DSs/LLspX70OlTasucWZeqQMSjpkz0OhnTDhJJxIZN0=; b=hHISaaSPOF65kO zhHm7oaRZPdSl7q0pGZdv5uBZ2DEYBIhEruYc20bloKALfes8wz6aXO/IeAxgOZfB8iltjCYJReOV iJQsoeacsYACtbJnVVwCxbZ8/mNVwyLo9+ZvXxMsY1+p3JtSJQTjElTeJZc6g48dax7aSFAJp2XpK obPQS5JjujM164+Z/a5VbQMASPScqfZYGHx484uP2JiZ5JYCDE0cDMTxwbSeLBnjoMyG+x6x+Jlqj xKrNUiBzh2IA8i9QTi0xzhRWB7n+yQ50n/bSbdXcVbbkoge1d9liRsS0qHI/DL5MgpLdWJp04QbGW GgxyQinKEDYsCWLtVYfQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mO6CL-007jVk-El; Wed, 08 Sep 2021 22:37:41 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mO6CG-007jTu-Hd for linux-arm-kernel@lists.infradead.org; Wed, 08 Sep 2021 22:37:39 +0000 Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 33B7E883; Thu, 9 Sep 2021 00:37:32 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1631140652; bh=1bGvqSulJnZHe9TfBwp2sU+A8oEVgTU8e0GgVdcU/ws=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=cef2d0qWc8/lv1lki7yGSBS6cZX5fhhyJWDWBDSt4oq6beKouAYBKJzPayLQxOM/9 31dAAad6plu2+pKQWCvzlYuR1nhBraJfy7JRu6F2fAajaiLYHoTaqPbXkjYu5HEMQk hNf/pqe830GoTcdoHx7+bjaYR3JlEeSM0ETYTP8E= Date: Thu, 9 Sep 2021 01:37:11 +0300 From: Laurent Pinchart To: Martin Kepplinger Cc: slongerbeam@gmail.com, p.zabel@pengutronix.de, mchehab@kernel.org, kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com, linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] media: imx: set a media_device bus_info string Message-ID: References: <20210908084746.243359-1-martin.kepplinger@puri.sm> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210908084746.243359-1-martin.kepplinger@puri.sm> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210908_153736_769443_3F3AB868 X-CRM114-Status: GOOD ( 15.63 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Martin, Thank you for the patch. On Wed, Sep 08, 2021 at 10:47:46AM +0200, Martin Kepplinger wrote: > some tools like v4l2-compliance let users select a media device based s/some/Some/ I'll fix then when applying the patch to my tree. > on the bus_info string which can be quite convenient. Use a unique > string for that. > > This also fixes the following v4l2-compliance warning: > warn: v4l2-test-media.cpp(52): empty bus_info > > Signed-off-by: Martin Kepplinger Reviewed-by: Laurent Pinchart > --- > drivers/staging/media/imx/imx-media-dev-common.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/staging/media/imx/imx-media-dev-common.c b/drivers/staging/media/imx/imx-media-dev-common.c > index d006e961d8f4..80b69a9a752c 100644 > --- a/drivers/staging/media/imx/imx-media-dev-common.c > +++ b/drivers/staging/media/imx/imx-media-dev-common.c > @@ -367,6 +367,8 @@ struct imx_media_dev *imx_media_dev_init(struct device *dev, > imxmd->v4l2_dev.notify = imx_media_notify; > strscpy(imxmd->v4l2_dev.name, "imx-media", > sizeof(imxmd->v4l2_dev.name)); > + snprintf(imxmd->md.bus_info, sizeof(imxmd->md.bus_info), > + "platform:%s", dev_name(imxmd->md.dev)); > > media_device_init(&imxmd->md); > -- Regards, Laurent Pinchart _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel