From: Jacob Chen <jacobchen110@gmail.com>
To: Baruch Siach <baruch@tkos.co.il>
Cc: "open list:ARM/Rockchip SoC..."
<linux-rockchip@lists.infradead.org>,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
"Linux Media Mailing List" <linux-media@vger.kernel.org>,
"Sakari Ailus" <sakari.ailus@linux.intel.com>,
"Hans Verkuil" <hans.verkuil@cisco.com>,
"Tomasz Figa" <tfiga@chromium.org>,
"Shunqian Zheng" <zhengsq@rock-chips.com>,
"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>,
钟以崇 <zyc@rock-chips.com>, "Eddie Cai" <eddie.cai.linux@gmail.com>,
"Jeffy Chen" <jeffy.chen@rock-chips.com>,
devicetree@vger.kernel.org, "Heiko Stuebner" <heiko@sntech.de>
Subject: Re: [PATCH v6 00/17] Rockchip ISP1 Driver
Date: Fri, 9 Mar 2018 13:05:28 +0800 [thread overview]
Message-ID: <CAFLEztT4XJ9QnvLciv62dFCfj-y7Kynd2R8HpVxXeUXObvp3GQ@mail.gmail.com> (raw)
In-Reply-To: <20180309040903.hhkhylvs6q6lvqjy@tarshish>
Hi Baruch,
2018-03-09 12:09 GMT+08:00 Baruch Siach <baruch@tkos.co.il>:
> Hi Jacob,
>
> On Fri, Mar 09, 2018 at 08:53:57AM +0800, Jacob Chen wrote:
>> 2018-03-08 20:02 GMT+08:00 Baruch Siach <baruch@tkos.co.il>:
>> > On Thu, Mar 08, 2018 at 05:47:50PM +0800, Jacob Chen wrote:
>> >> This patch series add a ISP(Camera) v4l2 driver for rockchip rk3288/rk3399
>> >> SoC.
>> >>
>> >> Wiki Pages:
>> >> http://opensource.rock-chips.com/wiki_Rockchip-isp1
>> >>
>> >> The deprecated g_mbus_config op is not dropped in V6 because i am waiting
>> >> tomasz's patches.
>> >
>> > Which tree is this series based on? On top of v4.16-rc4 I get the build
>> > failure below. The V4L2_BUF_TYPE_META_OUTPUT macro, for example, is not even
>> > in media_tree.git.
>>
>> This series is based on v4.16-rc4 with below patch.
>> https://patchwork.kernel.org/patch/9792001/
>
> This patch does not apply on v4.16-rc4. I also tried v2 of this patch with the
> same result:
>
> https://patchwork.linuxtv.org/patch/44682/
It need resolve merge conflict.
>
> Can you push your series to a public git repo branch?
>
Sure, I have push it to my github.
https://github.com/wzyy2/linux/tree/4.16-rc4
This commit might be a bit of a mess
https://github.com/wzyy2/linux/commit/ff68323c4804adc10f64836ea1be172c54a9d6c6
> Thanks,
> baruch
>
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1321:3: error: ‘const struct v4l2_ioctl_ops’ has no member named ‘vidioc_enum_fmt_meta_out’; did you mean ‘vidioc_enum_fmt_meta_cap’?
>> > .vidioc_enum_fmt_meta_out = rkisp1_params_enum_fmt_meta_out,
>> > ^~~~~~~~~~~~~~~~~~~~~~~~
>> > vidioc_enum_fmt_meta_cap
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1321:30: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
>> > .vidioc_enum_fmt_meta_out = rkisp1_params_enum_fmt_meta_out,
>> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1321:30: note: (near initialization for ‘rkisp1_params_ioctl.vidioc_g_std’)
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1322:3: error: ‘const struct v4l2_ioctl_ops’ has no member named ‘vidioc_g_fmt_meta_out’; did you mean ‘vidioc_g_fmt_meta_cap’?
>> > .vidioc_g_fmt_meta_out = rkisp1_params_g_fmt_meta_out,
>> > ^~~~~~~~~~~~~~~~~~~~~
>> > vidioc_g_fmt_meta_cap
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1322:27: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
>> > .vidioc_g_fmt_meta_out = rkisp1_params_g_fmt_meta_out,
>> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1322:27: note: (near initialization for ‘rkisp1_params_ioctl.vidioc_s_std’)
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1323:3: error: ‘const struct v4l2_ioctl_ops’ has no member named ‘vidioc_s_fmt_meta_out’; did you mean ‘vidioc_s_fmt_meta_cap’?
>> > .vidioc_s_fmt_meta_out = rkisp1_params_g_fmt_meta_out,
>> > ^~~~~~~~~~~~~~~~~~~~~
>> > vidioc_s_fmt_meta_cap
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1323:27: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
>> > .vidioc_s_fmt_meta_out = rkisp1_params_g_fmt_meta_out,
>> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1323:27: note: (near initialization for ‘rkisp1_params_ioctl.vidioc_querystd’)
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1324:3: error: ‘const struct v4l2_ioctl_ops’ has no member named ‘vidioc_try_fmt_meta_out’; did you mean ‘vidioc_try_fmt_meta_cap’?
>> > .vidioc_try_fmt_meta_out = rkisp1_params_g_fmt_meta_out,
>> > ^~~~~~~~~~~~~~~~~~~~~~~
>> > vidioc_try_fmt_meta_cap
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1324:29: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
>> > .vidioc_try_fmt_meta_out = rkisp1_params_g_fmt_meta_out,
>> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1324:29: note: (near initialization for ‘rkisp1_params_ioctl.vidioc_enum_input’)
>> > drivers/media/platform/rockchip/isp1/isp_params.c: In function ‘rkisp1_params_init_vb2_queue’:
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1462:12: error: ‘V4L2_BUF_TYPE_META_OUTPUT’ undeclared (first use in this function); did you mean ‘V4L2_BUF_TYPE_SDR_OUTPUT’?
>> > q->type = V4L2_BUF_TYPE_META_OUTPUT;
>> > ^~~~~~~~~~~~~~~~~~~~~~~~~
>> > V4L2_BUF_TYPE_SDR_OUTPUT
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1462:12: note: each undeclared identifier is reported only once for each function it appears in
>> > CC drivers/media/platform/rockchip/isp1/mipi_dphy_sy.o
>> > drivers/media/platform/rockchip/isp1/isp_params.c: In function ‘rkisp1_register_params_vdev’:
>> > drivers/media/platform/rockchip/isp1/isp_params.c:1507:43: error: ‘V4L2_CAP_META_OUTPUT’ undeclared (first use in this function); did you mean ‘V4L2_CAP_VBI_OUTPUT’?
>> > vdev->device_caps = V4L2_CAP_STREAMING | V4L2_CAP_META_OUTPUT;
>> > ^~~~~~~~~~~~~~~~~~~~
>> > V4L2_CAP_VBI_OUTPUT
>> >
>> > Thanks,
>> > baruch
>> >
>> >> Jacob Chen (12):
>> >> media: doc: add document for rkisp1 meta buffer format
>> >> media: rkisp1: add Rockchip MIPI Synopsys DPHY driver
>> >> media: rkisp1: add Rockchip ISP1 subdev driver
>> >> media: rkisp1: add ISP1 statistics driver
>> >> media: rkisp1: add ISP1 params driver
>> >> media: rkisp1: add capture device driver
>> >> media: rkisp1: add rockchip isp1 core driver
>> >> dt-bindings: Document the Rockchip ISP1 bindings
>> >> dt-bindings: Document the Rockchip MIPI RX D-PHY bindings
>> >> ARM: dts: rockchip: add isp node for rk3288
>> >> ARM: dts: rockchip: add rx0 mipi-phy for rk3288
>> >> MAINTAINERS: add entry for Rockchip ISP1 driver
>> >>
>> >> Jeffy Chen (1):
>> >> media: rkisp1: Add user space ABI definitions
>> >>
>> >> Shunqian Zheng (3):
>> >> media: videodev2.h, v4l2-ioctl: add rkisp1 meta buffer format
>> >> arm64: dts: rockchip: add isp0 node for rk3399
>> >> arm64: dts: rockchip: add rx0 mipi-phy for rk3399
>> >>
>> >> Wen Nuan (1):
>> >> ARM: dts: rockchip: Add dts mipi-dphy TXRX1 node for rk3288
>> >>
>> >> .../devicetree/bindings/media/rockchip-isp1.txt | 69 +
>> >> .../bindings/media/rockchip-mipi-dphy.txt | 90 +
>> >> Documentation/media/uapi/v4l/meta-formats.rst | 2 +
>> >> .../media/uapi/v4l/pixfmt-meta-rkisp1-params.rst | 20 +
>> >> .../media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst | 18 +
>> >> MAINTAINERS | 10 +
>> >> arch/arm/boot/dts/rk3288.dtsi | 33 +
>> >> arch/arm64/boot/dts/rockchip/rk3399.dtsi | 25 +
>> >> drivers/media/platform/Kconfig | 10 +
>> >> drivers/media/platform/Makefile | 1 +
>> >> drivers/media/platform/rockchip/isp1/Makefile | 8 +
>> >> drivers/media/platform/rockchip/isp1/capture.c | 1751 ++++++++++++++++++++
>> >> drivers/media/platform/rockchip/isp1/capture.h | 167 ++
>> >> drivers/media/platform/rockchip/isp1/common.h | 110 ++
>> >> drivers/media/platform/rockchip/isp1/dev.c | 626 +++++++
>> >> drivers/media/platform/rockchip/isp1/dev.h | 93 ++
>> >> drivers/media/platform/rockchip/isp1/isp_params.c | 1539 +++++++++++++++++
>> >> drivers/media/platform/rockchip/isp1/isp_params.h | 49 +
>> >> drivers/media/platform/rockchip/isp1/isp_stats.c | 508 ++++++
>> >> drivers/media/platform/rockchip/isp1/isp_stats.h | 58 +
>> >> .../media/platform/rockchip/isp1/mipi_dphy_sy.c | 868 ++++++++++
>> >> .../media/platform/rockchip/isp1/mipi_dphy_sy.h | 15 +
>> >> drivers/media/platform/rockchip/isp1/regs.c | 239 +++
>> >> drivers/media/platform/rockchip/isp1/regs.h | 1550 +++++++++++++++++
>> >> drivers/media/platform/rockchip/isp1/rkisp1.c | 1177 +++++++++++++
>> >> drivers/media/platform/rockchip/isp1/rkisp1.h | 105 ++
>> >> drivers/media/v4l2-core/v4l2-ioctl.c | 2 +
>> >> include/uapi/linux/rkisp1-config.h | 798 +++++++++
>> >> include/uapi/linux/videodev2.h | 4 +
>> >> 29 files changed, 9945 insertions(+)
>> >> create mode 100644 Documentation/devicetree/bindings/media/rockchip-isp1.txt
>> >> create mode 100644 Documentation/devicetree/bindings/media/rockchip-mipi-dphy.txt
>> >> create mode 100644 Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-params.rst
>> >> create mode 100644 Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/Makefile
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/capture.c
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/capture.h
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/common.h
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/dev.c
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/dev.h
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/isp_params.c
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/isp_params.h
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/isp_stats.c
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/isp_stats.h
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/mipi_dphy_sy.c
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/mipi_dphy_sy.h
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/regs.c
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/regs.h
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/rkisp1.c
>> >> create mode 100644 drivers/media/platform/rockchip/isp1/rkisp1.h
>> >> create mode 100644 include/uapi/linux/rkisp1-config.h
>
> --
> http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
> =}------------------------------------------------ooO--U--Ooo------------{=
> - baruch@tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
next prev parent reply other threads:[~2018-03-09 5:05 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-08 9:47 [PATCH v6 00/17] Rockchip ISP1 Driver Jacob Chen
2018-03-08 9:47 ` [PATCH v6 01/17] media: videodev2.h, v4l2-ioctl: add rkisp1 meta buffer format Jacob Chen
2019-04-26 20:34 ` Nicolas Dufresne
2018-03-08 9:47 ` [PATCH v6 02/17] media: doc: add document for " Jacob Chen
2018-03-09 13:37 ` Hans Verkuil
2018-03-08 9:47 ` [PATCH v6 03/17] media: rkisp1: Add user space ABI definitions Jacob Chen
2018-03-09 13:34 ` Hans Verkuil
2018-04-24 4:28 ` Tomasz Figa
2018-03-08 9:47 ` [PATCH v6 06/17] media: rkisp1: add ISP1 statistics driver Jacob Chen
2018-03-09 13:59 ` Hans Verkuil
2018-03-08 9:47 ` [PATCH v6 07/17] media: rkisp1: add ISP1 params driver Jacob Chen
2018-03-09 14:03 ` Hans Verkuil
2018-03-08 9:47 ` [PATCH v6 09/17] media: rkisp1: add rockchip isp1 core driver Jacob Chen
2018-03-11 6:58 ` Baruch Siach
2018-05-16 8:08 ` Tomasz Figa
2018-03-08 9:48 ` [PATCH v6 10/17] dt-bindings: Document the Rockchip ISP1 bindings Jacob Chen
[not found] ` <20180308094807.9443-1-jacob-chen-fyOeoxGR3m/QT0dZR+AlfA@public.gmane.org>
2018-03-08 9:47 ` [PATCH v6 04/17] media: rkisp1: add Rockchip MIPI Synopsys DPHY driver Jacob Chen
2018-05-16 5:20 ` Laurent Pinchart
2018-05-16 14:39 ` Jacob Chen
2018-05-16 14:53 ` Jacob Chen
2018-05-16 15:15 ` Tomasz Figa
2019-03-10 17:49 ` Laurent Pinchart
2019-03-11 9:37 ` Tomasz Figa
2018-03-08 9:47 ` [PATCH v6 05/17] media: rkisp1: add Rockchip ISP1 subdev driver Jacob Chen
2018-03-09 13:57 ` Hans Verkuil
2018-05-03 9:09 ` Baruch Siach
2018-05-07 6:13 ` Tomasz Figa
2018-05-07 6:38 ` Baruch Siach
2018-05-07 6:41 ` Tomasz Figa
2018-05-24 11:30 ` Baruch Siach
2018-05-25 4:35 ` Tomasz Figa
2018-03-08 9:47 ` [PATCH v6 08/17] media: rkisp1: add capture device driver Jacob Chen
2018-03-09 14:07 ` Hans Verkuil
2018-04-17 8:44 ` Tomasz Figa
2018-03-08 9:48 ` [PATCH v6 11/17] dt-bindings: Document the Rockchip MIPI RX D-PHY bindings Jacob Chen
2018-03-08 9:48 ` [PATCH v6 16/17] arm64: dts: rockchip: add rx0 mipi-phy for rk3399 Jacob Chen
2018-03-08 9:48 ` [PATCH v6 17/17] MAINTAINERS: add entry for Rockchip ISP1 driver Jacob Chen
2018-03-08 10:29 ` [PATCH v6 00/17] Rockchip ISP1 Driver Tomasz Figa
2018-03-08 9:48 ` [PATCH v6 12/17] ARM: dts: rockchip: add isp node for rk3288 Jacob Chen
2018-03-08 9:48 ` [PATCH v6 13/17] ARM: dts: rockchip: add rx0 mipi-phy " Jacob Chen
2018-03-08 9:48 ` [PATCH v6 14/17] ARM: dts: rockchip: Add dts mipi-dphy TXRX1 node " Jacob Chen
2018-03-08 9:48 ` [PATCH v6 15/17] arm64: dts: rockchip: add isp0 node for rk3399 Jacob Chen
2018-03-08 12:02 ` [PATCH v6 00/17] Rockchip ISP1 Driver Baruch Siach
2018-03-09 0:53 ` Jacob Chen
2018-03-09 4:09 ` Baruch Siach
2018-03-09 5:05 ` Jacob Chen [this message]
2018-03-09 5:54 ` Baruch Siach
2018-03-09 14:12 ` Hans Verkuil
2018-09-03 13:07 ` Hans Verkuil
2018-09-03 13:50 ` Tomasz Figa
2018-09-03 14:15 ` Heiko Stuebner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAFLEztT4XJ9QnvLciv62dFCfj-y7Kynd2R8HpVxXeUXObvp3GQ@mail.gmail.com \
--to=jacobchen110@gmail.com \
--cc=baruch@tkos.co.il \
--cc=devicetree@vger.kernel.org \
--cc=eddie.cai.linux@gmail.com \
--cc=hans.verkuil@cisco.com \
--cc=heiko@sntech.de \
--cc=jeffy.chen@rock-chips.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=mchehab@kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=tfiga@chromium.org \
--cc=zhengsq@rock-chips.com \
--cc=zyc@rock-chips.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).