[M] Change in coreboot[master]: chromeos/cr50_enable_update.c: Clear EC AP_IDLE flag

[Subrata Banik (Code Review)]

Attention is currently required from: Tarun Tuli, Derek Huang, Caveh Jalali, Nick Vaccaro, Daisuke Nojiri, Eric Lai, Boris Mittelberg.

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/70773 )

Change subject: chromeos/cr50_enable_update.c: Clear EC AP_IDLE flag
......................................................................

Patch Set 13:

(2 comments)

File src/mainboard/google/brya/Kconfig:

https://review.coreboot.org/c/coreboot/+/70773/comment/f336b023_e8078796 
PS12, Line 336: config CR50_RESET_CLEAR_EC_AP_IDLE_FLAG
> this should move to common code.

+2

File src/vendorcode/google/chromeos/cr50_enable_update.c:

https://review.coreboot.org/c/coreboot/+/70773/comment/ff63aaf9_7030e132 
PS13, Line 159: 		if (CONFIG(CR50_RESET_CLEAR_EC_AP_IDLE_FLAG)) {
              : 			/* Send EC command to clear AP_IDLE flag */
              : 			if (!google_chromeec_reboot(0, EC_REBOOT_NO_OP,
              : 						EC_REBOOT_FLAG_CLEAR_AP_IDLE |
              : 						EC_REBOOT_FLAG_ON_AP_SHUTDOWN))
              : 				printk(BIOS_INFO,
              : 				       "Successfully clear AP_IDLE flag");
              : 			else
              : 				printk(BIOS_ERR,
              : 				       "Failed to clear EC AP_IDLE flag");
              : 		}
suggestion: please create a helper function