Introduce Genesys GL3224 quirks
ClosedPublic
Actions

Authored by mw_semihalf.com on May 25 2017, 3:12 PM.

Details

Reviewers

zbb
mav
imp

Commits

rS319117: Introduce Genesys GL3224 quirks

Summary

The Genesys chip is failing when issueing READ_CAP(16) command.
Force a quirk to disable it and use READ_CAP(10) instead.

Also, depending on used firmware, GL3224 can be recognized
either as 'storage device' or 'mass storage class' -
enable both variants in scsi_quirk_table.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

mw_semihalf.com created this revision.May 25 2017, 3:12 PM

I have no objections about the first part, I saw enough USB devices with broken implementation, but what exactly the second part is about? What problem It causes? From one side MSC specification indeed does not specify REPORT LUNS command, but does it create some problems here that require the quirk?

GL3224 is a dual LUN, but fails to report them properly - the outcome without quirks are errors and undetected devices, connected to it.

OK. I have no objections. Though may be we could workaround REPORT LUNS globally for all USB MSC devices somehow.

This revision is now accepted and ready to land.May 25 2017, 5:39 PM

Closed by commit rS319117: Introduce Genesys GL3224 quirks (authored by wma). · Explain WhyMay 29 2017, 9:23 AM

This revision was automatically updated to reflect the committed changes.