[PATCH v3 07/11] platform/x86/intel/pmt: correct types

David E. Box david.e.box at linux.intel.com
Fri Jun 6 21:57:27 UTC 2025


On Thu, 2025-06-05 at 14:44 -0400, Michael J. Ruhl wrote:
> A couple of auto variables do not match the return types of some of
> the functions.
> 
> Update the mismatched types to match.
> 
> Signed-off-by: Michael J. Ruhl <michael.j.ruhl at intel.com>
> ---
>  drivers/platform/x86/intel/pmt/crashlog.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/platform/x86/intel/pmt/crashlog.c
> b/drivers/platform/x86/intel/pmt/crashlog.c
> index aa3f57fbe018..a51923d0be72 100644
> --- a/drivers/platform/x86/intel/pmt/crashlog.c
> +++ b/drivers/platform/x86/intel/pmt/crashlog.c
> @@ -136,7 +136,7 @@ static ssize_t
>  enable_show(struct device *dev, struct device_attribute *attr, char *buf)
>  {
>  	struct intel_pmt_entry *entry = dev_get_drvdata(dev);
> -	int enabled = !pmt_crashlog_disabled(entry);
> +	bool enabled = !pmt_crashlog_disabled(entry);
>  
>  	return sprintf(buf, "%d\n", enabled);
>  }
> @@ -167,7 +167,7 @@ static ssize_t
>  trigger_show(struct device *dev, struct device_attribute *attr, char *buf)
>  {
>  	struct intel_pmt_entry *entry;
> -	int trigger;
> +	bool trigger;
>  
>  	entry = dev_get_drvdata(dev);
>  	trigger = pmt_crashlog_complete(entry);

Reviewed-by: David E. Box <david.e.box at linux.intel.com>


More information about the Intel-xe mailing list