diff mbox series

arm: Enable VIDEO_BPP32 on pinebook.

Message ID 20200118111516.2869-1-vagrant@debian.org
State New
Headers show
Series arm: Enable VIDEO_BPP32 on pinebook. | expand

Commit Message

Vagrant Cascadian Jan. 18, 2020, 11:15 a.m. UTC
Video output on the pinebook LCD screen was broken by:

commit 2cc393f32fd9 ("video: make BPP and ANSI configs optional").

Enable VIDEO_BPP32 which was previously enabled by default when
DM_VIDEO was set.

Signed-off-by: Vagrant Cascadian <vagrant at debian.org>
---

 configs/pinebook_defconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Anatolij Gustschin Jan. 18, 2020, 11:27 a.m. UTC | #1
On Sat, 18 Jan 2020 03:15:15 -0800
Vagrant Cascadian vagrant at debian.org wrote:
...
>  configs/pinebook_defconfig | 1 +
>  1 file changed, 1 insertion(+)

Applied to u-boot-video/master, thanks!

--
Anatolij
Maxime Ripard Jan. 18, 2020, 11:29 a.m. UTC | #2
Hi,

On Sat, Jan 18, 2020 at 03:15:15AM -0800, Vagrant Cascadian wrote:
> Video output on the pinebook LCD screen was broken by:
>
> commit 2cc393f32fd9 ("video: make BPP and ANSI configs optional").
>
> Enable VIDEO_BPP32 which was previously enabled by default when
> DM_VIDEO was set.
>
> Signed-off-by: Vagrant Cascadian <vagrant at debian.org>

There's nothing really specific about the pinebook here, but it's
needed for pretty much all the boards using DM_VIDEO (on Allwinner at
least).

You should add a kconfig select / default instead

Maxime
Vagrant Cascadian Jan. 18, 2020, 8:03 p.m. UTC | #3
On 2020-01-18, Maxime Ripard wrote:
> On Sat, Jan 18, 2020 at 03:15:15AM -0800, Vagrant Cascadian wrote:
>> Video output on the pinebook LCD screen was broken by:
>>
>> commit 2cc393f32fd9 ("video: make BPP and ANSI configs optional").
>>
>> Enable VIDEO_BPP32 which was previously enabled by default when
>> DM_VIDEO was set.
>>
>> Signed-off-by: Vagrant Cascadian <vagrant at debian.org>
>
> There's nothing really specific about the pinebook here, but it's
> needed for pretty much all the boards using DM_VIDEO (on Allwinner at
> least).
>
> You should add a kconfig select / default instead

That would basically revert 2cc393f32fd9, and I figured there was a
reason for it...

It wouldn't surprise me that other systems are affected, but I only
notice this issue on the pinebook (most of the systems I use are
headless), where it definitely needed to be fixed somehow.

If there's a correct and more general fix, please propose it!


live well,
  vagrant
Tom Rini Jan. 18, 2020, 10 p.m. UTC | #4
On Sat, Jan 18, 2020 at 12:03:02PM -0800, Vagrant Cascadian wrote:
> On 2020-01-18, Maxime Ripard wrote:
> > On Sat, Jan 18, 2020 at 03:15:15AM -0800, Vagrant Cascadian wrote:
> >> Video output on the pinebook LCD screen was broken by:
> >>
> >> commit 2cc393f32fd9 ("video: make BPP and ANSI configs optional").
> >>
> >> Enable VIDEO_BPP32 which was previously enabled by default when
> >> DM_VIDEO was set.
> >>
> >> Signed-off-by: Vagrant Cascadian <vagrant at debian.org>
> >
> > There's nothing really specific about the pinebook here, but it's
> > needed for pretty much all the boards using DM_VIDEO (on Allwinner at
> > least).
> >
> > You should add a kconfig select / default instead
> 
> That would basically revert 2cc393f32fd9, and I figured there was a
> reason for it...
> 
> It wouldn't surprise me that other systems are affected, but I only
> notice this issue on the pinebook (most of the systems I use are
> headless), where it definitely needed to be fixed somehow.
> 
> If there's a correct and more general fix, please propose it!

Well, looking at 2cc393f32fd9 there are a number of platforms that
enable more than one mode.  But maybe we should always have at least one
mode?
Maxime Ripard Jan. 22, 2020, 5:43 p.m. UTC | #5
On Sat, Jan 18, 2020 at 05:00:52PM -0500, Tom Rini wrote:
> On Sat, Jan 18, 2020 at 12:03:02PM -0800, Vagrant Cascadian wrote:
> > On 2020-01-18, Maxime Ripard wrote:
> > > On Sat, Jan 18, 2020 at 03:15:15AM -0800, Vagrant Cascadian wrote:
> > >> Video output on the pinebook LCD screen was broken by:
> > >>
> > >> commit 2cc393f32fd9 ("video: make BPP and ANSI configs optional").
> > >>
> > >> Enable VIDEO_BPP32 which was previously enabled by default when
> > >> DM_VIDEO was set.
> > >>
> > >> Signed-off-by: Vagrant Cascadian <vagrant at debian.org>
> > >
> > > There's nothing really specific about the pinebook here, but it's
> > > needed for pretty much all the boards using DM_VIDEO (on Allwinner at
> > > least).
> > >
> > > You should add a kconfig select / default instead
> >
> > That would basically revert 2cc393f32fd9, and I figured there was a
> > reason for it...
> >
> > It wouldn't surprise me that other systems are affected, but I only
> > notice this issue on the pinebook (most of the systems I use are
> > headless), where it definitely needed to be fixed somehow.
> >
> > If there's a correct and more general fix, please propose it!
>
> Well, looking at 2cc393f32fd9 there are a number of platforms that
> enable more than one mode.  But maybe we should always have at least one
> mode?

Yeah, I guess that would make the most sense. My suggestions was
actually to enable it platform by platform, which wouldn't be a revert
and would fix all the sunxi boards at once.

Maxime
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200122/02e543a8/attachment.sig>
diff mbox series

Patch

diff --git a/configs/pinebook_defconfig b/configs/pinebook_defconfig
index 929434e25a..306a6bc6b9 100644
--- a/configs/pinebook_defconfig
+++ b/configs/pinebook_defconfig
@@ -22,3 +22,4 @@  CONFIG_SYS_USB_EVENT_POLL_VIA_INT_QUEUE=y
 # CONFIG_USB_GADGET is not set
 CONFIG_VIDEO_BRIDGE=y
 CONFIG_VIDEO_BRIDGE_ANALOGIX_ANX6345=y
+CONFIG_VIDEO_BPP32=y