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