Vojtech Pavlik wrote: > > On Sat, Nov 30, 2002 at 12:53:35PM -0500, Mark D. Studebaker wrote: > > > Vojtech Pavlik wrote: > > > > > > On Mon, Nov 25, 2002 at 07:07:41PM -0500, Mark D. Studebaker wrote: > > > > > > > everything should be in by now. > > > > > > Thanks. In the meantime I obtained a winbond w83791d CNR testing card > > > and implemented/fixed PEC and process calls, and block reads/writes in > > > the driver. ARP now works with smbus-arp.c. Patch attached. > > > > > > > Please review / update documentation for dependencies on i2c, etc. > > > > I already took a shot at README, INSTALL, QUICKSTART. > > > > > > Thanks. All looks OK. > > > > > > > Please test compilation/installation on 2.2/2.4. > > > > > > 2.2 doesn't (cannot) work, the driver uses too much of 2.4 > > > functionality. 2.2 doesn't support the x86_64 (Hammer) architecture > > > anyway ... > > > > > > > > > If it's too hard to make it work on kernel 2.2, > > I can try, but other stuff is more pressing than this. > > > we'll have to do something > > in Module.mk so it doesn't try to compile > > Hmm, yes. > > > - or else remove it from Module.mk completely. > > > If the 8111 is for hammer, then is there a CONFIG_xxx option for hammer > > in the kernel .config we can look for? > > Yes, it is, and the option would be CONFIG_X86_64. Anyway, you can run a > normal i386 linux on Hammer also, so it'd make sense to try it on i386 > systems as well. > > -- > Vojtech Pavlik > SuSE Labs To make things simple then, I'll put something in Module.mk that checks for CONFIG_X86_64. This will keep it from compiling in 2.2. If we think of something better, or in the future when we remove support for 2.2, we can remove the restriction. mds
