在 2017-09-22 19:57，Maxime Ripard 写道：
On Fri, Sep 22, 2017 at 06:41:40AM +0000, icen...@aosc.io wrote:
在 2017-09-22 03:18，Maxime Ripard 写道：
> On Thu, Sep 21, 2017 at 07:48:20AM +0000, Icenowy Zheng wrote:
> >
> >
> > 于 2017年9月21日 GMT+08:00 下午3:40:23, Maxime Ripard
> > <maxime.rip...@free-electrons.com> 写到:
> > >Hi,
> > >
> > >On Wed, Sep 20, 2017 at 04:18:19PM +0000, Icenowy Zheng wrote:
> > >> The sunxi DE1 video option used to be CONFIG_VIDEO, which has the
> > >same
> > >> name as the "Enable legacy video support" option in
> > >> drivers/video/Kconfig.
> > >>
> > >> Change the option name to CONFIG_VIDEO_SUNXI, which is really used by
> > >> Makefile under drivers/video/sunxi/, and defined in sunxi-common.h
> > >> when CONFIG_VIDEO is selected before this change. Now
> > >CONFIG_VIDEO_SUNXI
> > >> selects CONFIG_VIDEO and the usages of CONFIG_VIDEO in sunxi Kconfig
> > >and
> > >> config headers are all converted to use CONFIG_VIDEO_SUNXI.
> > >>
> > >> Signed-off-by: Icenowy Zheng <icen...@aosc.io>
> > >
> > >The patch is fine, but you also seem to imply that it's an issue. It's
> > >not, you can very well define in Kconfig the same symbol in two
> > >different places, and Kconfig will merge the two by itself.
> >
> > So I can drop this patch and simply add "imply VIDEO_DT_SIMPLEFB" in
> > the VIDEO option at sunxi Kconfig?
>
> I don't see anything wrong with this patch, there's no need to change
> it.
So this patchset can be applied if no further rejections?
Yes, I'm just waiting for some feedback from the video people.
I didn't To: or Cc: them...
But I think U-Boot has Patchwork, right?
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
