Commit 31a3faf3 authored by Linus Torvalds's avatar Linus Torvalds

Merge branch 'for-4.13-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/libata

Pull libata fixes from Tejun Heo:
 "Late fixes for libata. There's a minor platform driver fix but the
  important one is READ LOG PAGE.

  This is a new ATA command which is used to test some optional features
  but it broke probing of some devices - they locked up instead of
  failing the unknown command.

  Christoph tried blacklisting, but, after finding out there are
  multiple devices which fail this way, backed off to testing feature
  bit in IDENTIFY data first, which is a bit lossy (we can miss features
  on some devices) but should be a lot safer"

* 'for-4.13-fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/libata:
  Revert "libata: quirk read log on no-name M.2 SSD"
  libata: check for trusted computing in IDENTIFY DEVICE data
  libata: quirk read log on no-name M.2 SSD
  sata: ahci-da850: Fix some error handling paths in 'ahci_da850_probe()'
parents 785373b4 2aca3923
...@@ -216,12 +216,16 @@ static int ahci_da850_probe(struct platform_device *pdev) ...@@ -216,12 +216,16 @@ static int ahci_da850_probe(struct platform_device *pdev)
return rc; return rc;
res = platform_get_resource(pdev, IORESOURCE_MEM, 1); res = platform_get_resource(pdev, IORESOURCE_MEM, 1);
if (!res) if (!res) {
rc = -ENODEV;
goto disable_resources; goto disable_resources;
}
pwrdn_reg = devm_ioremap(dev, res->start, resource_size(res)); pwrdn_reg = devm_ioremap(dev, res->start, resource_size(res));
if (!pwrdn_reg) if (!pwrdn_reg) {
rc = -ENOMEM;
goto disable_resources; goto disable_resources;
}
da850_sata_init(dev, pwrdn_reg, hpriv->mmio, mpy); da850_sata_init(dev, pwrdn_reg, hpriv->mmio, mpy);
......
...@@ -2411,6 +2411,9 @@ static void ata_dev_config_trusted(struct ata_device *dev) ...@@ -2411,6 +2411,9 @@ static void ata_dev_config_trusted(struct ata_device *dev)
u64 trusted_cap; u64 trusted_cap;
unsigned int err; unsigned int err;
if (!ata_id_has_trusted(dev->id))
return;
if (!ata_identify_page_supported(dev, ATA_LOG_SECURITY)) { if (!ata_identify_page_supported(dev, ATA_LOG_SECURITY)) {
ata_dev_warn(dev, ata_dev_warn(dev,
"Security Log not supported\n"); "Security Log not supported\n");
......
...@@ -60,7 +60,8 @@ enum { ...@@ -60,7 +60,8 @@ enum {
ATA_ID_FW_REV = 23, ATA_ID_FW_REV = 23,
ATA_ID_PROD = 27, ATA_ID_PROD = 27,
ATA_ID_MAX_MULTSECT = 47, ATA_ID_MAX_MULTSECT = 47,
ATA_ID_DWORD_IO = 48, ATA_ID_DWORD_IO = 48, /* before ATA-8 */
ATA_ID_TRUSTED = 48, /* ATA-8 and later */
ATA_ID_CAPABILITY = 49, ATA_ID_CAPABILITY = 49,
ATA_ID_OLD_PIO_MODES = 51, ATA_ID_OLD_PIO_MODES = 51,
ATA_ID_OLD_DMA_MODES = 52, ATA_ID_OLD_DMA_MODES = 52,
...@@ -889,6 +890,13 @@ static inline bool ata_id_has_dword_io(const u16 *id) ...@@ -889,6 +890,13 @@ static inline bool ata_id_has_dword_io(const u16 *id)
return id[ATA_ID_DWORD_IO] & (1 << 0); return id[ATA_ID_DWORD_IO] & (1 << 0);
} }
static inline bool ata_id_has_trusted(const u16 *id)
{
if (ata_id_major_version(id) <= 7)
return false;
return id[ATA_ID_TRUSTED] & (1 << 0);
}
static inline bool ata_id_has_unload(const u16 *id) static inline bool ata_id_has_unload(const u16 *id)
{ {
if (ata_id_major_version(id) >= 7 && if (ata_id_major_version(id) >= 7 &&
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment