From: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
To: sakari.ailus@iki.fi
Cc: linux-media@vger.kernel.org
Subject: Re: [GIT PULL v4 for 4.21] META_OUTPUT buffer type and the ipu3 staging driver
Date: Fri, 14 Dec 2018 09:18:50 -0200 [thread overview]
Message-ID: <20181214091850.0ab0cd22@coco.lan> (raw)
In-Reply-To: <20181213101905.6ad7c481@coco.lan>
Hi Sakari,
Em Thu, 13 Dec 2018 10:19:05 -0200
Mauro Carvalho Chehab <mchehab+samsung@kernel.org> escreveu:
> Em Thu, 13 Dec 2018 14:03:40 +0200
> sakari.ailus@iki.fi escreveu:
>
> > Hi Mauro,
> >
> > Here's the ipu3 staging driver plus the META_OUTPUT buffer type needed to
> > pass the parameters for the device. If you think this there's still time to
> > get this to 4.21, then please pull. The non-staging patches have been
> > around for more than half a year and they're relatively simple.
> >
> > Note: DO NOT WORRY about the documentation build warnings, they'll be
> > adderessed by commit fdf8298f7ff167e4e7522465a3c6e6b908cdb2af from the
> > documentation tree (already in linux-next).
>
> I'm assuming you're talking about this patch:
>
> 3d9bfb19bd70 ("scripts/kernel-doc: Fix struct and struct field attribute processing")
>
> Ok, I'll remind about that when pulling from it.
Pulled, thanks.
After reviewing the patchset (considering that it is for staging), I
opted to merge it on a separate topic branch. On one ot the builds,
I got this:
WARNING: kernel-doc './scripts/kernel-doc -rst -enable-lineno ./include/uapi/linux/intel-ipu3.h' failed with return code 1
Maybe it is related to the troubles with parsing __attribute, so
I'll wait until upstream pulls from docs-next, in order to run
a test and be sure that it won't break documentation build nor
produce the warnings.
Also, for 4.21, all documentation files now have either a SPDX tag
or a dual-license text (for stuff under Documentation/media/uapi).
The IPU3 doc files added by this series doesn't have it - and one
of the new doc files still have a encoding line like:
.. -*- coding: utf-8; mode: rst -*-
Please remove it, and be sure that all new doc files under
Documentation/media/v4l-drivers will have a SPDX tag compatible
with GPL 2.0.
Ah, I made a notice about some gotos to some code that it is
inside a for() loop. That's very ugly and weird (and not sure if this
is correct according with C spec). Please fix that too.
Thanks!
Mauro
>
> >
> > Since the v1 pull request, this contains the content of the v9
> > patchset (since v8):
> >
> > - Addressed most of Laurent's comments on the driver documentation. Some
> > have been postponed and added to TODO.
> >
> > - Added a MAINTAINERS entry.
> >
> > - Removed uAPI definitions (formats etc.) added by the patches originally
> > not intended to be merged (documentation outside the staging tree).
> >
> > - Added a patch to fix a few compiler warnings (false positives) plus
> > fixed the firmware location.
> >
> > - checkpatch.pl warnings remain; those need to be fixed as well.
> >
> > since v2 pull request:
> >
> > - Use correct tag.
> >
> > since v3 pull request:
> >
> > - Remove extra Reviewed-by: tags.
> >
> > Please pull.
> >
> >
> > The following changes since commit e159b6074c82fe31b79aad672e02fa204dbbc6d8:
> >
> > media: vimc: fix start stream when link is disabled (2018-12-07 13:08:41 -0500)
> >
> > are available in the git repository at:
> >
> > ssh://linuxtv.org/git/sailus/media_tree.git tags/ipu3-v8-4.20-3-sign
> >
> > for you to fetch changes up to 48acf4640e698334bfd9dc41a94b17b568a97b31:
> >
> > staging/ipu3-imgu: Add MAINTAINERS entry (2018-12-13 13:04:00 +0200)
> >
> > ----------------------------------------------------------------
> > imgu staging driver v9
> >
> > ----------------------------------------------------------------
> > Cao,Bing Bu (1):
> > media: staging/intel-ipu3: Add dual pipe support
> >
> > Rajmohan Mani (1):
> > doc-rst: Add Intel IPU3 documentation
> >
> > Sakari Ailus (6):
> > v4l: Add support for V4L2_BUF_TYPE_META_OUTPUT
> > docs-rst: v4l: Document V4L2_BUF_TYPE_META_OUTPUT interface
> > ipu3-imgu: Fix compiler warnings
> > ipu3-imgu: Fix firmware binary location
> > staging/ipu3-imgu: Address documentation comments
> > staging/ipu3-imgu: Add MAINTAINERS entry
> >
> > Tomasz Figa (2):
> > media: staging/intel-ipu3: mmu: Implement driver
> > media: staging/intel-ipu3: Implement DMA mapping functions
> >
> > Yong Zhi (12):
> > media: staging/intel-ipu3: abi: Add register definitions and enum
> > media: staging/intel-ipu3: abi: Add structs
> > media: staging/intel-ipu3: css: Add dma buff pool utility functions
> > media: staging/intel-ipu3: css: Add support for firmware management
> > media: staging/intel-ipu3: css: Add static settings for image pipeline
> > media: staging/intel-ipu3: css: Compute and program ccs
> > media: staging/intel-ipu3: css: Initialize css hardware
> > media: staging/intel-ipu3: Add css pipeline programming
> > media: staging/intel-ipu3: Add v4l2 driver based on media framework
> > media: staging/intel-ipu3: Add imgu top level pci device driver
> > media: staging/intel-ipu3: Add Intel IPU3 meta data uAPI
> > media: v4l: Add Intel IPU3 meta buffer formats
> >
> > Documentation/media/uapi/v4l/buffer.rst | 3 +
> > Documentation/media/uapi/v4l/dev-meta.rst | 33 +-
> > Documentation/media/uapi/v4l/meta-formats.rst | 1 +
> > .../media/uapi/v4l/pixfmt-meta-intel-ipu3.rst | 178 +
> > Documentation/media/uapi/v4l/vidioc-querycap.rst | 3 +
> > Documentation/media/v4l-drivers/index.rst | 1 +
> > Documentation/media/v4l-drivers/ipu3.rst | 369 +
> > Documentation/media/videodev2.h.rst.exceptions | 2 +
> > MAINTAINERS | 8 +
> > drivers/media/common/videobuf2/videobuf2-v4l2.c | 1 +
> > drivers/media/v4l2-core/v4l2-compat-ioctl32.c | 2 +
> > drivers/media/v4l2-core/v4l2-dev.c | 12 +-
> > drivers/media/v4l2-core/v4l2-ioctl.c | 23 +
> > drivers/staging/media/Kconfig | 2 +
> > drivers/staging/media/Makefile | 1 +
> > drivers/staging/media/ipu3/Kconfig | 14 +
> > drivers/staging/media/ipu3/Makefile | 11 +
> > drivers/staging/media/ipu3/TODO | 34 +
> > drivers/staging/media/ipu3/include/intel-ipu3.h | 2785 ++++++
> > drivers/staging/media/ipu3/ipu3-abi.h | 2011 ++++
> > drivers/staging/media/ipu3/ipu3-css-fw.c | 265 +
> > drivers/staging/media/ipu3/ipu3-css-fw.h | 188 +
> > drivers/staging/media/ipu3/ipu3-css-params.c | 2943 ++++++
> > drivers/staging/media/ipu3/ipu3-css-params.h | 28 +
> > drivers/staging/media/ipu3/ipu3-css-pool.c | 100 +
> > drivers/staging/media/ipu3/ipu3-css-pool.h | 55 +
> > drivers/staging/media/ipu3/ipu3-css.c | 2391 +++++
> > drivers/staging/media/ipu3/ipu3-css.h | 213 +
> > drivers/staging/media/ipu3/ipu3-dmamap.c | 270 +
> > drivers/staging/media/ipu3/ipu3-dmamap.h | 22 +
> > drivers/staging/media/ipu3/ipu3-mmu.c | 561 ++
> > drivers/staging/media/ipu3/ipu3-mmu.h | 35 +
> > drivers/staging/media/ipu3/ipu3-tables.c | 9609 ++++++++++++++++++++
> > drivers/staging/media/ipu3/ipu3-tables.h | 66 +
> > drivers/staging/media/ipu3/ipu3-v4l2.c | 1419 +++
> > drivers/staging/media/ipu3/ipu3.c | 830 ++
> > drivers/staging/media/ipu3/ipu3.h | 168 +
> > include/media/v4l2-ioctl.h | 17 +
> > include/uapi/linux/videodev2.h | 2 +
> > 39 files changed, 24659 insertions(+), 17 deletions(-)
> > create mode 100644 Documentation/media/uapi/v4l/pixfmt-meta-intel-ipu3.rst
> > create mode 100644 Documentation/media/v4l-drivers/ipu3.rst
> > create mode 100644 drivers/staging/media/ipu3/Kconfig
> > create mode 100644 drivers/staging/media/ipu3/Makefile
> > create mode 100644 drivers/staging/media/ipu3/TODO
> > create mode 100644 drivers/staging/media/ipu3/include/intel-ipu3.h
> > create mode 100644 drivers/staging/media/ipu3/ipu3-abi.h
> > create mode 100644 drivers/staging/media/ipu3/ipu3-css-fw.c
> > create mode 100644 drivers/staging/media/ipu3/ipu3-css-fw.h
> > create mode 100644 drivers/staging/media/ipu3/ipu3-css-params.c
> > create mode 100644 drivers/staging/media/ipu3/ipu3-css-params.h
> > create mode 100644 drivers/staging/media/ipu3/ipu3-css-pool.c
> > create mode 100644 drivers/staging/media/ipu3/ipu3-css-pool.h
> > create mode 100644 drivers/staging/media/ipu3/ipu3-css.c
> > create mode 100644 drivers/staging/media/ipu3/ipu3-css.h
> > create mode 100644 drivers/staging/media/ipu3/ipu3-dmamap.c
> > create mode 100644 drivers/staging/media/ipu3/ipu3-dmamap.h
> > create mode 100644 drivers/staging/media/ipu3/ipu3-mmu.c
> > create mode 100644 drivers/staging/media/ipu3/ipu3-mmu.h
> > create mode 100644 drivers/staging/media/ipu3/ipu3-tables.c
> > create mode 100644 drivers/staging/media/ipu3/ipu3-tables.h
> > create mode 100644 drivers/staging/media/ipu3/ipu3-v4l2.c
> > create mode 100644 drivers/staging/media/ipu3/ipu3.c
> > create mode 100644 drivers/staging/media/ipu3/ipu3.h
> >
>
>
>
> Thanks,
> Mauro
Thanks,
Mauro
next prev parent reply other threads:[~2018-12-14 11:18 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-13 12:03 [GIT PULL v4 for 4.21] META_OUTPUT buffer type and the ipu3 staging driver sakari.ailus
2018-12-13 12:19 ` Mauro Carvalho Chehab
2018-12-14 11:18 ` Mauro Carvalho Chehab [this message]
2019-01-07 18:01 ` Mauro Carvalho Chehab
2019-01-07 18:11 ` Mauro Carvalho Chehab
2019-01-07 22:51 ` Sakari Ailus
2019-01-08 12:27 ` Mauro Carvalho Chehab
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=20181214091850.0ab0cd22@coco.lan \
--to=mchehab+samsung@kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=sakari.ailus@iki.fi \
/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).