This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#9330 - patch for openBSD support for ipodpatcher
|
DetailsThe following patches will allow someone running openBSD to run ipodpatcher. This is my first pass at these patches, so they cannot be integrated without some problems.
A few outstanding questions remain: 1) should I use /dev/sd0c or /dev/rsd0c? 2) why do byteorder(3) functions exist in fat32format.c? 3) how do the developers want to handle disk geometry? I ask the last question because my patch overwrites the existing ioctl with what works for openBSD, rather than separate it out. And the patch doesn't fully use what is available in struct disklabel. I plan to re-write this one. The fat32format.diff comments out byteorder(3) functions. The ipodio-posix.diff converts the file to openBSD disk geometry. The ipodpatcher.diff corrects the device used. I have only tested the patches above for ipodpatcher on iPod nano 1st gen. 
fat32format.diff
(0.6 KiB)
fat32format.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-)
ipodio-posix.diff
(1.5 KiB)
ipodio-posix.c | 22 ++++++++++++++-------- 1 file changed, 14 insertions(+), 8 deletions(-)
ipodpatcher.diff
(0.7 KiB)
ipodpatcher.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) |
This task depends upon
2) On Linux I don't see swap16/32() functions in byteorder manpage, maybe they are specific to OpenBSD, and should be ifdef'ed ?
Something like #ifndef swap16 rather than #ifndef __OpenBSD__ . What do you think ?
I am planning to clean up the patches. These patches are more of a proof of concept, so that the developers would know what needs to be in place for rbutilqt to build on openBSD. However, before I spend time cleaning these up, I need a static tarball of the source required to build rbutilqt as of it's latest release. Then, I can obtain feedback from other openBSD users, so I can fine tune these patches and make they don't break other builds. I will build my own tarball in mean time.
I do agree with your point two. Here's a link to the byteorder(3) man page:
http://www.openbsd.org/cgi-bin/man.cgi?query=byteorder&apropos=0&sektion=0&manpath=OpenBSD+Current&arch=i386&format=html
The last two functions are the ones in question.