Hi
> I have checked the adm80211 code as well, it seems to behave quite the > same, with the most notable difference the fact that adm80211 writes the > READ_OPCODE and the word index within a single command, while in > eeprom_93cx6 this is split into 2 seperate write commands. > I have not yet tested the exact impact for rt2x00 devices when they would > combine the write commands, but if that has no impact (or the adm80211 > suffers no impact when the write commands are being split) then rt2x00 and > adm80211 could indeed share the eeprom_93cx6 module. Hm, which way is the correct way? (is there a correct way?)
I have tested rt2x00 with combining the read command and word index writing, and it is working perfectly.
> If eeprom_93cx6 is moved to the /lib folder of the kernel, should it be > approved by a different maintainer first? > I don't think so, but I'm not sure. BTW, I think there can actually be another user of this code. Take a look at drivers/scsi/aic7xxx/aic7xxx_93cx6.c I'm not sure who the maintainer for this is. Should be okay to just have adm8211 and rt2x00 using eeprom_93cx6 for now, unless you have a spare aic7xxx supported device lying around. ;)
Nope, don't have that device. I'll create a patch that will place the eeprom module in the /lib folder. I'll send that to the list as soon as possible. Ivo - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
