From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 19146C43334 for ; Thu, 16 Jun 2022 00:29:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S243455AbiFPA3V (ORCPT ); Wed, 15 Jun 2022 20:29:21 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35972 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1347709AbiFPA20 (ORCPT ); Wed, 15 Jun 2022 20:28:26 -0400 Received: from esa6.hgst.iphmx.com (esa6.hgst.iphmx.com [216.71.154.45]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1160B12A98 for ; Wed, 15 Jun 2022 17:28:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1655339305; x=1686875305; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=uYVFjpxrkNkhxj61fyi8GC6kWmJYxCqHfowX+smn4Q8=; b=M9EOGZZH5PodXB2uiAyRquTe20ZUaoDnbajM3s4mvUrsGLAsY+Hn3KsI 5DwtaqFd6/b2tVihScs0YXgsFF1FqtNvV1qcS4+m7Q8Ph1zkuIgzbzM7D 18jK6j+b+X0euiRJpRGcTx50ixa7MB9jv9hGblRXZI0tmdtWCOs9X6EAq IXW1dW611lTcT9OLvYp6Pu3QTtFEh3+zfkMM/pdiaV4Lx29LSzuNhCX3C 9EYGdQK9Mu8E9kq9rQsWb1eGMvwawUXuHSwwQQTzggM3VGMRaef/JA+hJ Sy0/QA3XHoAJgqH22PJUnnzaAnb2uc7mek9QV87E2lwESI0ptpf8zE2uX A==; X-IronPort-AV: E=Sophos;i="5.91,302,1647273600"; d="scan'208";a="204038287" Received: from uls-op-cesaip02.wdc.com (HELO uls-op-cesaep02.wdc.com) ([199.255.45.15]) by ob1.hgst.iphmx.com with ESMTP; 16 Jun 2022 08:28:24 +0800 IronPort-SDR: tQQfQh9/pPkzQTO4KmhtEii2t+UBDBHz6WpqbLmBG2zo5dGz9mzwQyfvm5diUDjoIw7BxifNTa jOApa2g59TYu8BTqJIx+bpFL2p5zryUSNcQIh0YrcDt3JXVogbeg0P+VifbK3AlmfD+M6jexWv IZj/3StB6kvCHDFznqIbB+2oh2pne+tIruDBrkgjpz8cUqmjlki4sGpusOpRy0grvWnISwX/DQ m3LMhhfuXNYV2qK+a4zs356A34TweqD/Qqy0t57pJMOmpk1kZUAiVHOTIfMZP3BnnExvUCg/Mk /zqesN0pZ/QR5TLK1wnW7Kxj Received: from uls-op-cesaip01.wdc.com ([10.248.3.36]) by uls-op-cesaep02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 15 Jun 2022 16:46:46 -0700 IronPort-SDR: 5wqdbYTPm3Qw8F1Kf9fYT5AdP6oLbl/qfJAoxHvjeA9rLj/Qyi3pkjUUnrgBNh58QWFYu3qqJC b2uMALNvRUeV/8oQe2V2CsvONYlBpra1JS+aJGtxpLTQYUQAr4QUDF/dqwzANWw6ZuOH7PdNOn zYsnnC71NRGsXDx7B7T9jQosxHXyyZslrL4Tftrmhh2y85+mGl0krih7krIVA7S77B63I/WZeB VYxAM2Ygn3kxNAYR/n9CZJRlTGYAuto7fGO9hu22q5dbGb8RdAAmvwpTT15SgLbi4TatagOlnw nKs= WDCIronportException: Internal Received: from usg-ed-osssrv.wdc.com ([10.3.10.180]) by uls-op-cesaip01.wdc.com with ESMTP/TLS/ECDHE-RSA-AES128-GCM-SHA256; 15 Jun 2022 17:28:23 -0700 Received: from usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTP id 4LNjh64Ch2z1Rwrw for ; Wed, 15 Jun 2022 17:28:22 -0700 (PDT) Authentication-Results: usg-ed-osssrv.wdc.com (amavisd-new); dkim=pass reason="pass (just generated, assumed good)" header.d=opensource.wdc.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d= opensource.wdc.com; h=content-transfer-encoding:content-type :in-reply-to:organization:from:references:to:content-language :subject:user-agent:mime-version:date:message-id; s=dkim; t= 1655339301; x=1657931302; bh=uYVFjpxrkNkhxj61fyi8GC6kWmJYxCqHfow X+smn4Q8=; b=f1mDXGtbN3/m1SlybA62kaNKiCv2e1mA2QRjgr07kkClwUdJA9Z RVAlJ2Y3xnYcWSzvDEihYHI5nz/pGGOTrpGK1W2o4CmtdJwaJHjhMyeq/mzqWLG8 7QqQu2XjHIPMHsEFOpjS576QEvoPZYRPl4mHk3VgWXOP5Cn8s645F9Xckkf6DUDS tCDWMv8rFKHIt8qsJ8rh4Zlg/BE8jMpyFtBxQbXHrZ/7GoMZ9vvCvP4Kz8/8Ixgj aY4/20MVVqEnTUYkmTl62DJzFih7PVV48O8vt63j1/Vedr1EWgrcnTS+rC44PCAe 2d0ncZPolaZ9SqEkyl1BcacLpw0w6h3IToQ== X-Virus-Scanned: amavisd-new at usg-ed-osssrv.wdc.com Received: from usg-ed-osssrv.wdc.com ([127.0.0.1]) by usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id UbiZ95XI9KB5 for ; Wed, 15 Jun 2022 17:28:21 -0700 (PDT) Received: from [10.89.84.185] (c02drav6md6t.dhcp.fujisawa.hgst.com [10.89.84.185]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTPSA id 4LNjh36YgPz1Rvlc; Wed, 15 Jun 2022 17:28:19 -0700 (PDT) Message-ID: <903e273a-9dc5-f0df-5391-e96e63318323@opensource.wdc.com> Date: Thu, 16 Jun 2022 09:28:18 +0900 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Thunderbird/91.10.0 Subject: Re: [PATCH v4 12/23] ata: libahci: Extend port-cmd flags set with port capabilities Content-Language: en-US To: Serge Semin Cc: Serge Semin , Hans de Goede , Jens Axboe , Hannes Reinecke , Alexey Malahov , Pavel Parkhomenko , Rob Herring , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org References: <20220610081801.11854-1-Sergey.Semin@baikalelectronics.ru> <20220610081801.11854-13-Sergey.Semin@baikalelectronics.ru> <20220615205819.uiqptkqm5qfdvrbj@mobilestation> From: Damien Le Moal Organization: Western Digital Research In-Reply-To: <20220615205819.uiqptkqm5qfdvrbj@mobilestation> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-ide@vger.kernel.org On 2022/06/16 5:58, Serge Semin wrote: > On Tue, Jun 14, 2022 at 05:32:41PM +0900, Damien Le Moal wrote: >> On 6/10/22 17:17, Serge Semin wrote: >>> Currently not all of the Port-specific capabilities listed in the >> >> s/listed/are listed >> >>> PORT_CMD-enumeration. Let's extend that set with the Cold Presence >>> Detection and Mechanical Presence Switch attached to the Port flags [1] so >>> to closeup the set of the platform-specific port-capabilities flags. Note >>> these flags are supposed to be set by the platform firmware if there is >>> one. Alternatively as we are about to do they can be set by means of the >>> OF properties. >>> >>> While at it replace PORT_IRQ_DEV_ILCK with PORT_IRQ_DMPS and fix the >>> comment there. In accordance with [2] that IRQ flag is supposed to >>> indicate the state of the signal coming from the Mechanical Presence >>> Switch. >>> >>> [1] Serial ATA AHCI 1.3.1 Specification, p.27 >>> [2] Serial ATA AHCI 1.3.1 Specification, p.24, p.88 >>> >>> Signed-off-by: Serge Semin >>> Reviewed-by: Hannes Reinecke >>> >>> --- >>> >>> Changelog v4: >>> - Fix the DMPS macros name in the patch log. (@Sergei Shtylyov) >>> --- >>> drivers/ata/ahci.h | 7 ++++++- >>> 1 file changed, 6 insertions(+), 1 deletion(-) >>> >>> diff --git a/drivers/ata/ahci.h b/drivers/ata/ahci.h >>> index 7d834deefeb9..f501531bd1b3 100644 >>> --- a/drivers/ata/ahci.h >>> +++ b/drivers/ata/ahci.h >>> @@ -138,7 +138,7 @@ enum { >>> PORT_IRQ_BAD_PMP = (1 << 23), /* incorrect port multiplier */ >>> >>> PORT_IRQ_PHYRDY = (1 << 22), /* PhyRdy changed */ >>> - PORT_IRQ_DEV_ILCK = (1 << 7), /* device interlock */ >>> + PORT_IRQ_DMPS = (1 << 7), /* mechanical presence status */ >>> PORT_IRQ_CONNECT = (1 << 6), /* port connect change status */ >>> PORT_IRQ_SG_DONE = (1 << 5), /* descriptor processed */ >>> PORT_IRQ_UNK_FIS = (1 << 4), /* unknown FIS rx'd */ >>> @@ -166,6 +166,8 @@ enum { >>> PORT_CMD_ATAPI = (1 << 24), /* Device is ATAPI */ >>> PORT_CMD_FBSCP = (1 << 22), /* FBS Capable Port */ >>> PORT_CMD_ESP = (1 << 21), /* External Sata Port */ >>> + PORT_CMD_CPD = (1 << 20), /* Cold Presence Detection */ >>> + PORT_CMD_MPSP = (1 << 19), /* Mechanical Presence Switch */ >>> PORT_CMD_HPCP = (1 << 18), /* HotPlug Capable Port */ >>> PORT_CMD_PMP = (1 << 17), /* PMP attached */ >>> PORT_CMD_LIST_ON = (1 << 15), /* cmd list DMA engine running */ >>> @@ -181,6 +183,9 @@ enum { >>> PORT_CMD_ICC_PARTIAL = (0x2 << 28), /* Put i/f in partial state */ >>> PORT_CMD_ICC_SLUMBER = (0x6 << 28), /* Put i/f in slumber state */ >>> >>> + PORT_CMD_CAP = PORT_CMD_HPCP | PORT_CMD_MPSP | >>> + PORT_CMD_CPD | PORT_CMD_ESP | PORT_CMD_FBSCP, >> > >> What is this one for ? A comment above it would be nice. > > Isn't it obviously inferrable from the definition and the item name? I am guessing from the name. Am I guessing OK ? A comment would still be nice. Why just these bits ? There are more cap/support indicator bits in that port cmd bitfield. So why this particular set of bits ? What do they mean all together ? Sure I can go and read the specs to figure it out. But again, a comment would avoid readers of the code to have to decrypt all that. > > -Sergey > >> >>> + >>> /* PORT_FBS bits */ >>> PORT_FBS_DWE_OFFSET = 16, /* FBS device with error offset */ >>> PORT_FBS_ADO_OFFSET = 12, /* FBS active dev optimization offset */ >> >> >> -- >> Damien Le Moal >> Western Digital Research -- Damien Le Moal Western Digital Research