Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable specific warnings to mitigate compilation problems with KEEP_INTERMEDIATES=yes. #20339

Merged
merged 9 commits into from
Apr 5, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions keyboards/hazel/bad_wings/keymaps/default/config.h
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,11 @@

#define TAPPING_TERM 200
#define PERMISSIVE_HOLD
#define IGNORE_MOD_TAP_INTERRUPT
#define TAPPING_FORCE_HOLD
#define TAPPING_TERM_PER_KEY
#define ONESHOT_TAP_TOGGLE 10
#define ONESHOT_TIMEOUT 500
#define DOUBLE_TAP_SHIFT_TURNS_ON_CAPS_WORD
#define CIRQUE_PINNACLE_TAP_ENABLE
#define CIRQUE_PINNACLE_POSITION_MODE CIRQUE_PINNACLE_ABSOLUTE_MODE
#define CIRQUE_DEVICE_GESTURES_SCROLL_ENABLE
#define CIRQUE_PINNACLE_POSITION_MODE CIRQUE_PINNACLE_ABSOLUTE_MODE
#define CIRQUE_DEVICE_GESTURES_SCROLL_ENABLE
1 change: 1 addition & 0 deletions keyboards/mechlovin/mechlovin9/rev3/info.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
"rows": ["B12", "B13", "B14", "B15", "A1"],
"cols": ["B11", "B10", "B2", "B1", "B0", "A6", "A5", "A4", "A3", "C13", "B7", "B6", "B5", "B4", "B3"]
},
"diode_direction": "COL2ROW",
"indicators": {
"caps_lock": "B9"
}
Expand Down
3 changes: 3 additions & 0 deletions keyboards/splitkb/kyria/info.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,5 +9,8 @@
"development_board": "elite_c",
"split": {
"enabled": true
},
"build": {
"lto": true
}
}
4 changes: 4 additions & 0 deletions platforms/avr/_wait.h
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,11 @@
*/
#pragma once

// Need to disable GCC's "maybe-uninitialized" warning for this file, as it causes issues when running `KEEP_INTERMEDIATES=yes`.
#pragma GCC diagnostic push
#pragma GCC diagnostic ignored "-Wmaybe-uninitialized"
#include <util/delay.h>
#pragma GCC diagnostic pop

// http://ww1.microchip.com/downloads/en/devicedoc/atmel-0856-avr-instruction-set-manual.pdf
// page 22: Table 4-2. Arithmetic and Logic Instructions
Expand Down
6 changes: 6 additions & 0 deletions platforms/chibios/drivers/audio_dac_additive.c
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,10 @@
#include <ch.h>
#include <hal.h>

// Need to disable GCC's "tautological-compare" warning for this file, as it causes issues when running `KEEP_INTERMEDIATES=yes`. Corresponding pop at the end of the file.
#pragma GCC diagnostic push
#pragma GCC diagnostic ignored "-Wtautological-compare"

/*
Audio Driver: DAC

Expand Down Expand Up @@ -335,3 +339,5 @@ void audio_driver_start(void) {
active_tones_snapshot_length = 0;
state = OUTPUT_SHOULD_START;
}

#pragma GCC diagnostic pop
6 changes: 6 additions & 0 deletions platforms/chibios/drivers/audio_dac_basic.c
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,10 @@
#include "ch.h"
#include "hal.h"

// Need to disable GCC's "tautological-compare" warning for this file, as it causes issues when running `KEEP_INTERMEDIATES=yes`. Corresponding pop at the end of the file.
#pragma GCC diagnostic push
#pragma GCC diagnostic ignored "-Wtautological-compare"

/*
Audio Driver: DAC

Expand Down Expand Up @@ -247,3 +251,5 @@ void audio_driver_start(void) {
}
gptStartContinuous(&AUDIO_STATE_TIMER, 2U);
}

#pragma GCC diagnostic pop
6 changes: 3 additions & 3 deletions tmk_core/protocol/usb_hid/override_wiring.c
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
* To keep Timer0 for common/timer.c override arduino/wiring.c.
*/
#define __DELAY_BACKWARD_COMPATIBLE__
#include <util/delay.h>
#include "wait.h"
#include "platforms/timer.h"


Expand All @@ -16,11 +16,11 @@ unsigned long micros(void)
}
void delay(unsigned long ms)
{
_delay_ms(ms);
wait_ms(ms);
}
void delayMicroseconds(unsigned int us)
{
_delay_us(us);
wait_us(us);
}
void init(void)
{
Expand Down