Re: [patch] drm/omap: tiler: clear buffer properly

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Aug 22, 2013 at 8:42 AM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> We're taking the sizeof() the wrong thing so it doesn't clear the whole
> buffer.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

Oh, that was very certainly wrong before!

Acked-by: Rob Clark <robdclark@xxxxxxxxx>


> ---
> I can't compile this.
>
> diff --git a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
> index 9b794c9..acf6678 100644
> --- a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
> +++ b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
> @@ -871,7 +871,7 @@ int tiler_map_show(struct seq_file *s, void *arg)
>                 goto error;
>
>         for (lut_idx = 0; lut_idx < omap_dmm->num_lut; lut_idx++) {
> -               memset(map, 0, sizeof(h_adj * sizeof(*map)));
> +               memset(map, 0, h_adj * sizeof(*map));
>                 memset(global_map, ' ', (w_adj + 1) * h_adj);
>
>                 for (i = 0; i < omap_dmm->container_height; i++) {
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux