On Tue 2022-11-22 15:39:35, Shuah Khan wrote: > On 11/22/22 14:05, Pavel Machek wrote: > > Hi! > > > > > wm831x_status_probe() accesses status from chip_pdata before > > > initializing it. Fix it. > > > > > > Signed-off-by: Shuah Khan <skhan@xxxxxxxxxxxxxxxxxxx> > > > > Does it? ARRAY_SIZE() will be compile-time constant, no? > > > > What is the bug? Did you test the code? > > > > Is ARRAY_SIZE() safe when accessing the status chip_pdata->status? > I wasn't sure. If so, this change isn't necessary. I think so. Feel free to quite C standard to prove me wrong :-). Best regards, Pavel -- People of Russia, stop Putin before his war on Ukraine escalates.
Attachment:
signature.asc
Description: PGP signature
