Re: [PATCH 16/22] xen-blkfront: Make use of the new sg_map helper function

On 18/04/17 08:27 AM, Konrad Rzeszutek Wilk wrote: > Interesting that you didn't CC any of the maintainers. Could you > do that in the future please? Please read the cover letter. The distribution list for the patchset would have been way too large to cc every maintainer (even as limited as it

Re: [PATCH 16/22] xen-blkfront: Make use of the new sg_map helper function

On 18/04/17 09:50 AM, Konrad Rzeszutek Wilk wrote: > I am not sure if you know, but you can add on each patch the respective > maintainer via 'CC'. That way you can have certain maintainers CCed only > on the subsystems they cover. You put it after (or before) your SoB and > git send-email

Re: [PATCH 05/22] drm/i915: Make use of the new sg_map helper function

On 18/04/17 12:44 AM, Daniel Vetter wrote: > On Thu, Apr 13, 2017 at 04:05:18PM -0600, Logan Gunthorpe wrote: >> This is a single straightforward conversion from kmap to sg_map. >> >> Signed-off-by: Logan Gunthorpe > > Acked-by: Daniel Vetter > >

Re: [PATCH 16/22] xen-blkfront: Make use of the new sg_map helper function

On Tue, Apr 18, 2017 at 09:42:20AM -0600, Logan Gunthorpe wrote: > > > On 18/04/17 08:27 AM, Konrad Rzeszutek Wilk wrote: > > Interesting that you didn't CC any of the maintainers. Could you > > do that in the future please? > > Please read the cover letter. The distribution list for the

[PATCH] rc-core: use the full 32 bits for NEC scancodes

