Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1453043

Re: [PATCH V3 2/5] dts/imx6q-b850v3: Configure IPU assignment order

From Lucas Stach <l.stach@pengutronix.de>
Newsgroups linux.kernel
Subject Re: [PATCH V3 2/5] dts/imx6q-b850v3: Configure IPU assignment order
Date 2016-08-01 11:00 +0200
Message-ID <s1heN-40C-3@gated-at.bofh.it> (permalink)
References <rEyy5-6IE-3@gated-at.bofh.it> <s153X-4pP-9@gated-at.bofh.it> <s153X-4pP-11@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Am Sonntag, den 31.07.2016, 21:55 +0200 schrieb Peter Senna Tschudin:
> As the IPU has combined limitations across multiple crtcs, and as that
> can't be communicated to userspace at the moment, reorder the crtcs to
> allow support to two Full-HD monitors by avoiding assigning two
> monitors to a single IPU.
> 
> Cc: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> Cc: Philipp Zabel <p.zabel@pengutronix.de>
> Cc: Rob Herring <robh@kernel.org>
> Cc: Fabio Estevam <fabio.estevam@nxp.com>
> Signed-off-by: Peter Senna Tschudin <peter.senna@collabora.com>

NACK. This is a userspace issue. Changing the assignment order of the
CRTCs just shifts the failure to a userspace that want to use CRTC 0 and
2 now.

imx-drm just got atomic support and with the atomic check it should be
possible to inform userspace in a reasonable way about such issues.

Regards,
Lucas

> ---
> Unchanged from V2.
> 
> Changes from V1:
>  - New commit message
> 
>  arch/arm/boot/dts/imx6q-b850v3.dts | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/imx6q-b850v3.dts b/arch/arm/boot/dts/imx6q-b850v3.dts
> index 167f744..88a70de 100644
> --- a/arch/arm/boot/dts/imx6q-b850v3.dts
> +++ b/arch/arm/boot/dts/imx6q-b850v3.dts
> @@ -51,6 +51,11 @@
>  	chosen {
>  		stdout-path = &uart3;
>  	};
> +
> +	display-subsystem {
> +		compatible = "fsl,imx-display-subsystem";
> +		ports = <&ipu1_di0>, <&ipu2_di0>, <&ipu1_di1>, <&ipu2_di1>;
> +	};
>  };
>  
>  &clks {

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH V3 2/5] dts/imx6q-b850v3: Configure IPU assignment order Peter Senna Tschudin <peter.senna@collabora.com> - 2016-07-31 22:00 +0200
  Re: [PATCH V3 2/5] dts/imx6q-b850v3: Configure IPU assignment order Lucas Stach <l.stach@pengutronix.de> - 2016-08-01 11:00 +0200
    Re: [PATCH V3 2/5] dts/imx6q-b850v3: Configure IPU assignment order "Peter Senna Tschudin" <peter.senna@collabora.co.uk> - 2016-08-01 14:40 +0200
      Re: [PATCH V3 2/5] dts/imx6q-b850v3: Configure IPU assignment order Daniel Vetter <daniel@ffwll.ch> - 2016-08-02 15:20 +0200

csiph-web