Re: [PATCH v7 1/2] spi: Add Renesas R-Car Gen3 RPC-IF SPI controller driver

2019-01-30 Thread Boris Brezillon
On Wed, 30 Jan 2019 08:15:25 +0100 Marek Vasut wrote: > >> > So far as I know that HF is provided by Cypress only and > >> > any mass production product use the component which is provided by only > >> > one provider > >> > will be a big risk. > >> > > >> > Compare to HF, there are more provider

Re: [PATCH v5 1/2] spi: Add Renesas R-Car Gen3 RPC-IF SPI controller driver

2019-01-10 Thread Boris Brezillon
On Wed, 9 Jan 2019 21:47:48 +0300 Sergei Shtylyov wrote: > On 01/08/2019 07:16 AM, Mason Yang wrote: > > > Add a driver for Renesas R-Car Gen3 RPC-IF SPI controller. > > > > Signed-off-by: Mason Yang > >You now need to add: > > Signed-off-by: Sergei Shtylyov May I ask why?

Re: [v2,1/3] mtd: spi-nor: Add support for is25lp016d

2018-12-08 Thread Boris Brezillon
On Thu, 2018-11-08 at 17:04:41 UTC, Fabrizio Castro wrote: > The is25lp016d is found on the iwg23s from iWave, therefore > add driver support for it so that we can upstream board support. > > Signed-off-by: Fabrizio Castro Applied to http://git.infradead.org/linux-mtd.git spi-nor/next, thanks.

Re: [PATCH 1/2] spi: Add Renesas R-Car RPC SPI controller driver

