[U-Boot] [linux-sunxi] Re: [PATCH v3 2/3] video: add an option for video simplefb via DT

[Maxime Ripard]

On Wed, Sep 13, 2017 at 09:17:21PM +0800, icenowy at aosc.io wrote:
> 在 2017-09-13 20:04，Maxime Ripard 写道：
> > Hi,
> > 
> > On Wed, Sep 13, 2017 at 10:17:50AM +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>
> > > ---
> > >  drivers/video/Kconfig | 10 ++++++++++
> > >  1 file changed, 10 insertions(+)
> > > 
> > > diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
> > > index 082cc4a528..e4e71763f5 100644
> > > --- a/drivers/video/Kconfig
> > > +++ b/drivers/video/Kconfig
> > > @@ -601,4 +601,14 @@ 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"
> > > +	depends on VIDEO_DE2
> > > +	default y
> > 
> > SIMPLEFB is also used by other platforms, but most platforms also
> > won't use it.
> > 
> > Adding an imply VIDEO_DT_SIMPLEFB to VIDEO_DE2 would make everyone
> > happy I guess.
> 
> Then should I drop the "depends on VIDEO_DE2"?

Yes.

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/20170913/1d0eda6a/attachment.sig>