Hi, We are facing build failure due to the https://review.coreboot.org/c/coreboot/+/54050 # util/crossgcc: Update gcc to 11.2 Does anyone else face the same issue? Thanks, Selma.
Does anyone else face the same issue?
I have been using it to build all System76 boards since it was merged, built on an Ubuntu 21.10 host. I've only had builds fail with it when trying to build our currently released 4.14-based branch, which last merged upstream in January.
-- Tim Crawford System76 Kernel Engineer tcrawford@system76.com
Hi Selma,
It might help to know what kind of build failures these are. Could you post them somewhere (e.g. ticket.coreboot.org) for discussion?
Regards, Patrick
Am Sa., 25. Sept. 2021 um 02:22 Uhr schrieb Bensaid, Selma < selma.bensaid@intel.com>:
Hi,
We are facing build failure due to the https://review.coreboot.org/c/coreboot/+/54050 # util/crossgcc: Update gcc to 11.2
Does anyone else face the same issue?
Thanks,
Selma.
coreboot mailing list -- coreboot@coreboot.org To unsubscribe send an email to coreboot-leave@coreboot.org
Hi Patrick, Here the build error, I checked that the error does not occurs with gcc 12 revert. We are using Ubuntu 18.04 for our build nodes. Thanks, Selma.
src/drivers/bus/i2c/designware.c: In function 'i2c_set_bus_speed': src/drivers/bus/i2c/designware.c:322:53: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 322 | MIN_HS_SCL_HIGHTIME, ®s->hs_scl_hcnt, | ^~~~~~~~~~~~~~~~~~ src/drivers/bus/i2c/designware.c:323:52: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 323 | MIN_HS_SCL_LOWTIME, ®s->hs_scl_lcnt); | ^~~~~~~~~~~~~~~~~~ src/drivers/bus/i2c/designware.c:326:53: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 326 | MIN_FS_SCL_HIGHTIME, ®s->fs_scl_hcnt, | ^~~~~~~~~~~~~~~~~~ src/drivers/bus/i2c/designware.c:327:52: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 327 | MIN_FS_SCL_LOWTIME, ®s->fs_scl_lcnt); | ^~~~~~~~~~~~~~~~~~ src/drivers/bus/i2c/designware.c:330:53: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 330 | MIN_SS_SCL_HIGHTIME, ®s->ss_scl_hcnt, | ^~~~~~~~~~~~~~~~~~ src/drivers/bus/i2c/designware.c:331:52: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 331 | MIN_SS_SCL_LOWTIME, ®s->ss_scl_lcnt); | ^~~~~~~~~~~~~~~~~~ src/drivers/bus/i2c/designware.c: In function 'i2c_bus_initialized': src/drivers/bus/i2c/designware.c:365:44: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 365 | return i2c_speed_init_done(®s->hs_scl_hcnt, | ^~~~~~~~~~~~~~~~~~ src/drivers/bus/i2c/designware.c:366:49: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 366 | ®s->hs_scl_lcnt); | ^~~~~~~~~~~~~~~~~~ src/drivers/bus/i2c/designware.c:368:44: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 368 | return i2c_speed_init_done(®s->fs_scl_hcnt, | ^~~~~~~~~~~~~~~~~~ src/drivers/bus/i2c/designware.c:369:49: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 369 | ®s->fs_scl_lcnt); | ^~~~~~~~~~~~~~~~~~ src/drivers/bus/i2c/designware.c:371:36: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 371 | return i2c_speed_init_done(®s->ss_scl_hcnt, | ^~~~~~~~~~~~~~~~~~ src/drivers/bus/i2c/designware.c:372:41: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member] 372 | ®s->ss_scl_lcnt); | ^~~~~~~~~~~~~~~~~~ cc1: all warnings being treated as errors
From: Patrick Georgi pgeorgi@google.com Sent: Sunday, September 26, 2021 8:37 AM To: Bensaid, Selma selma.bensaid@intel.com Cc: coreboot@coreboot.org Subject: Re: [coreboot] Build failure due to gcc update to 11.2
Hi Selma,
It might help to know what kind of build failures these are. Could you post them somewhere (e.g. ticket.coreboot.orghttp://ticket.coreboot.org) for discussion?
Regards, Patrick
Am Sa., 25. Sept. 2021 um 02:22 Uhr schrieb Bensaid, Selma <selma.bensaid@intel.commailto:selma.bensaid@intel.com>: Hi, We are facing build failure due to the https://review.coreboot.org/c/coreboot/+/54050 # util/crossgcc: Update gcc to 11.2 Does anyone else face the same issue? Thanks, Selma.
_______________________________________________ coreboot mailing list -- coreboot@coreboot.orgmailto:coreboot@coreboot.org To unsubscribe send an email to coreboot-leave@coreboot.orgmailto:coreboot-leave@coreboot.org
-- Google Germany GmbH, ABC-Str. 19, 20354 Hamburg Registergericht und -nummer: Hamburg, HRB 86891, Sitz der Gesellschaft: Hamburg Geschäftsführer: Paul Manicle, Halimah DeLaine Prado
Hi Selma,
Am Mo., 27. Sept. 2021 um 17:54 Uhr schrieb Bensaid, Selma < selma.bensaid@intel.com>:
Here the build error, I checked that the error does not occurs with gcc 12 revert. We are using Ubuntu 18.04 for our build nodes.
src/drivers/bus/i2c/designware.c: In function 'i2c_set_bus_speed':
src/drivers/bus/i2c/designware.c:322:53: error: taking address of packed member of 'struct <anonymous>' may result in an unaligned pointer value [-Werror=address-of-packed-member]
322 | MIN_HS_SCL_HIGHTIME, ®s->hs_scl_hcnt,
|^~~~~~~~~~~~~~~~~~
That issue seems to occur in depthcharge (we don't have that path in coreboot itself), for which we have fixes: this one in particular is fixed by the commit at https://chromium-review.googlesource.com/c/chromiumos/platform/depthcharge/+... but a few more (including a change to disable that type of warning because vboot runs in a few more instance) in the patch train.
Patrick
-
Bensaid, Selma -
Patrick Georgi -
Tim Crawford