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: <a514ace3-ee63-7556-1b89-cd76e6333015@amd.com>
Date:   Mon, 29 Aug 2022 07:23:04 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.10.0
Subject: Re: [PATCH v7 2/3] thunderbolt: Modify tb_nvm major and minor size.
Content-Language: en-US
To:     Szuying Chen <chensiying21@gmail.com>, gregkh@linuxfoundation.org,
        mika.westerberg@linux.intel.com, andreas.noever@gmail.com,
        michael.jamet@intel.com, YehezkelShB@gmail.com,
        linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Cc:     Yd_Tseng@asmedia.com.tw, Chloe_Chen@asmedia.com.tw,
        Richard_Hsu@asmedia.com.tw
References: <20220829111059.665305-1-chensiying21@gmail.com>
 <20220829111059.665305-3-chensiying21@gmail.com>
From:   Mario Limonciello <mario.limonciello@amd.com>
In-Reply-To: <20220829111059.665305-3-chensiying21@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ejJGQ0VPS01zaGl1cU51eWNINkZLOStiWjEvR1puT1d0NmZLamVPS042ZWRJ?=
 =?utf-8?B?RDdjYXE3bGhOeEVHMWZGeUI2Q3lZMng5RnZod1FyK284NlNBZnNISnNyZDMr?=
 =?utf-8?B?VzhCQTlYTDJyZTR3eVVTZlVLQVJtQWhIbjVhbzlhWWxNYVhhSk5LZHQrRXdK?=
 =?utf-8?B?Z0puNjVmY2JUdXh4VTRtQzFYY0VyLzlqK3NtQ3RzT09jLzFJRjN6ZWR5a1ZM?=
 =?utf-8?B?MG5DdlNRdEhkdllZWElYOGRSdER2WlBrdktUZDBMaFZ6M0pMTlpsQmlzVkR1?=
 =?utf-8?B?M1h2eGtnN3QrTWFoczltcGs1VU9RNFhmZUpTeVd0Z05rUXdjd3lseTdOdnFE?=
 =?utf-8?B?Uy9hQWgzQXN5ejZVS2JzYTd4SFBlYnBaNFlDbFJYVnVSYytBWnp4ZEVDYWZJ?=
 =?utf-8?B?Vlp0Q25PZk5JQmxkRDA2UEhXZ3U1M0FsK0dma1c4MEhMcXNhTUpTRnJQa2Rh?=
 =?utf-8?B?QW5IZVRlSTFsV3FEMEMxQ2xLK1hXeVkzaHF2MXl1N09UclZEU3FPOFdLc3dQ?=
 =?utf-8?B?OUVKZ0h1YnNNOXRvb29QWjhCWENoUUdOcDcxbWZ1cEF0REc0ZGxLOTdpZ2JC?=
 =?utf-8?B?SThBMG1meDBnbHpnNzMrYXk1aXhpQUN0cG12bmZwUHliRmc3eXpjc0dNc1Y1?=
 =?utf-8?B?VlFid2d1KzhGZDdHVTNSVGlwNkNRZjkzbXdXR3VFOFYwSERPYTA5Q2E5L29F?=
 =?utf-8?B?SkVVOHQ1d1dsdHNMNjMzRUZIUFRzSFBXMUd1NDhiRG5kNTVvWWN0T0RIMS9y?=
 =?utf-8?B?YjNaRmFMcUpvaWl4NkxIQVZkUG44UGZXZm5nd0FaY3J3OVBpT29oT0JkYlBU?=
 =?utf-8?B?SWFKTjdiYko2TnExcnlwOU5QQkV0d2Y0OW9RZ1hYb25MdVVHNE84YlRNU1c0?=
 =?utf-8?B?dFNacVFBVHduTWJvcHI3THlzaDlGTk5odzRadTJHb0tjOW1FaEw3VUg2OFNr?=
 =?utf-8?B?R2Fwd0pEdDlmTVRQRk0zT0kwRVlkdVNETTJjMEduOXZ6TmpnNkdQMjVYeEl3?=
 =?utf-8?B?V0JBNnZHOXhQWS9lUDB1aE5VSFNtOGdWWkRJU1NxWFBwaE11NzlnQ1hKMk5D?=
 =?utf-8?B?M1dNVElxcTk3SG1uVS9kV1NPczQ2ZDk3a3hlcWxBMkpYRitSQzVuY0ttOWo3?=
 =?utf-8?B?VGpyTlhxb3FGd25hUlB1RGNkTXdpLzNZWGwzK2ZBMUR6UjNScmh0K1VNLzU3?=
 =?utf-8?B?dGlhTGV6QS9rQldxU3ZtYk9qa0JDY3JjRDlRWUtlc2RmcEhsb1NhZW9HMFQ0?=
 =?utf-8?B?MVBKTE5HeElBZE1QK2xaOEtUTVZVOEQzMEsrY3NaTDBINnhFaWh0cU52Qlo2?=
 =?utf-8?B?TG1EMDNJK2R3L0Q3d1hoYTZDenVuR2JUbGtNN3dHUXd3MnFnWlVlUXM1dW1K?=
 =?utf-8?B?d3ZLOHJjNkl5N3dYSnF1MTdFT2JZRHRPR3hyZUYvd3FrUU1Yam1lcXgxT3FX?=
 =?utf-8?B?ODVjSk9EUFB1ekNmT3VYeWhJZnZ1NzBPRHJ3UTcxanlETDBXMC9DNmd4a1Zh?=
 =?utf-8?B?dVp4WGJHaE5NdWhRNXlaNENiQ3BxRDQvZm9KWTZjNXJtYjBocU5aS0MxZlVC?=
 =?utf-8?B?SzVacGY3eWpmR01wQXJ1bVpCUHVRUnkwZkpMNnVheGZwSFQxTVJCdXRZbjE0?=
 =?utf-8?B?bStlbFpTUE1HbENhWENpNEN1R0YzNjBFMlFSdjZaWHNRd0V2a2IzS2xHdklY?=
 =?utf-8?B?bDJ6bzBwNUU2cGgwdTJIczVuK1lKSUhjZkUxSHg1bmtBQVVvN29DZldYa25T?=
 =?utf-8?B?aFNRK2NuajlNRXFRRUVraUZZcUVCSmQrVHROdTY1cmp3ZkFoWEdzS2lxSnIy?=
 =?utf-8?B?YmM4cTcwWi9kOTVCNFFIdHpPVHY4eElPNXV3VldSUjdOVXRKYUVpSFNXOTYr?=
 =?utf-8?B?Mi9SRWJ0Zy9TUERoQmVFOFFDUFB2NjhRSDRVSVZkTjJmMnhGVjZhNlpNL2ht?=
 =?utf-8?B?dTFJMFpRRDFITlNsVjEva08rUmNCUzlLNURBYm8zalFXaXNVb1Qyend2R1Jk?=
 =?utf-8?B?KzhPMmRkRmkrQy9aU1hzamZzN0NtVHNCdkJUcVJLRXpRM0k5WWtPbG5wVkpX?=
 =?utf-8?B?a1RwVDA2bFNFbXdFUkRPV2VPQndkZ0VvNThMaDkyZ2NlU2p3NWl0LzVMb05X?=
 =?utf-8?B?NCtINi82czBEU2pEQkJUQU1nMlRqU0l0MS9GZE9iRWxUcU1uejdGUW13ZFA3?=
 =?utf-8?Q?oImG7n8REXUBfEQRG9rvycQHQWujG9lXjOmqfZHNc92a?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4acc479f-1ed3-40a6-33dc-08da89b93ac2
X-MS-Exchange-CrossTenant-AuthSource: MN0PR12MB6101.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Aug 2022 12:23:07.5513
 (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: 3ga7SSgp4jS+x7fc/AzAmJOqcxp31SiD0OQSMswfb6e2lKWnjLvn3flQfzEie0JjS3LLcEdj+LFjNR+jZXdOsw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR12MB7135
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        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

On 8/29/22 06:10, Szuying Chen wrote:
> From: Szuying Chen <Chloe_Chen@asmedia.com.tw>
> 
> The patch modify tb_nvm->major and tb_nvm->minor size to u32 that support
> diffrent vendor's NVM version show.

s/diffrent/different/

I would suggest you explain the WHY of this patch.  I would have worded 
it something like this:

Intel's version can be stored in 2 bytes, but ASMedia's version requires 
8 bytes.  Extend the 'major' and 'minor' members of the tb_nvm structure 
to support both vendors.

> 
> Signed-off-by: Szuying Chen <Chloe_Chen@asmedia.com.tw>
> ---
> Modify tb_nvm->major and tb_nvm->minor size to u32.

The idea with the changelog below the cutline is supposed to help 
reviewers know what to focus on when reviewing from one patch to another.

In general it's best to specify what changed from which patch to which 
past the cut line.

For example if this was first patch version that introduced the change 
it should be something like:

v6->v7:
  * New patch based on suggestion by Mario


> 
>   drivers/thunderbolt/tb.h | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/thunderbolt/tb.h b/drivers/thunderbolt/tb.h
> index fc32737fcde4..9cf62d5f25d2 100644
> --- a/drivers/thunderbolt/tb.h
> +++ b/drivers/thunderbolt/tb.h
> @@ -50,8 +50,8 @@
>    */
>   struct tb_nvm {
>   	struct device *dev;
> -	u8 major;
> -	u8 minor;
> +	u32 major;
> +	u32 minor;
>   	int id;
>   	struct nvmem_device *active;
>   	struct nvmem_device *non_active;
> --
> 2.34.1
>