Skip to content
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

mtp-responder: Update comment / sync from stable branch #16

Conversation

JianyuWang0623
Copy link
Member

@JianyuWang0623 JianyuWang0623 commented Jan 8, 2025

Summary

  1. Update comment / sync from stable branch
  2. Move mtp_port.c and mtp_usb_driver_nuttx.c back to external - external/MTP: Pick commits of mtp_usb_driver_nuttx.c and mtp_port.c from mtp-responder/src external#12

Impact

MTP

Testing

CI

Donny9 and others added 2 commits January 8, 2025 10:41
Signed-off-by: dongjiuzhu1 <dongjiuzhu1@xiaomi.com>
Signed-off-by: wangjianyu3 <wangjianyu3@xiaomi.com>
@JianyuWang0623
Copy link
Member Author

JianyuWang0623 commented Jan 8, 2025

Check

$ git fetch gerrit
$ git fetch openvela

$ git log gerrit/****-******-4.0  --pretty=format:"%h %s" | while read commit; do commit_id=$(echo $commit | awk '{print $1}'); commit_msg=$(echo $commit | cut -d" " -f2-); echo "Commit Message: $commit_msg"; done | sort -u > /tmp/sync_mtp_4_0

$ git log gerrit/****-******-3.5  --pretty=format:"%h %s" | while read commit; do commit_id=$(echo $commit | awk '{print $1}'); commit_msg=$(echo $commit | cut -d" " -f2-); echo "Commit Message: $commit_msg"; done | sort -u > /tmp/sync_mtp_3_5

$ git log openvela/dev --pretty=format:"%h %s" | while read commit; do commit_id=$(echo $commit | awk '{print $1}'); commit_msg=$(echo $commit | cut -d" " -f2-); echo "Commit Message: $commit_msg"; done | sort -u > /tmp/sync_mtp_openvela

$ diff /tmp/sync_mtp_4_0 /tmp/sync_mtp_openvela | grep "< "
< Commit Message: external/mtp: adjust max io buffer size to reduce memory used 10M -> 2M # c48568f # No need
< Commit Message: external/mtp-responder: Make operation CopyObject optional              # PR #10
< Commit Message: mtp-responder: adjust config about device/vendor name and storage name  # 0c1733f # No need
< Commit Message: mtp-responder: fix compile break when disable mtp-responder.            # PR #16 

$ diff /tmp/sync_mtp_3_5 /tmp/sync_mtp_openvela | grep "< "
< Commit Message: external/mtp-responder: add debug log for transfer file from device to PC            # (revert)
< Commit Message: external/mtp-responder: Make operation CopyObject optional                           # PR #10
< Commit Message: mtp-responder: Fix strict-prototypes warnings                                        # 757ebfb # No need
< Commit Message: Revert "external/mtp-responder: add debug log for transfer file from device to PC"   # (revert)

$ rm /tmp/sync_mtp_4_0 /tmp/sync_mtp_3_5 /tmp/sync_mtp_openvela

Waiting to be merged: #10, #16

Signed-off-by: wangjianyu3 <wangjianyu3@xiaomi.com>
…ernal

No need prebuilt

Signed-off-by: wangjianyu3 <wangjianyu3@xiaomi.com>
@xiaoxiang781216 xiaoxiang781216 merged commit 9ab294b into open-vela:dev Jan 8, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants