[haiku-commits] Re: r36416 - haiku/trunk/src/add-ons/kernel/bus_managers/ata

  • From: Ingo Weinhold <ingo_weinhold@xxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 22 Apr 2010 21:42:17 +0200

On 2010-04-22 at 16:13:32 [+0200], axeld@xxxxxxxxxxxxxxxx wrote:
> Author: axeld
> Date: 2010-04-22 16:13:32 +0200 (Thu, 22 Apr 2010)
> New Revision: 36416
> Changeset: http://dev.haiku-os.org/changeset/36416/haiku
> 
> Modified:
>    haiku/trunk/src/add-ons/kernel/bus_managers/ata/ATAChannel.cpp
>    haiku/trunk/src/add-ons/kernel/bus_managers/ata/ATADevice.cpp
>    haiku/trunk/src/add-ons/kernel/bus_managers/ata/ATAPrivate.h
> Log:
> * Got rid of the ATA_BLOCK_SIZE define.
> * The logical and physical block sizes are now retrieved from the info 
> block,
>   though only the logical one is actually used (where ATA_BLOCK_SIZE was 
>   used
>   before).
> * Also added retrieving the offset of the first logical block inside the 
> first
>   physical one.
> * All this stupidity just for Windows XP, it's really sad.
> * Added "const" to some methods, removed some superfluous ';'.

Breaks CD boot under qemu (OpenSuse 11.2).

CU, Ingo

Other related posts: