[coreboot-gerrit] Change in coreboot[master]: mb/google/brya/var/baseboard/brask: Add power limits functions

[Alan Huang (Code Review)]

Attention is currently required from: Tim Wawrzynczak, Paul Menzel, Nick Vaccaro, Zhuohao Lee.
Alan Huang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58241 )

Change subject: mb/google/brya/var/baseboard/brask: Add power limits functions
......................................................................

Patch Set 18:

(3 comments)

Patchset:

PS18: 
Move function declarations to the front of function body.
Add blank line after '}'.

File src/mainboard/google/brya/variants/baseboard/brask/ramstage.c:

https://review.coreboot.org/c/coreboot/+/58241/comment/5eda0bc5_d8f6d3d6 
PS16, Line 92: if (!num_entries)
             : 		return;
             : 
             : 	const struct device *policy_dev = DEV_PTR(dptf_policy);
             : 	if (!policy_dev)
             : 		return;
             : 
             : 	size_t intel_idx, brask_idx;
             : 	if (!get_sku_index(limits, num_entries, &intel_idx, &brask_idx))
             : 		return;
             : 
             : 	config_t *conf = config_of_soc();
             : 	struct soc_power_limits_config *soc_config = &conf->power_limits_config[intel_idx];
             : 
             : 	enum usb_chg_type type;
             : 	u32 watts;
             : 	u16 volts_mv, current_ma;
             : 	u32 psyspl2, pl2;
             : 	soc_config->tdp_pl4 = 0;
             : 	u32 pl2_default = DIV_ROUND_UP(limits[brask_idx].pl2_max_power, MILLIWATTS_TO_WATTS);
             : 	int rv = google_chromeec_get_usb_pd_power_info(&type, &current_ma, &volts_mv);
> suggestion, slighly reorganize this for readability: […]

Done

https://review.coreboot.org/c/coreboot/+/58241/comment/f714a130_e0874b4c 
PS16, Line 131: 	/* voltage unit is milliVolts and current is in milliAmps */
> nit: blank line after `}`

Done