|
 |
2d52c31 |
From patchwork Mon Jun 24 22:13:15 2019
|
|
|
2d52c31 |
Content-Type: text/plain; charset="utf-8"
|
|
|
2d52c31 |
MIME-Version: 1.0
|
|
|
2d52c31 |
Content-Transfer-Encoding: 7bit
|
|
|
2d52c31 |
X-Patchwork-Submitter: Stefan Wahren <wahrenst@gmx.net>
|
|
|
2d52c31 |
X-Patchwork-Id: 11014257
|
|
|
2d52c31 |
Return-Path:
|
|
|
2d52c31 |
<linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org>
|
|
|
2d52c31 |
Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org
|
|
|
2d52c31 |
[172.30.200.125])
|
|
|
2d52c31 |
by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id AF9B913AF
|
|
|
2d52c31 |
for <patchwork-linux-arm@patchwork.kernel.org>;
|
|
|
2d52c31 |
Mon, 24 Jun 2019 22:19:04 +0000 (UTC)
|
|
|
2d52c31 |
Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1])
|
|
|
2d52c31 |
by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 9E48D288D0
|
|
|
2d52c31 |
for <patchwork-linux-arm@patchwork.kernel.org>;
|
|
|
2d52c31 |
Mon, 24 Jun 2019 22:19:04 +0000 (UTC)
|
|
|
2d52c31 |
Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486)
|
|
|
2d52c31 |
id 915A92890C; Mon, 24 Jun 2019 22:19:04 +0000 (UTC)
|
|
|
2d52c31 |
X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on
|
|
|
2d52c31 |
pdx-wl-mail.web.codeaurora.org
|
|
|
2d52c31 |
X-Spam-Level:
|
|
|
2d52c31 |
X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00,DKIM_SIGNED,
|
|
|
2d52c31 |
DKIM_VALID,FREEMAIL_FROM,MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED autolearn=ham
|
|
|
2d52c31 |
version=3.3.1
|
|
|
2d52c31 |
Received: from bombadil.infradead.org (bombadil.infradead.org
|
|
|
2d52c31 |
[198.137.202.133])
|
|
|
2d52c31 |
(using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits))
|
|
|
2d52c31 |
(No client certificate requested)
|
|
|
2d52c31 |
by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 334AB288D0
|
|
|
2d52c31 |
for <patchwork-linux-arm@patchwork.kernel.org>;
|
|
|
2d52c31 |
Mon, 24 Jun 2019 22:19:04 +0000 (UTC)
|
|
|
2d52c31 |
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;
|
|
|
2d52c31 |
d=lists.infradead.org; s=bombadil.20170209; h=Sender:
|
|
|
2d52c31 |
Content-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe:
|
|
|
2d52c31 |
List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date:
|
|
|
2d52c31 |
Subject:To:From:Reply-To:Content-ID:Content-Description:Resent-Date:
|
|
|
2d52c31 |
Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:
|
|
|
2d52c31 |
References:List-Owner; bh=pZ4lWwxNSX3iSSGrbQoSUtMz7+GLTRWyjXRJWpId+5s=; b=De9
|
|
|
2d52c31 |
hICr3uVCMJ9HF64mqdARRKXLqBMi85tC4LHoOtzHM3RV+H9ci2Le55P7omHm7Yexo43W0Wko4kaPH
|
|
|
2d52c31 |
I7tMpiA/5CYW7ZAHjla3mxLEMLMSPFwYqBufuR1/uS3vK9+Hgo7oDVhN7Yn5qAtAZfY+8LT69pryi
|
|
|
2d52c31 |
8yetpp6upD56usJT/1aV92r4F07oF/sMdP7cAHOLGl30vKZPt3LQhfYranu0A4HFbzFdCbkjjGTdp
|
|
|
2d52c31 |
gGRUMaGkoQYiDmD4x5mDTsJX7czSSpRM11kcnxCmP6U6Hy47tXSNnaIFyFX7ZNQtYA2id9LKdq9jk
|
|
|
2d52c31 |
4ru0cwPcvfDiKLWysBsVJN7nPIacCYQ==;
|
|
|
2d52c31 |
Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org)
|
|
|
2d52c31 |
by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux))
|
|
|
2d52c31 |
id 1hfXIl-0000Kg-MK; Mon, 24 Jun 2019 22:19:03 +0000
|
|
|
2d52c31 |
Received: from mout.gmx.net ([212.227.17.21])
|
|
|
2d52c31 |
by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux))
|
|
|
2d52c31 |
id 1hfXIh-0000KK-QL
|
|
|
2d52c31 |
for linux-arm-kernel@lists.infradead.org; Mon, 24 Jun 2019 22:19:01 +0000
|
|
|
2d52c31 |
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net;
|
|
|
2d52c31 |
s=badeba3b8450; t=1561414736;
|
|
|
2d52c31 |
bh=GysLsYTuZsd/CXGs4HdO1VCTJil5RrdvuS6/TiPl2bY=;
|
|
|
2d52c31 |
h=X-UI-Sender-Class:From:To:Cc:Subject:Date;
|
|
|
2d52c31 |
b=gU0KSa1e5eyFsVnSCujOiYDvflXyoskJ0tc60c59NJ4k7CL076dwvum1oH1DIzsmh
|
|
|
2d52c31 |
kYOWRAvu4FZu33Bcxdtv84pgSNSHQt3P9oT0/hULw264uwuuzyjRqjnljK+4ObsjaK
|
|
|
2d52c31 |
3VE9Nt7wobpNaeXddwRKPy0VwPBPS9Cyi6Ype8YA=
|
|
|
2d52c31 |
X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c
|
|
|
2d52c31 |
Received: from localhost.localdomain ([37.4.249.111]) by mail.gmx.com
|
|
|
2d52c31 |
(mrgmx103 [212.227.17.168]) with ESMTPSA (Nemesis) id
|
|
|
2d52c31 |
0MKYpv-1hguWc2RVf-001wNW; Tue, 25 Jun 2019 00:13:21 +0200
|
|
|
2d52c31 |
From: Stefan Wahren <wahrenst@gmx.net>
|
|
|
2d52c31 |
To: Eric Anholt <eric@anholt.net>,
|
|
|
2d52c31 |
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
|
|
|
2d52c31 |
Madhumitha Prabakaran <madhumithabiw@gmail.com>
|
|
|
2d52c31 |
Subject: [PATCH] staging: bcm2835-camera: Restore return behavior of
|
|
|
2d52c31 |
ctrl_set_bitrate()
|
|
|
2d52c31 |
Date: Tue, 25 Jun 2019 00:13:15 +0200
|
|
|
2d52c31 |
Message-Id: <1561414395-12518-1-git-send-email-wahrenst@gmx.net>
|
|
|
2d52c31 |
X-Mailer: git-send-email 2.7.4
|
|
|
2d52c31 |
X-Provags-ID: V03:K1:LGa8UAO+GzmbgvlLItsNffmxnvrhpUjzzhEbaoSn7FwAXOkBTQT
|
|
|
2d52c31 |
CXW3GFoFMF7ihP2FNuGybonNAe4XC1IG1wlefi99zYppeICDE1DXNEfp72vJtW+hdWbIWTv
|
|
|
2d52c31 |
rQGf9V1DQzVE6avn/XMprdK5kasTgjwFYL1ePyQdxClEBcHaYQFYCkmYu3B0wvQkayY3VwX
|
|
|
2d52c31 |
I5OkkpT21gvRetaja00uw==
|
|
|
2d52c31 |
X-UI-Out-Filterresults: notjunk:1;V03:K0:jQ7kLM3btHE=:Vw0mHeUG2/OLYWx2xsMv86
|
|
|
2d52c31 |
BDj9ACSIdpZZdSkWv37UT7mHK3JlRySOYYM5m/IzfAyzlNxXvPH0NCmzMLpF7RVotI9/DU3MN
|
|
|
2d52c31 |
sK7g5b3yd5sxlNf03d9ryZUUFRAlj2jydoVyVgTJzSvYhV34nomQaX3H8i3y2hWm0IIk5OMXK
|
|
|
2d52c31 |
2W48wL4meZ5yWH9Ixh5R3/XIuyvaa4ehoTOBhaSaNMJw6zMZdNid+HWdNCSQ3Qy0lb+qhrPRI
|
|
|
2d52c31 |
+Aw/9ucDR4FkJYdUV5yYt4QhAmTY0sRxRWvTHRlqSr1+I++L5aT77BsWGKsbXq97GcpDLrEHF
|
|
|
2d52c31 |
lbRPDeDeo0kL5i51A+gQ+0DzV38h/rQ7yz54At+uEHw/9AcnW9w4AkZitDqjHy9F9lJ2obV4P
|
|
|
2d52c31 |
jGslvavwp3O4vJrISGoxpX/OuJFfPraVf3B2KNhBIw95jWA7R0fmAp10p1S+roCwkGKEDyuha
|
|
|
2d52c31 |
/TVFLsCmVpe3/3vJyll11l2miG60XHLl0aShILfChHlnWWacTi9O8bQoU9XTigvALJBxYpjo+
|
|
|
2d52c31 |
SqG1z9IYnUYpUx15qXt4hc6UJvCeCFG+Jz9B/QBPuxlmcylDZ/31PnF1GbqrwpBmYePNYBBtU
|
|
|
2d52c31 |
6nWLKLb/7VfmDxG/5VEaBgI0tt/DiXQ5owRmtV19r0sdzqeWRtUvdj3/TMMULZi28PHQfz7Q8
|
|
|
2d52c31 |
nEbF/nUtSRUEZK/wCDWgEq4x2qfG9Yz1Kwi2l4KDsWFezbQtdH5H9UsyOYQiYZhztDskEIol7
|
|
|
2d52c31 |
dtHt9qF75NElTmLOI9vXl/mYGvYfCj04DQqqVvTDxYncRewKBFWr603hweU/nY/IDsFs6F/cV
|
|
|
2d52c31 |
BdBRS6XR/PxROK/NLOBscMDwwEw7qdKbc7Eqv7OTjynQ9yt1KcS7oIYXUyKrd4BvaQdFqMDrs
|
|
|
2d52c31 |
Km3scWeKBdfeJaLKCLPitZYVTPG8PJwPgi5SqUMqgJkGbYxeUP+kgZmvXRz+vu35sYGftgV/t
|
|
|
2d52c31 |
vo2FKmWeEtPyAbDBEbV6LgIA+tQkwk0M=
|
|
|
2d52c31 |
X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3
|
|
|
2d52c31 |
X-CRM114-CacheID: sfid-20190624_151900_149791_318B5C68
|
|
|
2d52c31 |
X-CRM114-Status: UNSURE ( 9.24 )
|
|
|
2d52c31 |
X-CRM114-Notice: Please train this message.
|
|
|
2d52c31 |
X-BeenThere: linux-arm-kernel@lists.infradead.org
|
|
|
2d52c31 |
X-Mailman-Version: 2.1.29
|
|
|
2d52c31 |
Precedence: list
|
|
|
2d52c31 |
List-Id: <linux-arm-kernel.lists.infradead.org>
|
|
|
2d52c31 |
List-Unsubscribe:
|
|
|
2d52c31 |
<http://lists.infradead.org/mailman/options/linux-arm-kernel>,
|
|
|
2d52c31 |
<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>
|
|
|
2d52c31 |
List-Archive: <http://lists.infradead.org/pipermail/linux-arm-kernel/>
|
|
|
2d52c31 |
List-Post: <mailto:linux-arm-kernel@lists.infradead.org>
|
|
|
2d52c31 |
List-Help: <mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>
|
|
|
2d52c31 |
List-Subscribe:
|
|
|
2d52c31 |
<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,
|
|
|
2d52c31 |
<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>
|
|
|
2d52c31 |
Cc: devel@driverdev.osuosl.org, linux-rpi-kernel@lists.infradead.org,
|
|
|
2d52c31 |
linux-arm-kernel@lists.infradead.org, Stefan Wahren <wahrenst@gmx.net>
|
|
|
2d52c31 |
MIME-Version: 1.0
|
|
|
2d52c31 |
Content-Type: text/plain; charset="us-ascii"
|
|
|
2d52c31 |
Sender: "linux-arm-kernel" <linux-arm-kernel-bounces@lists.infradead.org>
|
|
|
2d52c31 |
Errors-To:
|
|
|
2d52c31 |
linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org
|
|
|
2d52c31 |
X-Virus-Scanned: ClamAV using ClamSMTP
|
|
|
2d52c31 |
|
|
|
2d52c31 |
The commit 52c4dfcead49 ("Staging: vc04_services: Cleanup in
|
|
|
2d52c31 |
ctrl_set_bitrate()") changed the return behavior of ctrl_set_bitrate().
|
|
|
2d52c31 |
This breaks probing of bcm2835-camera:
|
|
|
2d52c31 |
|
|
|
2d52c31 |
bcm2835-v4l2: mmal_init: failed to set all camera controls: -3
|
|
|
2d52c31 |
Cleanup: Destroy video encoder
|
|
|
2d52c31 |
Cleanup: Destroy image encoder
|
|
|
2d52c31 |
Cleanup: Destroy video render
|
|
|
2d52c31 |
Cleanup: Destroy camera
|
|
|
2d52c31 |
bcm2835-v4l2: bcm2835_mmal_probe: mmal init failed: -3
|
|
|
2d52c31 |
bcm2835-camera: probe of bcm2835-camera failed with error -3
|
|
|
2d52c31 |
|
|
|
2d52c31 |
So restore the old behavior and fix this issue.
|
|
|
2d52c31 |
|
|
|
2d52c31 |
Fixes: 52c4dfcead49 ("Staging: vc04_services: Cleanup in ctrl_set_bitrate()")
|
|
|
2d52c31 |
Signed-off-by: Stefan Wahren <wahrenst@gmx.net>
|
|
|
2d52c31 |
---
|
|
|
2d52c31 |
drivers/staging/vc04_services/bcm2835-camera/controls.c | 8 +++++---
|
|
|
2d52c31 |
1 file changed, 5 insertions(+), 3 deletions(-)
|
|
|
2d52c31 |
|
|
|
2d52c31 |
--
|
|
|
2d52c31 |
2.7.4
|
|
|
2d52c31 |
|
|
|
2d52c31 |
diff --git a/drivers/staging/vc04_services/bcm2835-camera/controls.c b/drivers/staging/vc04_services/bcm2835-camera/controls.c
|
|
|
2d52c31 |
index d60e378..1c4c9e8 100644
|
|
|
2d52c31 |
--- a/drivers/staging/vc04_services/bcm2835-camera/controls.c
|
|
|
2d52c31 |
+++ b/drivers/staging/vc04_services/bcm2835-camera/controls.c
|
|
|
2d52c31 |
@@ -610,9 +610,11 @@ static int ctrl_set_bitrate(struct bm2835_mmal_dev *dev,
|
|
|
2d52c31 |
|
|
|
2d52c31 |
encoder_out = &dev->component[MMAL_COMPONENT_VIDEO_ENCODE]->output[0];
|
|
|
2d52c31 |
|
|
|
2d52c31 |
- return vchiq_mmal_port_parameter_set(dev->instance, encoder_out,
|
|
|
2d52c31 |
- mmal_ctrl->mmal_id, &ctrl->val,
|
|
|
2d52c31 |
- sizeof(ctrl->val));
|
|
|
2d52c31 |
+ vchiq_mmal_port_parameter_set(dev->instance, encoder_out,
|
|
|
2d52c31 |
+ mmal_ctrl->mmal_id, &ctrl->val,
|
|
|
2d52c31 |
+ sizeof(ctrl->val));
|
|
|
2d52c31 |
+
|
|
|
2d52c31 |
+ return 0;
|
|
|
2d52c31 |
}
|
|
|
2d52c31 |
|
|
|
2d52c31 |
static int ctrl_set_bitrate_mode(struct bm2835_mmal_dev *dev,
|