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

On 07/19/17 09:02, Rob Herring wrote: > On Wed, Jul 19, 2017 at 4:41 AM, Sylwester Nawrocki > wrote: >> On 07/18/2017 11:43 PM, Rob Herring wrote: >>> Now that we have a custom printf format specifier, convert users of >>> full_name to use %pOF instead. This is preparation

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

Hi Rob, Thank you for the patch. On Tuesday 18 Jul 2017 16:43:04 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

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

Rob Herring writes: > 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. For bcm283x: Acked-by: Eric Anholt signature.asc

[renesas-drivers:topic/rcar2-cpg-mssr-dt-v2 2/9] arch/arm/boot/dts/r8a7791.dtsi:13:48: fatal error: dt-bindings/clock/r8a7791-cpg-mssr.h: No such file or directory

tree: https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git topic/rcar2-cpg-mssr-dt-v2 head: 719d66878b5366748b076ab8b7e3a0ed6012033c commit: 00dc7a102c7df46d30520d53002c77d9c75ce862 [2/9] ARM: dts: r8a7791: Convert to new CPG/MSSR bindings config: arm-at91_dt_defconfig

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

On Wed, Jul 19, 2017 at 4:41 AM, Sylwester Nawrocki wrote: > On 07/18/2017 11:43 PM, 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

Re: [renesas-drivers:topic/rcar2-cpg-mssr-dt-v2 1/9] arch/arm/boot/dts/r8a7790.dtsi:13:48: fatal error: dt-bindings/clock/r8a7790-cpg-mssr.h: No such file or directory

On Wed, Jul 19, 2017 at 5:54 PM, kbuild test robot wrote: > tree: > https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git > topic/rcar2-cpg-mssr-dt-v2 > head: 719d66878b5366748b076ab8b7e3a0ed6012033c > commit:

[renesas-drivers:topic/rcar2-cpg-mssr-dt-v2 1/9] arch/arm/boot/dts/r8a7790.dtsi:13:48: fatal error: dt-bindings/clock/r8a7790-cpg-mssr.h: No such file or directory

tree: https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-drivers.git topic/rcar2-cpg-mssr-dt-v2 head: 719d66878b5366748b076ab8b7e3a0ed6012033c commit: 09edb9fc52b0323c57747413da258d35f6a80495 [1/9] ARM: dts: r8a7790: Convert to new CPG/MSSR bindings config: arm-at91_dt_defconfig

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

On Wed, Jul 19, 2017 at 1:22 AM, Uwe Kleine-König wrote: > On Tue, Jul 18, 2017 at 04:42:41PM -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 >

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

Hi Uli, > +struct max9260_device { > + struct serdev_device *serdev; > + u8 *rx_buf; > + int rx_len; > + int rx_state; > + wait_queue_head_t rx_wq; > + struct i2c_adapter adap; > +}; > + > +static void wait_for_transaction(struct max9260_device *dev) max9260_ prefix as

[PATCH v2 3/7] drm/rcar-du: Use new iterator macros, v2.

for_each_obj_in_state is about to be removed, so use the correct new iterator macros. Also look at new_plane_state instead of plane->state when looking up the hw planes in use. They should be the same except when reallocating, (in which case this code is skipped) and we should really stop looking

Re: [PATCH] watchdog: asm9260_wdt: don't round closest with get_timeleft

On 07/17/2017 10:12 AM, Wolfram Sang wrote: We should never return more time left than there actually is. So, switch to a plain divider instead of DIV_ROUND_CLOSEST. Signed-off-by: Wolfram Sang Reviewed-by: Guenter Roeck --- This is

Re: [PATCH v5 0/4] v4l2-async: add subnotifier registration for subdevices

Hi Niklas, On Wed, Jul 19, 2017 at 12:49:42PM +0200, Niklas Söderlund wrote: > Hi, > > I know Sakari have posted a series '[RFC 00/19] Async sub-notifiers and > how to use them' which address similar problems as this series. This is > not intended to compete whit his work and Sakari includes

Re: [PATCH v5 0/4] v4l2-async: add subnotifier registration for subdevices

