Re: [Nouveau] [PATCH 01/17] drm/nouveau/core: add missing header dependencies

From	Karol Herbst <karolherbst@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [Nouveau] [PATCH 01/17] drm/nouveau/core: add missing header dependencies
Date	2016-10-24 17:50 +0200
Message-ID	<svPFD-17R-11@gated-at.bofh.it> (permalink)
References	<sv169-1EI-5@gated-at.bofh.it> <sv8qZ-6pE-9@gated-at.bofh.it> <svPFD-17R-13@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

2016-10-24 9:13 GMT+02:00 Baoyou Xie <baoyou.xie@linaro.org>:
>
>
> On 23 October 2016 at 01:32, Karol Herbst <karolherbst@gmail.com> wrote:
>>
>> I think it would be better to squash those commits:
>> 1. for the includes
>> 2. for static declerations
>>
> OK, I have resent new patch that squash those commits.
>

thanks, this is much easier to review and keeps the git history clean :)
Will try to make a test with those patches over the next days, but it
looks fine as it is already.

>>
>> 2016-10-22 11:41 GMT+02:00 Baoyou Xie <baoyou.xie@linaro.org>:
>> > We get 2 warnings when building kernel with W=1:
>> > drivers/gpu/drm/nouveau/nvkm/core/firmware.c:34:1: warning: no previous
>> > prototype for 'nvkm_firmware_get' [-Wmissing-prototypes]
>> > drivers/gpu/drm/nouveau/nvkm/core/firmware.c:58:1: warning: no previous
>> > prototype for 'nvkm_firmware_put' [-Wmissing-prototypes]
>> >
>> > In fact, these functions are declared in
>> > drivers/gpu/drm/nouveau/include/nvkm/core/firmware.h.
>> > So this patch adds missing header dependencies.
>> >
>> > Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
>> > ---
>> >  drivers/gpu/drm/nouveau/nvkm/core/firmware.c | 1 +
>> >  1 file changed, 1 insertion(+)
>> >
>> > diff --git a/drivers/gpu/drm/nouveau/nvkm/core/firmware.c
>> > b/drivers/gpu/drm/nouveau/nvkm/core/firmware.c
>> > index 34ecd4a..058ff46 100644
>> > --- a/drivers/gpu/drm/nouveau/nvkm/core/firmware.c
>> > +++ b/drivers/gpu/drm/nouveau/nvkm/core/firmware.c
>> > @@ -20,6 +20,7 @@
>> >   * DEALINGS IN THE SOFTWARE.
>> >   */
>> >  #include <core/device.h>
>> > +#include <core/firmware.h>
>> >
>> >  /**
>> >   * nvkm_firmware_get - load firmware from the official nvidia/chip/
>> > directory
>> > --
>> > 2.7.4
>> >
>> > _______________________________________________
>> > Nouveau mailing list
>> > Nouveau@lists.freedesktop.org
>> > https://lists.freedesktop.org/mailman/listinfo/nouveau
>
>

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 01/17] drm/nouveau/core: add missing header dependencies Baoyou Xie <baoyou.xie@linaro.org> - 2016-10-22 11:50 +0200
  [PATCH 10/17] drm/nouveau/device: mark symbol static where possible Baoyou Xie <baoyou.xie@linaro.org> - 2016-10-22 11:50 +0200
  [PATCH 16/17] drm/nouveau/dispnv04: add missing header dependencies Baoyou Xie <baoyou.xie@linaro.org> - 2016-10-22 11:50 +0200
  [PATCH 08/17] drm/nouveau/volt: add missing header dependencies Baoyou Xie <baoyou.xie@linaro.org> - 2016-10-22 11:50 +0200
  Re: [Nouveau] [PATCH 01/17] drm/nouveau/core: add missing header dependencies Karol Herbst <karolherbst@gmail.com> - 2016-10-22 19:40 +0200
    Re: [Nouveau] [PATCH 01/17] drm/nouveau/core: add missing header dependencies Karol Herbst <karolherbst@gmail.com> - 2016-10-24 17:50 +0200

csiph-web