[PATCH] soundwire: qcom: Constify static structs
Vinod Koul
vkoul at kernel.org
Mon Jun 22 13:58:37 CEST 2020
On 10-06-20, 01:00, Rikard Falkeborn wrote:
> qcom_swrm_port_ops and qcom_swrm_ops are not modified and can be made
> const to allow the compiler to put them in read-only memory.
>
> Before:
> text data bss dec hex filename
> 18266 3056 256 21578 544a drivers/soundwire/qcom.o
>
> After:
> text data bss dec hex filename
> 18426 2896 256 21578 544a drivers/soundwire/qcom.o
Applied, thanks
>
> Signed-off-by: Rikard Falkeborn <rikard.falkeborn at gmail.com>
> ---
> drivers/soundwire/qcom.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/soundwire/qcom.c b/drivers/soundwire/qcom.c
> index a1c2a44a3b4d..915c2cf0c274 100644
> --- a/drivers/soundwire/qcom.c
> +++ b/drivers/soundwire/qcom.c
> @@ -406,13 +406,13 @@ static int qcom_swrm_port_enable(struct sdw_bus *bus,
> return ctrl->reg_write(ctrl, reg, val);
> }
>
> -static struct sdw_master_port_ops qcom_swrm_port_ops = {
> +static const struct sdw_master_port_ops qcom_swrm_port_ops = {
> .dpn_set_port_params = qcom_swrm_port_params,
> .dpn_set_port_transport_params = qcom_swrm_transport_params,
> .dpn_port_enable_ch = qcom_swrm_port_enable,
> };
>
> -static struct sdw_master_ops qcom_swrm_ops = {
> +static const struct sdw_master_ops qcom_swrm_ops = {
> .xfer_msg = qcom_swrm_xfer_msg,
> .pre_bank_switch = qcom_swrm_pre_bank_switch,
> };
> --
> 2.27.0
--
~Vinod
More information about the Alsa-devel
mailing list