[{"id":1769905,"web_url":"http://patchwork.ozlabs.org/comment/1769905/","msgid":"<20170918072906.52u5xcpbpv45xxyb@flea>","list_archive_url":null,"date":"2017-09-18T07:29:06","subject":"Re: [U-Boot] [PATCH v4 2/3] video: add an option for video simplefb\n\tvia DT","submitter":{"id":12916,"url":"http://patchwork.ozlabs.org/api/people/12916/","name":"Maxime Ripard","email":"maxime.ripard@free-electrons.com"},"content":"Hi,\n\nOn Fri, Sep 15, 2017 at 09:59:20AM +0800, Icenowy Zheng wrote:\n> Add an option to indicate that the video driver should setup a SimpleFB\n> node that passes the video framebuffer initialized by U-Boot to the\n> operating system kernel.\n> \n> Currently only the Allwinner DE2 driver uses this option.\n> \n> Signed-off-by: Icenowy Zheng \n> ---\n> Changes in v4:\n> - Remove the dependency of VIDEO_DT_SIMPLEFB.\n> \n> drivers/video/Kconfig | 8 ++++++++\n> 1 file changed, 8 insertions(+)\n> \n> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig\n> index 082cc4a528..547d4cc0c7 100644\n> --- a/drivers/video/Kconfig\n> +++ b/drivers/video/Kconfig\n> @@ -601,4 +601,12 @@ config VIDEO_DW_HDMI\n> \t rather requires a SoC-specific glue driver to call it), it\n> \t can not be enabled from the configuration menu.\n> \n> +config VIDEO_DT_SIMPLEFB\n> +\tbool \"Enable SimpleFB support for passing framebuffer to OS\"\n> +\thelp\n> +\t Enables the code to pass the framebuffer to the kernel as a\n> +\t simple framebuffer in the device tree.\n> +\t The video output is initialized by U-Boot, and kept by the\n> +\t kernel.\n> +\n\nYou should also remove it from the current users. It will generate a\nwarning otherwise.\n\nMaxime","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwd3261Qwz9ryQ\n\tfor ;\n\tMon, 18 Sep 2017 17:30:54 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 8A555C21F09; Mon, 18 Sep 2017 07:30:14 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 931C4C21F34;\n\tMon, 18 Sep 2017 07:30:08 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid A0D5CC21F09; Mon, 18 Sep 2017 07:29:12 +0000 (UTC)","from mail.free-electrons.com (mail.free-electrons.com [62.4.15.54])\n\tby lists.denx.de (Postfix) with ESMTP id EC5F9C21EFC\n\tfor ; Mon, 18 Sep 2017 07:29:06 +0000 (UTC)","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 5679920839; Mon, 18 Sep 2017 09:29:06 +0200 (CEST)","from localhost (LStLambert-657-1-97-87.w90-63.abo.wanadoo.fr\n\t[90.63.216.87])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id 2C9DB207FE;\n\tMon, 18 Sep 2017 09:29:06 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=none autolearn=unavailable\n\tautolearn_force=no version=3.4.0","Date":"Mon, 18 Sep 2017 09:29:06 +0200","From":"Maxime Ripard ","To":"Icenowy Zheng ","Message-ID":"<20170918072906.52u5xcpbpv45xxyb@flea>","References":"<20170915015921.4659-1-icenowy@aosc.io>\n\t<20170915015921.4659-3-icenowy@aosc.io>","MIME-Version":"1.0","In-Reply-To":"<20170915015921.4659-3-icenowy@aosc.io>","User-Agent":"NeoMutt/20170714 (1.8.3)","Cc":"u-boot@lists.denx.de, Jernej Skrabec ,\n\tJagan Teki , linux-sunxi@googlegroups.com","Subject":"Re: [U-Boot] [PATCH v4 2/3] video: add an option for video simplefb\n\tvia DT","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"multipart/mixed;\n\tboundary=\"===============7576189352860960824==\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1769906,"web_url":"http://patchwork.ozlabs.org/comment/1769906/","msgid":"<14A90207-2C62-48EA-AF30-A031A6DE7241@aosc.io>","list_archive_url":null,"date":"2017-09-18T07:31:54","subject":"Re: [U-Boot] [PATCH v4 2/3] video: add an option for video simplefb\n\tvia DT","submitter":{"id":71295,"url":"http://patchwork.ozlabs.org/api/people/71295/","name":"Icenowy Zheng","email":"icenowy@aosc.io"},"content":"于 2017年9月18日 GMT+08:00 下午3:29:06, Maxime Ripard 写到:\n>Hi,\n>\n>On Fri, Sep 15, 2017 at 09:59:20AM +0800, Icenowy Zheng wrote:\n>> Add an option to indicate that the video driver should setup a\n>SimpleFB\n>> node that passes the video framebuffer initialized by U-Boot to the\n>> operating system kernel.\n>> \n>> Currently only the Allwinner DE2 driver uses this option.\n>> \n>> Signed-off-by: Icenowy Zheng \n>> ---\n>> Changes in v4:\n>> - Remove the dependency of VIDEO_DT_SIMPLEFB.\n>> \n>> drivers/video/Kconfig | 8 ++++++++\n>> 1 file changed, 8 insertions(+)\n>> \n>> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig\n>> index 082cc4a528..547d4cc0c7 100644\n>> --- a/drivers/video/Kconfig\n>> +++ b/drivers/video/Kconfig\n>> @@ -601,4 +601,12 @@ config VIDEO_DW_HDMI\n>> \t rather requires a SoC-specific glue driver to call it), it\n>> \t can not be enabled from the configuration menu.\n>> \n>> +config VIDEO_DT_SIMPLEFB\n>> +\tbool \"Enable SimpleFB support for passing framebuffer to OS\"\n>> +\thelp\n>> +\t Enables the code to pass the framebuffer to the kernel as a\n>> +\t simple framebuffer in the device tree.\n>> +\t The video output is initialized by U-Boot, and kept by the\n>> +\t kernel.\n>> +\n>\n>You should also remove it from the current users. It will generate a\n>warning otherwise.\n\nI think the only current user is DE1.\n\nShould I also convert DE1 to use this option in this patch?\n\n>\n>Maxime","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwd4n3BGtz9ryQ\n\tfor ;\n\tMon, 18 Sep 2017 17:32:25 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid 1CC9BC21F20; Mon, 18 Sep 2017 07:32:17 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id E25A3C21D57;\n\tMon, 18 Sep 2017 07:32:14 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid 8085EC21D57; Mon, 18 Sep 2017 07:32:13 +0000 (UTC)","from elephant.birch.relay.mailchannels.net\n\t(elephant.birch.relay.mailchannels.net [23.83.209.57])\n\tby lists.denx.de (Postfix) with ESMTPS id 6E6FDC21C5C\n\tfor ; Mon, 18 Sep 2017 07:32:12 +0000 (UTC)","from relay.mailchannels.net (localhost [127.0.0.1])\n\tby relay.mailchannels.net (Postfix) with ESMTP id 870F45C0CAA;\n\tMon, 18 Sep 2017 07:32:09 +0000 (UTC)","from hermes.aosc.io (unknown [100.96.140.144])\n\t(Authenticated sender: lmn-TZDUIOWCRQMW)\n\tby relay.mailchannels.net (Postfix) with ESMTPA id DCF195C05E9;\n\tMon, 18 Sep 2017 07:32:08 +0000 (UTC)","from hermes.aosc.io (hermes.aosc.io [172.20.66.218])\n\t(using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384)\n\tby 0.0.0.0:2500 (trex/5.9.14); Mon, 18 Sep 2017 07:32:09 +0000","from localhost (localhost [127.0.0.1]) (Authenticated sender:\n\ticenowy@aosc.io)\n\tby hermes.aosc.io (Postfix) with ESMTPSA id 4104B47660;\n\tMon, 18 Sep 2017 07:32:02 +0000 (UTC)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-0.0 required=5.0 tests=RCVD_IN_DNSWL_NONE,\n\tRCVD_IN_MSPIKE_H3,\n\tRCVD_IN_MSPIKE_WL autolearn=unavailable autolearn_force=no\n\tversion=3.4.0","X-Sender-Id":["lmn-tzduiowcrqmw|x-authsender|icenowy@aosc.io","lmn-tzduiowcrqmw|x-authsender|icenowy@aosc.io"],"X-MC-Relay":"Neutral","X-MailChannels-SenderId":"lmn-tzduiowcrqmw|x-authsender|icenowy@aosc.io","X-MailChannels-Auth-Id":"lmn-TZDUIOWCRQMW","X-Blushing-Macabre":"58963b5f0a232619_1505719929402_399456170","X-MC-Loop-Signature":"1505719929402:3275879540","X-MC-Ingress-Time":"1505719929401","Date":"Mon, 18 Sep 2017 15:31:54 +0800","In-Reply-To":"<20170918072906.52u5xcpbpv45xxyb@flea>","References":"<20170915015921.4659-1-icenowy@aosc.io>\n\t<20170915015921.4659-3-icenowy@aosc.io>\n\t<20170918072906.52u5xcpbpv45xxyb@flea>","MIME-Version":"1.0","To":"Maxime Ripard ","From":"Icenowy Zheng ","Message-ID":"<14A90207-2C62-48EA-AF30-A031A6DE7241@aosc.io>","Cc":"u-boot@lists.denx.de, Jernej Skrabec ,\n\tJagan Teki , linux-sunxi@googlegroups.com","Subject":"Re: [U-Boot] [PATCH v4 2/3] video: add an option for video simplefb\n\tvia DT","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}},{"id":1769965,"web_url":"http://patchwork.ozlabs.org/comment/1769965/","msgid":"<20170918082210.qd5juwds6rl6qczc@flea.lan>","list_archive_url":null,"date":"2017-09-18T08:22:10","subject":"Re: [U-Boot] [PATCH v4 2/3] video: add an option for video simplefb\n\tvia DT","submitter":{"id":12916,"url":"http://patchwork.ozlabs.org/api/people/12916/","name":"Maxime Ripard","email":"maxime.ripard@free-electrons.com"},"content":"On Mon, Sep 18, 2017 at 03:31:54PM +0800, Icenowy Zheng wrote:\n> \n> \n> 于 2017年9月18日 GMT+08:00 下午3:29:06, Maxime Ripard 写到:\n> >Hi,\n> >\n> >On Fri, Sep 15, 2017 at 09:59:20AM +0800, Icenowy Zheng wrote:\n> >> Add an option to indicate that the video driver should setup a\n> >SimpleFB\n> >> node that passes the video framebuffer initialized by U-Boot to the\n> >> operating system kernel.\n> >> \n> >> Currently only the Allwinner DE2 driver uses this option.\n> >> \n> >> Signed-off-by: Icenowy Zheng \n> >> ---\n> >> Changes in v4:\n> >> - Remove the dependency of VIDEO_DT_SIMPLEFB.\n> >> \n> >> drivers/video/Kconfig | 8 ++++++++\n> >> 1 file changed, 8 insertions(+)\n> >> \n> >> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig\n> >> index 082cc4a528..547d4cc0c7 100644\n> >> --- a/drivers/video/Kconfig\n> >> +++ b/drivers/video/Kconfig\n> >> @@ -601,4 +601,12 @@ config VIDEO_DW_HDMI\n> >> \t rather requires a SoC-specific glue driver to call it), it\n> >> \t can not be enabled from the configuration menu.\n> >> \n> >> +config VIDEO_DT_SIMPLEFB\n> >> +\tbool \"Enable SimpleFB support for passing framebuffer to OS\"\n> >> +\thelp\n> >> +\t Enables the code to pass the framebuffer to the kernel as a\n> >> +\t simple framebuffer in the device tree.\n> >> +\t The video output is initialized by U-Boot, and kept by the\n> >> +\t kernel.\n> >> +\n> >\n> >You should also remove it from the current users. It will generate a\n> >warning otherwise.\n> \n> I think the only current user is DE1.\n\nI was talking about configs/sunxi-common.h\n\n> Should I also convert DE1 to use this option in this patch?\n\nThat, or just make it a default to ARCH_SUNXI in that patch.\n\nMaxime","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.denx.de\n\t(client-ip=81.169.180.215; helo=lists.denx.de;\n\tenvelope-from=u-boot-bounces@lists.denx.de;\n\treceiver=)","Received":["from lists.denx.de (dione.denx.de [81.169.180.215])\n\tby ozlabs.org (Postfix) with ESMTP id 3xwfBc198Cz9s7M\n\tfor ;\n\tMon, 18 Sep 2017 18:22:31 +1000 (AEST)","by lists.denx.de (Postfix, from userid 105)\n\tid BCE1AC21DA2; Mon, 18 Sep 2017 08:22:25 +0000 (UTC)","from lists.denx.de (localhost [IPv6:::1])\n\tby lists.denx.de (Postfix) with ESMTP id 5182CC21D57;\n\tMon, 18 Sep 2017 08:22:23 +0000 (UTC)","by lists.denx.de (Postfix, from userid 105)\n\tid B8D72C21D57; Mon, 18 Sep 2017 08:22:21 +0000 (UTC)","from mail.free-electrons.com (mail.free-electrons.com [62.4.15.54])\n\tby lists.denx.de (Postfix) with ESMTP id 681CAC21C5C\n\tfor ; Mon, 18 Sep 2017 08:22:21 +0000 (UTC)","by mail.free-electrons.com (Postfix, from userid 110)\n\tid BC76D20A4E; Mon, 18 Sep 2017 10:22:20 +0200 (CEST)","from localhost (LStLambert-657-1-97-87.w90-63.abo.wanadoo.fr\n\t[90.63.216.87])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id 924D52083E;\n\tMon, 18 Sep 2017 10:22:10 +0200 (CEST)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=0.0 required=5.0 tests=none autolearn=unavailable\n\tautolearn_force=no version=3.4.0","Date":"Mon, 18 Sep 2017 10:22:10 +0200","From":"Maxime Ripard ","To":"Icenowy Zheng ","Message-ID":"<20170918082210.qd5juwds6rl6qczc@flea.lan>","References":"<20170915015921.4659-1-icenowy@aosc.io>\n\t<20170915015921.4659-3-icenowy@aosc.io>\n\t<20170918072906.52u5xcpbpv45xxyb@flea>\n\t<14A90207-2C62-48EA-AF30-A031A6DE7241@aosc.io>","MIME-Version":"1.0","In-Reply-To":"<14A90207-2C62-48EA-AF30-A031A6DE7241@aosc.io>","User-Agent":"NeoMutt/20170714 (1.8.3)","Cc":"u-boot@lists.denx.de, Jernej Skrabec ,\n\tJagan Teki , linux-sunxi@googlegroups.com","Subject":"Re: [U-Boot] [PATCH v4 2/3] video: add an option for video simplefb\n\tvia DT","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.18","Precedence":"list","List-Id":"U-Boot discussion ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"multipart/mixed;\n\tboundary=\"===============5357603963571202177==\"","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" "}}]