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

sg3_utils: sg_sat_datetime command to get/set the date & time on ATA devices. #37

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jrbauer-wdc
Copy link

Similar to sg_timestamp for SCSI, sg_sat_datetime allows the date and time (timestamp) to be set-on and returned-from ATA devices. This provides the ability to syncronize the system clock with the device. The SET DATE & TIME EXT and READ LOG [DMA] EXT ATA commands are sent using ATA PASSTHROUGH SCSI commands to set and retrieve the date and time on the device.

ATA devices.

Similar to sg_timestamp for SCSI, sg_sat_datetime allows the
date and time (timestamp) to be set-on and returned-from
ATA devices.  This provides the ability to syncronize the
system clock with the device.  The SET DATE & TIME EXT and
READ LOG [DMA] EXT ATA commands are sent using ATA
PASSTHROUGH SCSI commands to set and retrieve the date and
time on the device.
@doug-gilbert
Copy link
Owner

Thanks, this is great. Keeping the options similar to sg_timestamp (for SCSI devices) should help users. I merged it upstream, made some formatting changes and referenced it from sg_timestamp's manpage. The result is now mirrored it here. There was a minor CI problem (see the Actions tab) which I fixed. Often get hiccups with the MinGW compiler

@jrbauer-wdc
Copy link
Author

Thanks for picking this up, Doug. I compiled and tested with Cygwin, but not MinGW. I'll make sure to compile and test with MinGW in future contributions. Thanks for fixing that up and for adding the reference in sg_timestamp.

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.

None yet

2 participants