Re: [PATCH v1 1/2] platform/x86/amd/pmf: Use existing input event codes to update system states

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Ilpo,

On 7/10/2024 16:36, Ilpo Järvinen wrote:
> On Tue, 2 Jul 2024, Shyam Sundar S K wrote:
> 
>> At present, the PMF driver employs custom system state codes to update
>> system states. It is recommended to replace these with existing input
>> event codes (KEY_SLEEP, KEY_SUSPEND, and KEY_SCREENLOCK) to align system
>> updates with the PMF-TA output actions.
>>
>> Co-developed-by: Patil Rajesh Reddy <Patil.Reddy@xxxxxxx>
>> Signed-off-by: Patil Rajesh Reddy <Patil.Reddy@xxxxxxx>
>> Signed-off-by: Shyam Sundar S K <Shyam-sundar.S-k@xxxxxxx>
>> ---
> 
>> @@ -475,6 +506,10 @@ int amd_pmf_init_smart_pc(struct amd_pmf_dev *dev)
>>  	if (pb_side_load)
>>  		amd_pmf_open_pb(dev, dev->dbgfs_dir);
>>  
>> +	ret = amd_pmf_register_input_device(dev);
>> +	if (ret)
>> +		goto error;
>> +
>>  	return 0;
>>  
>>  error:
>> @@ -488,6 +523,9 @@ void amd_pmf_deinit_smart_pc(struct amd_pmf_dev *dev)
>>  	if (pb_side_load && dev->esbin)
>>  		amd_pmf_remove_pb(dev);
>>  
>> +	if (dev->pmf_idev)
>> +		input_unregister_device(dev->pmf_idev);
>> +
> 
> Why is the ordering in the init and deinit asymmetric? Is that 
> intentional?
> 

No. This is not intentional. I will respin a new version to make it
symmetric.

Thanks,
Shyam




[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux