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.

Sign up for GitHub

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

Jump to bottom

don't response to SCSI_CMD_PREVENT_ALLOW_MEDIUM_REMOVAL #1

Closed
wants to merge 1 commit into from
Closed

don't response to SCSI_CMD_PREVENT_ALLOW_MEDIUM_REMOVAL #1

wants to merge 1 commit into from

Conversation

hathach
Copy link

@hathach hathach commented Jun 7, 2022

hathach/tinyusb#1327

  • response to SCSI_CMD_PREVENT_ALLOW_MEDIUM_REMOVAL as unsupported request in all msc examples. This is required for macos to send TEST_UNIT_READY regularly for "SDCard reader" application. Response OK to this request for non-removal device such as ram/flash disk can be beneficial since macos will skip TEST_UNIT_READY command and assume medium is always present (saved bandwidth).
  • tud_msc_set_sense(lun, SCSI_SENSE_NOT_READY, 0x3A, 0x00) is optional, since tinyusb stack will default to response with that when test unit ready callback return false

@hathach hathach closed this Jul 4, 2022
@hathach hathach mentioned this pull request Jul 4, 2022
Draft
dhalbert pushed a commit that referenced this pull request Nov 29, 2022
@jshimbo
Merge pull request #1 from jshimbo/jshimbo-patch-1
84da3b2 
Typo in docs for supervisor.ticks_ms()
dhalbert pushed a commit that referenced this pull request May 11, 2023
@ChenxiXiao
Merge pull request #1 from ChenxiXiao/ChenxiXiao-ZH_PINYIN
47a901d 
Update zh_Latn_pinyin.po
dhalbert pushed a commit that referenced this pull request May 26, 2023
@tkroo
Merge pull request #1 from tkroo/lolin-s3-mini
b56bc36 
Board files for LOLIN S3 MINI
dhalbert pushed a commit that referenced this pull request Oct 25, 2023
@MakerM0
Merge pull request #1 from MakerM0/magiclick_s3_n4r2
0bf4f19 
Add board def for MagiClick S3
dhalbert pushed a commit that referenced this pull request Feb 23, 2024
@mariopesch
Merge pull request #1 from mariopesch/add-sensebox-mcus2
74ad733 
Add senseBox MCU-S2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@hathach