[coreboot] [PATCH 2/2] cs5536: Remove redundant enable_ide variable from ide device.

Mart Raudsepp mart.raudsepp at artecdesign.ee
Fri Jan 9 15:45:05 CET 2009


Ühel kenal päeval, R, 2009-01-09 kell 15:32, kirjutas Peter Stuge:
> Mart Raudsepp wrote:
> > cs5536: Remove redundant enable_ide variable from ide device.
> 
> Patch itself is clean, but I don't know if it is desirable to change
> the behavior this way. I could ack but I am not 100% sure about the
> disabling/enabling stuff - I remember a lot of discussion about it
> but I don't remember the outcome. :\

The controversial part is about such a variable in the southbridge
parent device that would select ide or nand. E.g what
sb->enable_ide_nand_flash kind of is right now - zero means IDE and
non-zero means Flash in a given chip select. I haven't gotten around to
investigating if that chip select and its setup can be moved to the
newly created NAND device or not - once I do that "controversy" will
arise.
The ide->enable_ide statictree struct member that this patch kills just
duplicates existing ide->enabled struct member.

> Will a dts expert please chime in?

That'd be appreciated


Mart Raudsepp





More information about the coreboot mailing list