Re: [PATCH v2 00/29] gpu/drm: remove -Iinclude/drm compiler flags from Makefile

2017-05-19 Thread Masahiro Yamada 
Hi Daniel,


2017-05-18 14:38 GMT+09:00 Daniel Vetter :
> On Mon, Apr 24, 2017 at 01:50:18PM +0900, Masahiro Yamada wrote:
>> Many Makefiles needed to add -Iinclude/drm as an include path,
>> but the right thing to do is to include headers in the form
>>   #include 
>>
>> This series fixes the source files, then rip off -Iinclude/drm flags.
>>
>>
>> Masahiro Yamada (29):
>>   drm: make drm_panel.h self-contained
>>   drm/ttm: fix include notation and remove -Iinclude/drm flag
>>   drm/amd: fix include notation and remove -Iinclude/drm flag
>>   drm/ast: fix include notation and remove -Iinclude/drm flag
>>   drm/bochs: fix include notation and remove -Iinclude/drm flag
>>   drm/bridge: fix include notation and remove -Iinclude/drm flag
>>   drm/cirrus: fix include notation and remove -Iinclude/drm flag
>>   drm/hisilicon: fix include notation and remove -Iinclude/drm flag
>>   drm/mgag200: fix include notation and remove -Iinclude/drm flag
>>   drm/msm: fix include notation and remove -Iinclude/drm flag
>>   drm/nouveau: fix include notation and remove -Iinclude/drm flag
>>   drm/qxl: fix include notation and remove -Iinclude/drm flag
>>   drm/radeon: fix include notation and remove -Iinclude/drm flag
>>   drm/tilcdc: fix include notation and remove -Iinclude/drm flag
>>   drm/vc4: fix include notation and remove -Iinclude/drm flag
>>   drm/virtio: fix include notation and remove -Iinclude/drm flag
>>   drm/vmwgfx: fix include notation and remove -Iinclude/drm flag
>>   drm/gma500: remove unneeded -Iinclude/drm compiler flag
>>   drm/i810: remove unneeded -Iinclude/drm compiler flag
>>   drm/i2c: remove unneeded -Iinclude/drm compiler flag
>>   drm/mga: remove unneeded -Iinclude/drm compiler flag
>>   drm/omap: remove unneeded -Iinclude/drm compiler flag
>>   drm/r128: remove unneeded -Iinclude/drm compiler flag
>>   drm/savage: remove unneeded -Iinclude/drm compiler flag
>>   drm/sis: remove unneeded -Iinclude/drm compiler flag
>>   drm/tdfx: remove unneeded -Iinclude/drm compiler flag
>>   drm/udl: remove unneeded -Iinclude/drm compiler flag
>>   drm/vgem: remove unneeded -Iinclude/drm compiler flag
>>   drm/via: remove unneeded -Iinclude/drm compiler flag
>
> Ok, I think I merged them all except the vc4 one. Can you please
> rebase that one?
>

I had already submitted the one.

https://patchwork.kernel.org/patch/9734869/

and one more patch

https://patchwork.kernel.org/patch/9734827/



-- 
Best Regards
Masahiro Yamada
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH v2 00/29] gpu/drm: remove -Iinclude/drm compiler flags from Makefile

2017-05-17 Thread Daniel Vetter 
On Mon, Apr 24, 2017 at 01:50:18PM +0900, Masahiro Yamada wrote:
> Many Makefiles needed to add -Iinclude/drm as an include path,
> but the right thing to do is to include headers in the form
>   #include 
> 
> This series fixes the source files, then rip off -Iinclude/drm flags.
> 
> 
> Masahiro Yamada (29):
>   drm: make drm_panel.h self-contained
>   drm/ttm: fix include notation and remove -Iinclude/drm flag
>   drm/amd: fix include notation and remove -Iinclude/drm flag
>   drm/ast: fix include notation and remove -Iinclude/drm flag
>   drm/bochs: fix include notation and remove -Iinclude/drm flag
>   drm/bridge: fix include notation and remove -Iinclude/drm flag
>   drm/cirrus: fix include notation and remove -Iinclude/drm flag
>   drm/hisilicon: fix include notation and remove -Iinclude/drm flag
>   drm/mgag200: fix include notation and remove -Iinclude/drm flag
>   drm/msm: fix include notation and remove -Iinclude/drm flag
>   drm/nouveau: fix include notation and remove -Iinclude/drm flag
>   drm/qxl: fix include notation and remove -Iinclude/drm flag
>   drm/radeon: fix include notation and remove -Iinclude/drm flag
>   drm/tilcdc: fix include notation and remove -Iinclude/drm flag
>   drm/vc4: fix include notation and remove -Iinclude/drm flag
>   drm/virtio: fix include notation and remove -Iinclude/drm flag
>   drm/vmwgfx: fix include notation and remove -Iinclude/drm flag
>   drm/gma500: remove unneeded -Iinclude/drm compiler flag
>   drm/i810: remove unneeded -Iinclude/drm compiler flag
>   drm/i2c: remove unneeded -Iinclude/drm compiler flag
>   drm/mga: remove unneeded -Iinclude/drm compiler flag
>   drm/omap: remove unneeded -Iinclude/drm compiler flag
>   drm/r128: remove unneeded -Iinclude/drm compiler flag
>   drm/savage: remove unneeded -Iinclude/drm compiler flag
>   drm/sis: remove unneeded -Iinclude/drm compiler flag
>   drm/tdfx: remove unneeded -Iinclude/drm compiler flag
>   drm/udl: remove unneeded -Iinclude/drm compiler flag
>   drm/vgem: remove unneeded -Iinclude/drm compiler flag
>   drm/via: remove unneeded -Iinclude/drm compiler flag

Ok, I think I merged them all except the vc4 one. Can you please
rebase that one?

Thanks, Daniel

> 
>  drivers/gpu/drm/amd/amdgpu/Makefile  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu.h  | 10 -
>  drivers/gpu/drm/amd/amdgpu/amdgpu_dpm.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c  | 10 -
>  drivers/gpu/drm/amd/amdgpu/ci_dpm.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/ci_smc.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/cik.c |  2 +-
>  drivers/gpu/drm/amd/amdgpu/cik_ih.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/cz_ih.c   |  2 +-
>  drivers/gpu/drm/amd/amdgpu/dce_v10_0.c   |  2 +-
>  drivers/gpu/drm/amd/amdgpu/dce_v11_0.c   |  2 +-
>  drivers/gpu/drm/amd/amdgpu/dce_v6_0.c|  2 +-
>  drivers/gpu/drm/amd/amdgpu/dce_v8_0.c|  2 +-
>  drivers/gpu/drm/amd/amdgpu/dce_virtual.c |  2 +-
>  drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c|  2 +-
>  drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c|  2 +-
>  drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c|  2 +-
>  drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c|  2 +-
>  drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c|  2 +-
>  drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c|  2 +-
>  drivers/gpu/drm/amd/amdgpu/iceland_ih.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/kv_dpm.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/kv_smc.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/psp_v3_1.c|  2 +-
>  drivers/gpu/drm/amd/amdgpu/si.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/si_dpm.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/si_ih.c   |  2 +-
>  drivers/gpu/drm/amd/amdgpu/si_smc.c  |  2 +-
>  drivers/gpu/drm/amd/amdgpu/soc15.c   |  2 +-
>  drivers/gpu/drm/amd/amdgpu/tonga_ih.c|  2 +-
>  drivers/gpu/drm/amd/amdgpu/vega10_ih.c   |  2 +-
>  drivers/gpu/drm/amd/amdgpu/vi.c  |  2 +-
>  drivers/gpu/drm/amd/amdkfd/Makefile  |  2 +-
>  drivers/gpu/drm/amd/powerplay/Makefile   |  2 +-
>  drivers/gpu/drm/ast/Makefile |  2 --
>  drivers/gpu/drm/ast/ast_ttm.c|  3 ++-
>  drivers/gpu/drm/bochs/Makefile   |  1 -
>  drivers/gpu/drm/bochs/bochs.h|  4 ++--
>  drivers/gpu/drm/bridge/Makefile  |  2 --
>  drivers/gpu/drm/bridge/nxp-ptn3460.c | 12 +-
>  drivers/gpu/drm/bridge/parade-ps8622.c   | 10 -
>  drivers/gpu/drm/cirrus/Makefile  |  1 -
>  drivers/gpu/drm/cirrus/cirrus_ttm.c  |  3 ++-
>  drivers/gpu/drm/gma500/Makefile  |  1 -

[PATCH v2 00/29] gpu/drm: remove -Iinclude/drm compiler flags from Makefile

2017-04-24 Thread Masahiro Yamada 
Many Makefiles needed to add -Iinclude/drm as an include path,
but the right thing to do is to include headers in the form
  #include 

This series fixes the source files, then rip off -Iinclude/drm flags.


