2023-02-13 10:05:32

by Colin Ian King

Subject: [PATCH][next] drm/sti: Fix spelling mistake "top_cmd_offest" -> "top_cmd_offset"

There is a spelling mistake in a variable name. Fix it.

Signed-off-by: Colin Ian King <[email protected]>
---
drivers/gpu/drm/sti/sti_hqvdp.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/sti/sti_hqvdp.c b/drivers/gpu/drm/sti/sti_hqvdp.c
index 02b77279f6e4..711a6e333afc 100644
--- a/drivers/gpu/drm/sti/sti_hqvdp.c
+++ b/drivers/gpu/drm/sti/sti_hqvdp.c
@@ -797,7 +797,7 @@ static void sti_hqvdp_disable(struct sti_hqvdp *hqvdp)
static int sti_hqvdp_vtg_cb(struct notifier_block *nb, unsigned long evt, void *data)
{
struct sti_hqvdp *hqvdp = container_of(nb, struct sti_hqvdp, vtg_nb);
- int btm_cmd_offset, top_cmd_offest;
+ int btm_cmd_offset, top_cmd_offset;
struct sti_hqvdp_cmd *btm_cmd, *top_cmd;

if ((evt != VTG_TOP_FIELD_EVENT) && (evt != VTG_BOTTOM_FIELD_EVENT)) {
@@ -816,14 +816,14 @@ static int sti_hqvdp_vtg_cb(struct notifier_block *nb, unsigned long evt, void *
if (hqvdp->btm_field_pending) {
/* Create the btm field command from the current one */
btm_cmd_offset = sti_hqvdp_get_free_cmd(hqvdp);
- top_cmd_offest = sti_hqvdp_get_curr_cmd(hqvdp);
- if ((btm_cmd_offset == -1) || (top_cmd_offest == -1)) {
+ top_cmd_offset = sti_hqvdp_get_curr_cmd(hqvdp);
+ if ((btm_cmd_offset == -1) || (top_cmd_offset == -1)) {
DRM_DEBUG_DRIVER("Warning: no cmd, will skip field\n");
return -EBUSY;
}

btm_cmd = hqvdp->hqvdp_cmd + btm_cmd_offset;
- top_cmd = hqvdp->hqvdp_cmd + top_cmd_offest;
+ top_cmd = hqvdp->hqvdp_cmd + top_cmd_offset;

memcpy(btm_cmd, top_cmd, sizeof(*btm_cmd));

--
2.30.2

2023-02-13 18:07:35

by Alain Volmat

Subject: Re: [PATCH][next] drm/sti: Fix spelling mistake "top_cmd_offest" -> "top_cmd_offset"

Hi,

thanks for the patch.


On Mon, Feb 13, 2023 at 10:05:22AM +0000, Colin Ian King wrote:
> There is a spelling mistake in a variable name. Fix it.
>
> Signed-off-by: Colin Ian King <[email protected]>
> ---
> drivers/gpu/drm/sti/sti_hqvdp.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/sti/sti_hqvdp.c b/drivers/gpu/drm/sti/sti_hqvdp.c
> index 02b77279f6e4..711a6e333afc 100644
> --- a/drivers/gpu/drm/sti/sti_hqvdp.c
> +++ b/drivers/gpu/drm/sti/sti_hqvdp.c
> @@ -797,7 +797,7 @@ static void sti_hqvdp_disable(struct sti_hqvdp *hqvdp)
> static int sti_hqvdp_vtg_cb(struct notifier_block *nb, unsigned long evt, void *data)
> {
> struct sti_hqvdp *hqvdp = container_of(nb, struct sti_hqvdp, vtg_nb);
> - int btm_cmd_offset, top_cmd_offest;
> + int btm_cmd_offset, top_cmd_offset;
> struct sti_hqvdp_cmd *btm_cmd, *top_cmd;
>
> if ((evt != VTG_TOP_FIELD_EVENT) && (evt != VTG_BOTTOM_FIELD_EVENT)) {
> @@ -816,14 +816,14 @@ static int sti_hqvdp_vtg_cb(struct notifier_block *nb, unsigned long evt, void *
> if (hqvdp->btm_field_pending) {
> /* Create the btm field command from the current one */
> btm_cmd_offset = sti_hqvdp_get_free_cmd(hqvdp);
> - top_cmd_offest = sti_hqvdp_get_curr_cmd(hqvdp);
> - if ((btm_cmd_offset == -1) || (top_cmd_offest == -1)) {
> + top_cmd_offset = sti_hqvdp_get_curr_cmd(hqvdp);
> + if ((btm_cmd_offset == -1) || (top_cmd_offset == -1)) {
> DRM_DEBUG_DRIVER("Warning: no cmd, will skip field\n");
> return -EBUSY;
> }
>
> btm_cmd = hqvdp->hqvdp_cmd + btm_cmd_offset;
> - top_cmd = hqvdp->hqvdp_cmd + top_cmd_offest;
> + top_cmd = hqvdp->hqvdp_cmd + top_cmd_offset;
>
> memcpy(btm_cmd, top_cmd, sizeof(*btm_cmd));
>
> --

Acked-by: Alain Volmat <[email protected]>

Alain

> 2.30.2
>