Hi Jennifer, On Mon, Nov 29, 2010 at 10:06:18AM +0800, Jennifer Li (TP) wrote: > Hi Chris, > > 1. We don't need disable ADMA on the 8120 device. For safety, we have > prepared a device ID for device 8120. > > 2. 0x01 bit is a lock/unlock bit. The CAPABILITIES register could be > opened by the 0x01 bit. If we set the 0x01, the CAPABILITIES register > can be written. We have to set the 0x01 bit. Perfect, thanks for the answers. I've added a comment explaining the unlock bit. I've queued your patch for inclusion in the Linux 2.6.38 release now. It'd be great if you could test the 2.6.38-rc1 kernel when it's released (sometime around February) and check that everything's working on your hardware. Thanks, - Chris. -- Chris Ball <cjb@xxxxxxxxxx> <http://printf.net/> One Laptop Per Child -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html