From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anatolij Gustschin Date: Fri, 3 Jan 2020 17:16:49 +0100 Subject: [PATCH 1/5] video: x86: Enable 32-bit graphics by default In-Reply-To: References: <20191221011037.143972-1-sjg@chromium.org> <20191221011037.143972-2-sjg@chromium.org> <20200102153306.59fd28f9@crub> Message-ID: <20200103171649.714dc921@crub> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi all, On Fri, 3 Jan 2020 23:41:49 +0800 Bin Meng bmeng.cn at gmail.com wrote: > Hi Simon, > > On Fri, Jan 3, 2020 at 11:24 AM Simon Glass wrote: > > > > Hi Bin, > > > > On Thu, 2 Jan 2020 at 19:41, Bin Meng wrote: > > > > > > Hi Simon, > > > > > > On Fri, Jan 3, 2020 at 10:30 AM Simon Glass wrote: > > > > > > > > Hi Anatolij, > > > > > > > > On Thu, 2 Jan 2020 at 07:34, Anatolij Gustschin wrote: > > > > > > > > > > Hi Simon, > > > > > > > > > > On Fri, 20 Dec 2019 18:10:33 -0700 > > > > > Simon Glass sjg at chromium.org wrote: > > > > > > > > > > > Most x86 boards that use video make use of 32bpp graphics. Enable this by > > > > > > default. This fixes missing graphics output on some x86 boards. > > > > > > > > > > Must this patch be applied for v2010.01 release? > > > > > > > > I think so since these boards don't work for me at present. > > > > > > > > What do you think, Bin? > > > > > > > > > > Which board is currently broken due to missing this config? Should it > > > be safer to move this to board defconfig? > > > > I think any x86 board with graphics would need this. Those that don't > > enable VIDEO won't get the change anyway. > > OK, thanks. I had a test with QEMU x86 and looks good. > > Reviewed-by: Bin Meng > Tested-by: Bin Meng Thanks for testing. I've submitted a pull request for this series and Tom already merged the patches in master. -- Anatolij