Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SVG logo #266

Merged
merged 4 commits into from
Feb 14, 2022
Merged

SVG logo #266

merged 4 commits into from
Feb 14, 2022

Conversation

MaxKellermann
Copy link
Collaborator

For now, this is just for u-boot, but I'll integrate this in psplash as well, and of course in the new XCSoar-based graphical menu.

New look:
new

Old look:
old

Thanks @MacflySF27 and his daughter.

This reduces the BMP files from more than 1 MB to just 50 kB, and
speeds up u-boot by 100ms.
@MaxKellermann
Copy link
Collaborator Author

I added two more patches; the first implements BMP-RLE support for the Cubieboard 2 (which has a 32 bit display, and u-boot implements BMP-RLE only for 16 bit displays); the second patch enables RLE compression, reducing the /boot partition (and the image size) by several megabytes.

@mihu-ov
Copy link
Member

mihu-ov commented Feb 10, 2022

Looks good! Still the logo we are used to but with a refreshed look.

@MaxKellermann
Copy link
Collaborator Author

The look previously was rather ugly; not because the logo design was bad, but because it was a scaled bitmap, and thus was very blurry. Now all images get renderered at their native resolution.

(And, of course, all images get rendered automatically during the build. Nobody needs to do that manually. This PR deletes 56 logo files which were previously managed manually!)

@linuxianer99
Copy link
Member

@kedder Ok to merrge ?

From fdad291b7837db232a9a8d071d8d587d8777a485 Mon Sep 17 00:00:00 2001
From: Max Kellermann <[email protected]>
Date: Wed, 9 Feb 2022 17:55:27 +0100
Subject: [PATCH] video_bmp: implement BMP-RLE to 32 bit
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you planning to submit this upstream? It might be a PITA to fix this large patch once it stops applying cleanly.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, though upstream is at a newer version, Yocto is outdated. And I need to clean up the patch a bit, it's somewhat quick'n'dirty code.
(And upstream is a confusing mess of dozens of per-vendor copies....)

@linuxianer99 linuxianer99 merged commit 3fbfc2d into Openvario:master Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants