Re: [v3 0/4] Fixes for hdm docoder initialization from DVSEC ranges

[Yanfei Xu <yanfei.xu@intel.com>]

On 8/28/2024 12:08 AM, Jonathan Cameron wrote:
> On Tue, 13 Aug 2024 19:05:28 +0800
> Yanfei Xu <yanfei.xu@intel.com> wrote:
> 
>> The first and third patch are intended to fix potential issues regarding to
>> retrieve and record DVSEC ranges. The second and fourth are cleanup.
> 
> typo in patch title.  decoder
> 
> 
> Also, good to include cxl: as prefix to the patch title.
> 

Got it, will do.

>>
>> v2->v3:
>> - improved the commit message of patch1 to indicate potential impact of
>>    the change.  (Dan)
>> - Dropped un-appropriate "Fixes" tag. (Dan)
>> - Dropped the patch2 which is a code movement in original patchset.
>> - Separated the original patch3 into cleanup one and logic change one which
>>    are corresponding to patch2 and patch3 in current patchset. (Dan)
>>
>> v2:https://lore.kernel.org/linux-cxl/20240809093442.646545-1-yanfei.xu@intel.com/T/#t
>>
>> Yanfei Xu (4):
>>    cxl/pci: Fix to record only non-zero ranges
>>    cxl/pci: Remove duplicated implementation of waiting for
>>      memory_info_valid
>>    cxl/pci: Check Mem_info_valid bit for each applicable DVSEC
>>    cxl/pci: simplify the check of mem_enabled in cxl_hdm_decode_init()
>>
>>   drivers/cxl/core/pci.c        | 74 +++++++++--------------------------
>>   drivers/cxl/cxl.h             |  2 +-
>>   drivers/cxl/port.c            |  2 +-
>>   tools/testing/cxl/test/mock.c |  4 +-
>>   4 files changed, 23 insertions(+), 59 deletions(-)
>>
> 
>