Re: [PATCH] [media] st-hva: fix a copy-and-paste variable name error

[Jean Christophe TROTIN]

Thanks (and sorry for the delay of my answer),

Acked-by: Jean-Christophe Trotin 

On 09/19/2016 08:19 AM, Colin King wrote:
> From: Colin Ian King 
>
> The second check for an error on hva->lmi_err_reg appears
> to be a copy-and-paste error, it should be hva->emi_err_reg
> instead.
>
> Signed-off-by: Colin Ian King 
> ---
>  drivers/media/platform/sti/hva/hva-hw.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/platform/sti/hva/hva-hw.c 
> b/drivers/media/platform/sti/hva/hva-hw.c
> index d341d49..dcf362c 100644
> --- a/drivers/media/platform/sti/hva/hva-hw.c
> +++ b/drivers/media/platform/sti/hva/hva-hw.c
> @@ -245,7 +245,7 @@ static irqreturn_t hva_hw_err_irq_thread(int irq, void 
> *arg)
>   ctx->hw_err = true;
>   }
>
> - if (hva->lmi_err_reg) {
> + if (hva->emi_err_reg) {
>   dev_err(dev, "%s external memory interface error: 0x%08x\n",
>   ctx->name, hva->emi_err_reg);
>   ctx->hw_err = true;
>

Re: [PATCH] [media] st-hva: fix a copy-and-paste variable name error

[Jean Christophe TROTIN]

Thanks (and sorry for the delay of my answer),

Acked-by: Jean-Christophe Trotin 

On 09/19/2016 08:19 AM, Colin King wrote:
> From: Colin Ian King 
>
> The second check for an error on hva->lmi_err_reg appears
> to be a copy-and-paste error, it should be hva->emi_err_reg
> instead.
>
> Signed-off-by: Colin Ian King 
> ---
>  drivers/media/platform/sti/hva/hva-hw.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/platform/sti/hva/hva-hw.c 
> b/drivers/media/platform/sti/hva/hva-hw.c
> index d341d49..dcf362c 100644
> --- a/drivers/media/platform/sti/hva/hva-hw.c
> +++ b/drivers/media/platform/sti/hva/hva-hw.c
> @@ -245,7 +245,7 @@ static irqreturn_t hva_hw_err_irq_thread(int irq, void 
> *arg)
>   ctx->hw_err = true;
>   }
>
> - if (hva->lmi_err_reg) {
> + if (hva->emi_err_reg) {
>   dev_err(dev, "%s external memory interface error: 0x%08x\n",
>   ctx->name, hva->emi_err_reg);
>   ctx->hw_err = true;
>

[PATCH] [media] st-hva: fix a copy-and-paste variable name error

[Colin King]

From: Colin Ian King 

The second check for an error on hva->lmi_err_reg appears
to be a copy-and-paste error, it should be hva->emi_err_reg
instead.

Signed-off-by: Colin Ian King 
---
 drivers/media/platform/sti/hva/hva-hw.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/platform/sti/hva/hva-hw.c 
b/drivers/media/platform/sti/hva/hva-hw.c
index d341d49..dcf362c 100644
--- a/drivers/media/platform/sti/hva/hva-hw.c
+++ b/drivers/media/platform/sti/hva/hva-hw.c
@@ -245,7 +245,7 @@ static irqreturn_t hva_hw_err_irq_thread(int irq, void *arg)
ctx->hw_err = true;
}
 
-   if (hva->lmi_err_reg) {
+   if (hva->emi_err_reg) {
dev_err(dev, "%s external memory interface error: 0x%08x\n",
ctx->name, hva->emi_err_reg);
ctx->hw_err = true;
-- 
2.9.3

[PATCH] [media] st-hva: fix a copy-and-paste variable name error

[Colin King]

From: Colin Ian King 

The second check for an error on hva->lmi_err_reg appears
to be a copy-and-paste error, it should be hva->emi_err_reg
instead.

Signed-off-by: Colin Ian King 
---
 drivers/media/platform/sti/hva/hva-hw.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/platform/sti/hva/hva-hw.c 
b/drivers/media/platform/sti/hva/hva-hw.c
index d341d49..dcf362c 100644
--- a/drivers/media/platform/sti/hva/hva-hw.c
+++ b/drivers/media/platform/sti/hva/hva-hw.c
@@ -245,7 +245,7 @@ static irqreturn_t hva_hw_err_irq_thread(int irq, void *arg)
ctx->hw_err = true;
}
 
-   if (hva->lmi_err_reg) {
+   if (hva->emi_err_reg) {
dev_err(dev, "%s external memory interface error: 0x%08x\n",
ctx->name, hva->emi_err_reg);
ctx->hw_err = true;
-- 
2.9.3