Re: [PATCH 06/25] vmwgfx: Some comments and BUG_ON

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

 



On Thu, Sep 29, 2011 at 11:42 PM, Konrad Rzeszutek Wilk
<konrad.wilk@xxxxxxxxxx> wrote:
> On Wed, Sep 28, 2011 at 04:10:02PM +0200, Thomas Hellstrom wrote:
>> From: Jakob Bornecrantz <jakob@xxxxxxxxxx>
>>
>> Signed-off-by: Jakob Bornecrantz <jakob@xxxxxxxxxx>
>> Reviewed-by: Thomas Hellstrom <thellstrom@xxxxxxxxxx>
>> ---
>>  drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c |    5 +++++
>>  1 files changed, 5 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c
>> index df1c7c5..207f595 100644
>> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c
>> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c
>> @@ -680,6 +680,9 @@ static int vmw_resize_cmd_bounce(struct vmw_sw_context *sw_context,
>>   * Creates a fence object and submits a command stream marker.
>>   * If this fails for some reason, We sync the fifo and return NULL.
>>   * It is then safe to fence buffers with a NULL pointer.
>> + *
>> + * If @p_handle is not NULL @file_priv must also not be not NULL. Creates
>
> not be not?
> I think you meant:
>
> "must also not be NULL"
>
>
>> + * a userspace handle if @p_handle is not NULL, does not if otherwise.
>
> "does not if otherwise".. how about ", otherwise not."

Thanks I'll fix that.

Cheers Jakob.
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux