Rockbox Development > New Ports

Cowon D2

<< < (39/194) > >>

andiator:
@shotofadds
Yes it's me.
For the WPS and menu are the four direction buttons (up/down...) enough. Some plugins need more. And yes, the mappings have to be altered to be usefull.

cybergrind:
I tryed add some printf in disk.c and fat.c, and get log:

--- Code: ---mount
start mounting disks. fat_init. NUM_VOLUMES : 1

start disk_mount(0)

start fat_mount()
fat_mount() - BPB is not sane

start fat_mount()
fat_mount() - Couldn't read BPB (error code -1)

start fat_mount()
fat_mount() - BPB is not sane

start fat_mount()
This is not FAT32. Go away!

No partition foudn, trying to mount sector 0.
start fat_mount()
--- End code ---

after this I comment, near 360 line in fat.c:

--- Code: ---#else /* #ifdef HAVE_FAT16SUPPORT */
//        DEBUGF("This is not FAT32. Go away!\n");
//        printf("This is not FAT32. Go away!");
//        return -2;
--- End code ---

above this fragment we have such code:

--- Code: ---#ifdef HAVE_FAT16SUPPORT
        fat_bpb->is_fat16 = true;
        if (fat_bpb->dataclusters < 4085)
        { /* FAT12 */
            DEBUGF("This is FAT12. Go away!\n");
            printf("This is FAT12. Go away!");
            return -2;
        }
--- End code ---
maybe some defines is missed (2 GB model)?

shotofadds:
The first 4 "start fat_mount()" lines are because Rockbox attempts to mount each partition in the partition table in turn - but the D2 has no valid partition table, since it is formatted in "superfloppy" mode. The code in disk.c and fat.c is fine, don't worry about this.

The last line "No partition found, trying to mount sector 0. start fat_mount()" is the important one - what happens after this?

Are you familiar with the linux 'dd' utility (or it's Windows equivalent)? A dump of the first 512 bytes from your D2 would be useful at this point.

cybergrind:
Oh I'm sorry, not all log, last message

--- Code: ---This is not FAT32. Go away!
--- End code ---

When I added in config-cowond2.h:

--- Code: ---#define HAVE_FAT16SUPPORT
--- End code ---
All seems ok. Maybe this trouble appeared only in 2 Gb version, because defaulf system is fat16 =)

dump (attached) shows that filesystem is Fat16.

Another problem: something strange with screen during shutdown,
maybe we should add

--- Code: ---#define HAVE_LCD_SHUTDOWN
--- End code ---
into config-cowond2.h

and rename:

--- Code: ---static void lcd_display_off(void)
--- End code ---
to:

--- Code: ---void lcd_shutdown(void)
--- End code ---
in lcd-cowond2.c

and use it during shutdown.

P.S. I didn't familiar with ROCKBOX rules, maybe I should create and commit patches for this issues?

shotofadds:
I thought that might be the case: the missing log message was the important one ;D. I'll add in HAVE_FAT16SUPPORT next time I update the code.

Shutdown/poweroff does not currently work - this requires some more thought (more than just the LCD driver - powering off the CPU & hardware needs to be done too).

If you make patches, it's best to submit them to the tracker (Flyspray). But if they're just 1-liners like the above, post them here.

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version