C in funkcije ter #define

Pozdrav vsem,

prosil bi da mi pomagate rešiti nekakšen problem zaradi premalo programerskega znanja.

Imam h in c file za vpis v RAM mikroprocesorja, in sicer funkcija za vpis, pripravo in brianje so takšne: (funkc0)

//vpis v ram
int iap_copy_ram_to_flash(void* ram_address, void* flash_address, unsigned int count) 
.....
//priprava sektorja
int iap_prepare_sector(unsigned int sector_start, unsigned int sector_end)
.....
//brisanje sektorja
int iap_erase_sector(unsigned int sector_start, unsigned int sector_end)
.....

Potem pa iz mojega razreda kličem te funkcije tako: (funkc1)

//priprava sektorja
iap_status = (__e_iap_status) iap_prepare_sector(CONFIG_FLASH_SECTOR,
	    	            (CONFIG_FLASH_SECTOR + CONFIG_FLASH_SECTOR_SIZE));
	    	    if (iap_status != CMD_SUCCESS) while(1);
.....
//brisanje sektorja
iap_status = (__e_iap_status) iap_erase_sector(CONFIG_FLASH_SECTOR,
	    	            (CONFIG_FLASH_SECTOR + CONFIG_FLASH_SECTOR_SIZE));
	    	    if (iap_status != CMD_SUCCESS) while(1);
.....
//pisanje v sektor
iap_status = (__e_iap_status) iap_copy_ram_to_flash(&demo_messg,
	    	            (void *)CONFIG_FLASH_OFFSET, 256);
	    	    if (iap_status != CMD_SUCCESS) while(1);

Sedaj pa za test sem imel definirane spremenljivke in makroje tako:

#define CONFIG_FLASH_OFFSET            0x00006000				//preberi 26.3.6. table 371 obvezno !!!!!!			
#define CONFIG_FLASH_SECTOR            (CONFIG_FLASH_OFFSET >> 12)
#define CONFIG_FLASH_SECTOR_SIZE    1
static uint8_t demo_messg = 123;
__e_iap_status iap_status;

Zdaj bi pa jaz rad naredil nekakšno unierzalno funkcijo, ki bi imela vhodne parametre, naslov in podatek. Tako da bi rad samo poklical funkc1, v katero bi poslal parametre naslov in podatek ter bi funkc1 naredila vse tiste korake zgoraj ter klicala funkc0.

Funkc0 je splošna in je nebi rad spreminjal ker je najbolj preizkuena ter robustna.
Problem se pojavi ko želim računati ter definirati ter parametre

#define CONFIG_FLASH_OFFSET            0x00006000				//preberi 26.3.6. table 371 obvezno !!!!!!			
#define CONFIG_FLASH_SECTOR            (CONFIG_FLASH_OFFSET >> 12)
#define CONFIG_FLASH_SECTOR_SIZE    1
static uint8_t demo_messg = 123;

Jaz vbistvu sploh ne vem kakšnega tipa bi dal te parametre da bi jih v vhodo v funkc1 sprepoznalo ter poslalo naprej v funkc0.

Za lažje razumevanje prilagam še celotne c in h ter main programe.

main.c:

/*
*
*	shranjevanje skozi IAP v ram 
*	glede naslova moraš obvezno prebrati 26.3.6. FLASH CONFIGURATZION FOR LPC1114 SERIES TABEL 371 TAM
*	ntr piše kak je size naslovov. 
*/

#include "iap_driver.h"

#define CONFIG_FLASH_OFFSET            0x00006000				//preberi 26.3.6. table 371 obvezno !!!!!!			
#define CONFIG_FLASH_SECTOR            (CONFIG_FLASH_OFFSET >> 12)
#define CONFIG_FLASH_SECTOR_SIZE    1

static uint8_t demo_messg = 123;
char sv[] = "";
static uint8_t st;

int main(void) {
	__e_iap_status iap_status;
	iap_init();

	 while(1){ //infinite loop

 /* Prepare the sector for erase */
	    	    iap_status = (__e_iap_status) iap_prepare_sector(CONFIG_FLASH_SECTOR,
	    	            (CONFIG_FLASH_SECTOR + CONFIG_FLASH_SECTOR_SIZE));
	    	    if (iap_status != CMD_SUCCESS) while(1);

	    	    /* Erase the sector */
	    	    iap_status = (__e_iap_status) iap_erase_sector(CONFIG_FLASH_SECTOR,
	    	            (CONFIG_FLASH_SECTOR + CONFIG_FLASH_SECTOR_SIZE));
	    	    if (iap_status != CMD_SUCCESS) while(1);

	    	    /* Prepare the sector for writing */
	    	    iap_status = (__e_iap_status) iap_prepare_sector(CONFIG_FLASH_SECTOR,
	    	            (CONFIG_FLASH_SECTOR + CONFIG_FLASH_SECTOR_SIZE));
	    	    if (iap_status != CMD_SUCCESS) while(1);

	    	    /* write data to flash */
	    	    iap_status = (__e_iap_status) iap_copy_ram_to_flash(&demo_messg,
	    	            (void *)CONFIG_FLASH_OFFSET, 256);
	    	    if (iap_status != CMD_SUCCESS) while(1);


	    	    st = * ((uint8_t *) 0x00006000);					///branje iz flasha na omenjen naslov 

	    	    itoa(st, sv, 10);			//pretvorba v char
	    	    PotisniChar(0, 0, 1, sv);		//izpis na LCD

	}
}

iap_driver.c:

/*

 */

#include <LPC122x.h>
#include "iap_driver.h"

/*
 * The IAP funtion address in LPC11xx ROM
 */
#define IAP_ADDRESS            0x1FFF1FF1

/*
 * Command codes for IAP
 */
#define PREPARE_SECTOR        50
#define COPY_RAM_TO_FLASH    51
#define ERASE_SECTOR        52
#define BLANK_CHECK_SECTOR    53
#define READ_PART_ID        54
#define READ_BOOT_CODE_REV    55
#define COMPARE                56
#define REINVOKE_ISP        57
#define READ_UID            58

typedef unsigned int (*IAP)(unsigned int[], unsigned int[]);
static const IAP iap_entry = (IAP) IAP_ADDRESS;

/*---------------------------------------------------------------------------
 * Public functions
 */

/**
 * Init IAP driver
 * @return    0 for success
 */
int iap_init(void) {
    /* Need to update 'SystemCoreClock' according to the current clock settings
     * It's needed as IAP parameter
     */
    SystemCoreClockUpdate();
    return 0;
}

/**
 * Erase flash sector(s)
 *
 * @param sector_start  The start of the sector to be erased
 * @param sector_end    The end of the sector to be erased
 *
 * @return CMD_SUCCESS, BUSY, SECTOR_NOT_PREPARED_FOR_WRITE_OPERATION,
 *         or INVALID_SECTOR
 */
int iap_erase_sector(unsigned int sector_start, unsigned int sector_end) {
    unsigned int command[5];
    unsigned int result[4];

    command[0] = ERASE_SECTOR;
    command[1] = (unsigned int) sector_start;
    command[2] = (unsigned int) sector_end;
    command[3] = SystemCoreClock / 1000;
    iap_entry(command, result);

    return (int) result[0];
}

/**
 * Prepare flash sector(s) for erase / writing
 *
 * @param sector_start  The start of the sector to be prepared
 * @param sector_end    The end of the sector to be prepared
 *
 * @return CMD_SUCCESS, BUSY, or INVALID_SECTOR
 */
int iap_prepare_sector(unsigned int sector_start, unsigned int sector_end) {
    unsigned int command[5];
    unsigned int result[4];

    command[0] = PREPARE_SECTOR;
    command[1] = (unsigned int) sector_start;
    command[2] = (unsigned int) sector_end;
    iap_entry(command, result);

    return (int) result[0];
}

/**
 * Copy RAM contents into flash
 *
 * @param ram_address    RAM address to be copied
 *                       It should be in word boundary
 * @param flash_address  Flash address where the contents are to be copied
 *                       It should be within 256bytes boundary
 * @param count          Number of data to be copied (in bytes)
 *                       The options: 256, 512, 1024, 4096
 *
 * @return CMD_SUCCESS, BUSY, or INVALID_SECTOR
 */
int iap_copy_ram_to_flash(void* ram_address, void* flash_address,
        unsigned int count) {
    unsigned int command[5];
    unsigned int result[4];

    command[0] = COPY_RAM_TO_FLASH;
    command[1] = (unsigned int) flash_address;
    command[2] = (unsigned int) ram_address;
    command[3] = count;
    command[4] = SystemCoreClock / 1000;
    iap_entry(command, result);

    return (int) result[0];
}

ter iap_driver.h

/*

 */

#ifndef IAP_DRIVER_H_
#define IAP_DRIVER_H_

/*
 * IAP status codes
 */
typedef enum {
    CMD_SUCCESS = 0,
    INVALID_COMMAND,
    SRC_ADDR_ERROR,
    DST_ADDR_ERROR,
    SRC_ADDR_NOT_MAPPED,
    DST_ADDR_NOT_MAPPED,
    COUNT_ERROR,
    INVALID_SECTOR,
    SECTOR_NOT_BLANK,
    SECTOR_NOT_PREPARED_FOR_WRITE_OPERATION,
    COMPARE_ERROR,
    BUSY,
} __e_iap_status;

/**
 * Init IAP driver
 * @return    0 for success
 */
int iap_init(void);

/**
 * Erase flash sector(s)
 *
 * @param sector_start  The start of the sector to be erased
 * @param sector_end    The end of the sector to be erased
 *
 * @return CMD_SUCCESS, BUSY, SECTOR_NOT_PREPARED_FOR_WRITE_OPERATION,
 *         or INVALID_SECTOR
 */
int iap_erase_sector(unsigned int sector_start, unsigned int sector_end);

/**
 * Prepare flash sector(s) for erase / writing
 *
 * @param sector_start  The start of the sector to be prepared
 * @param sector_end    The end of the sector to be prepared
 *
 * @return CMD_SUCCESS, BUSY, or INVALID_SECTOR
 */
int iap_prepare_sector(unsigned int sector_start, unsigned int sector_end);

/**
 * Copy RAM contents into flash
 *
 * @param ram_address    RAM address to be copied
 *                       It should be in word boundary
 * @param flash_address  Flash address where the contents are to be copied
 *                       It should be within 256bytes boundary
 * @param count          Number of data to be copied (in bytes)
 *                       The options: 256, 512, 1024, 4096
 *
 * @return CMD_SUCCESS, BUSY, or INVALID_SECTOR
 */
int iap_copy_ram_to_flash(void* ram_address, void* flash_address,
        unsigned int count);

#endif /* IAP_DRIVER_H_ */

Najlepša hvala za pomoč, upam da mi boste lahko pomagali in se bil dovolj razumljiv sploh kaj hočem.
Zelo me bega spremenljivka tipa void *, ki je v funkk0. Kakšna je pravzaprav to spr.

Spremenljivka input_data je kazalec, probaj tak:

prestavljen_data = *(uint8_t*)input_data;

Zgornja koda spremeni tip spremenljivke input_data v kazalec na uint8_t ter vzame prvi zlog iz tega naslova.

Aha, jaz sem predvideval da se podatek nahaja na naslovu na katerega kaže input_data. Očitno ne. Torej če zdaj prav razumem je podatek, ki ga rabiš shranjen v input_data? Če ja, zakaj je input_data tipa void* ? Naj bo kar tipa uint8_t in ne rabiš nič typecastat.

input_data je kazalec, toej verjetno 32 bitov.
prestavljen_data pa 8 bitni int.

Zakaj bi tlačil prvo v drugega? Kaka logika bi to naj bila? Kaj s tem hoče koda narediti?

Sploh, ker potem prestavljen_data uporabiš samo enkrat, kot parameter za iap_copy_ram_to_flash().
Ne bi tam kar direktno input_data uporabil?

