30 Dec
2020
30 Dec
'20
12:56 p.m.
On Wed 2020-12-30 14:44:41, Laurent Pinchart wrote:
On Wed, Dec 30, 2020 at 02:32:46PM +0200, Andy Shevchenko wrote:
On Wed, Dec 30, 2020 at 2:25 PM Pavel Machek wrote:
commit a86cf9b29e8b12811cf53c4970eefe0c1d290476 upstream.
Validate media bus code, width and height when setting the subdev format.
This effectively reworks how setting subdev format is implemented in the driver.
Something is wrong here:
fmt->format.code = formats[0].mbus_code;for (i = 0; i < ARRAY_SIZE(formats); i++) {Looks like 'i = 1' should be...
if (formats[i].mbus_code == fmt->format.code) {More likely
if (formats[i].mbus_code == mbus_code) {I think.
That looks reasonable, but I don't have hardware to test.
Pavel, would you like to submit a patch ?
Done, should be in your inbox.
Best regards, Pavel
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany