@TCH:
Én azt értem, hogy ez itt nem úgy volt, de én általánosságban beszéltem, hogy attól, hogy az érték valid, még nem biztos, hogy jó is, mert 1-től 4095-ig minden is valid, de csak az jó, ami a valóságnak megfelel. Ha pl. el van szaródva az a flash (vagy akármi), ahonnan ezeket olvassa, akkor akár minden körben is jöhetnek random adatok és akkor egyik körben jó lesz a felbontás a drivernek, másik körben nem, te meg tépheted a hajad, hogy miért nem jó, ami jó volt, hiszen ellenőrizted. A konkrét érték önmagában is információ. Két konkrét érték pedig még több, pl. ha a flash halott és random biteket büfög vissza, az már a második tesztnél kiderül.

dev_err(dev, "Unsupported touchscreen parameters: %*ph\n",
        NVT_TS_PARAMS_SIZE, data->buf);

@TCH:
De nem is azt mondtam, hogy ezt így be kell commitolni, csak azt, hogy felesleges ugyanazt a kódot duplikálni és az egyik ágban kiszedni a wake ellenőrzését, ha az a feltétel, ami alapján kiválasztja a lefuttatandó ágat, az belerakható az érintett - és a feltétel nem teljesülése esetén kiszűrendő - ellenőrzésbe.