Mark Seaborn writes:
> Take a look at part of adfspart_check_ICS() in adfspart.c:
> >             if (p->size < 0 && adfspart_check_ICSLinux (dev, p->start)) {
>                     ^^^^^^^^^^^
> This will never be true because `size' is defined as an unsigned long. 
> Change the definition of struct ics_part and it's okay.

Ok, fixed.

> >                     if (p->size > 1)
> >                             add_partition (hd, minor, first_sector + p->start + 1, 
>p->size - 1);
> 
> In these two lines above, ``p->size'' must be changed into ``-p->size''.

Fixed as well.

> Once this is done, adfspart_check_ICSLinux() never returns true for me (it
> never matches the string `LinuxPart').  However, the fdisk code does match
> this, but seems to be doing the same thing, so I don't know what's going on
> there.  I can't find this string when I look at the disc with Zap.

Have you checked the sectors that are specified in the debug file
(Pipe:$.C_Debug.*) which are passed to blkio_read during the icside_readinfo()
function?
   _____
  |_____| ------------------------------------------------- ---+---+-
  |   |        Russell King       [EMAIL PROTECTED]      --- ---
  | | | |  http://www.arm.linux.org.uk/~rmk/armlinux.html    /  /  |
  | +-+-+                                                     --- -+-
  /   |               THE developer of ARM Linux              |+| /|\
 /  | | |                                                     ---  |
    +-+-+ -------------------------------------------------  /\\\  |
unsubscribe: body of `unsubscribe linux-arm' to [EMAIL PROTECTED]

Reply via email to