From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ben Skeggs <bskeggs@redhat.com>
Subject: Re: drm/nv50-/disp: audit and version DAC_LOAD method
Date: Wed, 13 Aug 2014 19:30:40 -0400 (EDT)
Message-ID: <367116909.42724148.1407972640288.JavaMail.zimbra@redhat.com>
References: <20140813112916.GA24203@mwanda>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <dri-devel-bounces@lists.freedesktop.org>
Received: from mx6-phx2.redhat.com (mx6-phx2.redhat.com [209.132.183.39])
 by gabe.freedesktop.org (Postfix) with ESMTP id 0AB0F6E5FF
 for <dri-devel@lists.freedesktop.org>; Wed, 13 Aug 2014 16:30:40 -0700 (PDT)
In-Reply-To: <20140813112916.GA24203@mwanda>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/dri-devel>,
 <mailto:dri-devel-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/dri-devel>
List-Post: <mailto:dri-devel@lists.freedesktop.org>
List-Help: <mailto:dri-devel-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/dri-devel>,
 <mailto:dri-devel-request@lists.freedesktop.org?subject=subscribe>
Errors-To: dri-devel-bounces@lists.freedesktop.org
Sender: "dri-devel" <dri-devel-bounces@lists.freedesktop.org>
To: Dan Carpenter <dan.carpenter@oracle.com>
Cc: dri-devel@lists.freedesktop.org
List-Id: dri-devel@lists.freedesktop.org

----- Original Message -----
> From: "Dan Carpenter" <dan.carpenter@oracle.com>
> To: bskeggs@redhat.com
> Cc: dri-devel@lists.freedesktop.org
> Sent: Wednesday, 13 August, 2014 9:29:16 PM
> Subject: re: drm/nv50-/disp: audit and version DAC_LOAD method
> 
> Hello Ben Skeggs,
> 
> The patch c4abd3178e11: "drm/nv50-/disp: audit and version DAC_LOAD
> method" from Aug 10, 2014, leads to the following static checker
> warning:
> 
> 	drivers/gpu/drm/nouveau/core/engine/disp/dacnv50.c:78 nv50_dac_sense()
> 	warn: 0xfff00000 is larger than 16 bits
There was a mess-up when I cleaned up the structs, I've got a fix in my tree that I'll get to Linus in -fixes.

Thanks,
Ben.

> 
> drivers/gpu/drm/nouveau/core/engine/disp/dacnv50.c
>     65  nv50_dac_sense(NV50_DISP_MTHD_V1)
>     66  {
>     67          union {
>     68                  struct nv50_disp_dac_load_v0 v0;
>     69          } *args = data;
>     70          const u32 doff = outp->or * 0x800;
>     71          u32 loadval;
>     72          int ret;
>     73
>     74          nv_ioctl(object, "disp dac load size %d\n", size);
>     75          if (nvif_unpack(args->v0, 0, 0, false)) {
>     76                  nv_ioctl(object, "disp dac load vers %d data %08x\n",
>     77                           args->v0.version, args->v0.data);
>     78                  if (args->v0.data & 0xfff00000)
>                             ^^^^^^^^^^^^^^^^^^^^^^^^^^
> This condition can't be true.  It's not clear what was intended.
> 
>     79                          return -EINVAL;
>     80                  loadval = args->v0.data;
>     81          } else
>     82                  return ret;
> 
> 
> regards,
> dan carpenter
>