Hi Tejun, On Wed, Nov 9, 2016 at 10:15 PM, Tejun Heo <tj@xxxxxxxxxx> wrote: > Hello, > > On Wed, Sep 14, 2016 at 04:15:00PM +0530, Rameshwar Sahu wrote: >> > @@ -821,8 +823,6 @@ static int xgene_ahci_probe(struct platform_device >> > *pdev) >> > dev_warn(&pdev->dev, "%s: Error reading >> > device info. Assume version1\n", >> > __func__); >> > version = XGENE_AHCI_V1; >> > - } else if (info->valid & ACPI_VALID_CID) { >> > - version = XGENE_AHCI_V2; > > Can you please explain this part a bit? Everything else looks good to > me. Here we should not assume XGENE_AHCI_V2 always in case of having valid _CID in ACPI table. I need to remove this assumption because V1_1 has also valid _CID for backward compatibly with v1. > > Thanks. > > -- > tejun -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html