Re: [PATCH] staging: fbtft: improve TODO comment clarity

From: Andy Shevchenko

Date: Mon Apr 13 2026 - 03:28:17 EST


On Sat, Apr 11, 2026 at 6:26 PM Chandra Mouli Baskaran
<bcmouli2006@xxxxxxxxx> wrote:
>
> From: Chandra Mouli <bcmouli2006@xxxxxxxxx>
>
> Clarify the TODO comment in fbtft_ops_damage_range() to
> better describe that the current implementation updates
> the entire displayinstead of only the modified region.

Please, spellcheck the commit messages...

There was an attempt to improve that. Do you have HW to test?


> - /* TODO: only mark changed area update all for now */
> + /* TODO: Optimize to update only the modified region instead
> + * of refreshing the entire display.
> + */

/*
* The style of multi-line comment is still wrong. You
* may use this example.
*/

And also note:

https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/tree/drivers/staging/fbtft/README?h=staging-next

--
With Best Regards,
Andy Shevchenko