Re: [OSS-Tools] [PATCH platsch V5 5/5] Add spinner executable for boot animation and text show

[Philipp Zabel <p.zabel@pengutronix.de>]

On So, 2024-06-23 at 11:52 +0000, LI Qingwu wrote:
> 
> > -----Original Message-----
> > From: Philipp Zabel <p.zabel@pengutronix.de>
> > Sent: Thursday, June 20, 2024 8:47 PM
> > To: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn>;
> > oss-tools@pengutronix.de; m.felsch@pengutronix.de
> > Cc: GEO-CHHER-bsp-development
> > <bsp-development.geo@leica-geosystems.com>
> > Subject: Re: [OSS-Tools] [PATCH platsch V5 5/5] Add spinner executable for boot
> > animation and text show
> > 
> > [你通常不会收到来自 p.zabel@pengutronix.de 的电子邮件。请访问
> > https://aka.ms/LearnAboutSenderIdentification，以了解这一点为什么很重要]
> > 
> > This email is not from Hexagon’s Office 365 instance. Please be careful while
> > clicking links, opening attachments, or replying to this email.
> > 
> > 
> > On Mi, 2024-06-19 at 12:22 +0200, LI Qingwu wrote:
> > > This commit introduces a new executable, spinner,
> > >     which supports two types of animations for boot sequences:
> > >     1 static PNG and text support.
> > >     2 rotates square PNG images per frame
> > >     3 shows a sequence of square images from a strip of PNG images.
> > 
> > I've tried this and the CPU load was a bit unexpected.
> > 
> > I suggest clipping the background_surface->cr blits in
> > on_draw_..._animation() and and the drawing_surface->disp_cr blit in the
> > drawing loop to the minimum rectangle necessary, commented below.
> 
> I just create a drawing_surface same size as the animation symbol instead of display,
> And 20fps with 800*600 display, CPU load of rotate animation goes from 16.6% to 4.3%
> And sequence animation goes from 12.3% to 0.7%

Yes, that's even better.

regards
Philipp