Re: sysless: Added-basic-support-for-NXP-LPC13xx-ARM-Cortex-M3.diff

Hoj, Michale, dik za pouceni.
Priste zkus sysless@pandora.cz :-)
Snad si tento slozity nazev zapamatuji.
Zaroven jsem vytvoril arch lpc_m3, Asi jsi myslel mach lpc_m3, ne?
Ano.
Zkus priste ten patch vygenerovat pomoci git format-patch a pripadne ho odeslat pomoci git send-email. Usnadnis mi tim praci: 1) budu moct patch okamzite aplikovat (pomoci git am), 2) bude tam diffstat (viz nize), takze jeste nez zacnu studovat ten patch, budu vedet jake soubory jsi pridal/menil/mazal.
Polepsim se. Ja se bal, jestli ten mail neni prave spatne ;-)
+++ b/app/arm/nazdar1311/Makefile
Chapu, ze mas rad cestinu, ale ja, kdyz vidim adresar s celkym nazvem v miste, kde se vetsina adresaru jmenuje anglicky, tak mam pocit, ze to je nejakej bordel, kterej tam kdosi zapomnel. Nechces to prejmenovat na hello? Budou ti vdecni i vsichni zahranicni studenti (a jejich vedouci), kteri k nam prijedou na staz a budem je nutit s tim pracovat.
Nikterak nelpim na nazvu cehokoli, natoz tohoto pasqilu, v syslessu, prejmenuj to popr. smaz dle sve libovule. Jinak, spise z hlediska psychohumoristickeho, rekl bych, ze "nazdar" v cizinci nemusi nutne pusobit dojmem hrubsiho chaosu, nez treba "pxmc" ci "kopete". (Natoz "bonjour" ci "salaam".)
+ LPC_SYSCON->SYSAHBCLKCTRL |= (1<<6); //enable clock/power for GPIO
Vyhledove bych se primlouval, aby 1<<6 bylo konstanta v headeru.
Mozna to v nejakem headeru i je, ten priklad jsem tam dal ciste jen proto, aby sel zkompilovat jeden hruznej program. Je v nem spousta balastu a je to eklhaft, ale dal jsem to tam, protoze v tu chvili jsem nemel nic jineho. Usoudil jsem, ze je to na podobne urovni nechutnosti jako ruzne nekdejsi arm_led_testy apod. Casem bych to bud nahradil nebo promazal.
+ LPC_IOCON->ARM_SWDIO_PIO1_3 |= 0x1;
dtto
To tam nejspis ani nic nedela, jak jsem posleze zjistil.
Nebylo by tohle lepsi spis jako knihovna?
Nebylo, protoze to neni uart, ale jakasi parodie na uart. Nejlepsi bude bud predelat spinkaouv uart_zen, nebo napsat neco vlastniho, na cemz budu v nejblizsich dnech snad delat.
Tahle funkce ma DOSovy konce radku, zbytek souboru ne. V ostatnich souborech je v koncich rakdu taky trochu zmanek.
Nevedel jsem, jak git zachazi s bilymi znaky, mel jsem za to, ze implicitne zmeny jen v bilych znacich ignoruje. Kazdopadne zkusim priste dodrzet vsude \n, abych v tom nedelal bordel.
A tolpc nam k tomu nedas?
Chtel jsem, aby tvorilo samostatny patch, protoze se jedna o ucelene dilo T.Baraka a P.Svobody. Neni to zatim v kompatibilnim stavu s nasim tolpc, tudiz nahradit ho nemuzeme, navic nemam sil to predelavat, takze jsem to chtel provizorne dat takhle bokem do arm_m3, aby slo nahravat do RAM, coz asi jiny program neumi. Kazdopadne s patchem jsem cekal na Tvou odpoved, abych vedel, jak ho spravne zformatovat. Ted uz to vim :-) Diky za revizi a zdravim, Marek
participants (1)
-
Marek Peca