Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH 0/2] staging: vboxvideo: Remove chekpatch issues
To:     Shayenne da Luz Moura <shayenneluzmoura@gmail.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Michael Thayer <michael.thayer@oracle.com>
Cc:     devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org,
        outreachy-kernel@googlegroups.com
References: <cover.1540490962.git.shayenneluzmoura@gmail.com>
From:   Hans de Goede <hdegoede@redhat.com>
Message-ID: <e59aeb92-b2f9-ae8d-f910-4410ba9c732a@redhat.com>
Date:   Fri, 26 Oct 2018 12:34:33 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.8.0
MIME-Version: 1.0
In-Reply-To: <cover.1540490962.git.shayenneluzmoura@gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hi,

On 25-10-18 21:40, Shayenne da Luz Moura wrote:
> This series cleans the following checkpatch.pl issues:
> 
> CHECK: Prefer kernel type 'u32' over 'uint32_t'
> CHECK: Avoid using bool structure members because of possible alignment issues
> 
> Shayenne da Luz Moura (2):
>    staging: vboxvideo: Change uint32_t to u32
>    staging: vboxvideo: Use unsigned int instead bool

Why am I receiving only the cover letter of this series? Please resend
so that I get all the patches.

Also at least the first patch is wrong, the drm headers use uint32_t
in the prototype definition of the callback functions we are defining,
so the vboxvideo code should use the same even if the compiler does not
warn about the callback implementation having different parameter types
in this case.

Regards,

Hans


> 
>   drivers/staging/vboxvideo/vbox_drv.h        | 14 +++++++-------
>   drivers/staging/vboxvideo/vbox_mode.c       |  2 +-
>   drivers/staging/vboxvideo/vboxvideo_guest.h |  2 +-
>   3 files changed, 9 insertions(+), 9 deletions(-)
>