From patchwork Thu Jul 5 06:54:20 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hans Verkuil X-Patchwork-Id: 1158141 Return-Path: X-Original-To: patchwork-linux-media@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id A348440ABE for ; Thu, 5 Jul 2012 06:55:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752778Ab2GEGz1 (ORCPT ); Thu, 5 Jul 2012 02:55:27 -0400 Received: from smtp-vbr2.xs4all.nl ([194.109.24.22]:2441 "EHLO smtp-vbr2.xs4all.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752689Ab2GEGzA (ORCPT ); Thu, 5 Jul 2012 02:55:00 -0400 Received: from alastor.dyndns.org (166.80-203-20.nextgentel.com [80.203.20.166] (may be forged)) (authenticated bits=0) by smtp-vbr2.xs4all.nl (8.13.8/8.13.8) with ESMTP id q656sQge045381 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=FAIL); Thu, 5 Jul 2012 08:54:27 +0200 (CEST) (envelope-from hverkuil@xs4all.nl) Received: from tschai.localnet (tschai.lan [192.168.1.10]) (Authenticated sender: hans) by alastor.dyndns.org (Postfix) with ESMTPSA id E925980012; Thu, 5 Jul 2012 08:54:19 +0200 (CEST) From: Hans Verkuil To: Sylwester Nawrocki Subject: Re: [GIT PULL FOR v3.6] mostly remove V4L2_FL_LOCK_ALL_FOPS Date: Thu, 5 Jul 2012 08:54:20 +0200 User-Agent: KMail/1.13.7 (Linux/3.4.0-rc7-tschai; KDE/4.8.4; x86_64; ; ) Cc: "linux-media" References: <201207041942.04606.hverkuil@xs4all.nl> <4FF49957.3070604@gmail.com> In-Reply-To: <4FF49957.3070604@gmail.com> MIME-Version: 1.0 Message-Id: <201207050854.20966.hverkuil@xs4all.nl> X-Virus-Scanned: by XS4ALL Virus Scanner Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org On Wed July 4 2012 21:28:23 Sylwester Nawrocki wrote: > Hi Hans, > > On 07/04/2012 07:42 PM, Hans Verkuil wrote: > > Hi Mauro, > > > > This pull request builds on (and includes) this core patch series: > > > > http://patchwork.linuxtv.org/patch/13180/ > > > > It is identical to the RFC patch series I posted before: > > > > http://www.mail-archive.com/linux-media@vger.kernel.org/msg48184.html > > > > ...except that I dropped the patches touching s5p-fimc and fimc-lite and the > > final patch removing the flag altogether. > > > > Sylwester posted patches for those two drivers, but they won't apply. Sylwester, > > can you rebase those patches? Once I have those, then I can make another pull > > request that fixes those two drivers and removes the flag completely. > > First of all, sorry about the trouble. These patches are available at: > > git://git.infradead.org/users/kmpark/linux-samsung v4l-fimc-next > > The first one was already included in a pull request with fixes for 3.5-rc: > http://patchwork.linuxtv.org/patch/13149. It is not really needed for > your V4L2_FL_LOCK_ALL_FOPS removal changeset. You only need the top one from > the above branch. > > The reason why those 2 patches won't apply to the current media tree is > a pending fixups pull request from over a month ago - > http://patchwork.linuxtv.org/patch/11503. These are mostly important fixes > for new code that went into 3.5, and I'm really not happy they're still not > upstream :( > > So I would propose you to merge v4l-fimc-fixes branch from > git://git.infradead.org/users/kmpark/linux-samsung, then there won't be > any merge conflicts. > > Please feel free to add the top patch: > "s5p-fimc: Remove V4L2_FL_LOCK_ALL_FOPS flag" > to your pull request including the patch removing V4L2_FL_LOCK_ALL_FOPS. > > Here is a gitweb link: > http://git.infradead.org/users/kmpark/linux-samsung/shortlog/refs/heads/v4l-fimc-next Hi Sylwester, It still doesn't apply. This patch starts with: diff --git a/drivers/media/video/s5p-fimc/fimc-capture.c b/drivers/media/video/s5p-fimc/fimc-capture.c index da2c40e..cb04a870 100644 --- a/drivers/media/video/s5p-fimc/fimc-capture.c +++ b/drivers/media/video/s5p-fimc/fimc-capture.c @@ -480,48 +480,59 @@ static int fimc_capture_set_default_format(struct fimc_dev *fimc); static int fimc_capture_open(struct file *file) { struct fimc_dev *fimc = video_drvdata(file); - int ret; + int ret = -EBUSY; The actual current source code starts with: static int fimc_capture_open(struct file *file) { struct fimc_dev *fimc = video_drvdata(file); int ret = v4l2_fh_open(file); Quite different. That 'int ret = v4l2_fh_open(file);' line was added June 10th, 2011, so I don't understand why that isn't in your git repository. Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html