FROMLIST: firmware: arm_scmi: remove legacy scmi_clk_ops protocol interface
Now that all the SCMI driver users have been migrated to the new interface remove the legacy interface and all the transient code. Signed-off-by: Cristian Marussi <cristian.marussi@arm.com> Bug: 171409184 Link: https://lore.kernel.org/linux-arm-kernel/20210202221555.41167-1-cristian.marussi@arm.com/ Change-Id: I8deb2f967d9333bb6f6cf88fb1bb4f7aed622608 Signed-off-by: Rishabh Bhatnagar <quic_rishabhb@quicinc.com>
This commit is contained in:
parent
bcd627f631
commit
cf6cde9f1d
@ -235,16 +235,6 @@ scmi_clock_rate_get(const struct scmi_protocol_handle *ph,
|
||||
return ret;
|
||||
}
|
||||
|
||||
static int
|
||||
__scmi_clock_rate_get(const struct scmi_handle *handle,
|
||||
u32 clk_id, u64 *value)
|
||||
{
|
||||
const struct scmi_protocol_handle *ph =
|
||||
scmi_map_protocol_handle(handle, SCMI_PROTOCOL_CLOCK);
|
||||
|
||||
return scmi_clock_rate_get(ph, clk_id, value);
|
||||
}
|
||||
|
||||
static int scmi_clock_rate_set(const struct scmi_protocol_handle *ph,
|
||||
u32 clk_id, u64 rate)
|
||||
{
|
||||
@ -280,15 +270,6 @@ static int scmi_clock_rate_set(const struct scmi_protocol_handle *ph,
|
||||
return ret;
|
||||
}
|
||||
|
||||
static int __scmi_clock_rate_set(const struct scmi_handle *handle,
|
||||
u32 clk_id, u64 rate)
|
||||
{
|
||||
const struct scmi_protocol_handle *ph =
|
||||
scmi_map_protocol_handle(handle, SCMI_PROTOCOL_CLOCK);
|
||||
|
||||
return scmi_clock_rate_set(ph, clk_id, rate);
|
||||
}
|
||||
|
||||
static int
|
||||
scmi_clock_config_set(const struct scmi_protocol_handle *ph, u32 clk_id,
|
||||
u32 config)
|
||||
@ -317,27 +298,11 @@ static int scmi_clock_enable(const struct scmi_protocol_handle *ph, u32 clk_id)
|
||||
return scmi_clock_config_set(ph, clk_id, CLOCK_ENABLE);
|
||||
}
|
||||
|
||||
static int __scmi_clock_enable(const struct scmi_handle *handle, u32 clk_id)
|
||||
{
|
||||
const struct scmi_protocol_handle *ph =
|
||||
scmi_map_protocol_handle(handle, SCMI_PROTOCOL_CLOCK);
|
||||
|
||||
return scmi_clock_enable(ph, clk_id);
|
||||
}
|
||||
|
||||
static int scmi_clock_disable(const struct scmi_protocol_handle *ph, u32 clk_id)
|
||||
{
|
||||
return scmi_clock_config_set(ph, clk_id, 0);
|
||||
}
|
||||
|
||||
static int __scmi_clock_disable(const struct scmi_handle *handle, u32 clk_id)
|
||||
{
|
||||
const struct scmi_protocol_handle *ph =
|
||||
scmi_map_protocol_handle(handle, SCMI_PROTOCOL_CLOCK);
|
||||
|
||||
return scmi_clock_disable(ph, clk_id);
|
||||
}
|
||||
|
||||
static int scmi_clock_count_get(const struct scmi_protocol_handle *ph)
|
||||
{
|
||||
struct clock_info *ci = ph->get_priv(ph);
|
||||
@ -345,14 +310,6 @@ static int scmi_clock_count_get(const struct scmi_protocol_handle *ph)
|
||||
return ci->num_clocks;
|
||||
}
|
||||
|
||||
static int __scmi_clock_count_get(const struct scmi_handle *handle)
|
||||
{
|
||||
const struct scmi_protocol_handle *ph =
|
||||
scmi_map_protocol_handle(handle, SCMI_PROTOCOL_CLOCK);
|
||||
|
||||
return scmi_clock_count_get(ph);
|
||||
}
|
||||
|
||||
static const struct scmi_clock_info *
|
||||
scmi_clock_info_get(const struct scmi_protocol_handle *ph, u32 clk_id)
|
||||
{
|
||||
@ -365,24 +322,6 @@ scmi_clock_info_get(const struct scmi_protocol_handle *ph, u32 clk_id)
|
||||
return clk;
|
||||
}
|
||||
|
||||
static const struct scmi_clock_info *
|
||||
__scmi_clock_info_get(const struct scmi_handle *handle, u32 clk_id)
|
||||
{
|
||||
const struct scmi_protocol_handle *ph =
|
||||
scmi_map_protocol_handle(handle, SCMI_PROTOCOL_CLOCK);
|
||||
|
||||
return scmi_clock_info_get(ph, clk_id);
|
||||
}
|
||||
|
||||
static const struct scmi_clk_ops clk_ops = {
|
||||
.count_get = __scmi_clock_count_get,
|
||||
.info_get = __scmi_clock_info_get,
|
||||
.rate_get = __scmi_clock_rate_get,
|
||||
.rate_set = __scmi_clock_rate_set,
|
||||
.enable = __scmi_clock_enable,
|
||||
.disable = __scmi_clock_disable,
|
||||
};
|
||||
|
||||
static const struct scmi_clk_proto_ops clk_proto_ops = {
|
||||
.count_get = scmi_clock_count_get,
|
||||
.info_get = scmi_clock_info_get,
|
||||
@ -397,7 +336,6 @@ static int scmi_clock_protocol_init(const struct scmi_protocol_handle *ph)
|
||||
u32 version;
|
||||
int clkid, ret;
|
||||
struct clock_info *cinfo;
|
||||
struct scmi_handle *handle;
|
||||
|
||||
ph->xops->version_get(ph, &version);
|
||||
|
||||
@ -424,11 +362,6 @@ static int scmi_clock_protocol_init(const struct scmi_protocol_handle *ph)
|
||||
}
|
||||
|
||||
cinfo->version = version;
|
||||
|
||||
/* Transient code for legacy ops interface */
|
||||
handle = scmi_map_scmi_handle(ph);
|
||||
handle->clk_ops = &clk_ops;
|
||||
|
||||
return ph->set_priv(ph, cinfo);
|
||||
}
|
||||
|
||||
|
@ -84,19 +84,6 @@ struct scmi_clk_proto_ops {
|
||||
int (*disable)(const struct scmi_protocol_handle *ph, u32 clk_id);
|
||||
};
|
||||
|
||||
struct scmi_clk_ops {
|
||||
int (*count_get)(const struct scmi_handle *hamdle);
|
||||
|
||||
const struct scmi_clock_info *(*info_get)
|
||||
(const struct scmi_handle *handle, u32 clk_id);
|
||||
int (*rate_get)(const struct scmi_handle *handle, u32 clk_id,
|
||||
u64 *rate);
|
||||
int (*rate_set)(const struct scmi_handle *handle, u32 clk_id,
|
||||
u64 rate);
|
||||
int (*enable)(const struct scmi_handle *handle, u32 clk_id);
|
||||
int (*disable)(const struct scmi_handle *handle, u32 clk_id);
|
||||
};
|
||||
|
||||
/**
|
||||
* struct scmi_perf_proto_ops - represents the various operations provided
|
||||
* by SCMI Performance Protocol
|
||||
@ -615,7 +602,6 @@ struct scmi_notify_ops {
|
||||
*
|
||||
* @dev: pointer to the SCMI device
|
||||
* @version: pointer to the structure containing SCMI version information
|
||||
* @clk_ops: pointer to set of clock protocol operations
|
||||
* @sensor_ops: pointer to set of sensor protocol operations
|
||||
* @reset_ops: pointer to set of reset protocol operations
|
||||
* @voltage_ops: pointer to set of voltage protocol operations
|
||||
@ -641,7 +627,6 @@ struct scmi_notify_ops {
|
||||
struct scmi_handle {
|
||||
struct device *dev;
|
||||
struct scmi_revision_info *version;
|
||||
const struct scmi_clk_ops *clk_ops;
|
||||
const struct scmi_sensor_ops *sensor_ops;
|
||||
const struct scmi_reset_ops *reset_ops;
|
||||
const struct scmi_voltage_ops *voltage_ops;
|
||||
|
Loading…
Reference in New Issue
Block a user