Changeset 8659a17 in avrstuff for V-USB_Dev/firmwares/herePic/driver/herepic.cpp

Timestamp:

Sep 3, 2012, 8:21:47 PM (12 years ago)

Author:

Adrien Destugues <pulkomandy@…>

Branches:

main

Children:

c8ec8ce

Parents:

f0689d3

Message:

Progress on the host driver. Still untested.

git-svn-id: ​svn://pulkomandy.tk/avrstuff@66 c6672c3c-f6b6-47f9-9001-1fd6b12fecbe

File:

• V-USB_Dev/firmwares/herePic/driver/herepic.cpp (modified) (5 diffs)

V-USB_Dev/firmwares/herePic/driver/herepic.cpp

@@ -15 +15 @@
 #include <USBKit.h>
 
+#include "../shared.h"
+#include "icsp.h"
 
-// We need a subclass of USBRoster to enumerate devices. The Deviceadded hook
-// will be called for all plugged devices. If we don't find the correct one,
-// it will also be called later, as devices are plugged. So we can wait for the
-// right device to come in !
+
+// We need a subclass of USBRoster to enumerate devices.
 class MyUSBRoster: public BUSBRoster {
         public:
@@ -35 +35 @@
                 }
 
+                /* The DeviceAdded hook will be called for all plugged devices.
+                If we don't find the correct one, the method will also be called later,
+                as devices are plugged. So we can wait for the right device to come in!
+                */
                 status_t DeviceAdded(BUSBDevice* device)
                 {
@@ -41 +45 @@
                                 && strcmp(device->ProductString(), "HerePic") == 0)
                         {
-                                std::cout << "Programmer found !" << std::endl;
-                                // TODO send message to main app to tell it we're ready to go.
                                 fDevice = device;
+                                // Notify main thread it's ok to continue
                                 sem_post(&fDeviceLock);
                         }
@@ -121 +124 @@
         } while(nextopt != -1);
 
-        if (action <= 0)
+        // If we found no valid action request, print usage and exit
+        if (action <= 0 || action == 'h')
         {
                 usage(argv[0]);
-                exit(action);
+                // Exit code is EXIT_SUCCESS if we reach this by -h option. Otherwise
+                // the arguments are incorrect
+                exit(action < 0 ? action : 0);
         }
 
-        // TODO take action depending on command request in CLI
         // TODO handle the boring usb stuff... (might be nice to abstract it so we
         // can use both libusb and haiku usb kit)
@@ -140 +145 @@
 
         int deviceAvailable;
+        // TODO we rely on the initial device scan being done when we get here. Is
+        // that always true ? If not, we may print this message in cases where it
+        // is not needed.
         sem_getvalue(&deviceLock, &deviceAvailable);
         if(deviceAvailable == 0)
         {
-                std::cout << "Programmer not connected. Waiting for it..." << std::endl;
+                std::cout << "Programmer hardware not detected. Please connect it now..." << std::endl;
         }
 
+        // Wait for the device
         sem_wait(&deviceLock);
 
-        // Here, perform the programming cycle !
+        ICSP icsp(*theDevice);
+
+        // take action depending on command request in CLI
         switch(action)
         {
                 case 'r':
+                        // read the chip
+                        icsp.Read(filename);
+                        break;
                 case 'w':
+                        icsp.Write(filename);
+                        // Here, perform the programming cycle !
+                        break;
                 case 'e':
-                        ;
+                        icsp.Erase();
+                        // erase the chip
+                        break;
         }
+
+        exit(0);
 }