[coreboot-gerrit] Change in coreboot[master]: mb/google/poppy: Fix race condition in acpi camera_pmic

[Ricardo Ribalda (Code Review)]

Ricardo Ribalda has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48631 )

Change subject: mb/google/poppy: Fix race condition in acpi camera_pmic
......................................................................

Patch Set 3:

(4 comments)

https://review.coreboot.org/c/coreboot/+/48631/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/48631/2//COMMIT_MSG@9 
PS2, Line 9: Newer kernels can re-schedule new acpi command calls during a Sleep().
> Is there a pointer to this change?

I would love to blame this:

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?i...

but it might be too old...

https://review.coreboot.org/c/coreboot/+/48631/2//COMMIT_MSG@9 
PS2, Line 9: Newer kernels can re-schedule new acpi command calls during a Sleep().
> Is there a pointer to this change?

Done

https://review.coreboot.org/c/coreboot/+/48631/2//COMMIT_MSG@28 
PS2, Line 28: c
> b

Done

https://review.coreboot.org/c/coreboot/+/48631/2//COMMIT_MSG@29 
PS2, Line 29: TEST=while true; do if ssh $DUT "dmesg | grep "failed to find sensor" "; then  break;  fi; ssh $DUT reboot; sleep 30 ; done
> BRANCH=poppy

Done