From mboxrd@z Thu Jan  1 00:00:00 1970
From: Peng Fan <van.freenix@gmail.com>
Date: Thu, 18 Aug 2016 14:19:30 +0800
Subject: [U-Boot] dm video question
In-Reply-To: <CAPnjgZ3_e0OTZoRXKH2V9L2XZosYK-2GtBRgMu2Mf3rBz99VPQ@mail.gmail.com>
References: <20160815100520.GA21874@linux-7smt.suse>
 <CAPnjgZ3_e0OTZoRXKH2V9L2XZosYK-2GtBRgMu2Mf3rBz99VPQ@mail.gmail.com>
Message-ID: <20160818061927.GA25295@linux-7smt.suse>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Simon,

On Tue, Aug 16, 2016 at 10:16:00PM -0600, Simon Glass wrote:
>Hi Peng,
>
>On 15 August 2016 at 04:05, Peng Fan <van.freenix@gmail.com> wrote:
>>
>> Hi Simon,
>>
>> I am trying to coverting mxsfb.c to support dm.
>>
>> But met the following issue.
>> "
>> mxsfb_lcd_bind: Frame buffer size 200400
>> Video device 'lcdif at 021c8000' cannot allocate frame buffer memory -ensure the device is set up before relocation
>> "
>>
>> I add "u-boot, dm-pre-reloc" in dts as the following, but no help.
>>
>
>Can you check that you have that property in each node leading to the
>root also? You can use fdtdump to display the tree.

I did not know that I need to add u-boot,dm-pre-reloc for the nodes
in the path of root -> lcdif.

Now I works. Thanks.

Still have a question.
When enabled DM_VIDEO, do you know how to let the screen show a logo, just like
what cfb_console supports?

Thanks,
PPeng.
>
>Did you add a bind() method to handle the pre-relocaiton setup?
>
>> &lcdif {
>>         pinctrl-names = "default";
>>         pinctrl-0 = <&pinctrl_lcdif_dat
>>                      &pinctrl_lcdif_ctrl
>>                      &pinctrl_lcdif_reset>;
>>         display = <&display0>;
>>         status = "okay";
>>         u-boot,dm-pre-reloc;
>>
>>         display0: display {
>>                 bits-per-pixel = <16>;
>>                 bus-width = <24>;
>>                 u-boot,dm-pre-reloc;
>>                 status = "okay";
>>
>>                 display-timings {
>>                         native-mode = <&timing0>;
>>                         u-boot,dm-pre-reloc;
>>                         timing0: timing0 {
>>                         u-boot,dm-pre-reloc;
>>                         clock-frequency = <9200000>;
>>                         hactive = <480>;
>>                         vactive = <272>;
>>                         hfront-porch = <8>;
>>                         hback-porch = <4>;
>>                         hsync-len = <41>;
>>                         vback-porch = <2>;
>>                         vfront-porch = <4>;
>>                         vsync-len = <10>;
>>
>>                         hsync-active = <0>;
>>                         vsync-active = <0>;
>>                         de-active = <1>;
>>                         pixelclk-active = <0>;
>>                         };
>>                 };
>>         };
>> };
>
>Regards,
>Simon