On 3/6/24 3:44 AM, Shyam Sundar S K wrote: > Update the APMF function index 2 for family 1Ah, that gets the > information of SBIOS requests (like the pending requests from BIOS, > custom notifications, updation of power limits etc). > > 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> > --- Looks good to me. Reviewed-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@xxxxxxxxxxxxxxx> > drivers/platform/x86/amd/pmf/acpi.c | 5 +++++ > drivers/platform/x86/amd/pmf/pmf.h | 13 +++++++++++++ > 2 files changed, 18 insertions(+) > > diff --git a/drivers/platform/x86/amd/pmf/acpi.c b/drivers/platform/x86/amd/pmf/acpi.c > index 1f287a147c57..13af83b187ac 100644 > --- a/drivers/platform/x86/amd/pmf/acpi.c > +++ b/drivers/platform/x86/amd/pmf/acpi.c > @@ -166,6 +166,11 @@ int apmf_get_auto_mode_def(struct amd_pmf_dev *pdev, struct apmf_auto_mode *data > return apmf_if_call_store_buffer(pdev, APMF_FUNC_AUTO_MODE, data, sizeof(*data)); > } > > +int apmf_get_sbios_requests_v2(struct amd_pmf_dev *pdev, struct apmf_sbios_req_v2 *req) > +{ > + return apmf_if_call_store_buffer(pdev, APMF_FUNC_SBIOS_REQUESTS, req, sizeof(*req)); > +} > + > int apmf_get_sbios_requests(struct amd_pmf_dev *pdev, struct apmf_sbios_req *req) > { > return apmf_if_call_store_buffer(pdev, APMF_FUNC_SBIOS_REQUESTS, > diff --git a/drivers/platform/x86/amd/pmf/pmf.h b/drivers/platform/x86/amd/pmf/pmf.h > index 51e31d008ef9..aeaaff290bff 100644 > --- a/drivers/platform/x86/amd/pmf/pmf.h > +++ b/drivers/platform/x86/amd/pmf/pmf.h > @@ -120,6 +120,18 @@ struct apmf_sbios_req { > u8 skin_temp_hs2; > } __packed; > > +struct apmf_sbios_req_v2 { > + u16 size; > + u32 pending_req; > + u8 rsd; > + u32 ppt_pmf; > + u32 ppt_pmf_apu_only; > + u32 stt_min_limit; > + u8 skin_temp_apu; > + u8 skin_temp_hs2; > + u32 custom_policy[10]; > +} __packed; > + > struct apmf_fan_idx { > u16 size; > u8 fan_ctl_mode; > @@ -621,6 +633,7 @@ void amd_pmf_init_auto_mode(struct amd_pmf_dev *dev); > void amd_pmf_deinit_auto_mode(struct amd_pmf_dev *dev); > void amd_pmf_trans_automode(struct amd_pmf_dev *dev, int socket_power, ktime_t time_elapsed_ms); > int apmf_get_sbios_requests(struct amd_pmf_dev *pdev, struct apmf_sbios_req *req); > +int apmf_get_sbios_requests_v2(struct amd_pmf_dev *pdev, struct apmf_sbios_req_v2 *req); > > void amd_pmf_update_2_cql(struct amd_pmf_dev *dev, bool is_cql_event); > int amd_pmf_reset_amt(struct amd_pmf_dev *dev); -- Sathyanarayanan Kuppuswamy Linux Kernel Developer