Mam zadeklarowaną strukturę oraz funkcje:
d_led.h
[syntax=c]//deklaracja struktury typu TLED_RGB
typedef struct {
volatile uint8_t color[6];//wartość PWM diod RGB: R=0, G=1, B=2, H=3, S=4, V=5
uint8_t paramiters_functions[10];//parametry wykorzystywane przez funkcje sterujące poszczególnymi diodami rgb
//paramiters_functions[0] = select_diode_rgb(used in the function: ????????_...) - zawiera informacje o diodzie z tabelą /*enum_5*/
//paramiters_functions[1] = select_color(used in the function: cyclic_change_colors_rgb_...) - wyświetlany kolor z tabeli COLORS
//paramiters_functions[2] = set_color (used in the function: cyclic_change_colors_rgb_...) - zawiera informacje o aktulanej wartości jasności koloru
//paramiters_functions[3] = flow_colors_h (used in the function: flow_colors_rgb_...) - komórka odpowiedzialna za wartość współczynnika H w stożku HSV
//paramiters_functions[4] = set_color_h (used in the function: flashing_one_color_rgb_...) - komórka zawierająca informację o wybranym kolorze przez użytkownika
//paramiters_functions[5] = multi_flashing_colors (used in the function: multi_flashing_color_rgb_...) - aktualnie wyświetlany kolor
//paramiters_functions[6] = fast_slow_change_multi_colors (used in the function: multi_flashing_color_rgb_...) - zwiększa lub spowalnia zmianę kolorów
//paramiters_functions[7] = rate_of_change (used in the function: multi_flashing_color_rgb_...) - zmienna umożliwiająca szybszą lub wolniejszą zmianę kolorów
//paramiters_functions[8] = common_flag_rgb (used in the function: multi_flashing_color_rgb_..., flashing_one_color_rgb_... oraz cyclic_change_colors_rgb_...)- bity tej zmiennej sterują odpowiednimi procesami w funkcjach - patrz tryb wyliczeniowy
//paramiters_functions[9] =
volatile uint16_t timer_set;//zapamiętany czas rozświetlania i ściemniania diody RGB, ładowany do licznika LED_timer
volatile uint16_t LED_timer;//czas odliczany w przerwaniu
volatile uint32_t period_time;//zmienna za pomocą której ustawiamy nowy czas szybkości rozświetlania i ściemniania diody
uint8_t chosen_function;//zmienna informująca o wybranej funkcji do sterowania ledem
volatile uint8_t value_max;//zmienna odpowiedzialna za maksymalny strumień świetlny
volatile uint8_t saturation_max;//zmienna odpowiadajaca za maksymalne nasycenie kolorów
} TLED_RGB;
TLED_RGB chimney, window_left, window_right, selected_rgb;// deklaracja DIOD RGB typu TLED_RGB
void change_paramiters_diode_rgb(TLED_RGB *diode_rgb, uint8_t command, uint8_t chose_led);//funkcja zmieniająca parametry wybranej diody rgb
void blink_selected_led(uint8_t *S, uint8_t *V, uint16_t *tim); //funkcja mrugnąca wybraną diodą[/syntax]
struktury przekazuję do funkcji change_paramiters_diode_rgb a w tej funkcji wywołuję drugą funkcję void blink_selected_led:
d_led.c (pokazuję tylko fragment tak aby nie zaciemniać
)[syntax=c]//****Funkcja mrugająca wybraną diodą RGB na biało****//
void blink_selected_led(uint8_t *S, uint8_t *V, uint16_t *tim){
*S = 0;
*V = 255;
ATOMIC_BLOCK(ATOMIC_RESTORESTATE) {*tim = 2000;}
}
//****funkcja zmieniająca parametry wybranej diody rgb na podstawie wciśniętego klawisza****//
void change_paramiters_diode_rgb(TLED_RGB *diode_rgb, uint8_t command, uint8_t chose_led ){
lcd_locate(0,6);
lcd_str(" ");
lcd_locate(0,6);
lcd_int(diode_rgb->paramiters_functions[select_diode_rgb] );
lcd_locate(0,8);
lcd_str(" ");
lcd_locate(0,8);
lcd_int( chose_led );
switch(command){
//**********************//
case KEY_1:
diode_rgb->chosen_function = cyclic_change_of_colors;
blink_selected_led(&diode_rgb->color[S], &diode_rgb->color[V], &diode_rgb->LED_timer);
delete_synchonization_setings(chose_led);
break;
.....
}[/syntax]
efektem tego są następujące warningi:
[syntax=c]make all
'Building file: ../D_LED/d_led.c'
'Invoking: AVR Compiler'
avr-gcc -Wall -Os -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega32 -DF_CPU=16000000UL -MMD -MP -MF"D_LED/d_led.d" -MT"D_LED/d_led.o" -c -o "D_LED/d_led.o" "../D_LED/d_led.c"
../D_LED/d_led.c: In function 'change_paramiters_diode_rgb':
../D_LED/d_led.c:598:4: warning: passing argument 1 of 'blink_selected_led' discards 'volatile' qualifier from pointer target type [enabled by default]
blink_selected_led(&diode_rgb->color[S], &diode_rgb->color[V], &diode_rgb->LED_timer);
^
../D_LED/d_led.c:155:6: note: expected 'uint8_t *' but argument is of type 'volatile uint8_t *'
void blink_selected_led(uint8_t *S, uint8_t *V, uint16_t *tim){
^
../D_LED/d_led.c:598:4: warning: passing argument 2 of 'blink_selected_led' discards 'volatile' qualifier from pointer target type [enabled by default]
blink_selected_led(&diode_rgb->color[S], &diode_rgb->color[V], &diode_rgb->LED_timer);
^
../D_LED/d_led.c:155:6: note: expected 'uint8_t *' but argument is of type 'volatile uint8_t *'
void blink_selected_led(uint8_t *S, uint8_t *V, uint16_t *tim){
^
../D_LED/d_led.c:598:4: warning: passing argument 3 of 'blink_selected_led' discards 'volatile' qualifier from pointer target type [enabled by default]
blink_selected_led(&diode_rgb->color[S], &diode_rgb->color[V], &diode_rgb->LED_timer);
^
../D_LED/d_led.c:155:6: note: expected 'uint16_t *' but argument is of type 'volatile uint16_t *'
void blink_selected_led(uint8_t *S, uint8_t *V, uint16_t *tim){
^[/syntax]
Może macie jakiś pomysł jak wyeliminować te warningi ?
W przerwaniu wywoływana jest tylko tablica color z powyższej struktury. Więc od razu zapytam, czy jest uzasadnione użycie specyfikatora volatile do pozostałych zmiennych z powyższej struktury. Rozumiem działanie tego specyfikatora, tylko mam za małe doświadczenie w programowaniu i jeszcze nie umiem ocenić, czy użycie go w tym miejscu wniesie coś bardziej pozytywnego do wykonywania się programu. A wiadomo, nadgorliwość jest gorsza od faszyzmu
Statystyki: Napisane przez amilo_pa — 15 sty 2017, o 11:46
]]>