Attention is currently required from: Felix Singer, Raul Rangel, Furquan Shaikh, Paul Menzel, Angel Pons, Subrata Banik, Kyösti Mälkki, Patrick Rudolph, Jason Glenesk, Matt Delco, Nico Huber, Marshall Dawson, Tim Wawrzynczak, Felix Held. build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57886 )
Change subject: acpigen,soc/amd,cpu/intel: rework static DWORD for CPPC table ......................................................................
Patch Set 5:
(109 comments)
File src/acpi/acpigen.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/3763025d_5da69539 PS5, Line 1802: if (entry->type == CPPC_TYPE_DWORD) { braces {} are not necessary for any arm of this statement
File src/cpu/intel/common/common_init.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/fb349aae_b0157aeb PS5, Line 108: config->entries[CPPC_HIGHEST_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 0, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/f94994cb_e386900d PS5, Line 108: config->entries[CPPC_HIGHEST_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 0, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/9979d194_075fa92b PS5, Line 108: config->entries[CPPC_HIGHEST_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 0, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/4b3bdf4b_8c4a812b PS5, Line 109: config->entries[CPPC_NOMINAL_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_PLATFORM_INFO, 8, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/4a54c6e9_81590b23 PS5, Line 109: config->entries[CPPC_NOMINAL_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_PLATFORM_INFO, 8, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/fd03c480_6ccbfbc6 PS5, Line 109: config->entries[CPPC_NOMINAL_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_PLATFORM_INFO, 8, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/8b90c6a3_26bf8532 PS5, Line 110: config->entries[CPPC_LOWEST_NONL_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 16, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/a7e90eca_6ecd2e11 PS5, Line 110: config->entries[CPPC_LOWEST_NONL_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 16, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/7636ea0a_90f289d2 PS5, Line 110: config->entries[CPPC_LOWEST_NONL_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 16, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/3186d9d6_dd536edb PS5, Line 111: config->entries[CPPC_LOWEST_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 24, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/e69886ac_d521e57c PS5, Line 111: config->entries[CPPC_LOWEST_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 24, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/c4089b01_7031b897 PS5, Line 111: config->entries[CPPC_LOWEST_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 24, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/c9353b81_bf308391 PS5, Line 112: config->entries[CPPC_GUARANTEED_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 8, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/d0726b0e_800e53db PS5, Line 112: config->entries[CPPC_GUARANTEED_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 8, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/07a4f70f_608018c1 PS5, Line 112: config->entries[CPPC_GUARANTEED_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_CAPABILITIES, 8, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/4cfdc4ba_275d09e1 PS5, Line 113: config->entries[CPPC_DESIRED_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 16, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/6f7c2111_56a49ce1 PS5, Line 113: config->entries[CPPC_DESIRED_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 16, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/da23c791_1f98c677 PS5, Line 113: config->entries[CPPC_DESIRED_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 16, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/c90a5a1d_74c5becb PS5, Line 114: config->entries[CPPC_MIN_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 0, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/765423d5_87e63829 PS5, Line 114: config->entries[CPPC_MIN_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 0, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/429e7b1a_52380e69 PS5, Line 114: config->entries[CPPC_MIN_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 0, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/2699d88a_169039cf PS5, Line 115: config->entries[CPPC_MAX_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 8, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/254c9dfb_6c2ecaa8 PS5, Line 115: config->entries[CPPC_MAX_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 8, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/37d46aab_001afa04 PS5, Line 115: config->entries[CPPC_MAX_PERF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 8, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/5e38bd27_970e8498 PS5, Line 116: config->entries[CPPC_PERF_REDUCE_TOLERANCE] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/86d1e194_3ad04bf9 PS5, Line 116: config->entries[CPPC_PERF_REDUCE_TOLERANCE] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/68c29775_14254735 PS5, Line 117: config->entries[CPPC_TIME_WINDOW] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/31336f4d_e94d78db PS5, Line 117: config->entries[CPPC_TIME_WINDOW] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/f509aea4_595d4bf3 PS5, Line 118: config->entries[CPPC_COUNTER_WRAP] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/5db53128_1cb47d91 PS5, Line 118: config->entries[CPPC_COUNTER_WRAP] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/21d9879d_797326dc PS5, Line 119: config->entries[CPPC_REF_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(IA32_MPERF, 0, 64) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/5ca4c702_ac97c54b PS5, Line 119: config->entries[CPPC_REF_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(IA32_MPERF, 0, 64) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/e4ce0424_ce11efe1 PS5, Line 119: config->entries[CPPC_REF_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(IA32_MPERF, 0, 64) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/71381b4d_74a36489 PS5, Line 120: config->entries[CPPC_DELIVERED_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(IA32_APERF, 0, 64) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/d7121e6d_b421b016 PS5, Line 120: config->entries[CPPC_DELIVERED_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(IA32_APERF, 0, 64) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/b221c018_7bb4ae5a PS5, Line 120: config->entries[CPPC_DELIVERED_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(IA32_APERF, 0, 64) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/94febc88_dd16036f PS5, Line 121: config->entries[CPPC_PERF_LIMITED] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_STATUS, 2, 1) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/8a7d713e_534f1d6f PS5, Line 121: config->entries[CPPC_PERF_LIMITED] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_STATUS, 2, 1) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/216ff305_344ba13f PS5, Line 121: config->entries[CPPC_PERF_LIMITED] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_STATUS, 2, 1) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/5429bffe_8f162ca4 PS5, Line 122: config->entries[CPPC_ENABLE] = CPPC_REG( ACPI_REG_MSR(IA32_PM_ENABLE, 0, 1) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/ec359d07_dfb7be1c PS5, Line 122: config->entries[CPPC_ENABLE] = CPPC_REG( ACPI_REG_MSR(IA32_PM_ENABLE, 0, 1) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/02712328_3bf6761d PS5, Line 122: config->entries[CPPC_ENABLE] = CPPC_REG( ACPI_REG_MSR(IA32_PM_ENABLE, 0, 1) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/3bbb71e5_cd5044b7 PS5, Line 128: config->entries[CPPC_AUTO_ACTIVITY_WINDOW] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 32, 10) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/391f14a4_0f4950a6 PS5, Line 128: config->entries[CPPC_AUTO_ACTIVITY_WINDOW] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 32, 10) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/19afa890_22650e87 PS5, Line 128: config->entries[CPPC_AUTO_ACTIVITY_WINDOW] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 32, 10) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/be541203_75dc45bd PS5, Line 129: config->entries[CPPC_PERF_PREF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 24, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/ef01eb55_9147eba5 PS5, Line 129: config->entries[CPPC_PERF_PREF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 24, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/e35dc658_41faf198 PS5, Line 129: config->entries[CPPC_PERF_PREF] = CPPC_REG( ACPI_REG_MSR(IA32_HWP_REQUEST, 24, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/d1324db9_c3399a88 PS5, Line 130: config->entries[CPPC_REF_PERF] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/96784e9a_580ed4c5 PS5, Line 130: config->entries[CPPC_REF_PERF] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/7dc7d433_4ea5c518 PS5, Line 135: config->entries[CPPC_LOWEST_FREQ] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/07749b71_8b5fe600 PS5, Line 135: config->entries[CPPC_LOWEST_FREQ] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/e1ca9ed2_1c592074 PS5, Line 136: config->entries[CPPC_NOMINAL_FREQ] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/c660767f_87960955 PS5, Line 136: config->entries[CPPC_NOMINAL_FREQ] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
File src/soc/amd/cezanne/cppc.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/78f0117f_476b7570 PS5, Line 18: config->entries[CPPC_HIGHEST_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_HIGHEST_PERF, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/7d459ad4_bd5bc6cf PS5, Line 18: config->entries[CPPC_HIGHEST_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_HIGHEST_PERF, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/a86dbf61_9c026224 PS5, Line 18: config->entries[CPPC_HIGHEST_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_HIGHEST_PERF, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/91311e9c_e358fc97 PS5, Line 19: config->entries[CPPC_NOMINAL_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_NOMINAL_PERF, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/91468dd8_3a5e3e34 PS5, Line 19: config->entries[CPPC_NOMINAL_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_NOMINAL_PERF, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/22810bca_8416d216 PS5, Line 19: config->entries[CPPC_NOMINAL_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_NOMINAL_PERF, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/152364c7_4c9a2625 PS5, Line 20: config->entries[CPPC_LOWEST_NONL_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_LOW_NON_LIN_PERF, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/380bcf7c_e506b8d8 PS5, Line 20: config->entries[CPPC_LOWEST_NONL_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_LOW_NON_LIN_PERF, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/df2c9ed1_26e11d47 PS5, Line 20: config->entries[CPPC_LOWEST_NONL_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_LOW_NON_LIN_PERF, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/96d9be1d_a50acd11 PS5, Line 21: config->entries[CPPC_LOWEST_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_LOWEST_PERF, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/7ee019a9_85d6a4ed PS5, Line 21: config->entries[CPPC_LOWEST_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_LOWEST_PERF, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/20490d72_b3bebdf9 PS5, Line 21: config->entries[CPPC_LOWEST_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_CAPABILITY_1, SHIFT_CPPC_CAPABILITY_1_LOWEST_PERF, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/4e098528_604c5966 PS5, Line 22: config->entries[CPPC_GUARANTEED_PERF] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/c4ce5f3e_67f50d7f PS5, Line 22: config->entries[CPPC_GUARANTEED_PERF] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/5b693a4f_343cf131 PS5, Line 23: config->entries[CPPC_DESIRED_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_DES_PERF, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/8534f687_aad8a65d PS5, Line 23: config->entries[CPPC_DESIRED_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_DES_PERF, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/1018401c_5ef9ecb9 PS5, Line 23: config->entries[CPPC_DESIRED_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_DES_PERF, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/db3b5470_91e0acda PS5, Line 24: config->entries[CPPC_MIN_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_MIN_PERF, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/da8ba296_f0f14093 PS5, Line 24: config->entries[CPPC_MIN_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_MIN_PERF, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/cf734e73_a3072e2d PS5, Line 24: config->entries[CPPC_MIN_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_MIN_PERF, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/8c09c61f_c723d580 PS5, Line 25: config->entries[CPPC_MAX_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_MAX_PERF, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/6487a621_f957f155 PS5, Line 25: config->entries[CPPC_MAX_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_MAX_PERF, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/f0fbd730_b33f8257 PS5, Line 25: config->entries[CPPC_MAX_PERF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_MAX_PERF, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/dd23da08_76935c71 PS5, Line 26: config->entries[CPPC_PERF_REDUCE_TOLERANCE] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/6e950615_d6531e29 PS5, Line 26: config->entries[CPPC_PERF_REDUCE_TOLERANCE] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/17346bc3_43c25f9e PS5, Line 27: config->entries[CPPC_TIME_WINDOW] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/386dd478_9b44e9a5 PS5, Line 27: config->entries[CPPC_TIME_WINDOW] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/fde15f9d_60b1e567 PS5, Line 28: config->entries[CPPC_COUNTER_WRAP] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/ba8ab5ed_e0d519f7 PS5, Line 28: config->entries[CPPC_COUNTER_WRAP] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/31a826dd_ca20273f PS5, Line 29: config->entries[CPPC_REF_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(MSR_MAX_PERFORMANCE_FREQUENCY_CLOCK_COUNT, 0, 64) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/ad909a55_cee1d50f PS5, Line 29: config->entries[CPPC_REF_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(MSR_MAX_PERFORMANCE_FREQUENCY_CLOCK_COUNT, 0, 64) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/9bbc2f52_77fc966c PS5, Line 29: config->entries[CPPC_REF_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(MSR_MAX_PERFORMANCE_FREQUENCY_CLOCK_COUNT, 0, 64) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/8f6fa5fa_64c4a26a PS5, Line 30: config->entries[CPPC_DELIVERED_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(MSR_ACTUAL_PERFORMANCE_FREQUENCY_CLOCK_COUNT, 0, 64) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/cdc9a4fa_b7e2c258 PS5, Line 30: config->entries[CPPC_DELIVERED_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(MSR_ACTUAL_PERFORMANCE_FREQUENCY_CLOCK_COUNT, 0, 64) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/aa8fed01_41bbbcec PS5, Line 30: config->entries[CPPC_DELIVERED_PERF_COUNTER] = CPPC_REG( ACPI_REG_MSR(MSR_ACTUAL_PERFORMANCE_FREQUENCY_CLOCK_COUNT, 0, 64) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/4dd22167_0cd24f78 PS5, Line 31: config->entries[CPPC_PERF_LIMITED] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_STATUS, 1, 1) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/0e6ce609_ac2fff71 PS5, Line 31: config->entries[CPPC_PERF_LIMITED] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_STATUS, 1, 1) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/a230a80a_8ccdd695 PS5, Line 31: config->entries[CPPC_PERF_LIMITED] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_STATUS, 1, 1) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/bd37f8e8_355f1af7 PS5, Line 32: config->entries[CPPC_ENABLE] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_ENABLE, 0, 1) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/3388c24b_88561d72 PS5, Line 32: config->entries[CPPC_ENABLE] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_ENABLE, 0, 1) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/b348a4ad_0837ae4a PS5, Line 32: config->entries[CPPC_ENABLE] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_ENABLE, 0, 1) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/da5c198d_7ca0d4ca PS5, Line 37: config->entries[CPPC_AUTO_SELECT] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/359a68e1_7a5e82a2 PS5, Line 37: config->entries[CPPC_AUTO_SELECT] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/2c1fd1fa_36c2e457 PS5, Line 38: config->entries[CPPC_AUTO_ACTIVITY_WINDOW] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/dc0ebf4b_68a1cfab PS5, Line 38: config->entries[CPPC_AUTO_ACTIVITY_WINDOW] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/f0aebeb2_c4c1b030 PS5, Line 39: config->entries[CPPC_PERF_PREF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_ENERGY_PERF_PREF, 8) ); line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/edbba8ad_d9118b33 PS5, Line 39: config->entries[CPPC_PERF_PREF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_ENERGY_PERF_PREF, 8) ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/db1f7bf8_e0b033eb PS5, Line 39: config->entries[CPPC_PERF_PREF] = CPPC_REG( ACPI_REG_MSR(MSR_CPPC_REQUEST, SHIFT_CPPC_REQUEST_ENERGY_PERF_PREF, 8) ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/d9c85b13_5244a92a PS5, Line 40: config->entries[CPPC_REF_PERF] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/ca33c04a_8d59294d PS5, Line 40: config->entries[CPPC_REF_PERF] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/eff86334_2fe0505e PS5, Line 45: config->entries[CPPC_LOWEST_FREQ] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/15fa305c_d6865cdc PS5, Line 45: config->entries[CPPC_LOWEST_FREQ] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/b0f5b227_7d846c9f PS5, Line 46: config->entries[CPPC_NOMINAL_FREQ] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited after that open parenthesis '('
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-129826): https://review.coreboot.org/c/coreboot/+/57886/comment/6b61318a_87afdb43 PS5, Line 46: config->entries[CPPC_NOMINAL_FREQ] = CPPC_REG( ACPI_REG_UNSUPPORTED ); space prohibited before that close parenthesis ')'