<p><a href="https://review.coreboot.org/26942">View Change</a></p><p>44 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/arch/riscv/include/mcall.h">File src/arch/riscv/include/mcall.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/arch/riscv/include/mcall.h@59">Patch Set #2, Line 59:</a> <code style="font-family:monospace,monospace">#define OTHER_HLS(id) ((hls_t*)((void *)HLS() + RISCV_PGSIZE * ((id) - HLS()->hart_id)))</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/arch/riscv/include/mcall.h@59">Patch Set #2, Line 59:</a> <code style="font-family:monospace,monospace">#define OTHER_HLS(id) ((hls_t*)((void *)HLS() + RISCV_PGSIZE * ((id) - HLS()->hart_id)))</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"(foo*)" should be "(foo *)"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/arch/riscv/trap_handler.c">File src/arch/riscv/trap_handler.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/arch/riscv/trap_handler.c@204">Patch Set #2, Line 204:</a> <code style="font-family:monospace,monospace">               void * badAddress = (void *) tf->badvaddr;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/arch/riscv/trap_handler.c@232">Patch Set #2, Line 232:</a> <code style="font-family:monospace,monospace">           void * badAddress = (void *) tf->badvaddr;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/cpu/amd/family_10h-family_15h/init_cpus.c">File src/cpu/amd/family_10h-family_15h/init_cpus.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/cpu/amd/family_10h-family_15h/init_cpus.c@380">Patch Set #2, Line 380:</a> <code style="font-family:monospace,monospace">        void * lower_stack_region_boundary = (void *)(bsp_stack_region_lower_boundary - max_ap_stack_region_size);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/cpu/amd/family_10h-family_15h/init_cpus.c@380">Patch Set #2, Line 380:</a> <code style="font-family:monospace,monospace">   void * lower_stack_region_boundary = (void *)(bsp_stack_region_lower_boundary - max_ap_stack_region_size);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/include/x86emu/regs.h">File src/device/oprom/include/x86emu/regs.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/include/x86emu/regs.h@309">Patch Set #2, Line 309:</a> <code style="font-family:monospace,monospace">        void *          private;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no space before tabs</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/include/x86emu/regs.h@309">Patch Set #2, Line 309:</a> <code style="font-family:monospace,monospace">       void *          private;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo *             bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/yabel/compat/of.h">File src/device/oprom/yabel/compat/of.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/yabel/compat/of.h@38">Patch Set #2, Line 38:</a> <code style="font-family:monospace,monospace">#define p32cast (int) (unsigned long) (void *)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Macros with complex values should be enclosed in parentheses</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/yabel/compat/of.h@62">Patch Set #2, Line 62:</a> <code style="font-family:monospace,monospace">int of_read (ihandle_t , void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space prohibited between function name and open parenthesis '('</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/yabel/compat/of.h@62">Patch Set #2, Line 62:</a> <code style="font-family:monospace,monospace">int of_read (ihandle_t , void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space prohibited before that ',' (ctx:WxW)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/yabel/compat/of.h@62">Patch Set #2, Line 62:</a> <code style="font-family:monospace,monospace">int of_read (ihandle_t , void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'ihandle_t' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/yabel/compat/of.h@62">Patch Set #2, Line 62:</a> <code style="font-family:monospace,monospace">int of_read (ihandle_t , void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'void *' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/yabel/compat/of.h@62">Patch Set #2, Line 62:</a> <code style="font-family:monospace,monospace">int of_read (ihandle_t , void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/yabel/compat/of.h@63">Patch Set #2, Line 63:</a> <code style="font-family:monospace,monospace">int of_write (ihandle_t, void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space prohibited between function name and open parenthesis '('</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/yabel/compat/of.h@63">Patch Set #2, Line 63:</a> <code style="font-family:monospace,monospace">int of_write (ihandle_t, void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'ihandle_t' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/yabel/compat/of.h@63">Patch Set #2, Line 63:</a> <code style="font-family:monospace,monospace">int of_write (ihandle_t, void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'void *' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/device/oprom/yabel/compat/of.h@63">Patch Set #2, Line 63:</a> <code style="font-family:monospace,monospace">int of_write (ihandle_t, void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'int' should also have an identifier name</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/drivers/aspeed/common/aspeed_coreboot.h">File src/drivers/aspeed/common/aspeed_coreboot.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/drivers/aspeed/common/aspeed_coreboot.h@76">Patch Set #2, Line 76:</a> <code style="font-family:monospace,monospace">        void * ptr = malloc(size);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/drivers/intel/fsp1_0/fsp_util.c">File src/drivers/intel/fsp1_0/fsp_util.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/drivers/intel/fsp1_0/fsp_util.c@103">Patch Set #2, Line 103:</a> <code style="font-family:monospace,monospace">volatile u8 *find_fsp ()</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space prohibited between function name and open parenthesis '('</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/drivers/intel/fsp1_0/fsp_util.c@224">Patch Set #2, Line 224:</a> <code style="font-family:monospace,monospace">                FspHobListPtr = (void *)*((u32*) cbmem_find(CBMEM_ID_HOB_POINTER));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/drivers/intel/fsp1_0/fsp_util.c@224">Patch Set #2, Line 224:</a> <code style="font-family:monospace,monospace">            FspHobListPtr = (void *)*((u32*) cbmem_find(CBMEM_ID_HOB_POINTER));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"(foo*)" should be "(foo *)"</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/drivers/intel/fsp1_0/fsp_util.c@297">Patch Set #2, Line 297:</a> <code style="font-family:monospace,monospace">   FspHobListPtr = (void *)*((u32*) cbmem_find(CBMEM_ID_HOB_POINTER));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"(foo*)" should be "(foo *)"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/drivers/xgi/common/xgi_coreboot.c">File src/drivers/xgi/common/xgi_coreboot.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/drivers/xgi/common/xgi_coreboot.c@133">Patch Set #2, Line 133:</a> <code style="font-family:monospace,monospace">      xgifb_info->video_vbase = hw_info->pjVideoMemoryAddress = (void *)(intptr_t)xgifb_info->video_base;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/northbridge/amd/amdht/ht_wrapper.c">File src/northbridge/amd/amdht/ht_wrapper.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/northbridge/amd/amdht/ht_wrapper.c@58">Patch Set #2, Line 58:</a> <code style="font-family:monospace,monospace">static const char *event_class_string_decodes[] = {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">static const char * array should probably be static const char * const</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/northbridge/amd/amdht/ht_wrapper.c@93">Patch Set #2, Line 93:</a> <code style="font-family:monospace,monospace">static const char *event_string_decode(uint32_t event) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">open brace '{' following function declarations go on the next line</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/northbridge/amd/pi/agesawrapper.c">File src/northbridge/amd/pi/agesawrapper.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/northbridge/amd/pi/agesawrapper.c@292">Patch Set #2, Line 292:</a> <code style="font-family:monospace,monospace">        const void * agesa;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/northbridge/intel/gm45/iommu.c">File src/northbridge/intel/gm45/iommu.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/northbridge/intel/gm45/iommu.c@57">Patch Set #2, Line 57:</a> <code style="font-family:monospace,monospace">             void * bar = (void *)pci_read_config32(igd, PCI_BASE_ADDRESS_0);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/northbridge/intel/nehalem/raminit.c">File src/northbridge/intel/nehalem/raminit.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/northbridge/intel/nehalem/raminit.c@2297">Patch Set #2, Line 2297:</a> <code style="font-family:monospace,monospace">       u8 *res_low, u8 *res_high, u8 val)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb800/Amd.h">File src/southbridge/amd/cimx/sb800/Amd.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb800/Amd.h@64">Patch Set #2, Line 64:</a> <code style="font-family:monospace,monospace">typedef AGESA_STATUS (*CALLOUT_ENTRY) (unsigned int Param1, unsigned int Param2, void * ConfigPtr);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb800/Amd.h@64">Patch Set #2, Line 64:</a> <code style="font-family:monospace,monospace">typedef AGESA_STATUS (*CALLOUT_ENTRY) (unsigned int Param1, unsigned int Param2, void * ConfigPtr);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb800/Amd.h@65">Patch Set #2, Line 65:</a> <code style="font-family:monospace,monospace">typedef AGESA_STATUS (*IMAGE_ENTRY) (IN OUT void * ConfigPtr);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb800/Amd.h@66">Patch Set #2, Line 66:</a> <code style="font-family:monospace,monospace">typedef AGESA_STATUS (*MODULE_ENTRY) (IN OUT void * ConfigPtr);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb800/AmdSbLib.h">File src/southbridge/amd/cimx/sb800/AmdSbLib.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb800/AmdSbLib.h@31">Patch Set #2, Line 31:</a> <code style="font-family:monospace,monospace">typedef void (*CIM_IMAGE_ENTRY) (void * pConfig);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb800/late.c">File src/southbridge/amd/cimx/sb800/late.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb800/late.c@46">Patch Set #2, Line 46:</a> <code style="font-family:monospace,monospace"> * prototype UINT32 (*SBCIM_HOOK_ENTRY)(UINT32 Param1, UINT32 Param2, void * pConfig)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb800/late.c@53">Patch Set #2, Line 53:</a> <code style="font-family:monospace,monospace">static u32 sb800_callout_entry(u32 func, u32 data, void * config)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb900/Amd.h">File src/southbridge/amd/cimx/sb900/Amd.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb900/Amd.h@64">Patch Set #2, Line 64:</a> <code style="font-family:monospace,monospace">typedef AGESA_STATUS (*CALLOUT_ENTRY) (unsigned int Param1, unsigned int Param2, void * ConfigPtr);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb900/Amd.h@64">Patch Set #2, Line 64:</a> <code style="font-family:monospace,monospace">typedef AGESA_STATUS (*CALLOUT_ENTRY) (unsigned int Param1, unsigned int Param2, void * ConfigPtr);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb900/Amd.h@65">Patch Set #2, Line 65:</a> <code style="font-family:monospace,monospace">typedef AGESA_STATUS (*IMAGE_ENTRY) (IN OUT void * ConfigPtr);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb900/Amd.h@66">Patch Set #2, Line 66:</a> <code style="font-family:monospace,monospace">typedef AGESA_STATUS (*MODULE_ENTRY) (IN OUT void * ConfigPtr);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb900/AmdSbLib.h">File src/southbridge/amd/cimx/sb900/AmdSbLib.h:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb900/AmdSbLib.h@31">Patch Set #2, Line 31:</a> <code style="font-family:monospace,monospace">typedef void (*CIM_IMAGE_ENTRY) (void * pConfig);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb900/late.c">File src/southbridge/amd/cimx/sb900/late.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb900/late.c@41">Patch Set #2, Line 41:</a> <code style="font-family:monospace,monospace"> * prototype UINT32 (*SBCIM_HOOK_ENTRY)(UINT32 Param1, UINT32 Param2, void * pConfig)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/cimx/sb900/late.c@48">Patch Set #2, Line 48:</a> <code style="font-family:monospace,monospace">u32 sb900_callout_entry(u32 func, u32 data, void * config)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/sr5650/sr5650.c">File src/southbridge/amd/sr5650/sr5650.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/2/src/southbridge/amd/sr5650/sr5650.c@339">Patch Set #2, Line 339:</a> <code style="font-family:monospace,monospace">                mmio_base = (void *)(pci_read_config32(iommu_dev, 0x44) & 0xffffc000);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/26942">change 26942</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/26942"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I8e4118c5c5d70719ad7dc5f9ff9f86d93fa498ac </div>
<div style="display:none"> Gerrit-Change-Number: 26942 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Elyes HAOUAS <ehaouas@noos.fr> </div>
<div style="display:none"> Gerrit-CC: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 07 Jun 2018 09:39:28 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>