Received: by 2002:a05:6a11:4021:0:0:0:0 with SMTP id ky33csp835920pxb; Wed, 22 Sep 2021 14:11:11 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzhkNdIQcmJlnIqrKP7UrgjWhjzbsCl4gXmv0asdaKbSwprlIyXhbCsP/5jrKguw2XR2G+S X-Received: by 2002:a17:906:a0a:: with SMTP id w10mr1453044ejf.416.1632345071183; Wed, 22 Sep 2021 14:11:11 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1632345071; cv=none; d=google.com; s=arc-20160816; b=A4ZghyaZ4mn5uIwXMaNolqz8ZebidXRrxn86VtdffGXmBPim++yZfqv5fZg26r8Vk4 +0uyCWGK+oinhpKpPkmP9Qlu8a/qCxI8B+DzCa4K44+RJS7n2LVKzaTO7S8ht52Ghfqo 5XEjDcdqzJxNT4zZzPecjbCzrvpODQW9TZSPzjKKhs7ZhFWjU1/3eOc65aMyx42T4trH 52MkNitFkjB+vhOyWIh5ToS4oxt8lE+kKAM933G+1vTmqr4o/tLiophyP9XAtyyGam8X LuTVZaR3FGiybBt1C01NoTfRY8A9DB0BSbQDFlEuAousrBvnb0gw9RKDPwfbVxDp+jv6 IM9w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:subject:cc:to:from:date:references:in-reply-to :message-id:mime-version:user-agent:dkim-signature:dkim-signature; bh=mtdkC9smfEGkMzwp5suu4GvLvKNjWQU2LGQHltLYlT4=; b=ZLYPTB+iWdAmfpi/sQHiF1peTsZDYF1uhnZ3RTgvYCwiUSIllhO5bFEyEbSUXINvU7 UEJt453YbknTETcqWAzdzchKBOyBnmSPJJZWg2rq4mvnDlUWwQETzwGyYKz89PZkn8Fw AZfqyOawkCXEASKmdZ0GQsUW01Vs5rT/IAbH9iUjHAdwIi7WLGB/5w4oVfY2MXeHRJeZ +z/HZFZpnoNgg3T2VbNIWOMiOyD5EgSucOe6ay5+bBBTsxChmuDShhbthBTrIEGpn8qX csTPiGH76H98O/NUOImMOJ/impU8ja8PkYBR33x5jGioKyYuGQVoGZOV1tDnVRzcbwYU nP1Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@svenpeter.dev header.s=fm2 header.b=S1hEFyFv; dkim=pass header.i=@messagingengine.com header.s=fm3 header.b=dO0DKpYX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=svenpeter.dev Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id jl19si3425721ejc.626.2021.09.22.14.10.46; Wed, 22 Sep 2021 14:11:11 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@svenpeter.dev header.s=fm2 header.b=S1hEFyFv; dkim=pass header.i=@messagingengine.com header.s=fm3 header.b=dO0DKpYX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=svenpeter.dev Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237833AbhIVVK1 (ORCPT + 99 others); Wed, 22 Sep 2021 17:10:27 -0400 Received: from new2-smtp.messagingengine.com ([66.111.4.224]:43399 "EHLO new2-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237759AbhIVVK0 (ORCPT ); Wed, 22 Sep 2021 17:10:26 -0400 Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailnew.nyi.internal (Postfix) with ESMTP id 603C0580595; Wed, 22 Sep 2021 17:08:55 -0400 (EDT) Received: from imap21 ([10.202.2.71]) by compute1.internal (MEProxy); Wed, 22 Sep 2021 17:08:55 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=svenpeter.dev; h=mime-version:message-id:in-reply-to:references:date:from:to :cc:subject:content-type; s=fm2; bh=mtdkC9smfEGkMzwp5suu4GvLvKNj WQU2LGQHltLYlT4=; b=S1hEFyFv9XbXLQhnLt70N1EPGX/H3bqcz7U/MTIXWzkw wjFnltIWbkCxSWXhkZ09pkN9PBdvJcXgXaaZDg43vKzH86JWQAFF0P16qRNxDusR DSOVexTv9E329Q/vNDDRarloQdeRlcCsgCvxbDGoHLAmzIAo6589gJodPaXg3cqL wSMdKh6hh65xGeeceNltm/HQZD/grayJnUa5Xn5ZEkFy2PDyG1+JWR/Rd1pQdkXa CDJI2hQvclzQdDj6qpE8xKRKDNH6/uiVm415vxyBkTGX3BpepJMpnbsYdsi0jH5h /gMQuiINTLrDZLwjGMHlSP1QAcsMFSlhqBGEE5X2qQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; bh=mtdkC9 smfEGkMzwp5suu4GvLvKNjWQU2LGQHltLYlT4=; b=dO0DKpYX8ksx/l1tM1e/3h 4PgDCXCTAx5CzpddSXz5rI6mss6MPKKpXPi+sO2vsKrEnAs2xOBhQNwT7iruX1Uk dBdHR87saCwTCBcBatfzVdNgri9VUL5I1lciIMr5Uh+ORBhf7Zp//GDFT8IIjSib f6AYJlTh3uONfUcAZqiGM27PGdIcHSvnWUjHG5p2dBT2GoePAQbmC+bji4hNov9a PpQeHJ8a2cBKg1shDxYtQtgQkuYr7c32DSjKdAiW6dytBHaCsT6HmW8lW1m4Hpcl wuJacvTn/fKTt+NyCiilH0n22pH37glqtJMhYCPSLm0YMr9ItK5sYF6lgN7OM/+g == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvtddrudeijedgudehjecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd enucfjughrpefofgggkfgjfhffhffvufgtsehttdertderredtnecuhfhrohhmpedfufhv vghnucfrvghtvghrfdcuoehsvhgvnhesshhvvghnphgvthgvrhdruggvvheqnecuggftrf grthhtvghrnhepgfeigeeiffeuhfettdejgfetjeetfeelfefgfefgvddvtdfghfffudeh vdefkeffnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomh epshhvvghnsehsvhgvnhhpvghtvghrrdguvghv X-ME-Proxy: Received: by mailuser.nyi.internal (Postfix, from userid 501) id 0652651C0060; Wed, 22 Sep 2021 17:08:53 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.5.0-alpha0-1302-gc62ab821ef-fm-20210921.001-gc62ab821 Mime-Version: 1.0 Message-Id: <86507f22-d824-4f7c-ba94-d3105c5206c2@www.fastmail.com> In-Reply-To: <20210922205458.358517-5-maz@kernel.org> References: <20210922205458.358517-1-maz@kernel.org> <20210922205458.358517-5-maz@kernel.org> Date: Wed, 22 Sep 2021 23:08:33 +0200 From: "Sven Peter" To: "Marc Zyngier" , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org Cc: "Bjorn Helgaas" , "Rob Herring" , "Lorenzo Pieralisi" , =?UTF-8?Q?Krzysztof_Wilczy=C5=84ski?= , "Alyssa Rosenzweig" , "Stan Skowronek" , "Mark Kettenis" , "Hector Martin" , "Robin Murphy" , kernel-team@android.com Subject: Re: [PATCH v4 04/10] PCI: apple: Add initial hardware bring-up Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Wed, Sep 22, 2021, at 22:54, Marc Zyngier wrote: > From: Alyssa Rosenzweig > [...] > + > + /* Use the first reg entry to work out the port index */ > + port->idx = idx >> 11; > + port->pcie = pcie; > + port->np = np; > + > + port->base = devm_platform_ioremap_resource(platform, port->idx + 2); > + if (IS_ERR(port->base)) > + return -ENODEV; > + > + rmw_set(PORT_APPCLK_EN, port + PORT_APPCLK); I think this should be rmw_set(PORT_APPCLK_EN, port->base + PORT_APPCLK); > + > + rmw_set(PORT_PERST_OFF, port->base + PORT_PERST); > + gpiod_set_value(reset, 1); > + > + ret = readl_relaxed_poll_timeout(port->base + PORT_STATUS, stat, > + stat & PORT_STATUS_READY, 100, 250000); > + if (ret < 0) { > + dev_err(pcie->dev, "port %pOF ready wait timeout\n", np); > + return ret; > + } > + > + /* Flush writes and enable the link */ > + dma_wmb(); I don't think this barrier is required. > + > + writel_relaxed(PORT_LTSSMCTL_START, port->base + PORT_LTSSMCTL); > + > + return 0; > +} > + [...] Looks good to me otherwise, Reviewed-by: Sven Peter Thanks, Sven