| Message ID | 20170908075016.18657-2-wens@csie.org (mailing list archive) |
|---|---|
| State | Mainlined, archived |
| Headers | show |
Hi, On Fri, Sep 08, 2017 at 03:50:09PM +0800, Chen-Yu Tsai wrote: > When the second display pipeline device nodes for the A31/A31s were > added, it was not known that the TCONs could (through either DRCs) > select either backend as their input. Thus in the endpoints connecting > these components together, the endpoint IDs were set to 0, while in > fact they should have been set to 1. > > Fixes: 9a26882a7378 ("ARM: dts: sun6i: Add second display pipeline device > nodes") > Signed-off-by: Chen-Yu Tsai <wens@csie.org> Should we CC stable on this one? Thanks! Maxime
On Fri, Sep 8, 2017 at 9:29 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > Hi, > > On Fri, Sep 08, 2017 at 03:50:09PM +0800, Chen-Yu Tsai wrote: >> When the second display pipeline device nodes for the A31/A31s were >> added, it was not known that the TCONs could (through either DRCs) >> select either backend as their input. Thus in the endpoints connecting >> these components together, the endpoint IDs were set to 0, while in >> fact they should have been set to 1. >> >> Fixes: 9a26882a7378 ("ARM: dts: sun6i: Add second display pipeline device >> nodes") >> Signed-off-by: Chen-Yu Tsai <wens@csie.org> > > Should we CC stable on this one? It wouldn't matter for old kernels, but I suppose we should get it right everywhere, so we should probably CC stable. Also I forgot to mention that this patch should go in -fixes, while the rest can go in -next. Thanks ChenYu
diff --git a/arch/arm/boot/dts/sun6i-a31.dtsi b/arch/arm/boot/dts/sun6i-a31.dtsi index f3d74dc5b292..00a4c7614e0a 100644 --- a/arch/arm/boot/dts/sun6i-a31.dtsi +++ b/arch/arm/boot/dts/sun6i-a31.dtsi @@ -311,8 +311,8 @@ #size-cells = <0>; reg = <0>; - tcon1_in_drc1: endpoint@0 { - reg = <0>; + tcon1_in_drc1: endpoint@1 { + reg = <1>; remote-endpoint = <&drc1_out_tcon1>; }; }; @@ -1038,8 +1038,8 @@ #size-cells = <0>; reg = <1>; - be1_out_drc1: endpoint@0 { - reg = <0>; + be1_out_drc1: endpoint@1 { + reg = <1>; remote-endpoint = <&drc1_in_be1>; }; }; @@ -1068,8 +1068,8 @@ #size-cells = <0>; reg = <0>; - drc1_in_be1: endpoint@0 { - reg = <0>; + drc1_in_be1: endpoint@1 { + reg = <1>; remote-endpoint = <&be1_out_drc1>; }; }; @@ -1079,8 +1079,8 @@ #size-cells = <0>; reg = <1>; - drc1_out_tcon1: endpoint@0 { - reg = <0>; + drc1_out_tcon1: endpoint@1 { + reg = <1>; remote-endpoint = <&tcon1_in_drc1>; }; };
When the second display pipeline device nodes for the A31/A31s were added, it was not known that the TCONs could (through either DRCs) select either backend as their input. Thus in the endpoints connecting these components together, the endpoint IDs were set to 0, while in fact they should have been set to 1. Fixes: 9a26882a7378 ("ARM: dts: sun6i: Add second display pipeline device nodes") Signed-off-by: Chen-Yu Tsai <wens@csie.org> --- arch/arm/boot/dts/sun6i-a31.dtsi | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-)