Using the full 32 bits for all kinds of NEC scancodes simplifies rc-core and the nec decoder without any loss of functionality. At the same time it ensures that scancodes for NEC16/NEC24/NEC32 do not overlap and removes lots of duplication (as you can see from the patch, the same NEC

Re: [PATCH 05/22] drm/i915: Make use of the new sg_map helper function

On Thu, Apr 13, 2017 at 04:05:18PM -0600, Logan Gunthorpe wrote: > This is a single straightforward conversion from kmap to sg_map. > > Signed-off-by: Logan Gunthorpe Acked-by: Daniel Vetter Probably makes sense to merge through some other tree,

Re: em28xx i2c writing error

On Mon, Apr 17, 2017 at 3:55 AM, Mauro Carvalho Chehab wrote: > Em Sat, 15 Apr 2017 20:28:20 +0200 > Anders Eriksson escreveu: > >> Hi Mauro, >> >> I've two devices using this driver, and whenever I have them both in >> use I eventually (between

Re: [PATCHv3 13/22] staging: android: ion: Use CMA APIs directly

On Mon, Apr 03, 2017 at 11:57:55AM -0700, Laura Abbott wrote: > When CMA was first introduced, its primary use was for DMA allocation > and the only way to get CMA memory was to call dma_alloc_coherent. This > put Ion in an awkward position since there was no device structure > readily available

RE: [PATCH 16/22] xen-blkfront: Make use of the new sg_map helper function

From: Logan Gunthorpe > Sent: 13 April 2017 23:05 > Straightforward conversion to the new helper, except due to > the lack of error path, we have to warn if unmapable memory > is ever present in the sgl. > > Signed-off-by: Logan Gunthorpe > --- >

[PATCH v3] [media] uvcvideo: Add iFunction or iInterface to device names.

Permits distinguishing between two /dev/videoX entries from the same physical UVC device (that naturally share the same iProduct name). This change matches current Windows behavior by prioritizing iFunction over iInterface, but unlike Windows it displays both iProduct and iFunction/iInterface

Re: [PATCH 6/6] drm: mali-dp: Add writeback connector

On Fri, Apr 14, 2017 at 11:47:00AM +0200, Boris Brezillon wrote: On Fri, 25 Nov 2016 16:49:04 + Brian Starkey wrote: +static int +malidp_mw_encoder_atomic_check(struct drm_encoder *encoder, + struct drm_crtc_state *crtc_state, +

Re: [PATCH 2/6] drm: writeback: Add out-fences for writeback connectors

On Fri, Apr 14, 2017 at 12:11:14PM +0200, Boris Brezillon wrote: On Fri, 25 Nov 2016 16:49:00 + Brian Starkey wrote: Add the OUT_FENCE_PTR property to writeback connectors, to enable userspace to get a fence which will signal once the writeback is complete. It is

RE: [PATCH v3 7/7] media: platform: rcar_drif: Add DRIF support

Hi Laurent, Many thanks for your time & the review comments. I have agreed to most of the comments and a few need further discussion. Could you please take a look at those? > On Tuesday 07 Feb 2017 15:02:37 Ramesh Shanmugasundaram wrote: > > This patch adds Digital Radio Interface (DRIF)

RE: [PATCH v3 5/7] doc_rst: media: New SDR formats PC16, PC18 & PC20

Hi Laurent, Thanks for the review comments. > On Tuesday 07 Feb 2017 15:02:35 Ramesh Shanmugasundaram wrote: > > This patch adds documentation for the three new SDR formats > > > > V4L2_SDR_FMT_PCU16BE > > V4L2_SDR_FMT_PCU18BE > > V4L2_SDR_FMT_PCU20BE > > > > Signed-off-by: Ramesh

Re: [RFC PATCH v3 0/6] Introduce writeback connectors

Hi Boris, On Fri, Apr 14, 2017 at 11:35:17AM +0200, Boris Brezillon wrote: Hi Brian, On Fri, 25 Nov 2016 16:48:58 + Brian Starkey wrote: Hi, This is v3 of my series introducing a new connector type: DRM_MODE_CONNECTOR_WRITEBACK See v1 and v2 here: [1] [2]

Re: [PATCH 1/6] drm: Add writeback connector type

Hi Boris, On Fri, Apr 14, 2017 at 12:08:23PM +0200, Boris Brezillon wrote: On Fri, 25 Nov 2016 16:48:59 + Brian Starkey wrote: diff --git a/drivers/gpu/drm/drm_connector.c b/drivers/gpu/drm/drm_connector.c index b5c6a8e..6bbd93f 100644 ---

[PATCHv4 00/12] Ion cleanup in preparation for moving out of staging

Hi, This is v4 of the series to cleanup to Ion. Greg took some of the patches that weren't CMA related already. There was a minor bisectability problem with the CMA APIs so this is a new version to address that. I also addressed some minor comments on the patch to collapse header files. Thanks,

[PATCHv4 11/12] staging: android: ion: Set query return value

This never got set in the ioctl. Properly set a return value of 0 on success. Signed-off-by: Laura Abbott --- drivers/staging/android/ion/ion.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c index

[PATCHv4 08/12] staging: android: ion: Rework heap registration/enumeration

The current model of Ion heap registration is based on the outdated model of board files. The replacement for board files (devicetree) isn't a good replacement for what Ion wants to do. In actuality, Ion wants to show what memory is available in the system for something else to figure out what to

[PATCHv4 07/12] staging: android: ion: Collapse internal header files

Ion current has ion_priv.h and ion.h as header files. ion.h was intended to be used for public APIs but Ion never ended up really having anything public. Combine the two headers so there is only one internal header. Signed-off-by: Laura Abbott --- v4: minor cleanup suggested

[PATCHv4 09/12] staging: android: ion: Drop ion_map_kernel interface

Nobody uses this interface externally. Drop it. Signed-off-by: Laura Abbott --- drivers/staging/android/ion/ion.c | 59 --- 1 file changed, 59 deletions(-) diff --git a/drivers/staging/android/ion/ion.c

[PATCHv4 12/12] staging/android: Update Ion TODO list

Most of the items have been taken care of by a clean up series. Remove the completed items and add a few new ones. Signed-off-by: Laura Abbott --- drivers/staging/android/TODO | 21 - 1 file changed, 4 insertions(+), 17 deletions(-) diff --git

[PATCHv4 10/12] staging: android: ion: Remove ion_handle and ion_client

ion_handle was introduced as an abstraction to represent a reference to a buffer via an ion_client. As frameworks outside of Ion evolved, the dmabuf emerged as the preferred standard for use in the kernel. This has made the ion_handle an unnecessary abstraction and prone to race conditions.

[PATCHv4 02/12] cma: Introduce cma_for_each_area

Frameworks (e.g. Ion) may want to iterate over each possible CMA area to allow for enumeration. Introduce a function to allow a callback. Signed-off-by: Laura Abbott --- include/linux/cma.h | 2 ++ mm/cma.c| 14 ++ 2 files changed, 16 insertions(+)

[PATCHv4 01/12] cma: Store a name in the cma structure

Frameworks that may want to enumerate CMA heaps (e.g. Ion) will find it useful to have an explicit name attached to each region. Store the name in each CMA structure. Signed-off-by: Laura Abbott --- arch/powerpc/kvm/book3s_hv_builtin.c | 3 ++- drivers/base/dma-contiguous.c

[PATCHv4 05/12] staging: android: ion: Break the ABI in the name of forward progress

Several of the Ion ioctls were designed in such a way that they necessitate compat ioctls. We're breaking a bunch of other ABIs and cleaning stuff up anyway so let's follow the ioctl guidelines and clean things up while everyone is busy converting things over anyway. As part of this, also remove

[PATCHv4 06/12] staging: android: ion: Get rid of ion_phys_addr_t

Once upon a time, phys_addr_t was not everywhere in the kernel. These days it is used enough places that having a separate Ion type doesn't make sense. Remove the extra type and just use phys_addr_t directly. Signed-off-by: Laura Abbott --- drivers/staging/android/ion/ion.h

[PATCHv4 04/12] staging: android: ion: Stop butchering the DMA address

Now that we have proper caching, stop setting the DMA address manually. It should be set after properly calling dma_map. Signed-off-by: Laura Abbott --- drivers/staging/android/ion/ion.c | 17 + 1 file changed, 1 insertion(+), 16 deletions(-) diff --git

[PATCHv4 03/12] staging: android: ion: Use CMA APIs directly

When CMA was first introduced, its primary use was for DMA allocation and the only way to get CMA memory was to call dma_alloc_coherent. This put Ion in an awkward position since there was no device structure readily available and setting one up messed up the coherency model. These days, CMA can

Re: Looking for device driver advice

Hi Niklas, On Sun, Apr 16, 2017 at 07:42:20PM +0200, Niklas Söderlund wrote: > Hi, > > On 2017-04-16 13:51:21 +0300, Sakari Ailus wrote: > > Hi Hans and Patrick, > > > > On Wed, Apr 12, 2017 at 01:37:33PM +0200, Hans Verkuil wrote: > > > Hi Patrick, > > > > > > On 04/10/2017 10:13 PM, Patrick

Re: [PATCH v6 17/39] platform: add video-multiplexer subdevice driver

Hi Pavel, On Fri, 2017-04-14 at 22:32 +0200, Pavel Machek wrote: > Hi! > > > > The MUX framework is already in linux-next. Could you use that instead of > > > adding new driver + bindings that are not compliant with the MUX > > > framework? > > > I don't think it'd be much of a change in terms

[PATCH for v4.12 2/3] cec.h: merge cec-edid.h into cec.h

From: Hans Verkuil Drop the separate cec-edid.h header and merge it into cec.h. There was really no need to have a separate header for this. Signed-off-by: Hans Verkuil --- MAINTAINERS | 1 -

[PATCH for v4.12 1/3] cec: Kconfig cleanup

From: Hans Verkuil The Kconfig options for the CEC subsystem were a bit messy. In addition there were two cec sources (cec-edid.c and cec-notifier.c) that were outside of the media/cec directory, which was weird. Move those sources to media/cec as well. The cec-edid and

[PATCH for v4.12 0/3] cec: clean up Kconfig, headers

From: Hans Verkuil This patch series cleans up the various CEC config options. In particular it adds a CEC_CORE config option which is what CEC drivers should depend on, and it removes the MEDIA_CEC_EDID config option which was rather pointless. Finally it adds a new

[PATCH for v4.12 3/3] cec: add MEDIA_CEC_RC config option

From: Hans Verkuil Add an explicit config option to select whether the CEC remote control messages are to be passed on to the RC subsystem or not. Signed-off-by: Hans Verkuil --- drivers/media/cec/Kconfig| 8 +++-

Re: [PATCH 16/22] xen-blkfront: Make use of the new sg_map helper function

On Tue, Apr 18, 2017 at 02:13:59PM +, David Laight wrote: > From: Logan Gunthorpe > > Sent: 13 April 2017 23:05 > > Straightforward conversion to the new helper, except due to > > the lack of error path, we have to warn if unmapable memory > > is ever present in the sgl. Interesting that you

[linuxtv-media:master 1530/1538] warning: (VIDEO_EM28XX_V4L2) selects VIDEO_OV2640 which has unmet direct dependencies (MEDIA_SUPPORT && ..)

tree: git://linuxtv.org/media_tree.git master head: ee0fe833d96793853335844b6d99fb76bd12cbeb commit: bb42fc4ad442d4de78b4a16233db98a5396988ff [1530/1538] [media] em28xx: add missing auto-selections for build config: x86_64-randconfig-s0-04190251 (attached as .config) compiler: gcc-4.4 (Debian

Re: [PATCH 1/6] drm: Add writeback connector type

Hi Brian, On Tue, 18 Apr 2017 18:34:43 +0100 Brian Starkey wrote: > >> @@ -214,6 +214,19 @@ struct drm_connector_state { > >>struct drm_encoder *best_encoder; > >> > >>struct drm_atomic_state *state; > >> + > >> + /** > >> + * @writeback_job: Writeback job for

Re: et8ek8 camera on Nokia N900: trying to understand what is going on with modes

Hi Pavel, On Wed, Apr 12, 2017 at 11:11:59PM +0200, Pavel Machek wrote: > Hi! > > 5Mpix mode does not work on N900, which is something I'd like to > understand. et8ek8_mode contains huge tables of register settings and > parameter values, but it seems that they are not really independend. > >

Re: [PATCH v6 17/39] platform: add video-multiplexer subdevice driver

Hi! > That self-referencing mux-controls property looks a bit superfluous: > > mux: video-multiplexer { > mux-controls = <>; > }; > > Other than that, I'm completely fine with splitting the compatible into > something like video-mux-gpio and video-mux-mmio and reusing

Re: [PATCH for v4.12 1/3] cec: Kconfig cleanup

2017-04-18 10:45 GMT+02:00 Hans Verkuil : > From: Hans Verkuil > > The Kconfig options for the CEC subsystem were a bit messy. In > addition there were two cec sources (cec-edid.c and cec-notifier.c) > that were outside of the media/cec directory, which

Re: [PATCH 40/40] media: imx: set and propagate empty field, colorimetry params

On Thu, 2017-04-13 at 09:40 -0700, Steve Longerbeam wrote: [...] > >> @@ -804,12 +804,29 @@ static void prp_try_fmt(struct prp_priv *priv, > >> >format.height, > >> infmt->height / 4, MAX_H_SRC, > >>

Re: dvb-tools: dvbv5-scan segfaults with DVB-T2 HD service that just started in Germany

On Thu, Mar 30, 2017 at 17:13:34 -0300, Mauro Carvalho Chehab wrote: > Hi Gregor, > > Em Wed, 29 Mar 2017 20:45:06 +0200 > Gregor Jasny escreveu: > > > Hello Mauro & list, > > > > could you please have a look at the dvbv5-scan crash report below? > >

Re: [PATCH v3] [media] staging: css2400: fix checkpatch error

On Wed, Mar 29, 2017 at 10:50:08AM +0300, Haim Daniel wrote: > isp_capture_defs.h: clean up ERROR: Macros with complex values should be > enclosed in parentheses > > Signed-off-by: Haim Daniel > --- > .../pci/atomisp2/css2400/css_2401_csi2p_system/hrt/isp_capture_defs.h

cron job: media_tree daily build: ERRORS

This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Wed Apr 19 05:00:16 CEST 2017 media-tree git hash:ee0fe833d96793853335844b6d99fb76bd12cbeb media_build

[PATCH] rc-core: use the full 32 bits for NEC scancodes in wakefilters

The new sysfs wakefilter API will expose the difference between the NEC protocols to userspace for no good reason and once exposed, it will be much more difficult to change the logic. By only allowing full NEC32 scancodes to be set, any heuristics in the kernel can be avoided. This is the

[PATCH] dma-buf: Rename dma-ops to prevent conflict with kunmap_atomic macro

Seeing the kunmap_atomic dma_buf_op shares the same name with a macro in higmem.h, the former can be aliased if any dma-buf user includes that header. I'm personally trying to include highmem.h inside scatterlist.h and this breaks the dma-buf code proper. Christoph Hellwig suggested [1] renaming

Re: [Intel-gfx] [PATCH] dma-buf: Rename dma-ops to prevent conflict with kunmap_atomic macro

Hi Logan, [auto build test ERROR on linus/master] [also build test ERROR on v4.11-rc7 next-20170418] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Logan-Gunthorpe/dma-buf-Rename-dma-ops