Commit c622cbe5 authored by Chakravarty, Souvik K's avatar Chakravarty, Souvik K Committed by Andy Shevchenko

platform/x86: intel_telemetry: Remove redundancies

This patch removes unnecessary header files and newlines.
It also fixes some alignment issues.
Signed-off-by: default avatarSouvik Kumar Chakravarty <souvik.k.chakravarty@intel.com>
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
parent ffd623d4
...@@ -23,7 +23,6 @@ ...@@ -23,7 +23,6 @@
*/ */
#include <linux/debugfs.h> #include <linux/debugfs.h>
#include <linux/device.h> #include <linux/device.h>
#include <linux/io.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/pci.h> #include <linux/pci.h>
#include <linux/seq_file.h> #include <linux/seq_file.h>
...@@ -32,11 +31,10 @@ ...@@ -32,11 +31,10 @@
#include <asm/cpu_device_id.h> #include <asm/cpu_device_id.h>
#include <asm/intel-family.h> #include <asm/intel-family.h>
#include <asm/intel_pmc_ipc.h> #include <asm/intel_pmc_ipc.h>
#include <asm/intel_punit_ipc.h>
#include <asm/intel_telemetry.h> #include <asm/intel_telemetry.h>
#define DRIVER_NAME "telemetry_soc_debugfs" #define DRIVER_NAME "telemetry_soc_debugfs"
#define DRIVER_VERSION "1.0.0" #define DRIVER_VERSION "1.0.0"
/* ApolloLake SoC Event-IDs */ /* ApolloLake SoC Event-IDs */
#define TELEM_APL_PSS_PSTATES_ID 0x2802 #define TELEM_APL_PSS_PSTATES_ID 0x2802
...@@ -246,7 +244,6 @@ static struct telem_ioss_pg_info telem_apl_ioss_pg_data[] = { ...@@ -246,7 +244,6 @@ static struct telem_ioss_pg_info telem_apl_ioss_pg_data[] = {
{"PRTC", 25}, {"PRTC", 25},
}; };
struct telemetry_debugfs_conf { struct telemetry_debugfs_conf {
struct telemetry_susp_stats suspend_stats; struct telemetry_susp_stats suspend_stats;
struct dentry *telemetry_dbg_dir; struct dentry *telemetry_dbg_dir;
...@@ -381,7 +378,6 @@ static int telem_pss_states_show(struct seq_file *s, void *unused) ...@@ -381,7 +378,6 @@ static int telem_pss_states_show(struct seq_file *s, void *unused)
TELEM_APL_MASK_PCS_STATE; TELEM_APL_MASK_PCS_STATE;
} }
TELEM_CHECK_AND_PARSE_EVTS(conf->pss_idle_id, TELEM_CHECK_AND_PARSE_EVTS(conf->pss_idle_id,
conf->pss_idle_evts - 1, conf->pss_idle_evts - 1,
pss_idle, evtlog[index].telem_evtlog, pss_idle, evtlog[index].telem_evtlog,
...@@ -401,7 +397,6 @@ static int telem_pss_states_show(struct seq_file *s, void *unused) ...@@ -401,7 +397,6 @@ static int telem_pss_states_show(struct seq_file *s, void *unused)
conf->pcs_s0ix_blkd_data, conf->pcs_s0ix_blkd_data,
TELEM_MASK_BYTE); TELEM_MASK_BYTE);
TELEM_CHECK_AND_PARSE_EVTS(conf->pss_wakeup_id, TELEM_CHECK_AND_PARSE_EVTS(conf->pss_wakeup_id,
conf->pss_wakeup_evts, conf->pss_wakeup_evts,
pss_s0ix_wakeup, pss_s0ix_wakeup,
...@@ -494,7 +489,6 @@ static const struct file_operations telem_pss_ops = { ...@@ -494,7 +489,6 @@ static const struct file_operations telem_pss_ops = {
.release = single_release, .release = single_release,
}; };
static int telem_ioss_states_show(struct seq_file *s, void *unused) static int telem_ioss_states_show(struct seq_file *s, void *unused)
{ {
struct telemetry_evtlog evtlog[TELEM_MAX_OS_ALLOCATED_EVENTS]; struct telemetry_evtlog evtlog[TELEM_MAX_OS_ALLOCATED_EVENTS];
...@@ -613,7 +607,7 @@ static int telem_soc_states_show(struct seq_file *s, void *unused) ...@@ -613,7 +607,7 @@ static int telem_soc_states_show(struct seq_file *s, void *unused)
(u64)(conf->suspend_stats.deep_res*10)/192); (u64)(conf->suspend_stats.deep_res*10)/192);
seq_printf(s, "TOTAL S0IX\t\t\t %10u\t %10llu\n", s0ix_total_ctr, seq_printf(s, "TOTAL S0IX\t\t\t %10u\t %10llu\n", s0ix_total_ctr,
(u64)(s0ix_total_res*10/192)); (u64)(s0ix_total_res*10/192));
seq_puts(s, "\n-------------------------------------------------\n"); seq_puts(s, "\n-------------------------------------------------\n");
seq_puts(s, "\t\tDEVICE STATES\n"); seq_puts(s, "\t\tDEVICE STATES\n");
seq_puts(s, "-------------------------------------------------\n"); seq_puts(s, "-------------------------------------------------\n");
...@@ -758,7 +752,6 @@ static const struct file_operations telem_pss_trc_verb_ops = { ...@@ -758,7 +752,6 @@ static const struct file_operations telem_pss_trc_verb_ops = {
.release = single_release, .release = single_release,
}; };
static int telem_ioss_trc_verb_show(struct seq_file *s, void *unused) static int telem_ioss_trc_verb_show(struct seq_file *s, void *unused)
{ {
u32 verbosity; u32 verbosity;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment