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

Marek Peca pecam1 at fel.cvut.cz
Sun Jun 20 20:50:15 CEST 2010


Hoj, Michale,

dik za pouceni.

> Priste zkus sysless at 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


More information about the Sysless mailing list