Hi, On Monday, September 23, 2013 04:53:52 PM Sergei Shtylyov wrote: > Hello. > > On 23-09-2013 1:51, Tejun Heo wrote: > > >> Not sure why you asked -- I'm not using this driver, neither I'm > > > Well, you have better grip of what's going on in the embedded world > > than me. I'm mostly curious whether adding dependency on PHY is okay. > > This driver already supports optional clock, the optional PHY support > seems analogous.
Right, this reminds me that PHY support should probably also be added to ahci_suspend() and ahci_resume(). Also please note the generic PHY framework is not yet merged in Linus' tree so this patch should probably be merged only after the generic PHY framework is in. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/