在 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.ripard@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 icenowy@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?
Maxime
-- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com