-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
inotify of new file uploaded? #46
Comments
The inotify and mtp events are already supported. |
I have issues too with inotify. The issue also seems to hang the Linux host (KIO) when copying a file on the gadget. Disabling the inotify function from the config files definitely fix it. Hope that helps. |
As far i know, the Interrupt EP is working properly, at least with the targets i am using. Which target have you ? (CPU, Kernel version...) |
I found a few bugs (missing This is a small capture I got.
This happens after the last block of data of the Anyway, I have not understood why the My board is a AllWinner A33 cpu board similar to the Olimex A33 board. USB connector is a USB-C so it has some difference. Kernel is 4.19, usb driver is sun4i-usb-phy. The mtp responder is setup using FunctionFS, not GadgetFS. At the moment, due to some limitation, I can't test it with GadgetFS and anyway we couldn't use it because of our USB setup (we need to provide multiple interface). It is possible that the problem is in our kernel, I can't exclude. Anyway, disabling inotify just fix the issue on our side and the responder works perfectly on both linux and MacOS. |
More precisely: we're configuring the gadget using ConfigFS. |
for sure the interrupt endpoints are supported by FunctionFS. What might not be supported is the MTP file transfer cancellation/interruption, which is a specific low level usb message, but you don't appear to use it in the present case. |
I see that inotify is being used. I'm looking to get notified anytime a new file has been added to the MTP. Is this functionality currently added, or is this something I'll need to add separately.
The text was updated successfully, but these errors were encountered: