[{"id":15446,"web_url":"https://patchwork.libcamera.org/comment/15446/","msgid":"<adc3a629-6b76-a62f-23d3-e69d5f6231b1@collabora.com>","date":"2021-03-03T14:34:40","subject":"Re: [libcamera-devel] [PATCH 0/2] libcamera: Update kernel headers\n\tto v5.12-rc1","submitter":{"id":46,"url":"https://patchwork.libcamera.org/api/people/46/","name":"Dafna Hirschfeld","email":"dafna.hirschfeld@collabora.com"},"content":"On 02.03.21 21:16, Laurent Pinchart wrote:\n> Hello,\n> \n> For development related to the simple pipeline handler, I need v5.12-rc1\n> of the media.h header. This series pulls the kernel headers from that\n> kernel, and update the rkisp1 IPA accordingly as the rkisp1 driver's\n> UAPI has changed upstream.\n> \n> There will be a bisection breakage (at runtime, not compile time)\n> between the two patches. I can squash them together if desired.\n> \n> Note that Dafna has posted a pstch series to support different hardware\n> revisions of the rkisp1 (see\n> https://patchwork.libcamera.org/cover/11383/). This series conflicts\n> with it, but doesn't take over that work, as it only updates the IPA to\n> avoid breakages without taking the hardware revision into account.\n> \n> Dafna, I don't mind merging your series first if you want to post a new\n> version before these two patches get reviewed, in which case I'll rebase\n> 1/2 and drop 2/2. You can also decide to rebase on top of this and make\n> it a dependency.\n\nHi, I'll rebase v3 on top of this patchset, this will save me some work:)\n\nThanks,\nDafna\n\n> \n> Please also note that this change requires kernel v5.11 or newer for the\n> rkisp1, or a backport of the rkisp1 driver to an older kernel.\n> \n> Laurent Pinchart (2):\n>    include: linux: Update Linux headers to v5.12-rc1\n>    ipa: rkisp1: Update to kernel header changes\n> \n>   include/linux/dma-heap.h         |   6 +-\n>   include/linux/drm_fourcc.h       | 530 +++++++++++++++++-\n>   include/linux/intel-ipu3.h       |  22 +-\n>   include/linux/media-bus-format.h |  14 +-\n>   include/linux/media.h            |   1 +\n>   include/linux/rkisp1-config.h    |  92 +++-\n>   include/linux/v4l2-controls.h    | 916 ++++++++++++++++++++++++-------\n>   include/linux/v4l2-mediabus.h    |  15 +-\n>   include/linux/v4l2-subdev.h      |  26 +-\n>   include/linux/videodev2.h        |  99 +++-\n>   src/ipa/rkisp1/rkisp1.cpp        |   2 +-\n>   11 files changed, 1445 insertions(+), 278 deletions(-)\n>","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 8E943BD1F1\n\tfor <parsemail@patchwork.libcamera.org>;\n\tWed,  3 Mar 2021 14:34:44 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 6AAFD68A9A;\n\tWed,  3 Mar 2021 15:34:44 +0100 (CET)","from bhuna.collabora.co.uk (bhuna.collabora.co.uk\n\t[IPv6:2a00:1098:0:82:1000:25:2eeb:e3e3])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 7C68368A7E\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tWed,  3 Mar 2021 15:34:43 +0100 (CET)","from [IPv6:2003:c7:cf38:3800:7d25:342b:1528:aa90]\n\t(p200300c7cf3838007d25342b1528aa90.dip0.t-ipconnect.de\n\t[IPv6:2003:c7:cf38:3800:7d25:342b:1528:aa90])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits))\n\t(No client certificate requested) (Authenticated sender: dafna)\n\tby bhuna.collabora.co.uk (Postfix) with ESMTPSA id 27A8E1F45635;\n\tWed,  3 Mar 2021 14:34:42 +0000 (GMT)"],"To":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>,\n\tlibcamera-devel@lists.libcamera.org","References":"<20210302201603.21566-1-laurent.pinchart@ideasonboard.com>","From":"Dafna Hirschfeld <dafna.hirschfeld@collabora.com>","Message-ID":"<adc3a629-6b76-a62f-23d3-e69d5f6231b1@collabora.com>","Date":"Wed, 3 Mar 2021 15:34:40 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101\n\tThunderbird/68.10.0","MIME-Version":"1.0","In-Reply-To":"<20210302201603.21566-1-laurent.pinchart@ideasonboard.com>","Content-Language":"en-US","Subject":"Re: [libcamera-devel] [PATCH 0/2] libcamera: Update kernel headers\n\tto v5.12-rc1","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","Content-Transfer-Encoding":"7bit","Content-Type":"text/plain; charset=\"us-ascii\"; Format=\"flowed\"","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}},{"id":15447,"web_url":"https://patchwork.libcamera.org/comment/15447/","msgid":"<YD+eywgZna70bJIT@pendragon.ideasonboard.com>","date":"2021-03-03T14:35:55","subject":"Re: [libcamera-devel] [PATCH 0/2] libcamera: Update kernel headers\n\tto v5.12-rc1","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Dafna,\n\nOn Wed, Mar 03, 2021 at 03:34:40PM +0100, Dafna Hirschfeld wrote:\n> On 02.03.21 21:16, Laurent Pinchart wrote:\n> > Hello,\n> > \n> > For development related to the simple pipeline handler, I need v5.12-rc1\n> > of the media.h header. This series pulls the kernel headers from that\n> > kernel, and update the rkisp1 IPA accordingly as the rkisp1 driver's\n> > UAPI has changed upstream.\n> > \n> > There will be a bisection breakage (at runtime, not compile time)\n> > between the two patches. I can squash them together if desired.\n> > \n> > Note that Dafna has posted a pstch series to support different hardware\n> > revisions of the rkisp1 (see\n> > https://patchwork.libcamera.org/cover/11383/). This series conflicts\n> > with it, but doesn't take over that work, as it only updates the IPA to\n> > avoid breakages without taking the hardware revision into account.\n> > \n> > Dafna, I don't mind merging your series first if you want to post a new\n> > version before these two patches get reviewed, in which case I'll rebase\n> > 1/2 and drop 2/2. You can also decide to rebase on top of this and make\n> > it a dependency.\n> \n> Hi, I'll rebase v3 on top of this patchset, this will save me some work:)\n\nIt will save me rebasing on top of your series, so it's good for me too\n:-)\n\nCould you review 2/2 ?\n\n> > Please also note that this change requires kernel v5.11 or newer for the\n> > rkisp1, or a backport of the rkisp1 driver to an older kernel.\n> > \n> > Laurent Pinchart (2):\n> >    include: linux: Update Linux headers to v5.12-rc1\n> >    ipa: rkisp1: Update to kernel header changes\n> > \n> >   include/linux/dma-heap.h         |   6 +-\n> >   include/linux/drm_fourcc.h       | 530 +++++++++++++++++-\n> >   include/linux/intel-ipu3.h       |  22 +-\n> >   include/linux/media-bus-format.h |  14 +-\n> >   include/linux/media.h            |   1 +\n> >   include/linux/rkisp1-config.h    |  92 +++-\n> >   include/linux/v4l2-controls.h    | 916 ++++++++++++++++++++++++-------\n> >   include/linux/v4l2-mediabus.h    |  15 +-\n> >   include/linux/v4l2-subdev.h      |  26 +-\n> >   include/linux/videodev2.h        |  99 +++-\n> >   src/ipa/rkisp1/rkisp1.cpp        |   2 +-\n> >   11 files changed, 1445 insertions(+), 278 deletions(-)\n> >","headers":{"Return-Path":"<libcamera-devel-bounces@lists.libcamera.org>","X-Original-To":"parsemail@patchwork.libcamera.org","Delivered-To":"parsemail@patchwork.libcamera.org","Received":["from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 639D9BD80C\n\tfor <parsemail@patchwork.libcamera.org>;\n\tWed,  3 Mar 2021 14:36:25 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 253C968A99;\n\tWed,  3 Mar 2021 15:36:25 +0100 (CET)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 8407468A7E\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tWed,  3 Mar 2021 15:36:24 +0100 (CET)","from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi\n\t[62.78.145.57])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 0D0B28CA;\n\tWed,  3 Mar 2021 15:36:23 +0100 (CET)"],"Authentication-Results":"lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"SD2nqdTR\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1614782184;\n\tbh=njlU3rQRCN2RIObcc2f3JQyCJZArWk35BWq557ecp9E=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=SD2nqdTRdAHfx2Vo6xUkzjuNWzauxaivSdQ8pA12F7Mcng9BMLdN3U7Cs9KSbQO8s\n\tNL+pO04+ZSeq5nk2/z9k45UL9Uewem+g0zbV2G0hZ1uQMgSwfaXDcnQDEPD9ldH/Jl\n\tfIhTENS/ufI5reaUq3jAx4F3Xo5mIPKJSPqyCY30=","Date":"Wed, 3 Mar 2021 16:35:55 +0200","From":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>","To":"Dafna Hirschfeld <dafna.hirschfeld@collabora.com>","Message-ID":"<YD+eywgZna70bJIT@pendragon.ideasonboard.com>","References":"<20210302201603.21566-1-laurent.pinchart@ideasonboard.com>\n\t<adc3a629-6b76-a62f-23d3-e69d5f6231b1@collabora.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<adc3a629-6b76-a62f-23d3-e69d5f6231b1@collabora.com>","Subject":"Re: [libcamera-devel] [PATCH 0/2] libcamera: Update kernel headers\n\tto v5.12-rc1","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","Cc":"libcamera-devel@lists.libcamera.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"}}]