2018-11-20 Thread Boris Brezillon
On Tue, 20 Nov 2018 14:09:05 +0100 Marek Vasut wrote: > On 11/20/2018 08:23 AM, masonccy...@mxic.com.tw wrote: > > Hi Marek, > > Hi, > > >> Marek Vasut > >> 2018/11/19 下午 10:12 > >> > >> To > >> > >> > + > >> > +static int rpc_spi_set_freq(struct rpc_spi *rpc, unsigned long freq) > >> > +{

Re: [PATCH v3] drm/bridge/sii902x: Add missing dependency on I2C_MUX

2018-11-20 Thread Boris Brezillon
On Tue, 20 Nov 2018 09:00:47 +0100 Boris Brezillon wrote: > On Mon, 19 Nov 2018 19:09:33 + > Fabrizio Castro wrote: > > > Hello Boris, > > > > > From: Boris Brezillon > > > Sent: 19 November 2018 19:07 > > > Subject: Re: [PATCH v

Re: [PATCH 1/2] spi: Add Renesas R-Car RPC SPI controller driver

2018-11-20 Thread Boris Brezillon
On Tue, 20 Nov 2018 09:01:29 +0100 Geert Uytterhoeven wrote: > > --- /dev/null > > +++ b/drivers/spi/spi-renesas-rpc.c > > @@ -0,0 +1,750 @@ > > > +static int rpc_spi_set_freq(struct rpc_spi *rpc, unsigned long freq) > > +{ > > + int ret; > > + > > + if (rpc->cur_speed_hz == freq)

Re: [PATCH v3] drm/bridge/sii902x: Add missing dependency on I2C_MUX

2018-11-20 Thread Boris Brezillon
On Mon, 19 Nov 2018 19:09:33 + Fabrizio Castro wrote: > Hello Boris, > > > From: Boris Brezillon > > Sent: 19 November 2018 19:07 > > Subject: Re: [PATCH v3] drm/bridge/sii902x: Add missing dependency on > > I2C_MUX > > > > On Mon, 19 Nov 2018

Re: [PATCH 2/2] dt-binding: spi: Document Renesas R-Car RPC controller bindings

2018-11-19 Thread Boris Brezillon
On Mon, 19 Nov 2018 23:29:00 +0100 Marek Vasut wrote: > On 11/19/2018 11:25 PM, Boris Brezillon wrote: > > On Mon, 19 Nov 2018 23:22:45 +0100 > > Marek Vasut wrote: > > > >> On 11/19/2018 11:19 PM, Boris Brezillon wrote: > >>> On Mon, 19 Nov 20

Re: [PATCH 2/2] dt-binding: spi: Document Renesas R-Car RPC controller bindings

2018-11-19 Thread Boris Brezillon
On Mon, 19 Nov 2018 23:22:45 +0100 Marek Vasut wrote: > On 11/19/2018 11:19 PM, Boris Brezillon wrote: > > On Mon, 19 Nov 2018 23:11:31 +0100 > > Marek Vasut wrote: > > > >> On 11/19/2018 04:21 PM, Boris Brezillon wrote: > >>> On Mon, 19 Nov 20

Re: [PATCH 2/2] dt-binding: spi: Document Renesas R-Car RPC controller bindings

2018-11-19 Thread Boris Brezillon
On Mon, 19 Nov 2018 23:11:31 +0100 Marek Vasut wrote: > On 11/19/2018 04:21 PM, Boris Brezillon wrote: > > On Mon, 19 Nov 2018 16:12:41 +0100 > > Marek Vasut wrote: > > > >> On 11/19/2018 03:43 PM, Boris Brezillon wrote: > >>> On Mon, 19 Nov 20

Re: [PATCH v3] drm/bridge/sii902x: Add missing dependency on I2C_MUX

2018-11-19 Thread Boris Brezillon
On Mon, 19 Nov 2018 18:59:04 + Fabrizio Castro wrote: > Hello Boris, > > > From: Boris Brezillon > > Sent: 19 November 2018 16:55 > > Subject: Re: [PATCH v3] drm/bridge/sii902x: Add missing dependency on > > I2C_MUX > > > > Hi Fabrizio,

Re: [PATCH v3] drm/bridge/sii902x: Add missing dependency on I2C_MUX

2018-11-19 Thread Boris Brezillon
Hi Fabrizio, On Mon, 19 Nov 2018 13:26:18 + Fabrizio Castro wrote: > kbuild test robot reports: > > >> ERROR: "i2c_mux_add_adapter" [drivers/gpu/drm/bridge/sii902x.ko] > undefined! > >> ERROR: "i2c_mux_alloc" [drivers/gpu/drm/bridge/sii902x.ko] > undefined! > >> ERROR: "i2c_mux_del_adap

Re: [PATCH 2/2] dt-binding: spi: Document Renesas R-Car RPC controller bindings

2018-11-19 Thread Boris Brezillon
On Mon, 19 Nov 2018 16:12:41 +0100 Marek Vasut wrote: > On 11/19/2018 03:43 PM, Boris Brezillon wrote: > > On Mon, 19 Nov 2018 15:14:07 +0100 > > Marek Vasut wrote: > > > >> On 11/19/2018 03:10 PM, Boris Brezillon wrote: > >>> On Mon, 19 Nov 20

Re: [PATCH 2/2] dt-binding: spi: Document Renesas R-Car RPC controller bindings

2018-11-19 Thread Boris Brezillon
On Mon, 19 Nov 2018 15:14:07 +0100 Marek Vasut wrote: > On 11/19/2018 03:10 PM, Boris Brezillon wrote: > > On Mon, 19 Nov 2018 14:49:31 +0100 > > Marek Vasut wrote: > > > >> On 11/19/2018 11:01 AM, Mason Yang wrote: > >>> Document the bindings u

Re: [PATCH 2/2] dt-binding: spi: Document Renesas R-Car RPC controller bindings

2018-11-19 Thread Boris Brezillon
On Mon, 19 Nov 2018 14:49:31 +0100 Marek Vasut wrote: > On 11/19/2018 11:01 AM, Mason Yang wrote: > > Document the bindings used by the Renesas R-Car D3 RPC controller. > > > > Signed-off-by: Mason Yang > > --- > > .../devicetree/bindings/spi/spi-renesas-rpc.txt| 33 > > ++

Re: [PATCH] drm/bridge: Fix 0-day build error

2018-11-19 Thread Boris Brezillon
On Mon, 19 Nov 2018 13:09:16 + Fabrizio Castro wrote: > Hi Boris, > > > From: Boris Brezillon > > Sent: 19 November 2018 12:51 > > Subject: Re: [PATCH] drm/bridge: Fix 0-day build error > > > > Hi Fabrizio, > > > > The prefix should

Re: [PATCH] drm/bridge: Fix 0-day build error

2018-11-19 Thread Boris Brezillon
Hi Fabrizio, The prefix should be "drm/bridge/sii902x:" and I'd prefer a short explanation of what is problematic in the subject rather than "Fix 0-day build error". Maybe something like "drm/bridge/sii902x: Add missing dependency on I2C_MUX" On Mon, 19 Nov 2018 12:44:23 + Fabrizio Castro w

Re: [PATCH 3/9] drm: replace "drm_dev_unref" function with "drm_dev_put"

2018-11-15 Thread Boris Brezillon
> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 4 ++-- For the Atmel HLCDC driver Acked-by: Boris Brezillon > drivers/gpu/drm/etnaviv/etnaviv_drv.c| 4 ++-- > drivers/gpu/drm/mxsfb/mxsfb_drv.c| 4 ++-- > drivers/gpu/drm/rcar-du/rcar_du_drv.c| 2

Re: [PATCH v3] drm/bridge/sii902x: Fix EDID readback

2018-11-15 Thread Boris Brezillon
On Thu, 15 Nov 2018 11:39:30 + Fabrizio Castro wrote: > Hello Boris, > > > From: Boris Brezillon > > Sent: 15 November 2018 10:16 > > Subject: Re: [PATCH v3] drm/bridge/sii902x: Fix EDID readback > > > > On Thu, 15 Nov 2018 11:13:52 +0100 > > Linus

Re: [PATCH v3] drm/bridge/sii902x: Fix EDID readback

2018-11-15 Thread Boris Brezillon
ce during the DDC Bus > > Request/Grant procedure and while talking to the monitor, we > > have to use the adapter locking primitives rather than the > > i2c-mux locking primitives. > > > > Signed-off-by: Fabrizio Castro > > > > --- > > v2->v3: > >

Re: [PATCH v3] drm/bridge/sii902x: Fix EDID readback

2018-11-07 Thread Boris Brezillon
ux locking primitives. > > Signed-off-by: Fabrizio Castro Acked-by: Boris Brezillon > > --- > v2->v3: > * Incorporated comments from Boris Brezillon and Peter Rosin > > v1->v2: > * Incorporated comments from Peter Rosin > > drivers/gpu/drm/bridge/sii902x

Re: [PATCH v2] drm/bridge/sii902x: Fix EDID readback

2018-11-05 Thread Boris Brezillon
Hi Fabrizio, Thanks for working on that. I think your solution fixes the issue we had on atmel boards [1] in a clean way (still have to find some time to test it). On Mon, 5 Nov 2018 13:56:48 + Fabrizio Castro wrote: > @@ -441,6 +558,9 @@ static int sii902x_remove(struct i2c_client *client

Re: [PATCH] spi: spi-mem: Fix inverted logic in op sanity check

2018-09-25 Thread Boris Brezillon
any buswidth is valid, instead of invalid. > > Fixes: 380583227c0c7f52 ("spi: spi-mem: Add extra sanity checks on the op > param") > Signed-off-by: Geert Uytterhoeven Reviewed-by: Boris Brezillon Sorry for the mess :-/, and thanks for reporting/fixing it. > --- &

Re: [PATCH] mtd: partitions: Handle add_mtd_device() failures gracefully

2018-04-26 Thread Boris Brezillon
On Thu, 26 Apr 2018 19:56:58 +0200 Geert Uytterhoeven wrote: > Hi Boris, > > On Thu, Apr 26, 2018 at 7:53 PM, Boris Brezillon > wrote: > > On Tue, 10 Apr 2018 15:26:20 +0200 > > Geert Uytterhoeven wrote: > >> On Mon, Apr 9, 2018 at 11:59 PM, Marek Vasut &

Re: [PATCH] mtd: partitions: Handle add_mtd_device() failures gracefully

2018-04-26 Thread Boris Brezillon
Hi Geert, Sorry for the late reply. On Tue, 10 Apr 2018 15:26:20 +0200 Geert Uytterhoeven wrote: > Hi Marek, > > On Mon, Apr 9, 2018 at 11:59 PM, Marek Vasut wrote: > > On 04/09/2018 02:25 PM, Geert Uytterhoeven wrote: > >> Currently add_mtd_device() failures are plainly ignored, which may

Re: [PATCH] drm/core: Remove drm_dev_unref() and it's uses

2018-04-26 Thread Boris Brezillon
| 4 ++-- > drivers/gpu/drm/armada/armada_drv.c| 6 +++--- > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 4 ++-- For atmel-hlcdc: Acked-by: Boris Brezillon

Re: [PATCH 30/61] mtd: nand: onenand: simplify getting .drvdata

2018-04-22 Thread Boris Brezillon
On Thu, 19 Apr 2018 16:06:00 +0200 Wolfram Sang wrote: > We should get drvdata from struct device directly. Going via > platform_device is an unneeded step back and forth. > > Signed-off-by: Wolfram Sang Applied to nand/next. Thanks, Boris > --- > > Build tested only. buildbot is happy. Pl

Re: [PATCH 29/61] mtd: devices: simplify getting .drvdata

2018-04-22 Thread Boris Brezillon
On Thu, 19 Apr 2018 16:05:59 +0200 Wolfram Sang wrote: > We should get drvdata from struct device directly. Going via > platform_device is an unneeded step back and forth. > > Signed-off-by: Wolfram Sang Applied to mtd/next. Thanks, Boris > --- > > Build tested only. buildbot is happy. Ple

Re: [PATCH] mtd: nand: sh_flctl: Use of_device_get_match_data() helper

2017-10-05 Thread Boris Brezillon
On Wed, 4 Oct 2017 14:19:03 +0200 Geert Uytterhoeven wrote: > Use the of_device_get_match_data() helper instead of open coding. > While at it, make config const so the cast can be dropped. > Applied. Thanks, Boris > Signed-off-by: Geert Uytterhoeven > --- > drivers/mtd/nand/sh_flctl.c | 9

Re: [PATCH] mtd: spi-nor: fix DMA unsafe buffer issue in spi_nor_read_sfdp()

2017-09-10 Thread Boris Brezillon
On Thu, 7 Sep 2017 10:00:50 +0200 Geert Uytterhoeven wrote: > Hi Cyrille, > > On Wed, Sep 6, 2017 at 11:45 PM, Cyrille Pitchen > wrote: > > spi_nor_read_sfdp() calls nor->read() to read the SFDP data. > > When the m25p80 driver is used (pretty common case), nor->read() is then > > implemented b

Re: [PATCH] mtd: spi-nor: fix DMA unsafe buffer issue in spi_nor_read_sfdp()

2017-09-07 Thread Boris Brezillon
On Thu, 7 Sep 2017 10:00:50 +0200 Geert Uytterhoeven wrote: > Hi Cyrille, > > On Wed, Sep 6, 2017 at 11:45 PM, Cyrille Pitchen > wrote: > > spi_nor_read_sfdp() calls nor->read() to read the SFDP data. > > When the m25p80 driver is used (pretty common case), nor->read() is then > > implemented b

Re: [PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Boris Brezillon
the encoder and optional previous bridge to perform that task, > update all the callers. > > Signed-off-by: Laurent Pinchart > --- > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 4 +- For atmel-hlcdc Acked-by: Boris Brezillon > drivers/gpu/drm/bridge/an

Re: [PATCH v2] pwm: improve args checking in pwm_apply_state()

2016-06-22 Thread Boris Brezillon
On Wed, 22 Jun 2016 12:16:59 -0700 Brian Norris wrote: > On Wed, Jun 22, 2016 at 10:04:22AM +0200, Boris Brezillon wrote: > > From 0610f7e24976e176054bce20445ff42d8aea9513 Mon Sep 17 00:00:00 2001 > > From: Boris Brezillon > > Date: Wed, 22 Jun 2016 09:25:14 +0200 > >

Re: [PATCH v2] pwm: improve args checking in pwm_apply_state()

2016-06-22 Thread Boris Brezillon
apply_state:502: period 0 -> 3 > > pwm_apply_state:503: duty_cycle 0 -> 3 > > tpu_pwm_config:267 > > pwm_apply_state:516: enabled 0 -> 0 > > pwm_apply_state:460: pwm backlight/2: period 3, duty_cycle 3 > > -pwm_apply_state:479: polarity 0 -&

Re: [PATCH v2] pwm: improve args checking in pwm_apply_state()

2016-06-21 Thread Boris Brezillon
On Tue, 21 Jun 2016 11:37:31 -0700 Brian Norris wrote: > Hi Geert, > > On Tue, Jun 21, 2016 at 04:42:04PM +0200, Geert Uytterhoeven wrote: > > On Fri, May 27, 2016 at 6:45 PM, Brian Norris > > wrote: > > > It seems like in the process of refactoring pwm_config() to utilize the > > > newly-in

[PATCH v2 02/20] drm: arc: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder(), and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/arc/arcpgu_hdmi.c |

[PATCH v2 01/20] drm/atomic: Fix remaining places where !funcs->best_encoder is valid

2016-06-07 Thread Boris Brezillon
gly. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/drm_atomic_helper.c | 4 +++- drivers/gpu/drm/drm_fb_helper.c | 13 - include/drm/drm_modeset_helper_vtables.h | 10 -- 3 files changed, 23 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/

[PATCH v2 04/20] drm: exynos: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
We have 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/exy

[PATCH v2 05/20] drm: fsl-dcu: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_rgb.c

[PATCH v2 07/20] drm: mediatek: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementation and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/mediatek/mtk_ds

[PATCH v2 06/20] drm: i915: Rely on the default ->best_encoder() behavior where appropriate

2016-06-07 Thread Boris Brezillon
For all outputs except dp_mst, we have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementation and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezil

[PATCH v2 00/20] drm/atomic: Provide default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
e applying the associated changes. Note that once patch 1 is applied, the other patches can be applied independently. Best Regards, Boris Changes since v1: - remove useless ->encoder backpointers in some implementations - documented the default behavior in the vtable doc - added R-b/A-b tags

[PATCH v2 03/20] drm: atmel-hlcdc: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_outpu

[PATCH v2 08/20] drm: msm: Rely on the default ->best_encoder() behavior where appropriate

2016-06-07 Thread Boris Brezillon
For all outputs except DSI we have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/

[PATCH v2 09/20] drm: rcar-du: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
All outputs have a 1:1 relationship between connectors and encoders, and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon Reviewed-by: Laur

[PATCH v2 11/20] drm: sti: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
All outputs have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/

[PATCH v2 15/20] drm: virtgpu: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
The virtgpu output exposes a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementation and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/

[PATCH v2 12/20] drm: sun4i: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
All outputs have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/su

[PATCH v2 19/20] drm/bridge: ps8622: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders, and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder(), and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/bridge/parade-ps8622.c |

[PATCH v2 17/20] drm/bridge: anx78xx: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders, and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder(), and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/bridge/analogix-anx78xx.c

[PATCH v2 14/20] drm: vc4: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
All outputs have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/

[PATCH v2 16/20] drm: omap: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementation and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon Reviewed-by: Laurent Pinch

[PATCH v2 20/20] drm/bridge: dw-hdmi: Use drm_atomic_helper_best_encoder()

2016-06-07 Thread Boris Brezillon
hen ->best_encoder() is NULL is only available when the DRM device is using the atomic helpers, and this bridge is compatible with non-atomic and atomic devices. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/bridge/dw-hdmi.c | 11 +-- 1 file changed, 1 insertion(+), 10 deletions(-)

[PATCH v2 18/20] drm/bridge: ptn3460: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders, and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder(), and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/bridge/nxp-ptn3460.c

[PATCH v2 10/20] drm: rockchip: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
All outputs have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon Acked-by: Mark

[PATCH v2 13/20] drm: tegra: Rely on the default ->best_encoder() behavior

2016-06-07 Thread Boris Brezillon
All outputs have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementation and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/te

Re: [PATCH 09/20] drm: rcar-du: Rely on the default ->best_encoder() behavior

2016-06-03 Thread Boris Brezillon
On Thu, 02 Jun 2016 23:57:14 +0300 Laurent Pinchart wrote: > Hi Boris, > > Thank you for the patch. > > On Thursday 02 Jun 2016 16:31:36 Boris Brezillon wrote: > > All outputs have a 1:1 relationship between connectors and encoders, > > and the driver is relying on

Re: [PATCH 01/20] drm/atomic: Fix remaining places where !funcs->best_encoder is valid

2016-06-03 Thread Boris Brezillon
On Thu, 2 Jun 2016 23:57:02 +0200 Daniel Vetter wrote: > On Thu, Jun 2, 2016 at 11:05 PM, Laurent Pinchart > wrote: > > Hi Boris, > > > > Thank you for the patch. > > > > On Thursday 02 Jun 2016 16:31:28 Boris Brezillon wrote: > >> Adapt drm_p

Re: [PATCH 01/20] drm/atomic: Fix remaining places where !funcs->best_encoder is valid

2016-06-03 Thread Boris Brezillon
On Fri, 03 Jun 2016 00:05:46 +0300 Laurent Pinchart wrote: > Hi Boris, > > Thank you for the patch. > > On Thursday 02 Jun 2016 16:31:28 Boris Brezillon wrote: > > Adapt drm_pick_crtcs() and update_connector_routing() to fallback to > > drm_atomic_helper_best_encod

[PATCH 05/20] drm: fsl-dcu: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_rgb.c

[PATCH 04/20] drm: exynos: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
We have 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/exy

[PATCH 01/20] drm/atomic: Fix remaining places where !funcs->best_encoder is valid

2016-06-02 Thread Boris Brezillon
Adapt drm_pick_crtcs() and update_connector_routing() to fallback to drm_atomic_helper_best_encoder() if funcs->best_encoder() is NULL so that DRM drivers can leave this hook unassigned if they know they want to use drm_atomic_helper_best_encoder(). Signed-off-by: Boris Brezillon --- driv

[PATCH 17/20] drm/bridge: anx78xx: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders, and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder(), and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/bridge/analogix-anx78xx.c

[PATCH 09/20] drm: rcar-du: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
All outputs have a 1:1 relationship between connectors and encoders, and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/rcar

[PATCH 08/20] drm: msm: Rely on the default ->best_encoder() behavior where appropriate

2016-06-02 Thread Boris Brezillon
For all outputs except DSI we have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/

[PATCH 06/20] drm: i915: Rely on the default ->best_encoder() behavior where appropriate

2016-06-02 Thread Boris Brezillon
For all outputs except dp_mst, we have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementation and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezil

[PATCH 10/20] drm: rockchip: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
All outputss have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/

[PATCH 07/20] drm: mediatek: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementation and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/mediatek/mtk_ds

[PATCH 11/20] drm: sti: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
All outputs have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/

[PATCH 12/20] drm: sun4i: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
All outputs have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/su

[PATCH 13/20] drm: tegra: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
All outputs have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementation and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/te

[PATCH 03/20] drm: atmel-hlcdc: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_outpu

[PATCH 16/20] drm: omap: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementation and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/omap

[PATCH 19/20] drm/bridge: ps8622: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders, and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder(), and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/bridge/parade-ps8622.c |

[PATCH 15/20] drm: virtgpu: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
The virtgpu output exposes a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementation and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/

[PATCH 14/20] drm: vc4: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
All outputs have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder() implementations and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/

[PATCH 18/20] drm/bridge: ptn3460: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders, and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder(), and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/bridge/nxp-ptn3460.c

[PATCH 20/20] drm/bridge: dw-hdmi: Use drm_atomic_helper_best_encoder()

2016-06-02 Thread Boris Brezillon
hen ->best_encoder() is NULL is only available when the DRM device is using the atomic helpers, and this bridge is compatible with non-atomic and atomic devices. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/bridge/dw-hdmi.c | 11 +-- 1 file changed, 1 insertion(+), 10 deletions(-)

[PATCH 02/20] drm: arc: Rely on the default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
We have a 1:1 relationship between connectors and encoders and the driver is relying on the atomic helpers: we can drop the custom ->best_encoder(), and let the core call drm_atomic_helper_best_encoder() for us. Signed-off-by: Boris Brezillon --- drivers/gpu/drm/arc/arcpgu_hdmi.c |

[PATCH 00/20] drm/atomic: Provide default ->best_encoder() behavior

2016-06-02 Thread Boris Brezillon
e applying the associated changes. Note that once patch 1 is applied, the other patches can be applied independently. Best Regards, Boris Boris Brezillon (20): drm/atomic: Fix remaining places where !funcs->best_encoder is valid drm: arc: Rely on the default ->best_encoder() behavior

Re: [PATCH 0/3 v5] drm: Introduce drm_connector_register_all() helper

2016-03-29 Thread Boris Brezillon
hes > * Updated kerneldoc descriptins of both register_ and unregister_all() > * Updated commit messages (mostly spellos and grammar issues) > > Changes v1 -> v2: > * Rename drm_connector_unplug_all() to drm_connector_unregister_all() > * Use drm_for_each_connector() ins

Re: [PATCH 1/4 v3] drm: Rename drm_connector_unplug_all() to drm_connector_unregister_all()

2016-03-23 Thread Boris Brezillon
y. > > And while at it remove potentially dangerous locking around > drm_connector_unregister() in rcar_du_remove() as mentioned > in kerneldoc for drm_connector_unregister_all(). > > Signed-off-by: Alexey Brodkin > Cc: Daniel Vetter > Cc: David Airlie > Cc: Boris Brez

Re: [PATCH 11/16] drm/atmel-hldcd: removed optional dummy crtc mode_fixup function.

2016-03-04 Thread Boris Brezillon
t; > I can fix the typos, but I'm meh on the whitespace change ;-) Imo that > doesn't need a resend. Oops, I forgot to answer to that one. I'm fine with the extra space removal too, you can apply it. Thanks, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com

Re: [PATCH 11/16] drm/atmel-hldcd: removed optional dummy crtc mode_fixup function.

2016-02-16 Thread Boris Brezillon
empty line after atmel_hlcdc_crtc_create() definition (which is correct, but I'm not sure it should be part of the same patch). Otherwise it looks good to me. Once you've fixed those 2 things, you can add my Acked-by: Boris Brezillon > > Signed-off-by: Carlos Palminha >