On Sun, Jun 01, 2025 at 07:23:34PM -0400, Sasha Levin wrote:
From: Jonas Karlman jonas@kwiboo.se
[ Upstream commit d5e0aa61470c48ddc04d433a00e79cef8716377a ]
Use bytesperline and buffer height to calculate the strides configured.
This does not really change anything other than ensuring the bytesperline that is signaled to userspace matches what is configured in HW.
Signed-off-by: Jonas Karlman jonas@kwiboo.se Reviewed-by: Nicolas Dufresne nicolas.dufresne@collabora.com Tested-by: Nicolas Dufresne nicolas.dufresne@collabora.com Tested-by: Christopher Obbard chris.obbard@collabora.com Signed-off-by: Nicolas Dufresne nicolas.dufresne@collabora.com Signed-off-by: Hans Verkuil hverkuil@xs4all.nl Signed-off-by: Sasha Levin sashal@kernel.org
### Key Reasons for NO Backport: 1. **Not a Bug Fix**: The commit message explicitly states "This does not really change anything other than ensuring the bytesperline that is signaled to userspace matches what is configured in HW." This is not fixing a functional bug but rather improving consistency between userspace and hardware configuration.
As the bot said "NO", why was this picked up?
thanks,
greg k-h