[Intel-gfx] [PATCH 1/5] Documentation: drm: Fix pdfdocs sect/title tags

2015-08-25 Thread Graham Whaley
On Tue, 2015-08-25 at 13:34 +0200, Daniel Vetter wrote:
> On Tue, Aug 25, 2015 at 10:26:41AM +0100, Graham Whaley wrote:
> > Building pdfdocs shows errors with !includes and s such as:
> >  jade:/Documentation/DocBook/drm.xml:666:11:E: document type does
> > not
> >  allow element "para" here; missing one of "glossary",
> > "bibliography",
> >  "index" start-tag
> > Fix by adding  items and add/shuffle ,  and
> > !include
> > items.
> > 
> > Signed-off-by: Graham Whaley 
> 
> The idea behind having both the overview sections and the detailed
> function references in the same section is to have them in the same
> section. Is there nothing we can do to salvage that? At least you
> seem to
> add a lot more sections here, but it doesn't look entirely
> consistent.
> -Daniel
Let me go re-visit and have a look. I suspect there is an underlying
rule or feature of the processing that either has restrictions on
include ordering or section depth when including or both. I'll search
to see if there are any examples that currently work as we'd like these
to.
Thanks for the feedback.

Graham
> 
> > ---
> >  Documentation/DocBook/drm.tmpl | 32 ++
> > --
> >  1 file changed, 22 insertions(+), 10 deletions(-)
> > 
> > diff --git a/Documentation/DocBook/drm.tmpl
> > b/Documentation/DocBook/drm.tmpl
> > index 66bc646..586f1b8 100644
> > --- a/Documentation/DocBook/drm.tmpl
> > +++ b/Documentation/DocBook/drm.tmpl
> > @@ -303,6 +303,9 @@ char *date;
> >
> >  !Edrivers/gpu/drm/drm_pci.c
> >  !Edrivers/gpu/drm/drm_platform.c
> > +
> > +
> > +  Calling Registration Directly
> >
> >  New drivers that no longer rely on the services provided
> > by the
> >  drm_bus structure can call the
> > low-level
> > @@ -2352,7 +2355,7 @@ void intel_crt_init(struct drm_device *dev)
> >
> >  
> >  
> > -  Atomic Modeset Helper Functions Reference
> > +  Atomic Modeset Helper Functions Introduction
> >
> > Overview
> >  !Pdrivers/gpu/drm/drm_atomic_helper.c overview
> > @@ -2365,14 +2368,20 @@ void intel_crt_init(struct drm_device *dev)
> > Atomic State Reset and Initialization
> >  !Pdrivers/gpu/drm/drm_atomic_helper.c atomic state reset and
> > initialization
> >
> > +
> > +
> > +  Atomic Modeset Helper Functions Reference
> >  !Iinclude/drm/drm_atomic_helper.h
> >  !Edrivers/gpu/drm/drm_atomic_helper.c
> >  
> >  
> > +  Modeset Helper Functions Introduction
> > +!Pdrivers/gpu/drm/drm_crtc_helper.c overview
> > +
> > +
> >Modeset Helper Functions Reference
> >  !Iinclude/drm/drm_crtc_helper.h
> >  !Edrivers/gpu/drm/drm_crtc_helper.c
> > -!Pdrivers/gpu/drm/drm_crtc_helper.c overview
> >  
> >  
> >Output Probing Helper Functions Reference
> > @@ -2432,8 +2441,8 @@ void intel_crt_init(struct drm_device *dev)
> >  
> >  
> >Plane Helper
> > Reference
> > -!Edrivers/gpu/drm/drm_plane_helper.c
> >  !Pdrivers/gpu/drm/drm_plane_helper.c overview
> > +!Edrivers/gpu/drm/drm_plane_helper.c
> >  
> >  
> >   Tile group
> > @@ -2449,6 +2458,9 @@ void intel_crt_init(struct drm_device *dev)
> >  Default bridge callback sequence
> >  !Pdrivers/gpu/drm/drm_bridge.c bridge callbacks
> >
> > +
> > +
> > +  Bridges Function Reference
> >  !Edrivers/gpu/drm/drm_bridge.c
> >  
> >
> > @@ -4114,19 +4126,19 @@ int num_ioctls;
> >
> >  GTT Fences and Swizzling
> >  !Idrivers/gpu/drm/i915/i915_gem_fence.c
> > -
> > -  Global GTT Fence Handling
> > +  
> > +  
> > +Global GTT Fence Handling
> >  !Pdrivers/gpu/drm/i915/i915_gem_fence.c fence register handling
> > -
> > -
> > -  Hardware Tiling and Swizzling Details
> > +  
> > +  
> > +Hardware Tiling and Swizzling Details
> >  !Pdrivers/gpu/drm/i915/i915_gem_fence.c tiling swizzling details
> > -
> >
> >
> >  Object Tiling IOCTLs
> > -!Idrivers/gpu/drm/i915/i915_gem_tiling.c
> >  !Pdrivers/gpu/drm/i915/i915_gem_tiling.c buffer object tiling
> > +!Idrivers/gpu/drm/i915/i915_gem_tiling.c
> >
> >
> >  Buffer Object Eviction
> > -- 
> > 2.4.3
> > 
> > ___
> > Intel-gfx mailing list
> > Intel-gfx at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 


