Re: [RFC PATCH 08/15] ata: ahci_platform: Manage SATA PHY

From: Bartlomiej Zolnierkiewicz
Date: Wed Sep 25 2013 - 08:17:07 EST



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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/