PaulStoffregen / USBHost_t36

USB Host Library for Teensy 3.6 and 4.0
170 stars 87 forks source link

Format cleanup - Plus Added support for MSC drives to enumerate partitions at MTP.Task() #92

Closed KurtE closed 2 years ago

KurtE commented 2 years ago

Hi Paul and @mjs513,

I thought I would issue a PR on this stuff, so you have a chance to look at it.

Some cleanup in Format code, like move secondary header file to sub-directory...

Lots more details up on the Forum postings, but trying to remove the need for Sketches to have to add code to check to see if there is a new File system that was plugged into USBHost. So I changed the linked list of USBDrive objects to keep in one list, and then at Task time, it checks to see if the status of these drives have changed, and takes care of doing the enumeration of partitions at this point. The removal was already done when we received the parent device was removed.

The USBFilesystem objects also are linked and allow to be queried to see if there state has changed and if ANY state has changed...

Also have parallel code in the MTP code that can detect when any state changed and if so work through and Add or Remove to MTP...

Not sure if you have time to look through this stuff here and forum or not, but thought I would mention the current stuff

mjs513 commented 2 years ago

@PaulStoffregen and @KurtE Things getting back to normal now :) More time to play.

Did confirm that merged changes still work using my convoluted test sketch. Have to say it is really nice not to have to add all that extra code to the sketch to determine when a new USB drive is inserted or removed.

KurtE commented 2 years ago

@PaulStoffregen @mjs513,

Paul,

Have you had a chance to look through this stuff? This is part of the stuff in the summary email.

Thanks Kurt