Re: [PATCH v3 2/5] ARM: dts: imx27: Add imx framebuffer device

[Alexander Shiyan]

> Signed-off-by: Markus Pargmann <mpa at pengutronix.de>
> ---
>  arch/arm/boot/dts/imx27.dtsi | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
> index 8603886..10dcbd3 100644
> --- a/arch/arm/boot/dts/imx27.dtsi
> +++ b/arch/arm/boot/dts/imx27.dtsi
> @@ -342,6 +342,15 @@
>  			reg = <0x10020000 0x20000>;
>  			ranges;
>  
> +			imxfb: fb at 10021000 {

If it can be fixed while applying, it would be right to use just "fb" here.
We now use a lot of devices without "imx"-prefix, so I do not see any need to use it here.

> +				compatible = "fsl,imx27-fb", "fsl,imx21-fb";
> +				interrupts = <61>;
> +				reg = <0x10021000 0x1000>;
> +				clocks = <&clks 36>, <&clks 65>, <&clks 59>;
> +				clock-names = "ipg", "ahb", "per";
> +				status = "disabled";
> +			};
> +

---