Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 9 Jun 2023 19:37:14 -0700
From:   Dan Williams <dan.j.williams@intel.com>
To:     Terry Bowman <terry.bowman@amd.com>, <alison.schofield@intel.com>,
        <vishal.l.verma@intel.com>, <ira.weiny@intel.com>,
        <bwidawsk@kernel.org>, <dan.j.williams@intel.com>,
        <dave.jiang@intel.com>, <Jonathan.Cameron@huawei.com>,
        <linux-cxl@vger.kernel.org>
CC:     <terry.bowman@amd.com>, <rrichter@amd.com>,
        <linux-kernel@vger.kernel.org>, <bhelgaas@google.com>
Subject: RE: [PATCH v5 17/26] cxl/port: Remove Component Register base
 address from struct cxl_dport
Message-ID: <6483e1dae75e4_e067a294bd@dwillia2-xfh.jf.intel.com.notmuch>
References: <20230607221651.2454764-1-terry.bowman@amd.com>
 <20230607221651.2454764-18-terry.bowman@amd.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20230607221651.2454764-18-terry.bowman@amd.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?PnljoSlyhR8vCfU2mRviB1UYZ6Abtw7Sh//DIMafQrdeLZisYa07RcJ2Utfk?=
 =?us-ascii?Q?2qMrddht+89u0xufMwti+s5Y/UcJxtUe8tRJcqock/SjF0ePvH5NCk69oZJb?=
 =?us-ascii?Q?4yIp+FXjzUwAQV7Wx/qhnzbuHZtSDgWLUxQ8HwnXXdiFIVALjMUc4v4CSHrU?=
 =?us-ascii?Q?buLOEmyThvYwoveDLzbpNPpX2bG17uHh8W4sBL5xDTpeNT1jGjW1JCxpPp0X?=
 =?us-ascii?Q?8egqO+fXI3mD5q0fb4onO6QL7Cbe1wPtUnrQCQ892Ivhi2EvXCbzJetbTd2s?=
 =?us-ascii?Q?O7exrR8zcU/fe+/+dVEJ1QPCmzGxk0OIfgHsNooHxfCb4zZTOHbpTFEWD78t?=
 =?us-ascii?Q?tj427JGGQObeSdDky+/jw0rgA/W6oJZTWVos7DfhnNmpyWFsb1KC6SOQtTO2?=
 =?us-ascii?Q?c1W1YCo+DKXI1OausYMvSKEBq/1i6ovLkk1PdcXFikF9zRFlF/kNHaKQ4XlL?=
 =?us-ascii?Q?KuQTMVXxXC73jpWs66vzTGjXkDKhzu7u9Dl8NNN9ZtlbIRjF/6DtusC/KRsZ?=
 =?us-ascii?Q?b4Y+pGwto2zGJzpK0Gxam5up4BW9MtmAsjJrGEG9z33GH1JufLOF7xJYQczm?=
 =?us-ascii?Q?N0NIoIFlI0XedV6bdAlsYWg9UMcpHW2xZQQBOAzu0akmLJnZuY2V1exUt4QD?=
 =?us-ascii?Q?6DwyARvSpQiRRQSCXIfrQiGQ3o0htNfq+K8Rfse+jtVhypnTtDKrWTJ/DXcG?=
 =?us-ascii?Q?4L7wMvlTsnHgA4gd5nXdqUIZeeodrPEBSHY6Vm+44X+jDwzXa6894ufH/ufP?=
 =?us-ascii?Q?7bl2PB33DhZFTjAV7gpIBWngHLhE1Dd2ohbkPAi5FfXAFPqgUX02t2nKS9ny?=
 =?us-ascii?Q?zSX+N2MWCzUZYyTNK1HBTS+NxzwkK0NE1AWUPJjyETqC4J5SgWY2d2IaMw/b?=
 =?us-ascii?Q?+jBvXY5UN+9ulsLYPhrNZOp69cs++HADI7BEbp/mqy269SG8DHoa/MqNV4rY?=
 =?us-ascii?Q?RoMiLEZt7yoDxUi/x1CmBO1U1loVXeC+em/bx28CyYud+TRAXsqDEBugc+wC?=
 =?us-ascii?Q?uJFLjrrGQmQsPCK1sGjtYiEhMi1Ap/7zeBZT0IXhPx3xiWx56vnWDTrQ56Xj?=
 =?us-ascii?Q?zdSWiEWX+sbwKyiHX54DS0MLNUTQH7xy7pYK5QuBXb4QvWIfJJE9kCa3iFHG?=
 =?us-ascii?Q?owUvipgpMsjpZ8JPmIYNHErTiyk7dKCvJTXpc/MNZnnTX92D/RhZqceE4sGJ?=
 =?us-ascii?Q?sOH3yCz/GPkbAN2F++Z8Q995EN+Q/4JChg8gW5BTfV1EKdM9Siy2XOu4MQRN?=
 =?us-ascii?Q?K2QfS/dZXp6w+9KgfHLSOb2GTVuzpxUaewPCmxtI1KNn3iavM/SkwclEG0gq?=
 =?us-ascii?Q?hDqgPaP8NUYzF3ahRaLWh5cFLw3jqnXo6b3qyKTFUwpOZv8jIJX67F+5Qd71?=
 =?us-ascii?Q?u8epQub3lBWRsMNS4stG3cciz41dAo2sr7UYbWzCCbulgjUpiZaysypz6AeG?=
 =?us-ascii?Q?QOd4THAZi7rZHd2iWZs0U2Zr/dnzCI5ZIQ8KciPsN3OiknRvoLnJRmGEY1cb?=
 =?us-ascii?Q?QW2alB3w1tDLgW/yYoQAuKoV48y6KctONyHOZSvuq9IlUwVp4J37fTtu4jT7?=
 =?us-ascii?Q?JKLUa2oA2tF8f7AGuSq/j3M5MTJYMaEOdlYs66G9oc1pAI4kT4W3M579Ob/v?=
 =?us-ascii?Q?Og=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 43fbe2c0-dd20-4c08-6f66-08db695b9c61
X-MS-Exchange-CrossTenant-AuthSource: PH8PR11MB8107.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Jun 2023 02:37:19.1039
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: TsYx3/wArEVrxCAKycg/fY3E4Qz+ZfUH6Q06KRoEMgP6W+UJ0HMP2O8eGNj55EEd5ZA9CFq0p2VD3IjsN3qxArGAZomDx4fgFG+44sQCc7M=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR11MB7245
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,
        T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Terry Bowman wrote:
> From: Robert Richter <rrichter@amd.com>
> 
> The Component Register base address @component_reg_phys is no longer
> used after the rework of the Component Register setup which now uses
> struct member @comp_map instead. Remove the base address.
> 
> Signed-off-by: Robert Richter <rrichter@amd.com>
> Signed-off-by: Terry Bowman <terry.bowman@amd.com>
> Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

As I mentioned earlier this could have happened several patches back, so
looks good, but consider moving it earlier in the series for v6.