| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <797c1e16-6393-8cdb-81c8-da25dc9be1fa@linaro.org>
Date: Mon, 28 Feb 2022 09:52:49 -0600
From: Alex Elder <elder@...aro.org>
To: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>,
mhi@...ts.linux.dev
Cc: quic_hemantk@...cinc.com, quic_bbhatt@...cinc.com,
quic_jhugo@...cinc.com, vinod.koul@...aro.org,
bjorn.andersson@...aro.org, dmitry.baryshkov@...aro.org,
quic_vbadigan@...cinc.com, quic_cang@...cinc.com,
quic_skananth@...cinc.com, linux-arm-msm@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 07/27] bus: mhi: host: Rename "struct mhi_tre" to
"struct mhi_ring_element"
On 2/28/22 6:43 AM, Manivannan Sadhasivam wrote:
> Structure "struct mhi_tre" is representing a generic MHI ring element and
> not specifically a Transfer Ring Element (TRE). Fix the naming.
>
> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Looks good.
Reviewed-by: Alex Elder <elder@...aro.org>
> ---
> drivers/bus/mhi/host/init.c | 6 +++---
> drivers/bus/mhi/host/internal.h | 2 +-
> drivers/bus/mhi/host/main.c | 20 ++++++++++----------
> 3 files changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/drivers/bus/mhi/host/init.c b/drivers/bus/mhi/host/init.c
> index ca068a017a42..016dcc35db80 100644
> --- a/drivers/bus/mhi/host/init.c
> +++ b/drivers/bus/mhi/host/init.c
> @@ -339,7 +339,7 @@ int mhi_init_dev_ctxt(struct mhi_controller *mhi_cntrl)
> er_ctxt->msivec = cpu_to_le32(mhi_event->irq);
> mhi_event->db_cfg.db_mode = true;
>
> - ring->el_size = sizeof(struct mhi_tre);
> + ring->el_size = sizeof(struct mhi_ring_element);
> ring->len = ring->el_size * ring->elements;
> ret = mhi_alloc_aligned_ring(mhi_cntrl, ring, ring->len);
> if (ret)
> @@ -371,7 +371,7 @@ int mhi_init_dev_ctxt(struct mhi_controller *mhi_cntrl)
> for (i = 0; i < NR_OF_CMD_RINGS; i++, mhi_cmd++, cmd_ctxt++) {
> struct mhi_ring *ring = &mhi_cmd->ring;
>
> - ring->el_size = sizeof(struct mhi_tre);
> + ring->el_size = sizeof(struct mhi_ring_element);
> ring->elements = CMD_EL_PER_RING;
> ring->len = ring->el_size * ring->elements;
> ret = mhi_alloc_aligned_ring(mhi_cntrl, ring, ring->len);
> @@ -598,7 +598,7 @@ int mhi_init_chan_ctxt(struct mhi_controller *mhi_cntrl,
>
> buf_ring = &mhi_chan->buf_ring;
> tre_ring = &mhi_chan->tre_ring;
> - tre_ring->el_size = sizeof(struct mhi_tre);
> + tre_ring->el_size = sizeof(struct mhi_ring_element);
> tre_ring->len = tre_ring->el_size * tre_ring->elements;
> chan_ctxt = &mhi_cntrl->mhi_ctxt->chan_ctxt[mhi_chan->chan];
> ret = mhi_alloc_aligned_ring(mhi_cntrl, tre_ring, tre_ring->len);
> diff --git a/drivers/bus/mhi/host/internal.h b/drivers/bus/mhi/host/internal.h
> index 1c7a48be033f..5860cd326db6 100644
> --- a/drivers/bus/mhi/host/internal.h
> +++ b/drivers/bus/mhi/host/internal.h
> @@ -168,7 +168,7 @@ struct mhi_ctxt {
> dma_addr_t cmd_ctxt_addr;
> };
>
> -struct mhi_tre {
> +struct mhi_ring_element {
> __le64 ptr;
> __le32 dword[2];
> };
> diff --git a/drivers/bus/mhi/host/main.c b/drivers/bus/mhi/host/main.c
> index 3e6e615466b7..dabf85b92a84 100644
> --- a/drivers/bus/mhi/host/main.c
> +++ b/drivers/bus/mhi/host/main.c
> @@ -554,7 +554,7 @@ static void mhi_recycle_ev_ring_element(struct mhi_controller *mhi_cntrl,
> }
>
> static int parse_xfer_event(struct mhi_controller *mhi_cntrl,
> - struct mhi_tre *event,
> + struct mhi_ring_element *event,
> struct mhi_chan *mhi_chan)
> {
> struct mhi_ring *buf_ring, *tre_ring;
> @@ -590,7 +590,7 @@ static int parse_xfer_event(struct mhi_controller *mhi_cntrl,
> case MHI_EV_CC_EOT:
> {
> dma_addr_t ptr = MHI_TRE_GET_EV_PTR(event);
> - struct mhi_tre *local_rp, *ev_tre;
> + struct mhi_ring_element *local_rp, *ev_tre;
> void *dev_rp;
> struct mhi_buf_info *buf_info;
> u16 xfer_len;
> @@ -689,7 +689,7 @@ static int parse_xfer_event(struct mhi_controller *mhi_cntrl,
> }
>
> static int parse_rsc_event(struct mhi_controller *mhi_cntrl,
> - struct mhi_tre *event,
> + struct mhi_ring_element *event,
> struct mhi_chan *mhi_chan)
> {
> struct mhi_ring *buf_ring, *tre_ring;
> @@ -753,12 +753,12 @@ static int parse_rsc_event(struct mhi_controller *mhi_cntrl,
> }
>
> static void mhi_process_cmd_completion(struct mhi_controller *mhi_cntrl,
> - struct mhi_tre *tre)
> + struct mhi_ring_element *tre)
> {
> dma_addr_t ptr = MHI_TRE_GET_EV_PTR(tre);
> struct mhi_cmd *cmd_ring = &mhi_cntrl->mhi_cmd[PRIMARY_CMD_RING];
> struct mhi_ring *mhi_ring = &cmd_ring->ring;
> - struct mhi_tre *cmd_pkt;
> + struct mhi_ring_element *cmd_pkt;
> struct mhi_chan *mhi_chan;
> u32 chan;
>
> @@ -791,7 +791,7 @@ int mhi_process_ctrl_ev_ring(struct mhi_controller *mhi_cntrl,
> struct mhi_event *mhi_event,
> u32 event_quota)
> {
> - struct mhi_tre *dev_rp, *local_rp;
> + struct mhi_ring_element *dev_rp, *local_rp;
> struct mhi_ring *ev_ring = &mhi_event->ring;
> struct mhi_event_ctxt *er_ctxt =
> &mhi_cntrl->mhi_ctxt->er_ctxt[mhi_event->er_index];
> @@ -961,7 +961,7 @@ int mhi_process_data_event_ring(struct mhi_controller *mhi_cntrl,
> struct mhi_event *mhi_event,
> u32 event_quota)
> {
> - struct mhi_tre *dev_rp, *local_rp;
> + struct mhi_ring_element *dev_rp, *local_rp;
> struct mhi_ring *ev_ring = &mhi_event->ring;
> struct mhi_event_ctxt *er_ctxt =
> &mhi_cntrl->mhi_ctxt->er_ctxt[mhi_event->er_index];
> @@ -1185,7 +1185,7 @@ int mhi_gen_tre(struct mhi_controller *mhi_cntrl, struct mhi_chan *mhi_chan,
> struct mhi_buf_info *info, enum mhi_flags flags)
> {
> struct mhi_ring *buf_ring, *tre_ring;
> - struct mhi_tre *mhi_tre;
> + struct mhi_ring_element *mhi_tre;
> struct mhi_buf_info *buf_info;
> int eot, eob, chain, bei;
> int ret;
> @@ -1256,7 +1256,7 @@ int mhi_send_cmd(struct mhi_controller *mhi_cntrl,
> struct mhi_chan *mhi_chan,
> enum mhi_cmd_type cmd)
> {
> - struct mhi_tre *cmd_tre = NULL;
> + struct mhi_ring_element *cmd_tre = NULL;
> struct mhi_cmd *mhi_cmd = &mhi_cntrl->mhi_cmd[PRIMARY_CMD_RING];
> struct mhi_ring *ring = &mhi_cmd->ring;
> struct device *dev = &mhi_cntrl->mhi_dev->dev;
> @@ -1518,7 +1518,7 @@ static void mhi_mark_stale_events(struct mhi_controller *mhi_cntrl,
> int chan)
>
> {
> - struct mhi_tre *dev_rp, *local_rp;
> + struct mhi_ring_element *dev_rp, *local_rp;
> struct mhi_ring *ev_ring;
> struct device *dev = &mhi_cntrl->mhi_dev->dev;
> unsigned long flags;
Powered by blists - more mailing lists