[U-Boot] [PATCH v4 2/3] video: add an option for video simplefb via DT

[Maxime Ripard]

On Mon, Sep 18, 2017 at 03:31:54PM +0800, Icenowy Zheng wrote:
> 
> 
> 于 2017年9月18日 GMT+08:00 下午3:29:06, Maxime Ripard <maxime.ripard at free-electrons.com> 写到:
> >Hi,
> >
> >On Fri, Sep 15, 2017 at 09:59:20AM +0800, Icenowy Zheng wrote:
> >> Add an option to indicate that the video driver should setup a
> >SimpleFB
> >> node that passes the video framebuffer initialized by U-Boot to the
> >> operating system kernel.
> >> 
> >> Currently only the Allwinner DE2 driver uses this option.
> >> 
> >> Signed-off-by: Icenowy Zheng <icenowy at aosc.io>
> >> ---
> >> Changes in v4:
> >> - Remove the dependency of VIDEO_DT_SIMPLEFB.
> >> 
> >>  drivers/video/Kconfig | 8 ++++++++
> >>  1 file changed, 8 insertions(+)
> >> 
> >> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
> >> index 082cc4a528..547d4cc0c7 100644
> >> --- a/drivers/video/Kconfig
> >> +++ b/drivers/video/Kconfig
> >> @@ -601,4 +601,12 @@ config VIDEO_DW_HDMI
> >>  	  rather requires a SoC-specific glue driver to call it), it
> >>  	  can not be enabled from the configuration menu.
> >>  
> >> +config VIDEO_DT_SIMPLEFB
> >> +	bool "Enable SimpleFB support for passing framebuffer to OS"
> >> +	help
> >> +	  Enables the code to pass the framebuffer to the kernel as a
> >> +	  simple framebuffer in the device tree.
> >> +	  The video output is initialized by U-Boot, and kept by the
> >> +	  kernel.
> >> +
> >
> >You should also remove it from the current users. It will generate a
> >warning otherwise.
> 
> I think the only current user is DE1.

I was talking about configs/sunxi-common.h

> Should I also convert DE1 to use this option in this patch?

That, or just make it a default to ARCH_SUNXI in that patch.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170918/b615068e/attachment.sig>