Re: sysless: Added-basic-support-for-NXP-LPC13xx-ARM-Cortex-M3.diff
Robothron 1715 (sysless@pandora.cz)
pecam1 at fel.cvut.cz
Sun Jun 20 20:50:05 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