Re: fatfs and io disk misc patches

[Savin Zlobec <savin at elatec dot si>]

Andrew Lunn wrote:

> On Tue, Jun 29, 2004 at 12:02:11PM +0300, Jani Monoses wrote:
> > > >      
> > > > +#define CYG_DISK_MAX_PART_NUM 4
> > > 
> > > Please could you make this a CDL option, and i think it should really
> > > be called something like CYGNUM_IO_DISK_PART_NUM
> >
> > Isn't this the constant 4, the number of primary partitions in an msdos
> > partition table, not something configurable?
>
> I don't know. Why are we limited to the restrictions of msdos anyway?
> I don't have any M$ OS on my disk, so why cannot i have 5 partitions?
>
> Even if it is a constant the name should change.
>  
Master boot record (first sector on the disk) contains the master 
partition table
and the master boot code. Master partition table has space for 4 
partitions - so
called primary partitions. If more partitions are desired on one disk, 
then  one
of the 4 primary partitions has to be designated as an extended 
partition and may
then be subdivided into multiple logical partitions.

The io_disk layer currently supports only primary partitions - this is 
where the
number 4 comes from.

I can make this an CDL option and set it to 4 by default, if the 
configured number
is 1, then only the first partition on the disk will be detected, if 2, 
then only the first
two ... and so on. Later when the support for extended partitions is 
added we can
increase the default.

Does this sound OK ?

savin