On Thu, Oct 11, 2007 at 09:39:54AM -0700, ron minnich wrote:
attached. I am working to integrate the SIS NB support in, as the discussion seems to have gone silent.
I'm not sure what the course of action is here -- shall we try to modify the code ourselves or wait for SiS developers? I don't want to duplicate any efforts and we probably cannot easily test our changes right now...
These files are now used in more than one place, so we copy them here, prior to removing them elsewhere.
Nope, please make the moving of the files one atomic commit, i.e. use 'svn mv foo.h bar.h' as appropriate. Further _changes_ to the files (apart from moving them) should be an extra patch/commit.
I don't see a need for a long debate about location, given that we plan to transition to v3 in the next 18 months.
As Stefan said, you didn't specify a location anyway. Maybe simply src/include/? We have spd.h there, too.
There is no copyright notice, and I can't figure out where they came from; I am not sure we should put one in.
Yes, we should, as in every other file. But if they don't have a GPL header right now, we should first move the files with 'svn mv' and add headers later.
Index: spd_ddr2.h
Do we really want this in an extra file, or just merge it into the already existing src/include/spd.h?
Index: amdk8.h Index: amdk8_f.h
Why move these files? And from where to where? They're likely K8 specific and located in src/northbridge/amd/amdk8/ (which sounds correct), so why move them?
Uwe.