ATNEL tech-forum

Re: Funkcja INLINE w funkcji INLINE

2014-06-12T20:49:22+01:00

Sterownik już ogarnięty. Inlinowanie tych najbardziej niskopoziomowych funkcji skracało wykonanie kodu o kilka cykli. Co już było dla sterownika zbyt krótko. Po prostu nie spełniało to kryteriów czasowych wymaganych przez układ LC 7981.
Tak czy inaczej dorobiłem do biblioteki obsługę busy flag A delay'e na poziomie 500ns muszę w pewnych miejscach pozostać.
Dorobię trochę funkcji i się podzielę. LC 7981 nie jest może jakiś bardzo popularny, ale można spotkać wyświetlacze z tym sterownikiem.

Statystyki: Napisane przez sq8dsr — 12 cze 2014, o 20:49

Re: Funkcja INLINE w funkcji INLINE

2014-06-11T18:20:05+01:00

Teraz wygląda tak:
[syntax=c]inline void lcd_strobe(void) {
// czekanie 3 cykle
asm("nop");
asm("nop");
asm("nop");

// impuls na linii E
lcd_en_high();
asm("nop");
asm("nop");
asm("nop");
lcd_en_low();
// koniec impulsu

//checking busy flag
lcd_rw_high();
LCD_DATA &= ~(1<<7);
LCD_DATA_DDR &= ~(1<<7);
while( (LCD_DATA_PIN & (1<<7)) );
lcd_rw_low();
LCD_DATA_DDR |= (1<<7);

}[/syntax]

I oczywiście działa.

------------------------ [ Dodano po: 12 minutach ]

mokrowski napisał(a):

Tyle pamiętam.... jak się rąbnołem w literce, to przepraszam...
Jak tym ,,gąbki nie wyciśniesz" to zostaje głębokie przeprojektowanie lub asembler i ,,babranie się w bitach" :-/

Jeju, ale tych atrybutów jest... Dobrze to wiedzieć. Ale ja zbym mało znam asemblera, żeby widzieć dokładnie istotę tych różnic.
Bo nawet jak nie piszemy w asm, to ostatecznie sposób pracy kompilatora się przekłada na to i trzeba wiedzieć, co do czego.
I tak jakoś (dzięki Mirkowi) nie przeraża mnie już widok pliku .lss

Ale obecnie jestem niemal pewny, że po inline'owaniu funkcji lcd_strobe() stan wysoki na linii E pojawiał się zbyt szybko i transmisja się "rozjeżdżała". Nie które bajty były odbierane poprawnie przez sterownik a niektóre nie.
Przyglądam się timeingom trasmisji z noty PDF:

Czas po którym (przy zapisie) powinno pojawić się E to jest minimum 90ns po ustawieniu lini RS i RW
Zanik E minimum 10ns przez zmianą na liniach RS i RW.
Ale procesor chodzi na 10MHz więc jeden cykl trwa 100ns. Więc teoretycznie nie powinno to przeszkadzać.

Statystyki: Napisane przez sq8dsr — 11 cze 2014, o 18:20

Re: Funkcja INLINE w funkcji INLINE

2014-06-11T17:52:02+01:00

Inline'owanie lcd_strobe() powoduje problem. Static usunąłem, on tam jest bo ta funkcja nie jest udostępniana poza poza plikiem.
Jak sądzę jest to związane właśnie z obsługą linii E. Widzę, że występują teraz poważne błędy w transmisji / odczycie danych przez sterownik.
Tak sobie myślę, że ten CALL (gdy nie ma inline) powoduje odsuniecie w czasie momentu w którym wystawiany jest sygnał E. I wtedy jest dobrze. Zapewne tam jest jakaś krytyczna zależność czasowa.

Statystyki: Napisane przez sq8dsr — 11 cze 2014, o 17:52

Re: Funkcja INLINE w funkcji INLINE

2014-06-11T17:41:47+01:00

sq8dsr napisał(a):

Autor tak dziwnie makra zdefiniował, że wyglądają jak funkcje.

a no właśnie bo jakby były dużymi literami napisane to od razu można byłoby się domyśleć że to proste makra i łatwiej myśleć o ich inline'owaniu

ale skoro to jednak takie makra proste to dziwne że jak dasz inline na siłę to coś nie działa.

Próbowałeś np najpierw jedną z tych funkcji (najprosztą) zinlajnować i sprawdzić postępy ? poza tym wywalasz wtedy ten specyfikator static ?

Statystyki: Napisane przez mirekk36 — 11 cze 2014, o 17:41

Re: Funkcja INLINE w funkcji INLINE

2014-06-11T17:28:26+01:00

To jest tylko makro.
[syntax=c]// Macros for (un)setting the control pins
#define lcd_rs_high() (LCD_CTRL |= (1 << LCD_RS))//!< Set the Register-Select pin high
#define lcd_rs_low() (LCD_CTRL &= ~(1 << LCD_RS))//!< Set the Register-Select pin low

#define lcd_rw_high() (LCD_CTRL |= (1 << LCD_RW))//!< Set the Read-Write-Select pin high
#define lcd_rw_low() (LCD_CTRL &= ~(1 << LCD_RW))//!< Set the Register-Select pin low

#define lcd_en_high() (LCD_CTRL |= (1 << LCD_EN))//!< Set the strobe pin high
#define lcd_en_low() (LCD_CTRL &= ~(1 << LCD_EN))//!< Set the strobe pin low[/syntax]

Biblioteka nie jest moja. Ja się staram ją nieco udoskonalić.
Autor tak dziwnie makra zdefiniował, że wyglądają jak funkcje.

Statystyki: Napisane przez sq8dsr — 11 cze 2014, o 17:28

Re: Funkcja INLINE w funkcji INLINE

2014-06-11T16:54:13+01:00

a co robią u ciebie takie funkcje jak np:

lcd_rw_low();

??

Statystyki: Napisane przez mirekk36 — 11 cze 2014, o 16:54

Re: Funkcja INLINE w funkcji INLINE

2014-06-11T16:45:09+01:00

Obecnie funkcje wyglądają tak:

[syntax=c]static void lcd_strobe(void) {
lcd_en_high();
lcd_en_low();

//checking busy flag
lcd_rw_high();
LCD_DATA &= ~(1<<7);
LCD_DATA_DDR &= ~(1<<7);
while( (LCD_DATA_PIN & (1<<7)) );
lcd_rw_low();
LCD_DATA_DDR |= (1<<7);

}[/syntax]

[syntax=c]static void lcd_write_command(uint8_t cmd, uint8_t data) {
lcd_rw_low();
lcd_rs_high();
LCD_DATA = cmd;
lcd_strobe();

lcd_rs_low();
LCD_DATA = data;
lcd_strobe();

}[/syntax]

Ja widać nie ma inline... Gdy je zrobiłem jako INLINE wyświetlacz zaczął się niepoprawnie zachowywać.
Zuważyłem to przy działaniu funkcji clear. Która wykonuje po prostu zapisanie spacjami całego obszaru pamięci sterownika. Rozpoczynając od pozycji 0,0. Kasowanie nie dochodzi do końca. Kilkanaście ostatnich bajtów pozostaje nietkniętych.
Gdy funkcje są bez INLINE, tak jak powyżej, wszystko jest OK.
Ma to jakiś związek z czasem trwania stanu wysokiego na linii E. Gdy tam wstawię opóźnienie o 3 cykle zegara wszystko działa dobrze.

Statystyki: Napisane przez sq8dsr — 11 cze 2014, o 16:45

Re: Funkcja INLINE w funkcji INLINE

2014-06-11T16:12:12+01:00

sq8dsr napisał(a):

Sprawdzam po prostu BF w pętli while

a, no to dokładnie najlepszy sposób niż delaje

Statystyki: Napisane przez mirekk36 — 11 cze 2014, o 16:12

Re: Funkcja INLINE w funkcji INLINE

2014-06-11T15:55:03+01:00

To prawda, kod urośnie mocno. Bo to jest wołane przecież przy każdej okazji. CALL to chyba 5 cykli i RET kolejne 5 cykli. Więc 10 cykli cykli zegarowych. To jest jakieś 500 - 600 ns. Najdłuższe czasy reakcji tego sterownika są na poziomie 250ns.

Te delay'e, szczególnie ten 30us, już usunąłem. Sprawdzam po prostu BF w pętli while. W trybie tekstowym (bo to ma własny tryb tekstowy, zresztą dość ciekawy) to nie jest tab bardzo odczuwalne, ale w trybie graficznym prędkość rysowania wzrosła masakrycznie.

Statystyki: Napisane przez sq8dsr — 11 cze 2014, o 15:55

Re: Funkcja INLINE w funkcji INLINE

2014-06-11T15:35:43+01:00

No to są funkcje KOBYŁY i powiem ci że dodatkowy narzut CALL jest niczym przy ich budowie

ale gdybyś się uparł to jest taki atrybut

[syntax=c]void lcd_strobe(void) __attribute((always_inline));
void lcd_strobe(void) { ...[/syntax]

Statystyki: Napisane przez mirekk36 — 11 cze 2014, o 15:35

Funkcja INLINE w funkcji INLINE

2014-06-11T14:59:07+01:00

Witam

Zagadnienie jak w temacie.
Staram się dopracować bibliotekę do obsługi wyświetlacza opartego o sterownik SANYO LC7981.
Biblioteka jest autorstwa Sebastian Schumb'a.

W kodzie znajduję że tak powiem elementarne najbardziej niskopoziomowe fukcje odpowiadające za wysłanie komendy do sterownika oraz za "pyknięcie" linią E (wykonanie komendy).
Biblioteka nie posiadała obsługi flagi zajętości. Więc ją dorobiłem.
Jako, że jest to wyświetlacz graficzny, szybkość wykonywania operacji ma znaczenie dużo większe niż w przypadku 44780.
Dodatkowo przeglądając notę sterownika okazało się czasy wykonania komend są dość krótkie (ns), więc obecność delay'ow na poziomie 30 us w tak elementarnych funkcjach wydaje się nie na miejscu.

Autor fukcje napisał tak:
[syntax=c]static inline void lcd_write_command(uint8_t cmd, uint8_t data) {
_delay_us(30);
lcd_rw_low();
lcd_rs_high();
LCD_DATA = cmd;
_delay_us(1);
lcd_strobe();

lcd_rs_low();
LCD_DATA = data;
_delay_us(1);
lcd_strobe();

}[/syntax]

[syntax=c]static inline void lcd_strobe(void) {
lcd_en_high();
_delay_us(1)
lcd_en_low();
}[/syntax]

Jak widać fukcje są rodzaju INLINE i to zagnieżdżone.
Niestety w pliku lss nie zostały one rozwinięte. I wykonywany jest rozkaz call.
Wiem, że specyfikator INLINE nie jest dla kompilatora obligatoryjny i może on go nie zrealizować.
Funkcję strobe() (gdzie umieściłem sprawdzanie BF) oczywiście mogę literalnie (ręcznie) wkleić do lcd_write_command().

Ale czy da się jakoś zmusić kompilator, żeby jednak wykonał kompilację z uwzględnieniem specyfikatora INLINE?

Statystyki: Napisane przez sq8dsr — 11 cze 2014, o 14:59