ATNEL tech-forum

Re: Przeportowanie AVR C++ na AVR C

2017-09-01T13:32:31+01:00

No to już coś wiadomo. Jak wieczorem znajdę chwile to się przyjrzę kodowi w czym rzecz.

PS. Sprawdziłem na symulatorze dla twoich ustawień i wygląda na to, że przerzuca stanami portów tak jak powinno. Coś musi być nie tak w połączeniach (może coś do tych portów jest jeszcze podłączone i zakłóca transmisję).

Statystyki: Napisane przez kijas1 — 1 wrz 2017, o 13:32

Re: Przeportowanie AVR C++ na AVR C

2017-09-01T13:17:41+01:00

Problem pojawia się przy zmianie pinów PB4 i PB5 na inne.
Pomiędzy sobą mogą być zamieniane, ale na inne już nie.

Statystyki: Napisane przez LA72 — 1 wrz 2017, o 13:17

Re: Przeportowanie AVR C++ na AVR C

2017-09-01T10:59:48+01:00

Proponuje zacząć zmiany od pojedynczych linii i obserwować co się dzieje. Może gdzieś w połączeniach coś się pomieszało.

Statystyki: Napisane przez kijas1 — 1 wrz 2017, o 10:59

Re: Przeportowanie AVR C++ na AVR C

2017-08-31T21:20:21+01:00

W menu.cpp zmieniłem na:
[syntax=cpp]void lcdInit(void)
{
// lcd.lcd_init(&PORTD, PD0, &PORTD, PD1, &PORTB, PB3, &PORTB, PB4, &PORTB, PB5);
lcd.lcd_init(&PORTD, PD3, &PORTD, PD4, &PORTD, PD2, &PORTD, PD1, &PORTD, PD0);[/syntax]
zgodnie z opisem w 5110.cpp:
[syntax=cpp]void LCD_5110::lcd_init(
volatile uint8_t *port_SCE, uint8_t dq_SCE,
volatile uint8_t *port_RST, uint8_t dq_RST,
volatile uint8_t *port_DC, uint8_t dq_DC,
volatile uint8_t *port_DATA, uint8_t dq_DATA,
volatile uint8_t *port_CLK, uint8_t dq_CLK
)[/syntax]
W main.cpp zmieniłem podciąganie dla przycisków:
[syntax=cpp]int main(void)
{
PORTB |= (1<PORTB |= (1<W header.h zmieniłem definicje dla przycisków:
[syntax=cpp]//buttons
#define BTN_LEFT_PRESSED!(PINB & (1<#define BTN_RIGHT_PRESSED!(PINB & (1<#define BTN_MIDDLE_PRESSED!(PINB & (1<#define BTN_DOWN_PRESSED!(PINB & (1<#define BTN_UP_PRESSED!(PINB & (1<
I niestety teraz mam ładny pusty ekran
Co robię źle tym razem.
Łatwiej jest projektować układy niż pisać do nich soft.

Statystyki: Napisane przez LA72 — 31 sie 2017, o 21:20

Re: Przeportowanie AVR C++ na AVR C

2017-08-31T21:06:27+01:00

W pliku menu.cpp masz funkcję lcdInit, a tam wywołaną metode lcd.lcd_init. Wpisz w niej swoje porty. Kolejność możesz podejrzeć w pliku 5110.h gdzie jest metoda lcd_init.

Statystyki: Napisane przez kijas1 — 31 sie 2017, o 21:06

Re: Przeportowanie AVR C++ na AVR C

2017-08-31T20:07:53+01:00

Witam.
Teraz pytanie z "innej mańki".
Jak zmienić rozłożenie pinów do wyświetlacza?
Dla wszystkich elementów znalazłem, ale z LCD mam problem.
Nowe połączenia chcę ustawić tak:
- PD0 = CLK
- PD1 = DIN
- PD2 = DC
- PD3 = CE
- PD4 = DRST
Domyślam się, że znajdują wiś w 5110.cpp.

Statystyki: Napisane przez LA72 — 31 sie 2017, o 20:07

Re: Przeportowanie AVR C++ na AVR C

2017-08-31T08:31:42+01:00

Już pisałem byś sprawdził czy na pewno masz te wywołania, ale samo się wyjaśniło.

Statystyki: Napisane przez kijas1 — 31 sie 2017, o 08:31

Re: Przeportowanie AVR C++ na AVR C

2017-08-31T07:32:44+01:00

Niestety ta porada nie przyniosła poprawy.
Znak zapytania jest na końcu. Tak jak powinien być.
Coś czuję, że problem leży w pliku menu.cpp.

Chociaż, z drugiej strony dane te są zapisane w main.cpp:
[syntax=cpp]{
case 0://charge
uint8_t cells, hrs;
switch(drawSelectionMenu(chemistryMenuName, chemistryMenuItems))
{
case 0://Li-Ion/SLA
maxC = drawNumericalMenu("max charging\ncurrent [A]", 1.00);
maxV = drawNumericalMenu("max battery\nvoltage [V]", 4.20);
constantCVCharge(maxC, maxV);
break;
case 1://NiCd/NiMH
maxC = drawNumericalMenu("charging\ncurrent [A]", 1.00);
cells = drawNumericalMenu("# of\ncells", 1.00);
hrs = drawNumericalMenu("charging\ntime [Hrs]", 7);
chargeNiMH(maxC, cells, hrs);
break;
case 2://constant voltage
maxV = drawNumericalMenu("output\nvoltage [V]", 5.00);
constantVoltage(maxV);
break;
case 3://constant current
maxC = drawNumericalMenu("output\ncurrent [A]", 1.00);
constantCurrent(maxC);
break;
case 4://return
break;
}
break;
case 1://discharge
disC = drawNumericalMenu("discharging\ncurrent [A]", 1.00);
minV = drawNumericalMenu("minimal\nvoltage [A]", 1.00);
discharge(disC, minV);
break;
case 2://power supply - constant current and voltage limited power supply (WARNING - long
maxV = drawNumericalMenu("output\nvoltage [V]", 5.00);
maxC = drawNumericalMenu("current\nlimit [A]", 1.00);
constantCV(maxV, maxC);
break;
case 3://about - just display a screen on the lcd
lcd.lcd_clear();
lcd.lcd_goto_xy_exact(0, 0);
lcd.lcd_str(" AVR Charger by M. Danek V1.10 (c)2016 madanek\n post.cz");
while (!BTN_MIDDLE_PRESSED);
while (BTN_MIDDLE_PRESSED);
break;
case 4://restart (set up a watchdog)
wdt_enable(WDTO_15MS);
while (1);
break;
}[/syntax]

P.S.1
Uff. Jednak zadziałało.
Przez "przypadek" usunęło mi wpis "-Wl,-u,vfprintf".
Dzięki.

P.S.2
Program jak na razie działa.
Są jednak żółte ostrzeżenia w Eclipse.

Statystyki: Napisane przez LA72 — 31 sie 2017, o 07:32

Re: Przeportowanie AVR C++ na AVR C

2017-08-30T22:12:45+01:00

Proponuję sprawdzić sobie to: http://mirekk36.blogspot.com/2013/04/ec ... float.html . Jeśli nie pomorze będziemy dalej kombinować.

Statystyki: Napisane przez kijas1 — 30 sie 2017, o 22:12

Re: Przeportowanie AVR C++ na AVR C

2017-08-30T17:51:57+01:00

Ruszyło.
Ten sam wpis musiałem dodać do:
- adc.h
- NiMh.h
oraz wspomnianego 5110.h.
Dzięki

P.S.
Niestety zbyt pięknie by było prawdziwie.
Przy oryginalnym pliku hex widoczne są domyślne wartości, które można zmieniać.

Przy pliku nowo kompilowanym tych wartości nie ma.

O dziwo wielkość plików wynikowych też się różni.
Oryginał ma 47kb, za nowy tylko 36kb.

Statystyki: Napisane przez LA72 — 30 sie 2017, o 17:51

Re: Przeportowanie AVR C++ na AVR C

2017-08-30T17:37:05+01:00

Na początek proponuję dołączyć do pliku 5110.h i spróbować skompilować. Projekt jest gotowy pod AS6, ale jeśli kolega woli Eclipse, którego nie używam, to będziemy razem walczyć.

Statystyki: Napisane przez kijas1 — 30 sie 2017, o 17:37

Re: Przeportowanie AVR C++ na AVR C

2017-08-30T17:05:16+01:00

Chodzi o to, że Eclipse ma problem z kilkoma liniami.
Konsola:

Code:

18:00:46 **** Build of configuration Release for project AVR450_v14_charger_cpp ****
make all 
'Building file: ../display/5110.cpp'
'Invoking: AVR C++ Compiler'
avr-g++ -Wall -Os -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -funsigned-char -funsigned-bitfields -fno-exceptions -mmcu=atmega328 -DF_CPU=8000000UL -MMD -MP -MF"display/5110.d" -MT"display/5110.o" -c -o "display/5110.o" "../display/5110.cpp"
In file included from ../display/5110.cpp:8:0:
../display/../../AVR450_v14_charger_cpp/display/5110.h:22:2: error: 'uint8_t' does not name a type
  uint8_t  m_dq_SCE, m_dq_RST, m_dq_DC, m_dq_DATA, m_dq_CLK;
  ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:23:11: error: 'uint8_t' does not name a type
  volatile uint8_t  *m_port_SCE, *m_port_RST, *m_port_DC, *m_port_DATA, *m_port_CLK;
           ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:27:11: error: 'uint8_t' does not name a type
  volatile uint8_t* Port2DDR(volatile uint8_t *port) {
           ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:35:12: error: 'uint8_t' does not name a type
   volatile uint8_t  *port_SCE, uint8_t  dq_SCE,
            ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:35:22: error: ISO C++ forbids declaration of 'port_SCE' with no type [-fpermissive]
   volatile uint8_t  *port_SCE, uint8_t  dq_SCE,
                      ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:35:32: error: 'uint8_t' has not been declared
   volatile uint8_t  *port_SCE, uint8_t  dq_SCE,
                                ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:36:12: error: 'uint8_t' does not name a type
   volatile uint8_t  *port_RST, uint8_t  dq_RST,
            ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:36:22: error: ISO C++ forbids declaration of 'port_RST' with no type [-fpermissive]
   volatile uint8_t  *port_RST, uint8_t  dq_RST,
                      ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:36:32: error: 'uint8_t' has not been declared
   volatile uint8_t  *port_RST, uint8_t  dq_RST,
                                ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:37:12: error: 'uint8_t' does not name a type
   volatile uint8_t  *port_DC, uint8_t  dq_DC,
            ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:37:22: error: ISO C++ forbids declaration of 'port_DC' with no type [-fpermissive]
   volatile uint8_t  *port_DC, uint8_t  dq_DC,
                      ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:37:31: error: 'uint8_t' has not been declared
   volatile uint8_t  *port_DC, uint8_t  dq_DC,
                               ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:38:12: error: 'uint8_t' does not name a type
   volatile uint8_t  *port_DATA, uint8_t  dq_DATA,
            ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:38:22: error: ISO C++ forbids declaration of 'port_DATA' with no type [-fpermissive]
   volatile uint8_t  *port_DATA, uint8_t  dq_DATA,
                      ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:38:33: error: 'uint8_t' has not been declared
   volatile uint8_t  *port_DATA, uint8_t  dq_DATA,
                                 ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:39:12: error: 'uint8_t' does not name a type
   volatile uint8_t  *port_CLK, uint8_t  dq_CLK
            ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:39:22: error: ISO C++ forbids declaration of 'port_CLK' with no type [-fpermissive]
   volatile uint8_t  *port_CLK, uint8_t  dq_CLK
                      ^
../display/../../AVR450_v14_charger_cpp/display/5110.h:39:32: error: 'uint8_t' has not been declared
   volatile uint8_t  *port_CLK, uint8_t  dq_CLK
                                ^
../display/5110.cpp:129:6: error: prototype for 'void LCD_5110::lcd_init(volatile uint8_t*, uint8_t, volatile uint8_t*, uint8_t, volatile uint8_t*, uint8_t, volatile uint8_t*, uint8_t, volatile uint8_t*, uint8_t)' does not match any in class 'LCD_5110'
 void LCD_5110::lcd_init(
      ^
In file included from ../display/5110.cpp:8:0:
../display/../../AVR450_v14_charger_cpp/display/5110.h:34:7: error: candidate is: void LCD_5110::lcd_init(volatile int*, int, volatile int*, int, volatile int*, int, volatile int*, int, volatile int*, int)
  void lcd_init(
       ^
../display/5110.cpp: In member function 'void LCD_5110::lcd_clear_area(unsigned char, unsigned char, unsigned char)':
../display/5110.cpp:211:37: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
     for(unsigned int i = start; i < end; i++) lcd_send(0, LCD_DATA);
                                     ^
../display/5110.cpp: In member function 'void LCD_5110::lcd_send(unsigned char, LCD_5110::LcdCmdData)':
../display/5110.cpp:281:15: error: 'm_port_DATA' was not declared in this scope
  Set(Port2DDR(m_port_DATA), m_dq_DATA, 1);
               ^
../display/5110.cpp:281:26: error: 'Port2DDR' was not declared in this scope
  Set(Port2DDR(m_port_DATA), m_dq_DATA, 1);
                          ^
../display/5110.cpp:281:29: error: 'm_dq_DATA' was not declared in this scope
  Set(Port2DDR(m_port_DATA), m_dq_DATA, 1);
                             ^
../display/5110.cpp:282:15: error: 'm_port_DC' was not declared in this scope
  Set(Port2DDR(m_port_DC), m_dq_DC, 1);
               ^
../display/5110.cpp:282:27: error: 'm_dq_DC' was not declared in this scope
  Set(Port2DDR(m_port_DC), m_dq_DC, 1);
                           ^
../display/5110.cpp:285:9: error: 'm_port_SCE' was not declared in this scope
     Set(m_port_SCE, m_dq_SCE, 0);//LCD_PORT &= ~LCD_SCE_PIN;
         ^
../display/5110.cpp:285:21: error: 'm_dq_SCE' was not declared in this scope
     Set(m_port_SCE, m_dq_SCE, 0);//LCD_PORT &= ~LCD_SCE_PIN;
                     ^
../display/5110.cpp:306:7: error: 'm_port_CLK' was not declared in this scope
   Set(m_port_CLK, m_dq_CLK, 1);//LCD_PORT |= LCD_CLK_PIN;
       ^
../display/5110.cpp:306:19: error: 'm_dq_CLK' was not declared in this scope
   Set(m_port_CLK, m_dq_CLK, 1);//LCD_PORT |= LCD_CLK_PIN;
                   ^
make: *** [display/5110.o] Błąd 1

18:00:48 Build Finished (took 2s.677ms)

oraz w liniach 5110.cpp:
[syntax=cpp]void LCD_5110::lcd_init(
volatile uint8_t *port_SCE, uint8_t dq_SCE,
volatile uint8_t *port_RST, uint8_t dq_RST,
volatile uint8_t *port_DC, uint8_t dq_DC,
volatile uint8_t *port_DATA, uint8_t dq_DATA,
volatile uint8_t *port_CLK, uint8_t dq_CLK
)[/syntax]
[syntax=cpp]Set(Port2DDR(m_port_DATA), m_dq_DATA, 1);
Set(Port2DDR(m_port_DC), m_dq_DC, 1);

// Enable display controller (active low)
Set(m_port_SCE, m_dq_SCE, 0);//LCD_PORT &= ~LCD_SCE_PIN;[/syntax]
i w 5110.h

Statystyki: Napisane przez LA72 — 30 sie 2017, o 17:05

Re: Przeportowanie AVR C++ na AVR C

2017-08-30T16:45:56+01:00

Witam. A dlaczego kolega chce go przeportować do C? Przejrzałem tak na szybko i wydaje mi się, że prędzej da się wytłumaczyć koledze czym jest klasa LCD, niż prowadzić za rękę w bezsensownym pisaniu tego w czystym C. Jeśli kolega ma jednak jakieś argumenty za tym żeby było C, to chętnie je wysłucham i przedyskutuje, bo na razie to wiemy tylko, że ma być C a czemu i po co to tajemnica.

Statystyki: Napisane przez kijas1 — 30 sie 2017, o 16:45

Przeportowanie AVR C++ na AVR C

2017-08-30T12:08:04+01:00

Witam.
Znalazłem w sieci ciekawy projekt ładowarki z LCD Nokii 5510.
http://community.atmel.com/projects/avr-universal-charger
Niestety kod źródłowy jest napisany w C++.
Proszę o pomoc w przeportowaniu kodu na C.

Za pomoc z góry dziękuje.

Statystyki: Napisane przez LA72 — 30 sie 2017, o 12:08