Hi Hans, Thanks for your feedback. On 2017-07-19 13:02:14 +0200, Hans Verkuil wrote: > On 19/07/17 12:49, Niklas Söderlund wrote: > > * Changes since v4 > > - Add patch which aborts v4l2_async_notifier_unregister() if the memory > > allocation for the device cache fails instead of trying to

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

Hi Rob, On mar., juil. 18 2017, 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

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

On 18/07/2017 at 23:43, 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: Kyungmin Park

Re: [PATCH v5 0/4] v4l2-async: add subnotifier registration for subdevices

On 19/07/17 12:49, Niklas Söderlund wrote: > * Changes since v4 > - Add patch which aborts v4l2_async_notifier_unregister() if the memory > allocation for the device cache fails instead of trying to do as much > as possible but still leave the system in a semi good state. Since you are

[PATCH v5 3/4] v4l: async: do not hold list_lock when re-probing devices

There is no good reason to hold the list_lock when re-probing 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. Signed-off-by: Niklas Söderlund

[PATCH v5 4/4] v4l: async: add subnotifier to subdevices

Add a subdevice specific notifier which can be used by a subdevice driver to complement the master device notifier to extend the subdevice discovery. The master device registers the subdevices closest to itself in its notifier while the subdevice(s) register notifiers for their closest

[PATCH v5 0/4] v4l2-async: add subnotifier registration for subdevices

Hi, I know Sakari have posted a series '[RFC 00/19] Async sub-notifiers and how to use them' which address similar problems as this series. This is not intended to compete whit his work and Sakari includes one of my v3 patches in his series. Never the less I post this updated series since it

[PATCH v5 2/4] v4l: async: abort if memory allocation fails when unregistering notifiers

Instead of trying to cope with the failed memory allocation and still leaving the kernel in a semi-broken state (the subdevices will be released but never re-probed) simply abort. The kernel have already printed a warning about allocation failure but keep the error printout to ease pinpointing the

[PATCH v5 1/4] v4l: async: fix unbind error in v4l2_async_notifier_unregister()

The call to v4l2_async_cleanup() will set sd->asd to NULL so passing it to notifier->unbind() have no effect and leaves the notifier confused. Call the unbind() callback prior to cleaning up the subdevice to avoid this. Signed-off-by: Niklas Söderlund ---

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

On 07/18/2017 11:42 PM, 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: Michael Turquette

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

On 07/18/2017 11:43 PM, 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: Scott Wood

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

On 07/18/2017 11:43 PM, 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: Kyungmin Park

Re: [PATCH v3 4/4] i2c: rcar: check for DMA-capable buffers

Hi Wolfram, On 2017-07-18 12:23:39 +0200, Wolfram Sang wrote: > Handling this is special for this driver. Because the hardware needs to > initialize the next message in interrupt context, we cannot use the > i2c_check_msg_for_dma() directly. This helper only works reliably in > process context.

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

On 07/18/2017 11:43 PM, 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 > --- >

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

On 07/18/2017 11:42 PM, 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: Michael Turquette

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

Hi Wolfram, On 2017-07-18 12:23:38 +0200, Wolfram Sang wrote: > 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

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

Hi Wolfram, On 2017-07-18 12:23:37 +0200, Wolfram Sang wrote: > Signed-off-by: Wolfram Sang Reviewed-by: Niklas Söderlund > --- > Changes since v2: > > * documentation updates. Hopefully better wording now > >

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

On Tue, Jul 18, 2017 at 11:42 PM, 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

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

Hi Rob, Thank you for the patch. On Tuesday 18 Jul 2017 16:43:13 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

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

On 2017-07-18 16:43:13 -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: Kyungmin

[PATCH v2 2/5] watchdog: renesas_wdt: check rate also for upper limit

When checking the clock rate, ensure also that counting all 16 bits takes at least one second to match the granularity of the framework. Reviewed-by: Guenter Roeck Signed-off-by: Wolfram Sang --- drivers/watchdog/renesas_wdt.c | 4 ++-- 1

[PATCH v2 3/5] watchdog: renesas_wdt: don't round closest with get_timeleft

We should never return more time left than there actually is. So, switch to a plain divider instead of DIV_ROUND_CLOSEST. Reviewed-by: Guenter Roeck Signed-off-by: Wolfram Sang --- drivers/watchdog/renesas_wdt.c | 2 +- 1 file changed, 1

[PATCH v2 4/5] watchdog: renesas_wdt: apply better precision

The error margin of the clks_per_second variable was too large and caused offsets when used with clock frequencies which left a remainder after applying the dividers. Now we always calculate directly using the clock rate and the divider using some helper macros. That also means that DIV_ROUND_UP

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

Because the smallest clock divider we can select is 1, 'clks_per_sec' must be the same type as 'rate'. Reviewed-by: Guenter Roeck Signed-off-by: Wolfram Sang --- Changes since V1: * use proper type also in the private struct

[PATCH v2 0/5] watchdog: renesas_wdt: improve precision

Currently, the renesas-rwdt driver is not precise with input clocks which have a remainder after the clock divisors are applied. This series should fix the situation and also pays attention to ensure variables have proper types and are divided properly. As a cherry on top, we also get a new

[PATCH v2 5/5] watchdog: renesas_wdt: add another divider option

If we set RWTCSRB to 0, we can gain 4096 as another divider value. This is supported by all R-Car Gen2 and Gen3 devices which we aim to support. Reviewed-by: Guenter Roeck Signed-off-by: Wolfram Sang --- drivers/watchdog/renesas_wdt.c | 8

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

On Tue, Jul 18, 2017 at 6:44 PM, Wolfram Sang wrote: > 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

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

On Tue, 2017-07-18 at 16:43 -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 PATCH 0/2] clk: renesas: rcar-gen3-cpg: refactor accessing the SD div table

> > Please let me know if this is what you had in mind and if you can think of > > other ways of testing this change. Glad you like it :) I forgot to mention that I checked 'clk_summary' with the different SD card types and the applied values were as before. signature.asc Description: PGP

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

Am Dienstag, 18. Juli 2017, 16:42:41 CEST schrieb Rob Herring: > 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

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

On Tue, Jul 18, 2017 at 11:43 PM, 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

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

On Tue, Jul 18, 2017 at 6:44 PM, Wolfram Sang wrote: > 'rate' is not used, so we can use 'parent_rate' directly. > > Signed-off-by: Wolfram Sang Reviewed-by: Geert Uytterhoeven i.e. will queue in

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

On Tue, Jul 18, 2017 at 11:42 PM, 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

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

On Tue, Jul 18, 2017 at 11:43 PM, 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

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

On 2017-07-17 17:24, Ulrich Hecht wrote: > Adds an interface for slave device multiplexing using the mux subsystem. > > Signed-off-by: Ulrich Hecht > --- > drivers/tty/serdev/Kconfig | 3 +++ > drivers/tty/serdev/Makefile | 1 + > drivers/tty/serdev/core.c |

[PATCH 2/2] usb: renesas_usbhs: gadget: disable all eps when the driver stops

A gadget driver will not disable eps immediately when ->disconnect() is called. But, since this driver assumes all eps stop after the ->disconnect(), unexpected behavior happens (especially in system suspend). So, this patch disables all eps in usbhsg_try_stop(). After disabling eps by

[PATCH 0/2] usb: renesas_usbhs: fixes for system suspend/resume

This patch set is based on the latest Felipe's usb.git / testing/fixes branch (the commit id = 781001ff9678e5df048cca3d4290c1638a28cafa). Yoshihiro Shimoda (2): usb: renesas_usbhs: fix usbhsc_resume() for !USBHSF_RUNTIME_PWCTRL usb: renesas_usbhs: gadget: disable all eps when the driver stops

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

On 07/18/2017 11:43 PM, 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: Linus Walleij

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

On Tue, Jul 18, 2017 at 04:42:41PM -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 Oh nice. If the commit adding %pOF is already set in stone, i'd suggest to mention the commit

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

On Tue, Jul 18, 2017 at 04:43:04PM -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: [PATCH] clk: Convert to using %pOF instead of full_name

On Tue, Jul 18, 2017 at 04:42:52PM -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: