Re: [RFC PATCH v2 8/8] [SCSI] sr: check and enable Zero-power ODD support
From: Alan Stern <stern@rowland.harvard.edu>
Date: 2012-03-01 16:02:25
Also in:
linux-acpi, linux-scsi, lkml
On Thu, 1 Mar 2012, Lin Ming wrote:
ZPODD(Zero Power Optical Disk Drive) is a new feature in SATA 3.1 specification. It provides a way to power off unused ODD. ZPODD support is checked in in sr_probe(). can_power_off flag is set during suspend if ZPODD is supported. ATA port's runtime suspend callback will actually power off the ODD and its runtime resume callback will actually power on the ODD. When ODD is powered off(D3Cold state), inserting disk will trigger a wakeup event(GPE). GPE AML handler notifies the associated device. Then ODD is resumed in the notify handler.
I have one stylistic comment on this patch...
quoted hunk ↗ jump to hunk
diff --git a/drivers/scsi/sr.h b/drivers/scsi/sr.h index 37c8f6b..39b3d8c 100644 --- a/drivers/scsi/sr.h +++ b/drivers/scsi/sr.h@@ -42,6 +42,9 @@ typedef struct scsi_cd { unsigned readcd_cdda:1; /* reading audio data using READ_CD */ unsigned media_present:1; /* media is present */ + unsigned zpodd:1; /* is ZPODD supported */ + unsigned zpodd_event:1; +
You should not expect your readers to understand what "ZPODD" means. drivers/scsi/sr.h is used by lots of different people, many of whom will have no idea what it refers to, especially since it is part of the SATA spec and not the SCSI spec. You should provide a brief explanation. Alan Stern