void * je kazalec na "nekaj", torej na objekt, ki nima tipa. Smisel takega kazalca je ravno v tem, da ni treba cast operatorja pri prirejanju...

In pa kako bi naredil da bi shranil polje int v spomin, polje veliko 336 bitov (42bajtov). Pač for zanka spomin povečuje pa shranjujem?
So še kakšne druge opcije.

Zlo na hitro sem pregledu vs post tko da mogoče sm ki spustil ma vseeno, SaveIntoFlashINT maš definirano kot:

void SaveIntoFlashINT(void* naslov, void* input_data)

Notr daš dva parametra, pointer na naslov in pointer na podatke. Ti funkcijo kličeš kot:

SaveIntoFlashINT(F_DRZAVA, 4);

Sklepam da je F_DRZAVA naslov, 4 pa je direktna vrednost podatka, kar pomoje ni prov ker glede na definicijo SaveIntoFlashINT bi moral dat notr pointer ki kaže na podatek 4 (mogoče se pa motim in je 4 dejansko pointer na podatke). Glede na definicijo iap_copy_ram_to_flash

int iap_copy_ram_to_flash(void* ram_address, void* flash_address, unsigned int count);

si jaz tko razlagam da se preprosto kliče na ta način:

iap_status = (__e_iap_status) iap_copy_ram_to_flash(input_data, (void *)naslov, 256);

V vsakem primeru lahko pišeš le bloke po 256 bajtov.

MrStein je 29. jul 2015 ob 15:00 izjavil:

V vsakem primeru lahko pišeš le bloke po 256 bajtov.

Ne vem če to drži. Ponavadi je omejitev v brisanju, ki je po podatkih iz datasheet-a 512 bytov. Pišeš pa lahko 4 byte naenkrat. Vsaj tako je delovalo na STM32F4, ko sem pred leti implementiral podobno zadevo (branje in zapisovanje konfiguracije kar v flash pomnilnik).

Edit: Sem se spomnil da sem zadevo uporabil tudi za nalaganje nove verzije programske kode preko HTTP/FTP protokola.

Zadeva teče na LPC1114, na njem imam Menu in bi rad da kar user nastai bi se to shranilo. Ker ta LPC nima EEPROM moram shranjevati preko IAP v ram (flash) ne vem zdaj kam. Shranjuje se na način prikazan tule, dostopaš pa do njega s kazalcem.
Vedno je treba pred pisanjem podatka pirpaviti sektor in ga nato zapisati, sedaj sem naredil to funkcijo, je ista kot zgoraj le brez brisanja na začetku:

void SaveIntoFlashINT(void* naslov, void* input_data){
	unsigned int cfs = ((unsigned int)naslov >> 12);
	unsigned int cfss = 1;
		static uint8_t prestavljen_data = 0;
	    // Prepare the sector for writing
	    iap_status = (__e_iap_status) iap_prepare_sector(cfs,
	            (cfs + cfss));
	    if (iap_status != CMD_SUCCESS) while(1);
	    prestavljen_data = input_data;
	    // write data to flash
	    iap_status = (__e_iap_status) iap_copy_ram_to_flash(&prestavljen_data,
	            (void *)naslov, 256);
	    if (iap_status != CMD_SUCCESS) while(1);
}

Problem pa je da ta LPC nima funkcije da bi izbrisal posamezen page v sektorju ampak moraš zbrisati celoten sektor. Sedaj pa se mi poraja vprašanje ali moram vedno zbrisati sektor pred pisanjem ali ne. Ker se mi v vpisovanje na isto mesto drugiš tretjič ... zgodi da nimam več noter številke ampak samo ničlo.
Recimo pri meniju imam ko pritisnem tipko menu da potrdim izbiro jezikqa primer za kurzor na prvem mestu.

switch (y) {
	case 1: {
	country = 0;
	SaveIntoFlashINT(F_DRZAVA, country);
	drzava = 0;		//english
	drzavaDow =* ((uint8_t *) F_DRZAVA);
	drzavaDow++;	//ker imamo prejšnje jezike nastavljene od 0 naprej morajo biti da deluje spodnja formula od 1 naprej da velja
	ZacetniZaslon();
....

Niti ne vem ali mi spravi v drzavaDow celo številko ali samo prvo oz. kaj drugega.

Idejo imam sedaj takšno:

void PrirpaviPodatek(unsgined int pod){
              var RAM |= pod << i;
              if(i < 32){i++;}else{
                           i = 0;
                            ShraniVFlash(pod);
              }
}

Samo to mi zdaj shrani največ 32 bitov, jaz jih pa moram 336 naenkrat. Ne vem kako naj spravim toliko bitov v FLASH.

Imam pa še eno opcijo, saj skozi IAP omogoče zapis 256B 512, 1024 ... B, zato se mora omogočati da ti tudi dobiš takšen pdoatek.
Nova ideja če bi postavil kazalec kar na array?
Zdaj imam tak array:
char* LunicaSoncek[48] = {};

Toda rabim pa samo polje veliko 336 bitov, ker se nastavi 336 enič ali nul, nič več ne znam definirati polja.
Sedaj ima narejeno tako da najbolj enostavo vnašam kar stringe "0" ali "1" kar pa je absolutno peveliko za shranjevanje.

Vesoljc tvoja ideja mi je všeč toda nisem sposoben narediti kaj takega.

Ja program mi trenutno zasede cca 22k in potem imam podatke na 6 sektorju to je na cca 24k naslovu. Razmišljal sem da bi podatke dal na 6 in 7 sektor, to sta zadnja dva.
En sektor je velik 4k, ima jih 8.

Fora je v tem da se bo program še dopolnjevla in za zdaj upal da mi ne bo zasedel do 6 sektorja. Ok po tvojem načinu bi lahko zato uporabil samo en sektor v katerega bi shranjeval podatke. recimo 7 sektor, za kar pa verjetno program ne bo dosegel.

V podatke moram shraniti Lunico ali Sonček (1 ali 0) na pol urni set, torej 48 nastavitev za en dan krat 7 je to 336 bitov. Plus shraniti moram še izbiro uporabnika za jezik, to pa so številke od 1 do 7. (engl, slo, ger, ita, fra, tur, rus.).

Malce nerodno si se tega lotil. Tvoje rutine so za IAP (in application programming). Zato tudi tako veliki bloki za shrannjevanje.
Poišči raje eno kodo za emulacijo EEPROM-a. Sam sem izhajal iz AN2594: EEPROM emulation in STM32F10x microcontrollers.

Sedaj če prav razumem rabim bitfield, ki je struktura v katero se iz RAM-a skopirajo podatki.

drVrh je 30. jul 2015 ob 12:35 izjavil:

Tale LPC nima EEPROM-a, ravno v tem poanta vsega ....

Saj poanta je ravno to, da emuliraš eeprom funkcionalnost v flashu. Poglej AN, ki sem ga omenil.

OK ga recimo offsetam za toliko mest tja, sja to vem.
recimo potem takole:

Izflasha izf;
izf.jezik = *(uint8_t *) F_ADDR_OF_DATA;
izf.LunicaSoncek =  *(uint8_t *) (F_ADDR_OF_DATA << 8); //naslov je šiftan za 8 bitov naprej.

Kako se vam zdi ?

Ko želim definirati strukturu mi vrže vn napake:

	struct Podatki{
		unsigned int Jezik			: 8;
		unsigned int Ponedeljek		: 48;
		unsigned int Torek			: 48;
		unsigned int Sreda 			: 48;
		unsigned int Cetrtek 		: 48;
		unsigned int Petek 			: 48;
		unsigned int Sobota 		: 48;
		unsigned int Nedelja 		: 48;
	};

width of 'Cetrtek' exceeds its type
prav tako za ponedeljek, torek, sreda, petek, sobota in nedelja. Za jezik ne vrže ven nič.

Zanima me ali je s for zanko možno prestavljati elemente v strukturi.

void LoadFromFlash(){
	
	//prvi pdoatek
	for(int a = 0; a < 8; a++){
		Podatki.Jezik = *(uint8_t *) F_DRZAVA + a;		
	}
	//drugi podatek
	for(int a = 0; a < )
V TEJ FOR ZANKI BI RAD NAPOLNIL DNEVE ... EN ELEMENT JE DOLG 24 BITOV EN DAN JE DOLG DVA ELEMNETA ... KO BI NAPOLNILO EN ELEMENT BI PRESTAVIL NA DRUGEGA 

}

Kot sem že napisal si delaš daleč preveč dela in odkrivaš toplo vodo. Uporabljaš IAP funkcionalnost za nekaj drugega in zato imaš toliko težav. Googlaj "EEPROM emulation in Flash" pa boš našel en kup zadetkov s kodo, ki dela točno to, kar ti želiš (shranjevanje podatkov v flash pomnilnik in branje iz njega).

Sem napisal funkcijo za loadanje iz flasha

void LoadFromFlash(){
	int stevc;

	memcpy(Podatki.Jezik, F_DRZAVA, 1);
	for(stevc = 0; stevc < 7;  stevc++){

		memcpy(Podatki.dawt[stevc].AM, F_DRZAVA, 3);
		memcpy(Podatki.dawt[stevc].PM, F_DRZAVA, 3);
	}
}

Strukturo sem naredil tako:

	typedef struct Dan{
		long AM : 24;
		long PM : 24;
	}Dan;

	struct Podatki{
		unsigned long Jezik	 : 8;
		struct Dan dawt[7];

	}Podatki;

Pri stavkih z memcpy mi vrže ven warning:

passing argument 1 of 'memcpy' makes pointer from integer without a cast

(4 x warning)

Kako se znebim tega warning-a, saj če dam pred Podatki.dawt[stevc].AM znak & mi napiše error, da ne gre dostopa do naslova bitfielda.

Zdravo,

želim sprogramirati strukturo s 24 bit fieldi, ko jo napolnim s 24 enicami, rezulat ni pravi.
struktura je:

union 
{
	struct
	{
		long AM :24;
	}dayAM;
	
	struct
	{
		long PM :24;
	}dayPM;
}tDay[7];

int main(){
	char str[80];
	char str1[50];
	long long kvoc = 0;
	int day = 0;
	int ura = 0;
	int binNum[24], i, j = 0;
	while(day < 7){	
		for(ura = 0; ura < 24; ura++){			
			tDay[day].dayAM.AM |= (1 << ura);			
		}
		if(ura >= 24){
			day++;
			ura = 0;
		}
	}
	
	tDay[1].dayAM.AM = (1 << 0 | 1 << 1 | 0 << 2);	
	day = 0;
	while(day < 7){
		printf( "Value : %u\n", tDay[day].dayAM.AM);
		kvoc = tDay[0].dayAM.AM;
		printf( "Velikost : %u\n", sizeof(kvoc));
		day++;
	}

V prvem izpisu se mi mora izpisati 16777215, saj je to pretvorba 24 enic v DEC zapis. Izpiše pa se mi: 4294967295
Kaj delam narobe?

Zanima me še ali je možno kako pogledati kakšno vrednost ima določen bitField, recimo if(bitField[15] == 1){....

Ugotovil sem da mi napolni vse 32 bitov z enicam, jaz pa ima for zanko narejeno samo do 24, ne znajdem se zakaj.

union 
{
	struct
	{
		long AM :24;
	}dayAM;	
	struct
	{
		long PM :24;
	}dayPM;
}tDay[7];

	while(day < 7){	
		for(ura = 0; ura < 24; ura++){			
			tDay[day].dayAM.AM |= (1 << ura);			
		}
		if(ura >= 24){
			day++;
			ura = 0;
		}
	}

drVrh je 12. avg 2015 ob 13:44 izjavil:

Ko želim podatke shrnaiti v dayAM.AM, se mi podatke shraniju tudi v dayPM.PM, zakaj?

Ker si dayAM in dayPM delita isti prostor - uporabljaš namreč union.