[{"id":2912311,"web_url":"http://patchwork.ozlabs.org/comment/2912311/","msgid":"<611c8eeb-fa7e-286c-bef2-0e24cc2b6b2e@opensource.wdc.com>","list_archive_url":null,"date":"2022-06-14T08:45:35","subject":"Re: [PATCH v4 18/23] ata: libahci_platform: Add function returning a\n clock-handle by id","submitter":{"id":82259,"url":"http://patchwork.ozlabs.org/api/people/82259/","name":"Damien Le Moal","email":"damien.lemoal@opensource.wdc.com"},"content":"On 6/10/22 17:17, Serge Semin wrote:\n> Since all the clocks are retrieved by the method\n> ahci_platform_get_resources() there is no need for the LLD (glue) drivers\n> to be looking for some particular of them in the kernel clocks table\n> again. Instead we suggest to add a simple method returning a\n> device-specific clock with passed connection ID if it is managed to be\n> found. Otherwise the function will return NULL. Thus the glue-drivers\n> won't need to either manually touching the hpriv->clks array or calling\n> clk_get()-friends. The AHCI platform drivers will be able to use the new\n> function right after the ahci_platform_get_resources() method invocation\n> and up to the device removal.\n> \n> Note the method is left unused here, but will be utilized in the framework\n> of the DWC AHCI SATA driver being added in the next commit.\n> \n> Signed-off-by: Serge Semin \n> \n> ---\n> \n> Changelog v2:\n> - Fix some grammar mistakes in the method description.\n> \n> Changelog v4:\n> - Add a note regarding the new method usage.\n> ---\n> drivers/ata/libahci_platform.c | 27 +++++++++++++++++++++++++++\n> include/linux/ahci_platform.h | 3 +++\n> 2 files changed, 30 insertions(+)\n> \n> diff --git a/drivers/ata/libahci_platform.c b/drivers/ata/libahci_platform.c\n> index 8b542a8bc487..418961f954af 100644\n> --- a/drivers/ata/libahci_platform.c\n> +++ b/drivers/ata/libahci_platform.c\n> @@ -95,6 +95,33 @@ void ahci_platform_disable_phys(struct ahci_host_priv *hpriv)\n> }\n> EXPORT_SYMBOL_GPL(ahci_platform_disable_phys);\n> \n> +/**\n> + * ahci_platform_find_clk - Find platform clock\n> + * @hpriv: host private area to store config values\n> + * @con_id: clock connection ID\n> + *\n> + * This function returns a pointer to the clock descriptor of the clock with\n> + * the passed ID.\n> + *\n> + * RETURNS:\n> + * Pointer to the clock descriptor on success otherwise NULL\n> + */\n> +struct clk *ahci_platform_find_clk(struct ahci_host_priv *hpriv, const char *con_id)\n> +{\n> +\tstruct clk *clk = NULL;\n> +\tint i;\n> +\n> +\tfor (i = 0; i < hpriv->n_clks; i++) {\n> +\t\tif (!strcmp(hpriv->clks[i].id, con_id)) {\n> +\t\t\tclk = hpriv->clks[i].clk;\n\n\t\t\treturn hpriv->clks[i].clk;\n\n> +\t\t\tbreak;\n> +\t\t}\n> +\t}\n> +\n> +\treturn clk;\n\nAnd then \"return NULL;\" here and you do not need the clk variable.\n\n> +}\n> +EXPORT_SYMBOL_GPL(ahci_platform_find_clk);\n> +\n> /**\n> * ahci_platform_enable_clks - Enable platform clocks\n> * @hpriv: host private area to store config values\n> diff --git a/include/linux/ahci_platform.h b/include/linux/ahci_platform.h\n> index 6d7dd472d370..3418980b0341 100644\n> --- a/include/linux/ahci_platform.h\n> +++ b/include/linux/ahci_platform.h\n> @@ -13,6 +13,7 @@\n> \n> #include \n> \n> +struct clk;\n> struct device;\n> struct ata_port_info;\n> struct ahci_host_priv;\n> @@ -21,6 +22,8 @@ struct scsi_host_template;\n> \n> int ahci_platform_enable_phys(struct ahci_host_priv *hpriv);\n> void ahci_platform_disable_phys(struct ahci_host_priv *hpriv);\n> +struct clk *\n> +ahci_platform_find_clk(struct ahci_host_priv *hpriv, const char *con_id);\n\nPlease make this:\n\nstruct clk *ahci_platform_find_clk(struct ahci_host_priv *hpriv,\n\n const char *con_id);\n\n> int ahci_platform_enable_clks(struct ahci_host_priv *hpriv);\n> void ahci_platform_disable_clks(struct ahci_host_priv *hpriv);\n> int ahci_platform_deassert_rsts(struct ahci_host_priv *hpriv);","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["bilbo.ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n unprotected) header.d=wdc.com header.i=@wdc.com header.a=rsa-sha256\n header.s=dkim.wdc.com header.b=Fs56aKPD;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=opensource.wdc.com header.i=@opensource.wdc.com\n header.a=rsa-sha256 header.s=dkim header.b=iMAF22D0;\n\tdkim-atps=neutral","ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=2620:137:e000::1:20; helo=out1.vger.email;\n envelope-from=linux-ide-owner@vger.kernel.org; receiver=)","usg-ed-osssrv.wdc.com (amavisd-new); dkim=pass\n reason=\"pass (just generated, assumed good)\"\n header.d=opensource.wdc.com"],"Received":["from out1.vger.email (out1.vger.email [IPv6:2620:137:e000::1:20])\n\tby bilbo.ozlabs.org (Postfix) with ESMTP id 4LMhry26l9z9sGH\n\tfor ; Tue, 14 Jun 2022 18:47:30 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n id S1354836AbiFNIr0 (ORCPT );\n Tue, 14 Jun 2022 04:47:26 -0400","from lindbergh.monkeyblade.net ([23.128.96.19]:46352 \"EHLO\n lindbergh.monkeyblade.net\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n with ESMTP id S241547AbiFNIpm (ORCPT\n ); Tue, 14 Jun 2022 04:45:42 -0400","from esa2.hgst.iphmx.com (esa2.hgst.iphmx.com [68.232.143.124])\n by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7CE74B862\n for ; Tue, 14 Jun 2022 01:45:41 -0700 (PDT)","from uls-op-cesaip01.wdc.com (HELO uls-op-cesaep01.wdc.com)\n ([199.255.45.14])\n by ob1.hgst.iphmx.com with ESMTP; 14 Jun 2022 16:45:40 +0800","from uls-op-cesaip02.wdc.com ([10.248.3.37])\n by uls-op-cesaep01.wdc.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256;\n 14 Jun 2022 01:08:39 -0700","from usg-ed-osssrv.wdc.com ([10.3.10.180])\n by uls-op-cesaip02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256;\n 14 Jun 2022 01:45:40 -0700","from usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1])\n by usg-ed-osssrv.wdc.com (Postfix) with ESMTP id 4LMhpq4wfhz1SVny\n for ; Tue, 14 Jun 2022 01:45:39 -0700 (PDT)","from usg-ed-osssrv.wdc.com ([127.0.0.1])\n by usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1])\n (amavisd-new, port 10026)\n with ESMTP id xwPKRntCVUmh for ;\n Tue, 14 Jun 2022 01:45:38 -0700 (PDT)","from [10.225.163.77] (unknown [10.225.163.77])\n by usg-ed-osssrv.wdc.com (Postfix) with ESMTPSA id 4LMhpn0mkyz1Rvlc;\n Tue, 14 Jun 2022 01:45:36 -0700 (PDT)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=simple/simple;\n d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com;\n t=1655196341; x=1686732341;\n h=message-id:date:mime-version:subject:to:cc:references:\n from:in-reply-to:content-transfer-encoding;\n bh=730WLOxUDl/tCCXOxzEMJE/Etnsc1xvpiRS8SMWclV4=;\n b=Fs56aKPDnaxtAYnHtHu8a9ZvoIXAGUoGxn5KknLa6Uy67nMnuCZY932a\n TCYUC7ELi3CJvzGyJq7J5t1OS4W4hVqw9q4krwYNUe7+PgiDmRy7OXQpa\n Z+m+ObVNiK66ByO8+3T5RdoZDsGCC7i4QGuwoOrF1rtlwaHFcyIyLsWUc\n KeoKtdmofizCM91oiu7GSppAh8wy/Ux2ibZRQozz09vBae6SksiMlgVst\n 1i2tqwfsZEJkE266I6RO1uuh0CgM2YkfRD62NedjGHVy0W5iplgI8oeYG\n eHbo0ib4Ggp6bEKfyp8FGB7pzAqsiX8WXs9wojSMgcL0NcBOk6jrNDWvn\n g==;","v=1; a=rsa-sha256; c=relaxed/simple; d=\n opensource.wdc.com; h=content-transfer-encoding:content-type\n :in-reply-to:organization:from:references:to:content-language\n :subject:user-agent:mime-version:date:message-id; s=dkim; t=\n 1655196338; x=1657788339; bh=730WLOxUDl/tCCXOxzEMJE/Etnsc1xvpiRS\n 8SMWclV4=; b=iMAF22D08tLP22EwyW2EGAImbpK8eQcBbDj01cr5hIdTvMvn9pl\n Z/xRZHFpQs+YVoEdTBqfq0NqDouapToDlh8A9TSpdO9Dp0Bk8kza4wB95o4kOUCh\n bKBxD34JxEGV1T/fUMszf/o+rIlyL4nCfwxD4dH2MwSnQPm5FH5bRPjMTE2Y3dAz\n derj6ZqUDHj4Y6mieLfh0do48fDsyz3OA4etLnu5zXg85LW/mwcXOZT/3HKB1QS+\n O7n1pg/UpQLtzJY5Ikore7lRjNXFehJmKUBmoSYW0mLnHU77J9XxbPrakqxVcf9q\n vTyT1w9p0RSWdf30ruP4Q40mQQEr7dEcIIQ=="],"X-IronPort-AV":"E=Sophos;i=\"5.91,299,1647273600\";\n d=\"scan'208\";a=\"307388535\"","IronPort-SDR":["\n YlIUwcs6rI1GUUIji4BSR+ko3orYyYospjhoXsdmu5S2LJlL5ecAO4Z1JQyUclHNfqGXlxUH7/\n I1T4AiVJiRN6x4Cjt8G6/Pr6KaaAH43PMza1R5fEBEDVwG9uuvAyJxMqbl/al45fGCDWlWibia\n twUUlqeSPWrl6D4bvwSKBR+wCYXrXynVcq3zemv26uQu4MWunCtwHaBJfW+s//61iPUeYcA3+D\n O73N9SLq778wjQBL8IIVA8O8ZXzN4xkVc9bA0Pi8gjFHIp5fQoyKfnIOFfPJRqX4/CtuWPZ9zX\n 3onCY44D/coPrNI++h3j0Uj/","\n lOZq6pczJPYqlhAKWMRvRJrADWRL2gXLrWbl+8wgyT+38wPtUs0fENsJqAkQ+SNNly7BsejLtR\n X5s3lzTokl5jHKjjd/dUWZot3X5CY7MRb9HCKqEUz271Ww+WfG2K0+Yjwin2P5tDzaduvIXgYd\n h3i7q9diTxa2i/oto2BA2/66D6Wjae439znx1oLEwh5h0+LpJ2L+BijlWoFDRcImNM6Mzana2a\n 7z+RLnoLhNHT2thIyqQlFnJ+2zre3o0wwnIhWA2eOFqIhmyr+AABWPbtwSojnfuWFfzwznKfjE\n J+w="],"WDCIronportException":"Internal","X-Virus-Scanned":"amavisd-new at usg-ed-osssrv.wdc.com","Message-ID":"<611c8eeb-fa7e-286c-bef2-0e24cc2b6b2e@opensource.wdc.com>","Date":"Tue, 14 Jun 2022 17:45:35 +0900","MIME-Version":"1.0","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101\n Thunderbird/91.10.0","Subject":"Re: [PATCH v4 18/23] ata: libahci_platform: Add function returning a\n clock-handle by id","Content-Language":"en-US","To":"Serge Semin ,\n Hans de Goede ,\n Jens Axboe , Hannes Reinecke ","Cc":"Serge Semin ,\n Alexey Malahov ,\n Pavel Parkhomenko ,\n Rob Herring , linux-ide@vger.kernel.org,\n linux-kernel@vger.kernel.org, devicetree@vger.kernel.org","References":"<20220610081801.11854-1-Sergey.Semin@baikalelectronics.ru>\n <20220610081801.11854-19-Sergey.Semin@baikalelectronics.ru>","From":"Damien Le Moal ","Organization":"Western Digital Research","In-Reply-To":"<20220610081801.11854-19-Sergey.Semin@baikalelectronics.ru>","Content-Type":"text/plain; charset=UTF-8","Content-Transfer-Encoding":"7bit","X-Spam-Status":"No, score=-5.6 required=5.0 tests=BAYES_00,DKIM_SIGNED,\n DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_MED,\n SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable\n autolearn_force=no version=3.4.6","X-Spam-Checker-Version":"SpamAssassin 3.4.6 (2021-04-09) on\n lindbergh.monkeyblade.net","Precedence":"bulk","List-ID":"","X-Mailing-List":"linux-ide@vger.kernel.org"}},{"id":2913747,"web_url":"http://patchwork.ozlabs.org/comment/2913747/","msgid":"<20220615212443.2zkirzppeby25xs6@mobilestation>","list_archive_url":null,"date":"2022-06-15T21:24:43","subject":"Re: [PATCH v4 18/23] ata: libahci_platform: Add function returning a\n clock-handle by id","submitter":{"id":70038,"url":"http://patchwork.ozlabs.org/api/people/70038/","name":"Serge Semin","email":"fancer.lancer@gmail.com"},"content":"On Tue, Jun 14, 2022 at 05:45:35PM +0900, Damien Le Moal wrote:\n> On 6/10/22 17:17, Serge Semin wrote:\n> > Since all the clocks are retrieved by the method\n> > ahci_platform_get_resources() there is no need for the LLD (glue) drivers\n> > to be looking for some particular of them in the kernel clocks table\n> > again. Instead we suggest to add a simple method returning a\n> > device-specific clock with passed connection ID if it is managed to be\n> > found. Otherwise the function will return NULL. Thus the glue-drivers\n> > won't need to either manually touching the hpriv->clks array or calling\n> > clk_get()-friends. The AHCI platform drivers will be able to use the new\n> > function right after the ahci_platform_get_resources() method invocation\n> > and up to the device removal.\n> > \n> > Note the method is left unused here, but will be utilized in the framework\n> > of the DWC AHCI SATA driver being added in the next commit.\n> > \n> > Signed-off-by: Serge Semin \n> > \n> > ---\n> > \n> > Changelog v2:\n> > - Fix some grammar mistakes in the method description.\n> > \n> > Changelog v4:\n> > - Add a note regarding the new method usage.\n> > ---\n> > drivers/ata/libahci_platform.c | 27 +++++++++++++++++++++++++++\n> > include/linux/ahci_platform.h | 3 +++\n> > 2 files changed, 30 insertions(+)\n> > \n> > diff --git a/drivers/ata/libahci_platform.c b/drivers/ata/libahci_platform.c\n> > index 8b542a8bc487..418961f954af 100644\n> > --- a/drivers/ata/libahci_platform.c\n> > +++ b/drivers/ata/libahci_platform.c\n> > @@ -95,6 +95,33 @@ void ahci_platform_disable_phys(struct ahci_host_priv *hpriv)\n> > }\n> > EXPORT_SYMBOL_GPL(ahci_platform_disable_phys);\n> > \n> > +/**\n> > + * ahci_platform_find_clk - Find platform clock\n> > + * @hpriv: host private area to store config values\n> > + * @con_id: clock connection ID\n> > + *\n> > + * This function returns a pointer to the clock descriptor of the clock with\n> > + * the passed ID.\n> > + *\n> > + * RETURNS:\n> > + * Pointer to the clock descriptor on success otherwise NULL\n> > + */\n> > +struct clk *ahci_platform_find_clk(struct ahci_host_priv *hpriv, const char *con_id)\n> > +{\n> > +\tstruct clk *clk = NULL;\n> > +\tint i;\n> > +\n> > +\tfor (i = 0; i < hpriv->n_clks; i++) {\n> > +\t\tif (!strcmp(hpriv->clks[i].id, con_id)) {\n> > +\t\t\tclk = hpriv->clks[i].clk;\n> \n> \t\t\treturn hpriv->clks[i].clk;\n> \n> > +\t\t\tbreak;\n> > +\t\t}\n> > +\t}\n> > +\n> > +\treturn clk;\n> \n\n> And then \"return NULL;\" here and you do not need the clk variable.\n\nOk.\n\n> \n> > +}\n> > +EXPORT_SYMBOL_GPL(ahci_platform_find_clk);\n> > +\n> > /**\n> > * ahci_platform_enable_clks - Enable platform clocks\n> > * @hpriv: host private area to store config values\n> > diff --git a/include/linux/ahci_platform.h b/include/linux/ahci_platform.h\n> > index 6d7dd472d370..3418980b0341 100644\n> > --- a/include/linux/ahci_platform.h\n> > +++ b/include/linux/ahci_platform.h\n> > @@ -13,6 +13,7 @@\n> > \n> > #include \n> > \n> > +struct clk;\n> > struct device;\n> > struct ata_port_info;\n> > struct ahci_host_priv;\n> > @@ -21,6 +22,8 @@ struct scsi_host_template;\n> > \n> > int ahci_platform_enable_phys(struct ahci_host_priv *hpriv);\n> > void ahci_platform_disable_phys(struct ahci_host_priv *hpriv);\n> > +struct clk *\n> > +ahci_platform_find_clk(struct ahci_host_priv *hpriv, const char *con_id);\n> \n\n> Please make this:\n> \n> struct clk *ahci_platform_find_clk(struct ahci_host_priv *hpriv,\n> \n> const char *con_id);\n\nOk.\n\n-Sergey\n\n> \n> > int ahci_platform_enable_clks(struct ahci_host_priv *hpriv);\n> > void ahci_platform_disable_clks(struct ahci_host_priv *hpriv);\n> > int ahci_platform_deassert_rsts(struct ahci_host_priv *hpriv);\n> \n> \n> -- \n> Damien Le Moal\n> Western Digital Research","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["bilbo.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256\n header.s=20210112 header.b=o+B+vgHF;\n\tdkim-atps=neutral","ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org\n (client-ip=2620:137:e000::1:20; helo=out1.vger.email;\n envelope-from=linux-ide-owner@vger.kernel.org; receiver=)"],"Received":["from out1.vger.email (out1.vger.email [IPv6:2620:137:e000::1:20])\n\tby bilbo.ozlabs.org (Postfix) with ESMTP id 4LNdcP4gKrz9sGF\n\tfor ; Thu, 16 Jun 2022 07:24:53 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n id S1349580AbiFOVYu (ORCPT );\n Wed, 15 Jun 2022 17:24:50 -0400","from lindbergh.monkeyblade.net ([23.128.96.19]:48576 \"EHLO\n lindbergh.monkeyblade.net\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n with ESMTP id S1349490AbiFOVYt (ORCPT\n ); Wed, 15 Jun 2022 17:24:49 -0400","from mail-lf1-x12b.google.com (mail-lf1-x12b.google.com\n [IPv6:2a00:1450:4864:20::12b])\n by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 38E78554BE;\n Wed, 15 Jun 2022 14:24:47 -0700 (PDT)","by mail-lf1-x12b.google.com with SMTP id h23so20851951lfe.4;\n Wed, 15 Jun 2022 14:24:47 -0700 (PDT)","from mobilestation ([95.79.189.214])\n by smtp.gmail.com with ESMTPSA id\n s27-20020a056512203b00b00478a311d399sm1937222lfs.0.2022.06.15.14.24.44\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Wed, 15 Jun 2022 14:24:44 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=gmail.com; s=20210112;\n h=date:from:to:cc:subject:message-id:references:mime-version\n :content-disposition:in-reply-to;\n bh=peqy01VggIT1uRQjQeqqFB+gR0sAWrSD7edJX84V7Mk=;\n b=o+B+vgHFKQ+VDXcmSN9mEZWAWh7aDrnnd1rJGysYV1AEXtvhrezJvOqiwVD6ZooNdY\n fVgZkrXFVaXhAkugd06guj3J+AOJeGbnO0xp8kBgb46lAVwuKON+veuEsGuQBnyx4Qa1\n wOLs6UAtYZyW7Rk7B9ydvVBYuNKpK9XBeHr7STrMR3EnviI6r3pd/i2otXmhmCnWATlg\n zgLwXQvATPjnwND/jo1Yc6EoTjVCx8W4qB2ycwDNIimx4JBWE9tbinfAnAXIZVyRSoyO\n kDdc28/PWakbvEcNAKTyFIK+W/83C5XrIDmHJjd5+rvzMLKs+A9znWwvg7hpY93OUrpI\n pgJg==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20210112;\n h=x-gm-message-state:date:from:to:cc:subject:message-id:references\n :mime-version:content-disposition:in-reply-to;\n bh=peqy01VggIT1uRQjQeqqFB+gR0sAWrSD7edJX84V7Mk=;\n b=54AKWZq46zZi7zCsv4NGqytVTOHitHDQ6BIN0oAArz78iVPlknzi133umJa1XrevCu\n Jg9Lk4ZCEolVpV5lnK4mQszI4jFxbI+kWdf/vkYnkug9Dhz1OUScJV05n0jGqG+6Uy96\n p8eDP3YjvjB1IFRPyWApGocM2EUxSsybgB0j5klIWKmmuPxZCGQbF6DBnwYSoRKTS/uq\n Lhc1c6YxiM4klD3hy5TfW+zOUr9hoY2f1xcKg2RZq3cbGRXTfpdVG+hFPv/fUAMToQDW\n S08eC6QQPI1oHn9k1RByOCYIAl4k9ZwEBpqF9fDICPOjTBDJIsIJwSTWEKOQTtdeFegC\n e2+Q==","X-Gm-Message-State":"AJIora+4PQv5olAJd+rQC4i3gtwgX1KYjIL4XNdZE5VIr/7/+KxISjpz\n Fe3xjV0wsDGdQNRRrqg5LQw=","X-Google-Smtp-Source":"\n AGRyM1vl9Th7OIF5POLvrtEojwMO8Ll6jPy4AU2qh4nUeFA48cQqgBOvYwNwOk3+CxDKMFVCXXgqOw==","X-Received":"by 2002:a05:6512:22ce:b0:479:ef3:141c with SMTP id\n g14-20020a05651222ce00b004790ef3141cmr771288lfu.393.1655328285558;\n Wed, 15 Jun 2022 14:24:45 -0700 (PDT)","Date":"Thu, 16 Jun 2022 00:24:43 +0300","From":"Serge Semin ","To":"Damien Le Moal ","Cc":"Serge Semin ,\n Hans de Goede ,\n Jens Axboe , Hannes Reinecke ,\n Alexey Malahov ,\n Pavel Parkhomenko ,\n Rob Herring , linux-ide@vger.kernel.org,\n linux-kernel@vger.kernel.org, devicetree@vger.kernel.org","Subject":"Re: [PATCH v4 18/23] ata: libahci_platform: Add function returning a\n clock-handle by id","Message-ID":"<20220615212443.2zkirzppeby25xs6@mobilestation>","References":"<20220610081801.11854-1-Sergey.Semin@baikalelectronics.ru>\n <20220610081801.11854-19-Sergey.Semin@baikalelectronics.ru>\n <611c8eeb-fa7e-286c-bef2-0e24cc2b6b2e@opensource.wdc.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<611c8eeb-fa7e-286c-bef2-0e24cc2b6b2e@opensource.wdc.com>","X-Spam-Status":"No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,\n DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM,\n RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE\n autolearn=ham autolearn_force=no version=3.4.6","X-Spam-Checker-Version":"SpamAssassin 3.4.6 (2021-04-09) on\n lindbergh.monkeyblade.net","Precedence":"bulk","List-ID":"","X-Mailing-List":"linux-ide@vger.kernel.org"}}]