On Sat, Jul 18, 2026 at 03:38:52PM +0500, Muhammad Bilal wrote:
On Sat, Jul 18, 2026, Dan Carpenter wrote:
quoted
Looks reasonable. Add a Fixes tag.
You have this hardware and are finding these with testing?
It would be better to move the source delta calculation into
the the sm750_hw_imageblit() since it's just width rounded up
(or rounded down if you want to be wrong I suppose).
Thanks for the review. Sending v2 as a separate message since it
came out as a fuller rework rather than a small diff on v1:
- Added Fixes: 81dee67e215b2 ("staging: sm750fb: add sm750 to
staging"), traced via git blame. The bug has been there,
unchanged, since the original 2015 staging import.
- No, I don't have this hardware. This was found by auditing
sm750_hw_imageblit()'s internal stride calculation against what
its caller was passing in, and confirmed with a clean "-Werror"
build. No runtime testing on real sm750 silicon.
You really need to put this into the commit message.
regards,
dan carpenter