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;
Message-ID: <440970b7-2220-97de-e0ea-ab639024db98@amd.com>
Date:   Thu, 8 Jun 2023 16:50:43 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.11.0
Subject: Re: [PATCH v5 07/26] cxl/acpi: Moving add_host_bridge_uport() around
Content-Language: en-US
To:     Dan Williams <dan.j.williams@intel.com>,
        alison.schofield@intel.com, vishal.l.verma@intel.com,
        ira.weiny@intel.com, bwidawsk@kernel.org, dave.jiang@intel.com,
        Jonathan.Cameron@huawei.com, linux-cxl@vger.kernel.org
Cc:     rrichter@amd.com, linux-kernel@vger.kernel.org, bhelgaas@google.com
References: <20230607221651.2454764-1-terry.bowman@amd.com>
 <20230607221651.2454764-8-terry.bowman@amd.com>
 <648233e5ea44e_1433ac29493@dwillia2-xfh.jf.intel.com.notmuch>
From:   Terry Bowman <Terry.Bowman@amd.com>
In-Reply-To: <648233e5ea44e_1433ac29493@dwillia2-xfh.jf.intel.com.notmuch>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?QmZ3UDFKYVhlVktPM3Q2WnRGaHNNM0dCM1hUdG1rQks5Z01IQThBMDZFUFh0?=
 =?utf-8?B?UlNIR05VbUR1QzhYYTVKZzNNaWxkYjdzSTJ2cGlkY29DUTRPNTZHMm9HR3Zl?=
 =?utf-8?B?cStxN3REVmZ0M2xkUCtER3Vpb3RaWnlLWGF1TlZxTnJTTkFONGNqaVFVeVZu?=
 =?utf-8?B?cTJ3Y2N0bDFqSGJEbzJic0pWM3E1dEFFYmd5V2hpYXFBaUg4UkN1eFF5c1Fp?=
 =?utf-8?B?d1lSUis3WjY1ZVFpeUdGN2JyemxHTTdkKzVBV1JsdWY1V3RxMVVYSURiYSt6?=
 =?utf-8?B?TzgzT1c2QmdMR1lOVzFQMW9ycWJ6WEhlU1FiZEsyZFpSL0hOd0QwUU1oQWVi?=
 =?utf-8?B?a1lieHJOcU5ra3RTWnE1M1djdVRmMCt5bEd0bVUyaGxzN0tTQkRmdDRKSWdM?=
 =?utf-8?B?KzZrdzY4Zm0vSzFxQ2U0MnV1d3Roa3QrYktOdHlPTktHMDFLZWROUTU3V29W?=
 =?utf-8?B?d0hJRmVGOXdQVGszL3hRaEhZUVlsSXVuZUtKWk95ZFo0Zi9FYlBNTlBlc2hR?=
 =?utf-8?B?K2VWQ1V6bC9rbGJLcEo5TjQ3VDFPZ0ZuSzdSV3gvK0VkelI0VTFTemdrVHNj?=
 =?utf-8?B?b09abWRNc3NDMWtycE8vSVluN2VVZm54c3dsT21YYVJzeC93SHZGNWFEQTda?=
 =?utf-8?B?dS9qNGxucVhTVEJ3Rng1Z3oydklxM3ZYdVBTRXFvbm1SZHJ4LzR3M3R3b1J1?=
 =?utf-8?B?T21jRDA5Q1dSRE1UL1UvaURZSzRqOHNqdXllUFBwU3ZsczlMaGZ3SXY1c3E0?=
 =?utf-8?B?MCtER0VrRlZDcjhoczhFaStPb09wU3RDcDFwYkVlVVFMemlFbmViQ1V3ZjY5?=
 =?utf-8?B?S1p1aGVObysyZDJsOENZb0F4U2NSbEFPeWQ0Vk5MTXhiaGowWGZhQ2NoSjJx?=
 =?utf-8?B?QStmQW5XdTI0cjBVcmhSZmZqakkzWTE4ZCtWYWxNM25wazNONGtZeDE3RlJF?=
 =?utf-8?B?Qjk3OVdaanRJdGt4azJNd251MURoeFZ0OTF3U09YYnZ1UUVMTXFjR0lnelB5?=
 =?utf-8?B?dzMzQXhBY2gwakVQNTFiVVVxTlp5TnpGR1VtNi82OHpPRWpTZk51bWxBR3du?=
 =?utf-8?B?Q3FUZTA4M3ZPRCtMWllsWEZUQ2hEdWFPSXRCMnJKdWJEb2Z5WlBWUEp3eWVw?=
 =?utf-8?B?M0NsdEU2RnY3dWZDcEkrZC9RZExxUnZ1b0NpOEJ3a0p0QmwzbUFsNGwrZ1E1?=
 =?utf-8?B?UmdrTmdpM1pBdnNBMmIyOERUUnpaeUtsWDZvTlhsRUROZU9GdldzVFlCcmI3?=
 =?utf-8?B?WktTSE1PbjMyWUxXQUtrVXRjU29aZUVvMWRhUmwvYTl0RWgwY1VEb05EWlNt?=
 =?utf-8?B?RHBpSzN5V1JmMUFpOER0Um5xWHdLL1M5Z0ZtRnUxRS93c0YzQnlpdklRM2xB?=
 =?utf-8?B?SmoxNU40Q001WXN1N3duUGdYK2djU2JUcU5JS3hVczIxMHF5eEJXWFJ1WmlZ?=
 =?utf-8?B?dWUvMnVrVndtQWRlMVZYa1lFWHVUM0t3Wk5YaHBHc0cwVU1oeFdRRk5PZUkv?=
 =?utf-8?B?Q0xNaVZCNkJONTFsVFpyZGFQN2hvbUlORUVERFl4aXFVV0hQOTM1WVhEZlJq?=
 =?utf-8?B?SkpkUDFyN0NNT24yYzZPMSsvS2FkVlpCekVzUjdHcngxVmdRaXptNmtnNE9P?=
 =?utf-8?B?alNjdWpuZ2hyaVJuV3J0M3RVRCs2TFhrUGE1MW04V0RBSDNyUFNmM0trbDZU?=
 =?utf-8?B?RGVXSnlLOHhrZWc5czh1YUcyMXNWajY5VjhIRlJzZS8zSU5oR0pWbnRDVEVB?=
 =?utf-8?B?bTlxNnFzTFZ0aWlGK29yakhYbzRQbDB3MEtIRzJCZWRlcDRqUy80M1hSSUNz?=
 =?utf-8?B?SldTb3U3c0dKY1Y5Nmt3a00yZUw5bC9OVEcvc0tUWXUrVUs4UUtkRXFzOWRj?=
 =?utf-8?B?T2pybGlqVS9jUFk5K2xiVHkwZy8xQmJVOHMrSzRhbXZ5WTBZajdZQ1BCaXVv?=
 =?utf-8?B?S3hiZWZxdXFIc2ZEWThqMnBaNW1BdU13NVRmdzRCM2Y4a0FXNHVHZXdjZEtX?=
 =?utf-8?B?bkYxNWNFSnlWSEc1TWZxZEcvSVpXSHdnOXJCdXIrZnVvUnhydW1IOC9IbVM1?=
 =?utf-8?B?d1lXTmFhRmtNWmdBSDZjLzJoV2Ivb0lBUWdDamZrd25sanQvUExXdFFudkxh?=
 =?utf-8?B?TjJUVWFsdDNRZkl2M09GWk92dVFKVmt3RFlBZ2I5b212V0IxSXQwc0RrRS9U?=
 =?utf-8?Q?/6gF6+neVxoChfrFwGLtrzwoQRHMpcka5OxKLmsG+8Lh?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a634119c-1e23-4c6c-b3a7-08db686a6a77
X-MS-Exchange-CrossTenant-AuthSource: DS0PR12MB6390.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Jun 2023 21:50:46.6418
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: CeUU8yuuajHKSHqqH9UYawtIXl2rrEX2sWq3mDnoiXD6UpF8B0eqdpq3y3xGKAi/VLVUNnEcce1SQzadYBcOCA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB6718
X-Spam-Status: No, score=-1.2 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,NICE_REPLY_A,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=no 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

Thanks Dan. I'll update.

Regards,
Terry

On 6/8/23 15:02, Dan Williams wrote:
> Minor quibble with the gerund phrase and this not being clear about what
> it is doing. Just say:
> 
> "cxl/acpi: Move add_host_bridge_uport() after cxl_get_chbs()"
> 
> Terry Bowman wrote:
>> From: Robert Richter <rrichter@amd.com>
>>
>> Just moving code to reorder functions to later share cxl_get_chbs()
>> with add_host_bridge_uport().
>>
>> This makes changes in the next patch visible. No other changes at all.
> 
> Other than subject line, looks good.