Re: [PATCH v1 3/4] drm/mga: drop use of drmP.h

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

 



Hi

Am 08.06.19 um 10:19 schrieb Sam Ravnborg:
> Drop the use of the deprecated drmP.h header file.
> Clean up list of include files and sort them.
> 
> Signed-off-by: Sam Ravnborg <sam@xxxxxxxxxxxx>
> Cc: Thomas Zimmermann <tzimmermann@xxxxxxx>
> Cc: Daniel Vetter <daniel@xxxxxxxx>
> Cc: David Airlie <airlied@xxxxxxxx>
> ---
>  drivers/gpu/drm/mga/mga_dma.c   | 2 --
>  drivers/gpu/drm/mga/mga_drv.c   | 7 +++----
>  drivers/gpu/drm/mga/mga_ioc32.c | 3 +--
>  drivers/gpu/drm/mga/mga_irq.c   | 2 --
>  drivers/gpu/drm/mga/mga_state.c | 2 --
>  drivers/gpu/drm/mga/mga_warp.c  | 4 +---
>  6 files changed, 5 insertions(+), 15 deletions(-)
> 
> diff --git a/drivers/gpu/drm/mga/mga_dma.c b/drivers/gpu/drm/mga/mga_dma.c
> index 123be2f3a151..139554a3733d 100644
> --- a/drivers/gpu/drm/mga/mga_dma.c
> +++ b/drivers/gpu/drm/mga/mga_dma.c
> @@ -37,8 +37,6 @@
>  
>  #include <linux/delay.h>
>  
> -#include <drm/drmP.h>
> -#include <drm/mga_drm.h>
>  #include "mga_drv.h"
>  
>  #define MGA_DEFAULT_USEC_TIMEOUT	10000
> diff --git a/drivers/gpu/drm/mga/mga_drv.c b/drivers/gpu/drm/mga/mga_drv.c
> index 6e1d1054ad06..71128e6f6ae9 100644
> --- a/drivers/gpu/drm/mga/mga_drv.c
> +++ b/drivers/gpu/drm/mga/mga_drv.c
> @@ -31,12 +31,11 @@
>  
>  #include <linux/module.h>
>  
> -#include <drm/drmP.h>
> -#include <drm/mga_drm.h>
> -#include "mga_drv.h"
> -
> +#include <drm/drm_drv.h>
>  #include <drm/drm_pciids.h>
>  
> +#include "mga_drv.h"
> +

Unless it violates an existing convention, I'd suggest the following
order for headers (here and in the other patches):

 c file header ("mga_foo.h" for mga_foo.c)
 driver header ("mga_drv.h")
 public linux/drm headers
 other driver headers ("mga_nonfoo.h", etc)

I know that it's nitpicking, but this maximizes the chance that the
driver's header files include all dependencies they need.

Best regards
Thomas


>  static struct pci_device_id pciidlist[] = {
>  	mga_PCI_IDS
>  };
> diff --git a/drivers/gpu/drm/mga/mga_ioc32.c b/drivers/gpu/drm/mga/mga_ioc32.c
> index 245fb2e359cf..6ccd270789c6 100644
> --- a/drivers/gpu/drm/mga/mga_ioc32.c
> +++ b/drivers/gpu/drm/mga/mga_ioc32.c
> @@ -30,10 +30,9 @@
>   * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
>   * IN THE SOFTWARE.
>   */
> +
>  #include <linux/compat.h>
>  
> -#include <drm/drmP.h>
> -#include <drm/mga_drm.h>
>  #include "mga_drv.h"
>  
>  typedef struct drm32_mga_init {
> diff --git a/drivers/gpu/drm/mga/mga_irq.c b/drivers/gpu/drm/mga/mga_irq.c
> index c6a3fab5b0c4..7c48f4d090ac 100644
> --- a/drivers/gpu/drm/mga/mga_irq.c
> +++ b/drivers/gpu/drm/mga/mga_irq.c
> @@ -31,8 +31,6 @@
>   *    Eric Anholt <anholt@xxxxxxxxxxx>
>   */
>  
> -#include <drm/drmP.h>
> -#include <drm/mga_drm.h>
>  #include "mga_drv.h"
>  
>  u32 mga_get_vblank_counter(struct drm_device *dev, unsigned int pipe)
> diff --git a/drivers/gpu/drm/mga/mga_state.c b/drivers/gpu/drm/mga/mga_state.c
> index 296a1db7e5ee..77a0b006f066 100644
> --- a/drivers/gpu/drm/mga/mga_state.c
> +++ b/drivers/gpu/drm/mga/mga_state.c
> @@ -32,8 +32,6 @@
>   *    Gareth Hughes <gareth@xxxxxxxxxxx>
>   */
>  
> -#include <drm/drmP.h>
> -#include <drm/mga_drm.h>
>  #include "mga_drv.h"
>  
>  /* ================================================================
> diff --git a/drivers/gpu/drm/mga/mga_warp.c b/drivers/gpu/drm/mga/mga_warp.c
> index 0b76352260a9..b5ef1d2c8b1c 100644
> --- a/drivers/gpu/drm/mga/mga_warp.c
> +++ b/drivers/gpu/drm/mga/mga_warp.c
> @@ -29,11 +29,9 @@
>  
>  #include <linux/firmware.h>
>  #include <linux/ihex.h>
> -#include <linux/platform_device.h>
>  #include <linux/module.h>
> +#include <linux/platform_device.h>
>  
> -#include <drm/drmP.h>
> -#include <drm/mga_drm.h>
>  #include "mga_drv.h"
>  
>  #define FIRMWARE_G200 "matrox/g200_warp.fw"
> 

-- 
Thomas Zimmermann
Graphics Driver Developer
SUSE Linux GmbH, Maxfeldstrasse 5, 90409 Nuernberg, Germany
GF: Felix Imendörffer, Mary Higgins, Sri Rasiah
HRB 21284 (AG Nürnberg)

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://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