Re: [PATCH] pinctrl: Convert to using %pOF instead of full_name

On Tue, Jul 18, 2017 at 04:43:23PM -0500, Rob Herring wrote: > Now that we have a custom printf format specifier, convert users of > full_name to use %pOF instead. This is preparation to remove storing > of the full path string for each node. > > Signed-off-by: Rob Herring > Cc:

Re: [RFC v2 4/6] serial: core: support deferring serdev controller registration

On Mon, Jul 17, 2017 at 10:24 AM, Ulrich Hecht wrote: > serdev controllers may depend on other devices (such as multiplexers) > and thus require deferred probing support. I wonder if instead of deferring, we could just delay binding the devices. There's really no

Re: [PATCH 1/5] watchdog: renesas_wdt: avoid (theoretical) type overflow

On 07/18/2017 12:21 AM, Wolfram Sang wrote: - unsigned long rate; - unsigned int clks_per_sec; + unsigned long rate, clks_per_sec; If you make this change, you should also update rwdt_priv.clks_per_sec (yes I know it's removed in a later patch in this series). Right. I will

Re: [RFC v2 0/6] serdev multiplexing support

On Mon, Jul 17, 2017 at 10:24 AM, Ulrich Hecht wrote: > Hi! > > This is a new attempt to add multiplexer support to serdev. It is now based > on the mux subsystem, making it more generic than the previous iteration > ("[RFC 0/4] serdev GPIO-based multiplexing

Re: [RFC v2 2/6] serdev: add method to set parity

On Mon, Jul 17, 2017 at 10:24 AM, Ulrich Hecht wrote: > Adds serdev_device_set_parity() and an implementation for ttyport. > > Signed-off-by: Ulrich Hecht > --- > drivers/tty/serdev/core.c | 12 >

Re: [RFC v2 3/6] serdev: add multiplexer support

On Mon, Jul 17, 2017 at 10:24 AM, Ulrich Hecht wrote: > Adds an interface for slave device multiplexing using the mux subsystem. > > Signed-off-by: Ulrich Hecht > --- > drivers/tty/serdev/Kconfig | 3 +++ >

[PATCH] clk: Convert to using %pOF instead of full_name

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node. Signed-off-by: Rob Herring Cc: Michael Turquette Cc: Stephen Boyd

[PATCH] media: Convert to using %pOF instead of full_name

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node. Signed-off-by: Rob Herring Cc: Kyungmin Park Cc: Andrzej Hajda

[PATCH] pinctrl: Convert to using %pOF instead of full_name

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node. Signed-off-by: Rob Herring Cc: Linus Walleij Cc: Lee Jones

[PATCH] soc: Convert to using %pOF instead of full_name

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node. Signed-off-by: Rob Herring Cc: Scott Wood Cc: Qiang Zhao Cc:

[PATCH] ARM: Convert to using %pOF instead of full_name

Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node. Signed-off-by: Rob Herring Cc: Russell King Cc: Kukjin Kim

Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

Hi Hans, On 2017-07-18 17:06:15 +0200, Hans Verkuil wrote: > On 18/07/17 16:47, Niklas Söderlund wrote: > >>> void v4l2_async_notifier_unregister(struct v4l2_async_notifier *notifier) > >>> { > >>> - struct v4l2_subdev *sd, *tmp; > >>> + struct v4l2_subdev *sd, *tmp, **subdev; > >>> unsigned

[RFC PATCH 0/2] clk: renesas: rcar-gen3-cpg: refactor accessing the SD div table

Here is a patch series trying to implement the conclusions about accessing the SD divider table, coming from this discussion [1]. Patch 1 is a tiny, yet seperate cleanup. Patch 2 is the bulk of the work doing the checks only when needed. I think the code looks much better now. I tested this on a

[RFC PATCH 1/2] clk: renesas: rcar-gen3-cpg: drop superfluous variable

'rate' is not used, so we can use 'parent_rate' directly. Signed-off-by: Wolfram Sang --- drivers/clk/renesas/rcar-gen3-cpg.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/clk/renesas/rcar-gen3-cpg.c

[RFC PATCH 2/2] clk: renesas: rcar-gen3-cpg.c: refactor checks for accessing the div table

Do the checks for accessing the SD divider table only when the rate gets updated, namely on init and set_rate. In all other cases, reuse the last value. This simplifies code, runtime load, and error reporting. Signed-off-by: Wolfram Sang ---

Re: [PATCH v4 2/3] v4l: async: do not hold list_lock when reprobing devices

On 2017-07-18 16:50:15 +0200, Hans Verkuil wrote: > On 18/07/17 16:39, Niklas Söderlund wrote: > > Hi Hans, > > > > Thanks for your feedback. > > > > On 2017-07-18 16:22:14 +0200, Hans Verkuil wrote: > >> On 17/07/17 18:59, Niklas Söderlund wrote: > >>> There is no good reason to hold the

Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

On 18/07/17 16:47, Niklas Söderlund wrote: >>> void v4l2_async_notifier_unregister(struct v4l2_async_notifier *notifier) >>> { >>> - struct v4l2_subdev *sd, *tmp; >>> + struct v4l2_subdev *sd, *tmp, **subdev; >>> unsigned int notif_n_subdev = notifier->num_subdevs; >>> unsigned int

Re: [PATCH v4 2/3] v4l: async: do not hold list_lock when reprobing devices

On 18/07/17 16:39, Niklas Söderlund wrote: > Hi Hans, > > Thanks for your feedback. > > On 2017-07-18 16:22:14 +0200, Hans Verkuil wrote: >> On 17/07/17 18:59, Niklas Söderlund wrote: >>> There is no good reason to hold the list_lock when reprobing the devices >>> and it prevents a clean

Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

Hi Hans, Thanks for your feedback. On 2017-07-18 16:22:43 +0200, Hans Verkuil wrote: > On 17/07/17 18:59, Niklas Söderlund wrote: > > Add a subdevice specific notifier which can be used by a subdevice > > driver to compliment the master device notifier to extend the subdevice > > compliment ->

Re: [PATCH v4 2/3] v4l: async: do not hold list_lock when reprobing devices

Hi Hans, Thanks for your feedback. On 2017-07-18 16:22:14 +0200, Hans Verkuil wrote: > On 17/07/17 18:59, Niklas Söderlund wrote: > > There is no good reason to hold the list_lock when reprobing the devices > > and it prevents a clean implementation of subdevice notifiers. Move the > > actual

Re: [PATCH v4 2/2] media: entity: Add media_entity_get_fwnode_pad() function

Hi Niklas, Small spelling error discovered in here: On 15/06/17 10:17, Niklas Söderlund wrote: > This is a wrapper around the media entity get_fwnode_pad operation. > > Signed-off-by: Niklas Söderlund > Acked-by: Sakari Ailus

[PATCH v2 3/3] ARM: dts: r8a7794: Add SMP support

From: Sergei Shtylyov Add the device tree node for the Advanced Power Management Unit (APMU). Use the "enable-method" prop to point out that the APMU should be used for the SMP support. Signed-off-by: Sergei Shtylyov

[PATCH v2 1/3] ARM: Add definition for monitor mode

provides *_MODE definitions for the various processor modes, but monitor mode was missing. Add MON_MODE to avoid code using the hardcoded value. Suggested-by: Marc Zyngier Signed-off-by: Geert Uytterhoeven --- ARM maintainers: Please provide your

[PATCH v2 2/3] ARM: shmobile: rcar-gen2: Make sure CNTVOFF is initialized on CA7/15

On Cortex-A7, the arch timer CNTVOFF register is uninitialized. Ideally it should be initialized by the boot loader, but it isn't. For the boot CPU, CNTVOFF is initialized by Linux since commit 9ce3fa6816c2fb59 ("ARM: shmobile: rcar-gen2: Add CA7 arch_timer initialization for r8a7794"). For

[PATCH v2 0/3] ARM: renesas: Enable SMP on R-Car E2

Hi all, This patch series enables SMP on R-Car E2 (r8a7794). The main hurdle here is that R-Car Gen2 boot loaders do not initialize the arch_timer CNTVOFF register, which thus needs workarounds on Linux. - The first patch adds a definition for MON_MODE, as suggested by Marc

Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

On 17/07/17 18:59, Niklas Söderlund wrote: > Add a subdevice specific notifier which can be used by a subdevice > driver to compliment the master device notifier to extend the subdevice compliment -> complement Just one character difference, but a wildly different meaning :-) Although it was

Re: [PATCH v4 2/3] v4l: async: do not hold list_lock when reprobing devices

On 17/07/17 18:59, Niklas Söderlund wrote: > There is no good reason to hold the list_lock when reprobing the devices > and it prevents a clean implementation of subdevice notifiers. Move the > actual release of the devices outside of the loop which requires the > lock to be held. > >

RE: [PATCH] ARM: shmobile: rcar-gen2: Correct arch timer frequency on RZ/G1E

It looks good to me. Regards, Biju > -Original Message- > From: Geert Uytterhoeven [mailto:geert+rene...@glider.be] > Sent: 18 July 2017 14:29 > To: Simon Horman ; Magnus Damm > > Cc: Sergei Shtylyov ; Biju

[PATCH v2] ARM: shmobile: rcar-gen2: Correct arch timer frequency on RZ/G1E

According to the datasheet, the frequency of the ARM architecture timer on RZ/G1E depends on the frequency of the ZS clock, just like on R-Car E2 and V2H. Signed-off-by: Geert Uytterhoeven --- Untested due to lack of hardware. v2: - Rebased to avoid dependencies. ---

[PATCH] ARM: shmobile: rcar-gen2: Correct arch timer frequency on RZ/G1E

According to the datasheet, the frequency of the ARM architecture timer on RZ/G1E depends on the frequency of the ZS clock, just like on R-Car E2 and V2H. Signed-off-by: Geert Uytterhoeven --- Untested due to lack of hardware. v2: - Rebased to avoid dependencies. ---

Re: [PATCH 3/3] usb: gadget: udc: renesas_usb3: protect usb3_ep->started in usb3_start_pipen()

On Tue, Jul 18, 2017 at 2:26 PM, Yoshihiro Shimoda wrote: > This patch fixes an issue that unexpected behavior happens when > both the interrupt handler and renesas_usb3_ep_enable() are called. > In this case, since usb3_start_pipen() checked the

Re: [PATCH 1/3] usb: gadget: udc: renesas_usb3: fix free size in renesas_usb3_dma_free_prd()

Hi Shimoda-san, On Tue, Jul 18, 2017 at 2:26 PM, Yoshihiro Shimoda wrote: > The commit 2d4aa21a73ba ("usb: gadget: udc: renesas_usb3: add support > for dedicated DMAC") has a bug in the renesas_usb3_dma_free_prd(). > The size of dma_free_coherent() should be the

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

On Tue, Jul 18, 2017 at 2:47 PM, Laurent Pinchart wrote: > On Tuesday 18 Jul 2017 14:08:39 Daniel Vetter wrote: >> On Tue, Jul 18, 2017 at 01:14:12PM +0300, Laurent Pinchart wrote: >> > On Tuesday 18 Jul 2017 09:05:22 Maxime Ripard wrote: >> >> On Fri, Jul 14,

Re: [PATCH v2 00/14] Renesas R-Car VSP: Add H3 ES2.0 support

On 26/06/17 20:12, Laurent Pinchart wrote: > Hello, > > This patch series implements support for the R-Car H3 ES2.0 SoC in the VSP > and DU drivers. > > Compared to the H3 ES1.1, the H3 ES2.0 has a new VSP2-DL instance that > includes two blending units, a BRU and a BRS. The BRS is similar to

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

Hi Daniel, On Tuesday 18 Jul 2017 14:08:39 Daniel Vetter wrote: > On Tue, Jul 18, 2017 at 01:14:12PM +0300, Laurent Pinchart wrote: > > On Tuesday 18 Jul 2017 09:05:22 Maxime Ripard wrote: > >> On Fri, Jul 14, 2017 at 02:43:12AM +0300, Laurent Pinchart wrote: > >>> On Thursday 13 Jul 2017

[PATCH 1/3] usb: gadget: udc: renesas_usb3: fix free size in renesas_usb3_dma_free_prd()

The commit 2d4aa21a73ba ("usb: gadget: udc: renesas_usb3: add support for dedicated DMAC") has a bug in the renesas_usb3_dma_free_prd(). The size of dma_free_coherent() should be the same with dma_alloc_coherent() Otherwise, this code causes a WARNING by mm/page_alloc.c when

[PATCH 3/3] usb: gadget: udc: renesas_usb3: protect usb3_ep->started in usb3_start_pipen()

This patch fixes an issue that unexpected behavior happens when both the interrupt handler and renesas_usb3_ep_enable() are called. In this case, since usb3_start_pipen() checked the usb3_ep->started, but the flags was not protected. So, this patch protects the flag by usb3->lock. Since

[PATCH 2/3] usb: gadget: udc: renesas_usb3: fix zlp transfer by the dmac

The dedicated dmac can transfer a zero-length-packet (zlp) if some bits of the USB_COM_CON register. However, the commit 2d4aa21a73ba ("usb: gadget: udc: renesas_usb3: add support for dedicated DMAC") didn't set the bits to 1. So, this patch fixes it. Fixes: 2d4aa21a73b ("usb: gadget: udc:

[PATCH 0/3] usb: gadget: udc: renesas_usb3: fixes for v4.13-rc1

This patch is based on the latest Felipe's usb.git / testing/fixes branch (The commit id = 4a71fcb8ac5f94c07bf47a43b13258a52e4fe3ad). Yoshihiro Shimoda (3): usb: gadget: udc: renesas_usb3: fix free size in renesas_usb3_dma_free_prd() usb: gadget: udc: renesas_usb3: fix zlp transfer by the

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

On Tue, Jul 18, 2017 at 01:14:12PM +0300, Laurent Pinchart wrote: > Hi Maxime, > > On Tuesday 18 Jul 2017 09:05:22 Maxime Ripard wrote: > > On Fri, Jul 14, 2017 at 02:43:12AM +0300, Laurent Pinchart wrote: > > > On Thursday 13 Jul 2017 16:41:13 Maxime Ripard wrote: > > >> The current

Re: [PATCH v6] media: platform: Renesas IMR driver

On 08/07/17 15:31, Konstantin Kozhevnikov wrote: > Hello all, > > the sample is made publicly available, and can be taken from > https://github.com/CogentEmbedded/imr-sv-utest/blob/master/utest/utest-imr.c. > > It doesn't show how luminance/chrominance correction actually works, however. >

renesas-drivers-2017-07-18-v4.13-rc1

I have pushed renesas-drivers-2017-07-18-v4.13-rc1 to https://git.kernel.org/cgit/linux/kernel/git/geert/renesas-drivers.git This tree is meant to ease development of platform support and drivers for Renesas ARM SoCs. It is created by merging (a) the for-next branches of various subsystem trees

[PATCH v3 3/4] i2c: sh_mobile: use helper to decide if DMA is useful

This ensures that we fall back to PIO if the buffer is too small for DMA being useful. Otherwise, we use DMA. A bounce buffer might be applied if the original message buffer is not DMA safe Signed-off-by: Wolfram Sang --- drivers/i2c/busses/i2c-sh_mobile.c | 8

[PATCH v3 2/4] i2c: add docs to clarify DMA handling

Signed-off-by: Wolfram Sang --- Changes since v2: * documentation updates. Hopefully better wording now Documentation/i2c/DMA-considerations | 38 1 file changed, 38 insertions(+) create mode 100644

[PATCH v3 1/4] i2c: add helpers to ease DMA handling

One helper checks if DMA is suitable and optionally creates a bounce buffer, if not. The other function returns the bounce buffer and makes sure the data is properly copied back to the message. Signed-off-by: Wolfram Sang --- Changes since v2: * rebased to

[PATCH v3 0/4] i2c: document DMA handling and add helpers for it

So, after revisiting old mail threads and taking part in a similar discussion on the USB list, here is what I cooked up to document and ease DMA handling for I2C within Linux. Please have a look at the documentation introduced in patch 2 for further details. All patches have been tested with a

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

Hi Maxime, On Tuesday 18 Jul 2017 09:05:22 Maxime Ripard wrote: > On Fri, Jul 14, 2017 at 02:43:12AM +0300, Laurent Pinchart wrote: > > On Thursday 13 Jul 2017 16:41:13 Maxime Ripard wrote: > >> The current drm_atomic_helper_commit_tail helper works only if the CRTC > >> is accessible, and

Re: [PATCH V4 2/2] mfd: Add ROHM BD9571MWV-M MFD PMIC driver

On Mon, 17 Jul 2017, Marek Vasut wrote: > Add the MFD part of the ROHM BD9571MWV-M PMIC driver and MAINTAINERS > entry. The MFD part only specifies the regmap bits for the PMIC and > binds the subdevs together. > > Signed-off-by: Marek Vasut > Cc:

Re: [PATCH v7 2/3] media: i2c: adv748x: add adv748x driver

Hi Kieran, A few more minor matters that you might want to address on top of Hans's pull request. On Thu, Jul 06, 2017 at 12:01:16PM +0100, Kieran Bingham wrote: ... > +static int adv748x_afe_g_input_status(struct v4l2_subdev *sd, u32 *status) > +{ > + struct adv748x_afe *afe =

[R-CAR H3] Can't Modify U-Boot Environment Variables on Updated Firmware

Hello, I understand that this is the mailing list for support for the R-Car line of boards. If this information is incorrect, please direct me, if possible, to the correct resource. I have posted my issue to various rcar-related repos on github as well as the e-linux talk page. Issue is as

Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

Hi Geert, Thanks for your feedback. On 2017-07-18 09:11:19 +0200, Geert Uytterhoeven wrote: > Hi Niklas, > > On Mon, Jul 17, 2017 at 6:59 PM, Niklas Söderlund > wrote: > > Add a subdevice specific notifier which can be used by a subdevice > > driver to

Re: [PATCH 4/4] drm/sun4i: make sure we don't have a commit pending

On Tue, Jul 18, 2017 at 9:07 AM, Maxime Ripard wrote: > On Mon, Jul 17, 2017 at 02:57:19PM +0800, Chen-Yu Tsai wrote: >> On Mon, Jul 17, 2017 at 2:55 PM, Maxime Ripard >> wrote: >> > On Fri, Jul 14, 2017 at 04:56:01PM +0800,

Re: [renesas-drivers:topic/renesas-overlays 1/86] drivers//of/configfs.c:48:2: error: implicit declaration of function 'of_fdt_unflatten_tree'

On Tue, Jul 18, 2017 at 8:44 AM, Geert Uytterhoeven wrote: > On Mon, Jul 17, 2017 at 7:58 PM, kbuild test robot > wrote: >> tree: >> https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git >> topic/renesas-overlays >> head:

Re: [PATCH 1/5] watchdog: renesas_wdt: avoid (theoretical) type overflow

> > - unsigned long rate; > > - unsigned int clks_per_sec; > > + unsigned long rate, clks_per_sec; > > If you make this change, you should also update rwdt_priv.clks_per_sec > (yes I know it's removed in a later patch in this series). Right. I will change but also wait a bit

Re: [PATCH v4 3/3] v4l: async: add subnotifier to subdevices

Hi Niklas, On Mon, Jul 17, 2017 at 6:59 PM, Niklas Söderlund wrote: > Add a subdevice specific notifier which can be used by a subdevice > driver to compliment the master device notifier to extend the subdevice > discovery. > > The master device registers

Re: [PATCH 4/4] drm/sun4i: make sure we don't have a commit pending

On Mon, Jul 17, 2017 at 02:57:19PM +0800, Chen-Yu Tsai wrote: > On Mon, Jul 17, 2017 at 2:55 PM, Maxime Ripard > wrote: > > On Fri, Jul 14, 2017 at 04:56:01PM +0800, Chen-Yu Tsai wrote: > >> Hi, > >> > >> On Thu, Jul 13, 2017 at 10:41 PM, Maxime Ripard > >>

Re: [PATCH 1/4] drm/atomic: implement drm_atomic_helper_commit_tail for runtime_pm users

Hi Laurent, On Fri, Jul 14, 2017 at 02:43:12AM +0300, Laurent Pinchart wrote: > Hi Maxime, > > Thank you for the patch. > > On Thursday 13 Jul 2017 16:41:13 Maxime Ripard wrote: > > The current drm_atomic_helper_commit_tail helper works only if the CRTC is > > accessible, and documents an

Re: [PATCH 1/5] watchdog: renesas_wdt: avoid (theoretical) type overflow

Hi Wolfram, On Mon, Jul 17, 2017 at 5:12 PM, Wolfram Sang wrote: > Because the smallest clock divider we can select is 1, 'clks_per_sec' > must be the same type as 'rate'. > > Signed-off-by: Wolfram Sang Thanks for your patch!

Re: [RFC v2 6/6] ARM: dts: blanche: add SCIF1 and MAX9260 deserializer

Hi Uli, On Mon, Jul 17, 2017 at 5:24 PM, Ulrich Hecht wrote: > Adds serial port SCIF1 and the MAX9260 deserializers connected to it. > > Signed-off-by: Ulrich Hecht Thanks for your patch! > arch/arm/boot/dts/r8a7792-blanche.dts

Re: [RFC v2 5/6] max9260: add driver for i2c over GMSL passthrough

Hi Ulrich, On Mon, Jul 17, 2017 at 5:24 PM, Ulrich Hecht wrote: > This driver implements tunnelling of i2c requests over GMSL via a > MAX9260 deserializer. It provides an i2c adapter that can be used > to reach devices on the far side of the link. > >

Re: [renesas-drivers:topic/renesas-overlays 1/86] drivers//of/configfs.c:48:2: error: implicit declaration of function 'of_fdt_unflatten_tree'

On Mon, Jul 17, 2017 at 7:58 PM, kbuild test robot wrote: > tree: > https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git > topic/renesas-overlays > head: 8361d7e7432d1937f94e1863211fb3f852e37f36 > commit: 1ba23467c1d052c0b5c35436c034f0fb7103eeaf