[Intel-gfx] [PATCH 1/5] Documentation: drm: Fix pdfdocs sect/title tags

2015-08-25 Thread Daniel Vetter
On Tue, Aug 25, 2015 at 10:26:41AM +0100, Graham Whaley wrote:
> Building pdfdocs shows errors with !includes and s such as:
>  jade:/Documentation/DocBook/drm.xml:666:11:E: document type does not
>  allow element "para" here; missing one of "glossary", "bibliography",
>  "index" start-tag
> Fix by adding  items and add/shuffle ,  and !include
> items.
> 
> Signed-off-by: Graham Whaley 

The idea behind having both the overview sections and the detailed
function references in the same section is to have them in the same
section. Is there nothing we can do to salvage that? At least you seem to
add a lot more sections here, but it doesn't look entirely consistent.
-Daniel

> ---
>  Documentation/DocBook/drm.tmpl | 32 ++--
>  1 file changed, 22 insertions(+), 10 deletions(-)
> 
> diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl
> index 66bc646..586f1b8 100644
> --- a/Documentation/DocBook/drm.tmpl
> +++ b/Documentation/DocBook/drm.tmpl
> @@ -303,6 +303,9 @@ char *date;
>
>  !Edrivers/gpu/drm/drm_pci.c
>  !Edrivers/gpu/drm/drm_platform.c
> +
> +
> +  Calling Registration Directly
>
>  New drivers that no longer rely on the services provided by the
>  drm_bus structure can call the low-level
> @@ -2352,7 +2355,7 @@ void intel_crt_init(struct drm_device *dev)
>
>  
>  
> -  Atomic Modeset Helper Functions Reference
> +  Atomic Modeset Helper Functions Introduction
>
>   Overview
>  !Pdrivers/gpu/drm/drm_atomic_helper.c overview
> @@ -2365,14 +2368,20 @@ void intel_crt_init(struct drm_device *dev)
>   Atomic State Reset and Initialization
>  !Pdrivers/gpu/drm/drm_atomic_helper.c atomic state reset and initialization
>
> +
> +
> +  Atomic Modeset Helper Functions Reference
>  !Iinclude/drm/drm_atomic_helper.h
>  !Edrivers/gpu/drm/drm_atomic_helper.c
>  
>  
> +  Modeset Helper Functions Introduction
> +!Pdrivers/gpu/drm/drm_crtc_helper.c overview
> +
> +
>Modeset Helper Functions Reference
>  !Iinclude/drm/drm_crtc_helper.h
>  !Edrivers/gpu/drm/drm_crtc_helper.c
> -!Pdrivers/gpu/drm/drm_crtc_helper.c overview
>  
>  
>Output Probing Helper Functions Reference
> @@ -2432,8 +2441,8 @@ void intel_crt_init(struct drm_device *dev)
>  
>  
>Plane Helper Reference
> -!Edrivers/gpu/drm/drm_plane_helper.c
>  !Pdrivers/gpu/drm/drm_plane_helper.c overview
> +!Edrivers/gpu/drm/drm_plane_helper.c
>  
>  
> Tile group
> @@ -2449,6 +2458,9 @@ void intel_crt_init(struct drm_device *dev)
>Default bridge callback sequence
>  !Pdrivers/gpu/drm/drm_bridge.c bridge callbacks
>
> +
> +
> +  Bridges Function Reference
>  !Edrivers/gpu/drm/drm_bridge.c
>  
>
> @@ -4114,19 +4126,19 @@ int num_ioctls;
>
>  GTT Fences and Swizzling
>  !Idrivers/gpu/drm/i915/i915_gem_fence.c
> -
> -  Global GTT Fence Handling
> +  
> +  
> +Global GTT Fence Handling
>  !Pdrivers/gpu/drm/i915/i915_gem_fence.c fence register handling
> -
> -
> -  Hardware Tiling and Swizzling Details
> +  
> +  
> +Hardware Tiling and Swizzling Details
>  !Pdrivers/gpu/drm/i915/i915_gem_fence.c tiling swizzling details
> -
>
>
>  Object Tiling IOCTLs
> -!Idrivers/gpu/drm/i915/i915_gem_tiling.c
>  !Pdrivers/gpu/drm/i915/i915_gem_tiling.c buffer object tiling
> +!Idrivers/gpu/drm/i915/i915_gem_tiling.c
>
>
>  Buffer Object Eviction
> -- 
> 2.4.3
> 
> ___
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch


[PATCH 1/5] Documentation: drm: Fix pdfdocs sect/title tags

2015-08-25 Thread Graham Whaley
Building pdfdocs shows errors with !includes and s such as:
 jade:/Documentation/DocBook/drm.xml:666:11:E: document type does not
 allow element "para" here; missing one of "glossary", "bibliography",
 "index" start-tag
Fix by adding  items and add/shuffle ,  and !include
items.

Signed-off-by: Graham Whaley 
---
 Documentation/DocBook/drm.tmpl | 32 ++--
 1 file changed, 22 insertions(+), 10 deletions(-)

diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl
index 66bc646..586f1b8 100644
--- a/Documentation/DocBook/drm.tmpl
+++ b/Documentation/DocBook/drm.tmpl
@@ -303,6 +303,9 @@ char *date;
   
 !Edrivers/gpu/drm/drm_pci.c
 !Edrivers/gpu/drm/drm_platform.c
+
+
+  Calling Registration Directly
   
 New drivers that no longer rely on the services provided by the
 drm_bus structure can call the low-level
@@ -2352,7 +2355,7 @@ void intel_crt_init(struct drm_device *dev)
   
 
 
-  Atomic Modeset Helper Functions Reference
+  Atomic Modeset Helper Functions Introduction
   
Overview
 !Pdrivers/gpu/drm/drm_atomic_helper.c overview
@@ -2365,14 +2368,20 @@ void intel_crt_init(struct drm_device *dev)
Atomic State Reset and Initialization
 !Pdrivers/gpu/drm/drm_atomic_helper.c atomic state reset and initialization
   
+
+
+  Atomic Modeset Helper Functions Reference
 !Iinclude/drm/drm_atomic_helper.h
 !Edrivers/gpu/drm/drm_atomic_helper.c
 
 
+  Modeset Helper Functions Introduction
+!Pdrivers/gpu/drm/drm_crtc_helper.c overview
+
+
   Modeset Helper Functions Reference
 !Iinclude/drm/drm_crtc_helper.h
 !Edrivers/gpu/drm/drm_crtc_helper.c
-!Pdrivers/gpu/drm/drm_crtc_helper.c overview
 
 
   Output Probing Helper Functions Reference
@@ -2432,8 +2441,8 @@ void intel_crt_init(struct drm_device *dev)
 
 
   Plane Helper Reference
-!Edrivers/gpu/drm/drm_plane_helper.c
 !Pdrivers/gpu/drm/drm_plane_helper.c overview
+!Edrivers/gpu/drm/drm_plane_helper.c
 
 
  Tile group
@@ -2449,6 +2458,9 @@ void intel_crt_init(struct drm_device *dev)
 Default bridge callback sequence
 !Pdrivers/gpu/drm/drm_bridge.c bridge callbacks
   
+
+
+  Bridges Function Reference
 !Edrivers/gpu/drm/drm_bridge.c
 
   
@@ -4114,19 +4126,19 @@ int num_ioctls;
   
 GTT Fences and Swizzling
 !Idrivers/gpu/drm/i915/i915_gem_fence.c
-
-  Global GTT Fence Handling
+  
+  
+Global GTT Fence Handling
 !Pdrivers/gpu/drm/i915/i915_gem_fence.c fence register handling
-
-
-  Hardware Tiling and Swizzling Details
+  
+  
+Hardware Tiling and Swizzling Details
 !Pdrivers/gpu/drm/i915/i915_gem_fence.c tiling swizzling details
-
   
   
 Object Tiling IOCTLs
-!Idrivers/gpu/drm/i915/i915_gem_tiling.c
 !Pdrivers/gpu/drm/i915/i915_gem_tiling.c buffer object tiling
+!Idrivers/gpu/drm/i915/i915_gem_tiling.c
   
   
 Buffer Object Eviction
-- 
2.4.3