From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Subject: Re: [PATCH] ahci: imx: manage only sata_ref_clk in imx_sata_enable[disable] Date: Fri, 30 May 2014 09:55:10 +0200 Message-ID: <5388395E.80500@redhat.com> References: <1401289539-3485-1-git-send-email-shawn.guo@freescale.com> <5386026C.1010603@redhat.com> <538610D5.7070908@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Return-path: Received: from mx1.redhat.com ([209.132.183.28]:58185 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750970AbaE3Hzk (ORCPT ); Fri, 30 May 2014 03:55:40 -0400 In-Reply-To: Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Fabio Estevam Cc: Shawn Guo , Tejun Heo , Fabio Estevam , linux-ide@vger.kernel.org, "linux-arm-kernel@lists.infradead.org" , Bartlomiej Zolnierkiewicz Hi, On 05/28/2014 06:48 PM, Fabio Estevam wrote: > On Wed, May 28, 2014 at 1:37 PM, Hans de Goede wrote: > >> Ah yes I see, the ahci_imx suspend/resume handler now no longer >> touches the controller, but the ata core is still trying to use it. >> >> Ok, in that case I agree that your original patch is the best >> solution. >> >> Your original patch is: >> >> Acked-by: Hans de Goede > > I think you meant Shawn's patch, not mine :-) Right, so to be extra clear, the patch titled: "ahci: imx: manage only sata_ref_clk in imx_sata_enable[disable]" is: Acked-by: Hans de Goede Regards, Hans From mboxrd@z Thu Jan 1 00:00:00 1970 From: hdegoede@redhat.com (Hans de Goede) Date: Fri, 30 May 2014 09:55:10 +0200 Subject: [PATCH] ahci: imx: manage only sata_ref_clk in imx_sata_enable[disable] In-Reply-To: References: <1401289539-3485-1-git-send-email-shawn.guo@freescale.com> <5386026C.1010603@redhat.com> <538610D5.7070908@redhat.com> Message-ID: <5388395E.80500@redhat.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, On 05/28/2014 06:48 PM, Fabio Estevam wrote: > On Wed, May 28, 2014 at 1:37 PM, Hans de Goede wrote: > >> Ah yes I see, the ahci_imx suspend/resume handler now no longer >> touches the controller, but the ata core is still trying to use it. >> >> Ok, in that case I agree that your original patch is the best >> solution. >> >> Your original patch is: >> >> Acked-by: Hans de Goede > > I think you meant Shawn's patch, not mine :-) Right, so to be extra clear, the patch titled: "ahci: imx: manage only sata_ref_clk in imx_sata_enable[disable]" is: Acked-by: Hans de Goede Regards, Hans