Masahiro Yamada (29):
  drm: make drm_panel.h self-contained
  drm/ttm: fix include notation and remove -Iinclude/drm flag
  drm/amd: fix include notation and remove -Iinclude/drm flag
  drm/ast: fix include notation and remove -Iinclude/drm flag
  drm/bochs: fix include notation and remove -Iinclude/drm flag
  drm/bridge: fix include notation and remove -Iinclude/drm flag
  drm/cirrus: fix include notation and remove -Iinclude/drm flag
  drm/hisilicon: fix include notation and remove -Iinclude/drm flag
  drm/mgag200: fix include notation and remove -Iinclude/drm flag
  drm/msm: fix include notation and remove -Iinclude/drm flag
  drm/nouveau: fix include notation and remove -Iinclude/drm flag
  drm/qxl: fix include notation and remove -Iinclude/drm flag
  drm/radeon: fix include notation and remove -Iinclude/drm flag
  drm/tilcdc: fix include notation and remove -Iinclude/drm flag
  drm/vc4: fix include notation and remove -Iinclude/drm flag
  drm/virtio: fix include notation and remove -Iinclude/drm flag
  drm/vmwgfx: fix include notation and remove -Iinclude/drm flag
  drm/gma500: remove unneeded -Iinclude/drm compiler flag
  drm/i810: remove unneeded -Iinclude/drm compiler flag
  drm/i2c: remove unneeded -Iinclude/drm compiler flag
  drm/mga: remove unneeded -Iinclude/drm compiler flag
  drm/omap: remove unneeded -Iinclude/drm compiler flag
  drm/r128: remove unneeded -Iinclude/drm compiler flag
  drm/savage: remove unneeded -Iinclude/drm compiler flag
  drm/sis: remove unneeded -Iinclude/drm compiler flag
  drm/tdfx: remove unneeded -Iinclude/drm compiler flag
  drm/udl: remove unneeded -Iinclude/drm compiler flag
  drm/vgem: remove unneeded -Iinclude/drm compiler flag
  drm/via: remove unneeded -Iinclude/drm compiler flag

 drivers/gpu/drm/amd/amdgpu/Makefile  |  2 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu.h  | 10 -
 drivers/gpu/drm/amd/amdgpu/amdgpu_dpm.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c  | 10 -
 drivers/gpu/drm/amd/amdgpu/ci_dpm.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/ci_smc.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/cik.c |  2 +-
 drivers/gpu/drm/amd/amdgpu/cik_ih.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/cz_ih.c   |  2 +-
 drivers/gpu/drm/amd/amdgpu/dce_v10_0.c   |  2 +-
 drivers/gpu/drm/amd/amdgpu/dce_v11_0.c   |  2 +-
 drivers/gpu/drm/amd/amdgpu/dce_v6_0.c|  2 +-
 drivers/gpu/drm/amd/amdgpu/dce_v8_0.c|  2 +-
 drivers/gpu/drm/amd/amdgpu/dce_virtual.c |  2 +-
 drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c|  2 +-
 drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c|  2 +-
 drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c|  2 +-
 drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c|  2 +-
 drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c|  2 +-
 drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c|  2 +-
 drivers/gpu/drm/amd/amdgpu/iceland_ih.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/kv_dpm.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/kv_smc.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/psp_v3_1.c|  2 +-
 drivers/gpu/drm/amd/amdgpu/si.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/si_dpm.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/si_ih.c   |  2 +-
 drivers/gpu/drm/amd/amdgpu/si_smc.c  |  2 +-
 drivers/gpu/drm/amd/amdgpu/soc15.c   |  2 +-
 drivers/gpu/drm/amd/amdgpu/tonga_ih.c|  2 +-
 drivers/gpu/drm/amd/amdgpu/vega10_ih.c   |  2 +-
 drivers/gpu/drm/amd/amdgpu/vi.c  |  2 +-
 drivers/gpu/drm/amd/amdkfd/Makefile  |  2 +-
 drivers/gpu/drm/amd/powerplay/Makefile   |  2 +-
 drivers/gpu/drm/ast/Makefile |  2 --
 drivers/gpu/drm/ast/ast_ttm.c|  3 ++-
 drivers/gpu/drm/bochs/Makefile   |  1 -
 drivers/gpu/drm/bochs/bochs.h|  4 ++--
 drivers/gpu/drm/bridge/Makefile  |  2 --
 drivers/gpu/drm/bridge/nxp-ptn3460.c | 12 +-
 drivers/gpu/drm/bridge/parade-ps8622.c   | 10 -
 drivers/gpu/drm/cirrus/Makefile  |  1 -
 drivers/gpu/drm/cirrus/cirrus_ttm.c  |  3 ++-
 drivers/gpu/drm/gma500/Makefile  |  1 -
 drivers/gpu/drm/hisilicon/hibmc/Makefile |  1 -
 drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c  |  2 +-
 drivers/gpu/drm/i2c/Makefile |  2 --
 drivers/gpu/drm/i810/Makefile|  1 -
 drivers/gpu/drm/mga/Makefile |  1 -
 drivers/gpu/drm/mgag200/Makefile