readme and less verbose messages

README

@@ -1,16 +1,14 @@
 +++ Logitech Solar k750 Userspace "Driver" for Linux
 
-This python code represents an initial stab at reverse-engineering a
-solar-powered (photovaltic) keyboard.  Predictably, Logitech provides
-Windoze and Crackintosh versions of its software, but no Linux drivers.
+Userspace driver for Logitech solar-powered (photovaltic) keyboard.
+Predictably, Logitech provides Windoze and Crackintosh versions of its
+software, but no Linux drivers.
 
 Alas, I am not a hardware hacker but I spent some time reading the USB
 spec and wireshark usb packet dumps (did you know libpcap is compatible
 with USB? I didn't!) and fuzzing the hardware with hex values using
 pyusb.  To my surprise, it kinda works.
 
-I'm still tweaking the output but I think this is good enough for a
-public domain rev-eng hack.  Cheers,
-
-Noah K. Tilton
-November 23, 2011
+Note:  drop the udev rule in ./rules.d into your distro's custom rules
+directory (presently /etc/udev/rules.d/ on Archlinux).  May require
+editing/rebooting; YMMV.

logitech_k750.py

@@ -43,18 +43,18 @@ class USB(object):
                 (self.vendor_id, self.product_id))
 
     def detach(self, n):
-        print "detaching.... ",
+        #print "detaching.... ",
         try:
             self.device.detach_kernel_driver(interface=n)
-            print "Detached %s." % n
+            #print "Detached %s." % n
         except usb.core.USBError as e:
             print "couldn't detach, %s" % e
 
     def attach(self, n):
-        print "attaching....",
+        #print "attaching....",
         try:
             self.device.attach_kernel_driver(interface=n)
-            print "Attached %s." % n
+            #print "Attached %s." % n
         except usb.core.USBError as e:
             print "couldn't attach, %s" % e
 
@@ -74,11 +74,11 @@ class USB(object):
 
 if __name__ == '__main__':
 
-    while True:
     # find
     kb = USB( vendor_id=0x046d, product_id=0xc52b)
     kb.open( device_index=0, interface_indices=(2,0,), endpoint_index=0 )
 
+    kb.detach(2)
     kb.claim(2)
 
     # fuzz (ymmv here -- I used wireshark, and there is a lot of
@@ -96,13 +96,17 @@ if __name__ == '__main__':
                 kb.interface.bInterfaceNumber,
                 6000))
 
-        print "Charge: %s Lux: %s (%s | %s)" % \
-                (data[4],
-                int(round(((255*data[5])+data[6])/538.0, 2)*100),
-                data[5],
-                data[6])
+
+    charge  = data[4]
+    lux     = int(round(((255*data[5])+data[6])/538.0, 2)*100)
+    print "%s,%s" % (charge, lux)
+    #"Charge: %s Lux: %s (%s | %s)" % \
+    #(data[4],
+    #data[5],
+    #data[6])
 
     kb.release(2)
+    kb.attach(2)
     time.sleep(1)
 
 # Python™ ftw!