Franck_ | fixed, see mail list | 01:31 |
---|---|---|
Franck_ | bye | 01:31 |
jonibo | olofk_htc: why do you think that or1ksim.cfg belongs in the kernel??? | 12:28 |
jonibo | dependencies are top-down, not bottom-up | 12:29 |
juliusb | that was my thinking too. I can't recally seeing qemu configs or arm simulator configs in their branches | 12:29 |
jonibo | there should be an or1ksim.cfg for the a sane kernel configuration in or1ksim itself | 12:30 |
juliusb | well, not their branches, in their arch paths | 12:30 |
juliusb | jonibo: that's a good point, actually! | 12:30 |
juliusb | we should have some preset defaults built in I reckon | 12:30 |
juliusb | one for newlib bare metal software, one for kernel | 12:30 |
jonibo | yeah, or1ksim should actually default to sane defaults even without a config file | 12:30 |
juliusb | currently it assumes you don't want any memory - a bad assumption | 12:30 |
jonibo | heh | 12:30 |
juliusb | mmm, mabye I can fix this | 12:30 |
juliusb | i think that'd be extremely handy for all or1k hackers | 12:30 |
olofk_htc | jonibo: I just asked because some instructions assume that the file is there | 12:33 |
olofk_htc | And since I have always cloned your git tree, I assumed it would be included upstream | 12:34 |
jonibo | right... no, I removed it when pushing upstream | 12:35 |
jonibo | you can always get it back, though, if you have both git repos locally | 12:35 |
olofk_htc | I see your points. We should probably vad some default configs to or1ksim instead | 12:35 |
jonibo | for example, assuming you have two remotes 'upstream' and 'jonas', then you can pull out the or1ksim.cfg file like this: | 12:36 |
jonibo | git checkout jonas -- arch/openrisc/or1ksim.cfg | 12:36 |
jonibo | (I hope that's right... off the top of my head) | 12:36 |
olofk_htc | It's ok for me. Just need to direct the noobs to a good config file :) | 12:36 |
jonibo | yeah, i'd throw one into or1ksim proper... I think that's the best approach... or just make ork1sim work 'sanely' without a config file, like juliusb said | 12:37 |
olofk_htc | Missed that, but I agree. Especially the memory should have a sane default | 12:38 |
juliusb | the most common use cases of teh simulator are running bare-metal and linux, so I reckon a default for both of those would be good | 12:40 |
olofk_htc | What would be the difference between those two cases? | 12:42 |
jonibo | For the Linux case, you needsat least 16MB or you can't boot the statically linked busybox image that everyone's using | 12:43 |
olofk_htc | Aha | 12:43 |
jonibo | i imagine you can get by with less for a bare-metal app, but a sane default should probably represent something that you can actually purchase on the market | 12:43 |
olofk_htc | But a standard set of peripherals could be the same, I guess | 12:44 |
jonibo | yeah, for sure... | 12:44 |
jonibo | just the minimum really | 12:44 |
olofk_htc | Ethernet, uart, gpio(?)... | 12:45 |
jonibo | really only UART, I'd say | 12:46 |
jonibo | but there's an argument to be made for including ethernet | 12:46 |
juliusb | well maybe keep it in sync with the defconfig of the kernel? | 12:46 |
jonibo | yeah, exactly... I think that's UART and ethernet only | 12:47 |
juliusb | olofk_htc: difference wouldn't be much I guess | 12:47 |
juliusb | but memory size and peripherals would probably be it | 12:47 |
juliusb | newlib's or1ksim default is pretty basic - doesn't even rely on a UART being there | 12:49 |
juliusb | and I guess makes an assumption about the clock frequency | 12:49 |
Franck_ | Hi there | 12:49 |
juliusb | Franck_: hi | 12:49 |
jonibo | hi Franck_ | 12:49 |
olofk_htc | Couldn't we just det 16MB default if that's required for busybox? It can always be overridden in the cfg | 12:50 |
jonibo | saw your email | 12:50 |
jonibo | you don't need to be pulling in libgcc.c | 12:50 |
jonibo | there's something wrong with the search path to libgcc.a in your toolchain | 12:50 |
Franck_ | the kernel does it too | 12:50 |
jonibo | try running GCC in verbose mode (-v argument) to see where it's trying to find the library | 12:50 |
jonibo | the kernel's a special case | 12:50 |
Franck_ | uboot too :) | 12:50 |
jonibo | also a special case :) | 12:50 |
Franck_ | so as barebox :) | 12:51 |
jonibo | yeah, you may be right, come to think of it | 12:51 |
jonibo | I don't know enough about barebox | 12:51 |
juliusb | olofk_htc: I reckon perhaps we find a default case that will be good for both bare-metal and the kernel defconfig | 12:51 |
Franck_ | barebox = uboot | 12:51 |
jonibo | ...but it sounds like a 'bootloader' type app | 12:51 |
jonibo | ok | 12:51 |
juliusb | perhaps overspecced for bare-metal but that's fine | 12:51 |
jonibo | juliusb: overspecced is fine... it's better that it runs than that you get hit by strange errors like aborting during boot | 12:52 |
juliusb | ya I agree | 12:52 |
olofk_htc | I'm not sure we even support a board with 8MB, except for a smaller variant of ordb1 | 12:53 |
Franck_ | jonibo: so you confirm I have no choice | 12:53 |
jonibo | you're using -nostdinc? | 12:55 |
Franck_ | yes | 12:55 |
jonibo | yeah, you've got a 'freestanding' application, so you'll need to pull in support libraries by hand | 12:55 |
jonibo | i.e. you've got no choice | 12:55 |
juliusb | I suspected that, too | 12:55 |
Franck_ | I'll try to get libgcc compile in my appli | 12:55 |
Franck_ | or a least missing functions | 12:55 |
--- Log closed Wed Dec 14 15:20:41 2011 | ||
--- Log opened Wed Dec 14 15:21:19 2011 | ||
-!- Irssi: #openrisc: Total of 10 nicks [0 ops, 0 halfops, 0 voices, 10 normal] | 15:21 | |
-!- Irssi: Join to #openrisc was synced in 156 secs | 15:23 | |
--- Log closed Wed Dec 14 15:29:08 2011 | ||
--- Log opened Wed Dec 14 15:29:22 2011 | ||
-!- Irssi: #openrisc: Total of 10 nicks [0 ops, 0 halfops, 0 voices, 10 normal] | 15:29 | |
-!- Irssi: Join to #openrisc was synced in 22 secs | 15:29 | |
Tartarus | Hey all | 22:27 |
Tartarus | Anyone around with a board that can confirm USB works in U-Boot still? | 22:28 |
Tartarus | And preferably that can also build u-boot if I point at a github or so :) | 22:28 |
stekern | Tartarus: It worked last I tested, what is the issue? | 22:29 |
Tartarus | stekern: The changes to make it work on openrisc broke ARM, and I've patched things so ARM works again | 22:30 |
Tartarus | And I hope, didn't break OR | 22:30 |
stekern | ah, Tom Rini? | 22:31 |
stekern | didn't make the connection, Stefan here | 22:32 |
Tartarus | Yes, hi :) | 22:32 |
stekern | I'll be happy to test your changes | 22:32 |
Tartarus | k, thanks | 22:32 |
Tartarus | kicking github now | 22:32 |
Tartarus | (and work proxy, sigh) | 22:33 |
Tartarus | Sigh, here's the patch stekern: https://dl.dropbox.com/s/2ml741qzwcykime/0001-WIP-USB-Use-get-set-_unaligned_le16-around-wMaxPacke.patch?dl=1 | 22:36 |
stekern | great, thanks, I'll just set up the board with usb and test it | 22:37 |
Tartarus | thanks | 22:38 |
Tartarus | stekern: FYI there's also #u-boot where some of the other custodians hang out | 22:39 |
stekern | ah, I'll join (although I'm just a wannabe custodian so far ;)) | 22:41 |
Tartarus | heh, well now that /next is open maybe Wolfgang will merge the OR code | 22:42 |
Tartarus | And he's in there so you can ask him :) | 22:42 |
stekern | right after breaking ARM is probably the right moment asking for favors? :P | 22:43 |
stekern | I'm getting a compile error in usb.c:384 | 23:04 |
stekern | breaking out a ep_wMaxPacketSize = get_unaligned_le16(..) and then doing the le16_to_cpus on that fixes that | 23:06 |
Tartarus | hm | 23:07 |
stekern | you're missing an & there too, but just adding that didn't remove all errors | 23:07 |
Tartarus | Er, what's the compiler error, first? | 23:08 |
stekern | otherwise all looks good, I've got USB up and running without any bus errors with your patch applied and mine reverted | 23:08 |
Tartarus | Ah, we can back out yours as well, OK | 23:09 |
Tartarus | I hadn't, just built on top :) | 23:09 |
Tartarus | So, lemme spin up a v2 quick | 23:09 |
stekern | oh, the error vanished in my screen backlog | 23:10 |
stekern | w8 I'll bring it back | 23:10 |
* Tartarus assumes an ICE | 23:11 | |
stekern | http://pastebin.com/ | 23:12 |
stekern | http://pastebin.com/fzBh04T9 | 23:12 |
Tartarus | k | 23:14 |
Tartarus | http://pastebin.com/0yah5gsN | 23:14 |
Tartarus | is what you have now? | 23:14 |
Tartarus | I need to step away for a few, but if so, Tested-by: here and I'll put that in the patch once I figure out how to fix the checkpatch warnings | 23:18 |
stekern | http://pastebin.com/cu3RqGdr | 23:18 |
Tartarus | Der, I thought I tried that and had a warning, but I don't | 23:19 |
Tartarus | thanks | 23:19 |
stekern | but yes, you can add; Tested-by: Stefan Kristiansson <stefan.kristiansson@saunalahti.fi> on it | 23:21 |
stekern | thanks for cleaning up the mess btw | 23:23 |
Generated by irclog2html.py 2.15.2 by Marius Gedminas - find it at mg.pov.lt!