BALATON Zoltan <bala...@eik.bme.hu> writes: > There are several definitions of MAX_IDE_BUS in different boards (some > of them unused) with the same value. Move it to include/hw/ide/internal.h > to have it in a central place. > > Signed-off-by: BALATON Zoltan <bala...@eik.bme.hu>
This one feels a bit questionable. The number of (PATA) IDE buses provides by a host bus adapter depends on the HBA. It happens to be 2 for all HBAs we implement, but it could really be anything. Similar for SATA, where the common number is 6, but could really be anything. I can't see offhand whether any HBA we implement provides a different number. By moving MAX_IDE_BUS to include/hw/ide/internal.h, you bake the accidental commonality into the interface to the IDE core. I'd prefer not to.