ATNEL tech-forum

Re: SPI-Konfiguracja ATmega32-sprawdzenie poprawności

2023-11-04T22:15:17+01:00

moscow napisał(a):

Być może dobrą pointą będzie cytat z Blue Booka - str. 365 liniki 12-13 od góry
"Ty możesz korzystać z zapisu, który bardziej przypadnie ci do gustu"

Dlatego ja NIKOGO nie zamierzam przekonywać na siłę co chyba widzisz ... mam nadzieję, ale proszę uszanuj mój tok dydaktyki na tym forum nie polecaj tego do czego ja świadomie zniechęcam bo to zaciemnia obraz ok? Tylko o to chodzi a nie żebym ja tobie wmawiał że źle robisz jak tego używasz czy coś ok ? tym bardziej że chyba się znamy mniej więcej i rozumiesz moje podejście do nauczania języka C embedded

------------------------ [ Dodano po: kilkunastu sekundach ]

mario2015 napisał(a):

Zobacz listingi współczesnych programistów i wskaż firmy, które piszą kod w "starym stylu", który i tak ciężko się czyta.

BARDZO DOBRZE powiedziane !

Statystyki: Napisane przez mirekk36 — 4 lis 2023, o 22:15

Re: SPI-Konfiguracja ATmega32-sprawdzenie poprawności

2023-11-04T20:49:56+01:00

Quote:

"Ty możesz korzystać z zapisu, który bardziej przypadnie ci do gustu"

I ja to rozumiem i na pewno Mirek.
Przyjrzyj się jednak, który kod jest wtedy czytelniejszy i nie chodzi o przyzwyczajenia.
Zobacz listingi współczesnych programistów i wskaż firmy, które piszą kod w "starym stylu", który i tak ciężko się czyta.

Statystyki: Napisane przez mario2015 — 4 lis 2023, o 20:49

Re: SPI-Konfiguracja ATmega32-sprawdzenie poprawności

2023-11-04T20:03:34+01:00

mirekk36 napisał(a):

moscow napisał(a):

3) Być może warto zastanowić się nad użyciem makr np. makra "Byte Value" _BV i zamiast pisać

[...] toż ja walczę z nimi od Bluebooka a KAŻDY kto się z BB wychował na przesunięciach sam wie dlaczego tak to zwalczam - a tu proszę kolega moscow niefrasobliwie poleca ...

Być może dobrą pointą będzie cytat z Blue Booka - str. 365 liniki 12-13 od góry
"Ty możesz korzystać z zapisu, który bardziej przypadnie ci do gustu"

Statystyki: Napisane przez moscow — 4 lis 2023, o 20:03

Re: SPI-Konfiguracja ATmega32-sprawdzenie poprawności

2023-11-04T18:35:44+01:00

Dokładnie Mirku. Tak jak piszesz, że to kocie BV() ja też ich nie nawidzę.

Statystyki: Napisane przez mario2015 — 4 lis 2023, o 18:35

Re: SPI-Konfiguracja ATmega32-sprawdzenie poprawności

2023-11-04T14:45:14+01:00

moscow napisał(a):

3) Być może warto zastanowić się nad użyciem makr np. makra "Byte Value" _BV i zamiast pisać

O nie! że też ja tego nie zauważyłem .... BARDZO ale to BARDZO proszę nie polecać na tym forum tych MEGA MEGA KOCICH makr, toż ja walczę z nimi od Bluebooka a KAŻDY kto się z BB wychował na przesunięciach sam wie dlaczego tak to zwalczam - a tu proszę kolega moscow niefrasobliwie poleca ... Więc moja gorąca prośba - proszę tego więcej nie robić tu na forum ok?

makra _BV() są kocie, mega nieczytelne, i zwykle oznaczają że ktoś kto się nimi posługuje ma troszkę problemy ze zrozumieniem przesunięć, no nie każdy wiadomo ale tak zwykle jest i tu nawet nie ma co dyskutować .

A cieszę się, że koledze autorowi już weszły w krew przesunięcia i sam z obrzydzeniem jak widzę patrzy na kocie BV()

Statystyki: Napisane przez mirekk36 — 4 lis 2023, o 14:45

Re: SPI-Konfiguracja ATmega32-sprawdzenie poprawności

2023-11-04T11:15:39+01:00

Dzięki moscow. Z niektórych porad na pewno skorzystam, ponieważ słyszałem o nich ale normalnie w świecie o nich zapomniałem.
Nie skorzystam na pewno z pkt.3 , ponieważ do mnie bardziej przemawia taki zapis:
[syntax=c]SPSR |= (1<Tu od razu wiem, że w rejestrze SPSR ustawiam jedynkę w bicie SPI2X

Mirku, tobie też dziękuję. Wujek google miałem właśnie na myśli ten tlanslator:
https://translate.google.pl/
i nie jest on zbyt precyzyjny ale jest pomocny.
Na pewno sprawdzę ten drugi, który podałeś i chyba będę z niego korzystał.
Jeszcze raz dzięki i pozdrawiam.

Statystyki: Napisane przez mario2015 — 4 lis 2023, o 11:15

Re: SPI-Konfiguracja ATmega32-sprawdzenie poprawności

2023-11-03T22:11:38+01:00

mario2015 napisał(a):

powinno być dobrze ale kiepsko u mnie z angielskim a wujek google pomaga ale tylko troszeczkę

Ale po co ci wujek google ? tzn wiadomo też się przydaje no ale przecież TYLE RAZY wszędzie przypominam o byle translatorach internetowych, i aż wstyd w dzisiejszych czasach z nich nie korzystać tym bardziej, jeśli nie zna się do końca angielskiego, eeeeeh ale proszę jeszcze raz, dwie stronki. Pierwsza to PODSTAWA PODSTAW

https://translate.google.pl/

druga o wiele lepsza:

https://www.deepl.com/translator

więc na przyszłość - proszę nie pisz że nie masz jak dowiedzieć się o co chodzi w PDF po angielsku bo można sobie całą stronę skopiować z PDF i wrzucić do translatora. Lepszy techniczny jest ten drugi

Statystyki: Napisane przez mirekk36 — 3 lis 2023, o 22:11

Re: SPI-Konfiguracja ATmega32-sprawdzenie poprawności

2023-11-03T21:00:02+01:00

mario2015 napisał(a):

Chodziło mi tylko o sprawdzenie, czy taka inicjalizacja jest poprawna a moja funkcja wygląda tak:
}[/syntax]

Z punktu widzenia technicznego - wygląda na to, że jest poprawnie - to znaczy sprawdziłem nawet parę podstawowych scenariuszy czy działa używając układu Max7219 i procka ATmega32. I u mnie działa
Zrobiłem tylko taką modyfikację - akurat do swojego sprawdzenia:
[syntax=c]SPCR |= (1<Warto być może się zastanowić nad jakąś customizacją z tymi parametrami CPOL/CPHA (możliwość ich włączania) , bo po SPI dla każdego układu to może wyglądać inacze.

PS.
Wiadomo - o gustach się nie dyskutuje, ale można by też zrobić nieco stylowego liftingu - jeśli będziesz miał ochotę skorzystać to niżej kilka uwag

1) W definicjach warto użyć przedrostka który będzie mówił czego dotyczy definicja, zmienna itd.
zamiast CLK_DIV można użyć nazwy typu SPI_CLK_DIV
W takim przypadku jak będziesz później dołączał jakiś inny kod, to bardziej precyzyjna nazwa sprawi, że są mniejsze szanse na konflikt definicji/zmiennych .

2) Użycie zwykłego cyferka do definicji generalnie zadziała, ale mogą być kłopoty z czytelnościa - w przypadku wykorzystania takiego jak u Ciebie można się zastaowić nad rozwiązaniem w tym stylu
Zamiast używać "#define CLK_DIV numerek" można spróbować zrobić kilka definicji zawczasu
[syntax=c]#define SPI_CLK_DIV_2 2
#define SPI_CLK_DIV_4 4
#define SPI_CLK_DIV_8 8
itd.
póżniej użyć np.
#define SPI_CLK_DIV SPI_CLK_DIV_2

oraz
#if SPI_CLK_DIV == SPI_CLK_DIV_2
...
#endif[/syntax]
od razu widać co jest grane i czego dotyczy. Jeśli będzie to po prostu jakaś liczba nie związana do tego z wartością - za parę tygodni być może trzeba będzie przeczytać cały kod, żeby dojść o co chodziło (to znaczy zakładam, że wiele osób ma lepszą pamięć ode mnie, ale akurat u mnie to może nawet nie być parę tygodni )

3) Być może warto zastanowić się nad użyciem makr np. makra "Byte Value" _BV i zamiast pisać
[syntax=c]SPSR |= (1<zrobić to tak
SPSR |= _BV(SPI2X);[/syntax]

przynajmniej dla mnie tak jest czytelniek, bo wiemy jaka była intencja

podobnie staram się robić kiedy mam jakąś operację w kodzie - zamiast używać bezpośrednio wartości - robię sobię definicje
[syntax=c]zamiast
uint8_t valHi = val & 240;

#define KMC_4_OUT_OF_8_BIT_MASK_HI 0xf0u
uint8_t valHi = val & KMC_4_OUT_OF_8_BIT_MASK_HI;

zamiast
divided = val / 8;
lub
divded = val >> 3;

#define KMC_DIV_BY_8 0x03u
divided = val >> KMC_DIV_BY_8[/syntax]

widać od razu wtedy po kodzie o co koderowi chodziło.

No dobra, bo się za bardzo rozpisałem
Mam nadzieję, że to co napisałem wyżej bedzie pomocne

Statystyki: Napisane przez moscow — 3 lis 2023, o 21:00

Re: SPI-Konfiguracja ATmega32-sprawdzenie poprawności

2023-11-03T10:20:04+01:00

Nie chodzi o to, że coś mi nie działa.
Po prostu chcę sobie sam napisać tę bibliotekę, żeby lepiej wszystko zrozumieć, dlatego nie używam gotowca.
Podpieram się BB i poradnikami Mirka.
Co do instrukcji "switch", to wkleiłem nie ten kod, bo ten właściwy mam na #if-ach.
Chodziło mi tylko o sprawdzenie, czy taka inicjalizacja jest poprawna a moja funkcja wygląda tak:
[syntax=c]void InitSpi(void)
{
//ustawienie kierunku wyjściowego dla linii MOSI, SCK i CS
DDRB |= (1<//aktywacja SPI, tryb pracy Master
SPCR |= (1<
#if CLK_DIV == 1
// dzielnik przez 2
SPSR |= (1<#endif
#if CLK_DIV == 2
// dzielnik przez 4
#endif
#if CLK_DIV == 3
// dzielnik przez 8
SPCR |= (1<#endif
#if CLK_DIV == 4
// dzielnik przez 16
SPCR |= (1<#endif
#if CLK_DIV == 5
// dzielnik przez 32
SPCR |= (1<SPSR |= (1<#endif
#if CLK_DIV == 6
// dzielnik przez 64
SPCR |= (1<#endif
#if CLK_DIV == 7
// dzielnik przez 128
SPCR |= (1<#endif
#if CLK_DIV == 8
// dzielnik przez 64
SPCR |= (1<SPSR |= (1<#endif
}[/syntax]

Statystyki: Napisane przez mario2015 — 3 lis 2023, o 10:20

Re: SPI-Konfiguracja ATmega32-sprawdzenie poprawności

2023-11-02T20:35:21+01:00

W teorii wygląda "legitnie", choć użycie switch/case w przypadku kiedy wartość na wejściu w formie definicji może mieć niespecjalny sens (pewnie kompilator i tak to sobie zoptymalizuje, ale to inna sprawa).

Z tym działaniem to warto sprawdzić, czy inicjalizacja nie powinna zawierać bitów CPOL i/lub CPHA.
Do końca nie wiem, co może u Ciebie nie działać, ale poniżej podaję swoje dwie funkcje, których użyłem do sterowania układem Max7219 / 72221 - może się przydadzą jako referencja

[syntax=c]#include
#include

#define KM_SPI_DDR -> port SPI dla danego procka
#define KM_SPI_DDR_CS_OUT -> bit na porcie, który ma sterować CS
#define KM_SPI_PIN_SCK -> bit na porcie, na którym w procku jest sygnał SCK
#define KM_SPI_PIN_MOSI -> bit na porcie, na którym w procku jest sygnał MOSI

#define KM_SPI_PINS_SENDER(_BV(KM_SPI_PIN_SCK) | _BV(KM_SPI_PIN_MOSI))

#define KM_SPI_MASTER_SPEED_FOSC_DIV_4_A(KMC_ALL_BITS_CLEARED)
#define KM_SPI_MASTER_SPEED_FOSC_DIV_4_B(KMC_ALL_BITS_CLEARED)
#define KM_SPI_MASTER_SPEED_FOSC_DIV_16_A(_BV(SPR0))
#define KM_SPI_MASTER_SPEED_FOSC_DIV_16_B(KMC_ALL_BITS_CLEARED)
#define KM_SPI_MASTER_SPEED_FOSC_DIV_64_A(_BV(SPR1))
#define KM_SPI_MASTER_SPEED_FOSC_DIV_64_B(KMC_ALL_BITS_CLEARED)
#define KM_SPI_MASTER_SPEED_FOSC_DIV_128_A(_BV(SPR0)|_BV(SPR0))
#define KM_SPI_MASTER_SPEED_FOSC_DIV_128_B(KMC_ALL_BITS_CLEARED)
#define KM_SPI_MASTER_SPEED_FOSC_DIV_2_AKM_SPI_MASTER_SPEED_FOSC_DIV_4_A
#define KM_SPI_MASTER_SPEED_FOSC_DIV_2_B(_BV(SPI2X))
#define KM_SPI_MASTER_SPEED_FOSC_DIV_8_AKM_SPI_MASTER_SPEED_FOSC_DIV_16_A
#define KM_SPI_MASTER_SPEED_FOSC_DIV_8_B(_BV(SPI2X))
#define KM_SPI_MASTER_SPEED_FOSC_DIV_32KM_SPI_MASTER_SPEED_FOSC_DIV_64_A
#define KM_SPI_MASTER_SPEED_FOSC_DIV_32_B(_BV(SPI2X))
#define KM_SPI_MASTER_MASK_AKM_SPI_MASTER_SPEED_FOSC_DIV_128_A
#define KM_SPI_MASTER_MASK_BKM_SPI_MASTER_SPEED_FOSC_DIV_32_B

// użycie np: kmInitSpiSender_U(KM_SPI_MASTER_SPEED_FOSC_DIV_8_A, KM_SPI_MASTER_SPEED_FOSC_DIV_8_B);
void kmInitSpiSender_U(uint8_t clockRateA, uint8_t clockRateB) {
KM_SPI_DDR |= KM_SPI_PINS_SENDER;
KM_SPI_DDR_CS_OUT |= KM_SPI_MASK_CS_OUT;

SPCR = _BV(SPE) | _BV(MSTR) | _BV(CPOL) | clockRateA;
SPSR = (SPSR & ~KM_SPI_MASTER_MASK_B) | clockRateB;
}

void kmSpiSenderPut2Bytes_U(const uint8_t deviceNo, const uint8_t data1, const uint8_t data2) {
KM_SPI_PORT_CS_OUT &= ~(_BV(KM_SPI_CS_0));
SPDR = data1;
loop_until_bit_is_set(SPSR, SPIF);
SPDR = data2;
loop_until_bit_is_set(SPSR, SPIF);
KM_SPI_PORT_CS_OUT |= _BV(KM_SPI_CS_0);
}[/syntax]

Statystyki: Napisane przez moscow — 2 lis 2023, o 20:35

SPI-Konfiguracja ATmega32-sprawdzenie poprawności

2023-11-02T17:36:14+01:00

Witam wszystkich.
Chciałbym uruchomić sobie wyświetlacz ze sterownikiem ST7735S, który działa po SPI.
Problem polega na tym, że jeszcze nigdy nic "sam" nie robiłem po SPI. Zawsze gotowce.
Teraz chciałbym sobie napisać najprostszą bibliotekę SPI dla Atmega32 na podstawie BB i GB.
Nie po to aby wyważać otwarte drzwi, tylko, żeby to dobrze wszystko zrozumieć.
Mógłbym kupić profesjonalną bibliotekę Mirka, tyle tylko, że nie miał bym pojęcia jak to działa w środku.
Ja chcę to wszystko sprawdzić osobiście, żeby się tego nauczyć.
Chciałbym żeby mi ktoś sprawdził, czy funkcję, którą sobie napisałem do inicjalizacji SPI, ma prawo działać.
Chodzi o to, że w tej funkcji użyłem instrukcji SWITCH, która w zależności od zadanego parametru, ma ustawić szybkość magistrali.
Wszystko ma działać na sprzętowym SPI. Według noty pdf dla ATmega32, powinno być dobrze ale kiepsko u mnie z angielskim a wujek google pomaga ale tylko troszeczkę.
Poniżej mój kod do funkcji. Chodzi mi o inicjalizację SPI. Proszę o sprawdzenie.

[syntax=c]#define CLK_DIV 1

void InitSpi(void)
{
//ustawienie kierunku wyjściowego dla linii MOSI, SCK i CS
DDRB |= (1<//aktywacja SPI, tryb pracy Master
SPCR |= (1<
switch(CLK_DIV)
{
case 1:
{
// dzielnik przez 2
SPSR |= (1<}
break;

case 2:
{
// dzielnik przez 4
}
break;

case 3:
{
// dzielnik przez 8
SPCR |= (1<}
break;

case 4:
{
// dzielnik przez 16
SPCR |= (1<}
break;

case 5:
{
// dzielnik przez 32
SPCR |= (1<SPSR |= (1<}
break;

case 6:
{
// dzielnik przez 64
SPCR |= (1<}
break;

case 7:
{
// dzielnik przez 128
SPCR |= (1<}
break;

case 8:
{
// dzielnik przez 64
SPCR |= (1<SPSR |= (1<}
break;
}
}[/syntax]

Statystyki: Napisane przez mario2015 — 2 lis 2023, o 17:36