ide-disk: move IDE_DFLAG_DOORLOCKING flag handling to idedisk_set_doorlock()
There should be no functional changes caused by this patch. Acked-by: Borislav Petkov <petkovbb@gmail.com> Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
This commit is contained in:
@@ -817,12 +817,21 @@ static ide_driver_t idedisk_driver = {
|
|||||||
static int idedisk_set_doorlock(ide_drive_t *drive, int on)
|
static int idedisk_set_doorlock(ide_drive_t *drive, int on)
|
||||||
{
|
{
|
||||||
ide_task_t task;
|
ide_task_t task;
|
||||||
|
int ret;
|
||||||
|
|
||||||
|
if ((drive->dev_flags & IDE_DFLAG_DOORLOCKING) == 0)
|
||||||
|
return 0;
|
||||||
|
|
||||||
memset(&task, 0, sizeof(task));
|
memset(&task, 0, sizeof(task));
|
||||||
task.tf.command = on ? ATA_CMD_MEDIA_LOCK : ATA_CMD_MEDIA_UNLOCK;
|
task.tf.command = on ? ATA_CMD_MEDIA_LOCK : ATA_CMD_MEDIA_UNLOCK;
|
||||||
task.tf_flags = IDE_TFLAG_TF | IDE_TFLAG_DEVICE;
|
task.tf_flags = IDE_TFLAG_TF | IDE_TFLAG_DEVICE;
|
||||||
|
|
||||||
return ide_no_data_taskfile(drive, &task);
|
ret = ide_no_data_taskfile(drive, &task);
|
||||||
|
|
||||||
|
if (ret)
|
||||||
|
drive->dev_flags &= ~IDE_DFLAG_DOORLOCKING;
|
||||||
|
|
||||||
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int idedisk_open(struct inode *inode, struct file *filp)
|
static int idedisk_open(struct inode *inode, struct file *filp)
|
||||||
@@ -845,9 +854,7 @@ static int idedisk_open(struct inode *inode, struct file *filp)
|
|||||||
* since the open() has already succeeded,
|
* since the open() has already succeeded,
|
||||||
* and the door_lock is irrelevant at this point.
|
* and the door_lock is irrelevant at this point.
|
||||||
*/
|
*/
|
||||||
if ((drive->dev_flags & IDE_DFLAG_DOORLOCKING) &&
|
idedisk_set_doorlock(drive, 1);
|
||||||
idedisk_set_doorlock(drive, 1))
|
|
||||||
drive->dev_flags &= ~IDE_DFLAG_DOORLOCKING;
|
|
||||||
check_disk_change(inode->i_bdev);
|
check_disk_change(inode->i_bdev);
|
||||||
}
|
}
|
||||||
return 0;
|
return 0;
|
||||||
@@ -862,11 +869,8 @@ static int idedisk_release(struct inode *inode, struct file *filp)
|
|||||||
if (idkp->openers == 1)
|
if (idkp->openers == 1)
|
||||||
ide_cacheflush_p(drive);
|
ide_cacheflush_p(drive);
|
||||||
|
|
||||||
if ((drive->dev_flags & IDE_DFLAG_REMOVABLE) && idkp->openers == 1) {
|
if ((drive->dev_flags & IDE_DFLAG_REMOVABLE) && idkp->openers == 1)
|
||||||
if ((drive->dev_flags & IDE_DFLAG_DOORLOCKING) &&
|
idedisk_set_doorlock(drive, 0);
|
||||||
idedisk_set_doorlock(drive, 0))
|
|
||||||
drive->dev_flags &= ~IDE_DFLAG_DOORLOCKING;
|
|
||||||
}
|
|
||||||
|
|
||||||
idkp->openers--;
|
idkp->openers--;
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user