Re: [Intel-gfx] [maintainer-tools PATCH 3/5] doc: use window.onload to call WaveDrom.ProcessAll()

2017-08-10 Thread Jani Nikula
On Thu, 10 Aug 2017, Rodrigo Vivi  wrote:
> I don't know WaveDrom much... but it works so
>
> Acked-by: Rodrigo Vivi 

Thanks, pushed the first three.

BR,
Jani.

>
> On Wed, Aug 9, 2017 at 2:08 PM, Jani Nikula  wrote:
>> Simplify the build by doing the WaveDrom processing from the rst file
>> (albeit raw html block) instead of post-processing the output html.
>>
>> Signed-off-by: Jani Nikula 
>> ---
>>  Makefile   | 6 --
>>  drm-intel-timeline.rst | 4 
>>  drm-misc-timeline.rst  | 4 
>>  3 files changed, 8 insertions(+), 6 deletions(-)
>>
>> diff --git a/Makefile b/Makefile
>> index 8bbabae56741..7059eec42720 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -12,15 +12,9 @@ all: drm-intel.html dim.html drm-misc.html
>>  %.html: %.rst
>> rst2html $< > $@
>>
>> -# the sed bit here is a hack to make wavedrom process the timeline
>>  drm-intel.html: drm-intel.rst drm-intel-flow.svg drm-intel-timeline.rst 
>> drm-intel-timeline.json
>> -   rst2html $< > $@
>> -   sed -i 's/>
>> -# the sed bit here is a hack to make wavedrom process the timeline
>>  drm-misc.html: drm-misc.rst drm-misc-timeline.rst drm-misc-timeline.json 
>> drm-misc-commit-flow.svg
>> -   rst2html $< > $@
>> -   sed -i 's/>
>>  dim.html: dim.rst
>>
>> diff --git a/drm-intel-timeline.rst b/drm-intel-timeline.rst
>> index e1766a5df98b..3ab39afd5788 100644
>> --- a/drm-intel-timeline.rst
>> +++ b/drm-intel-timeline.rst
>> @@ -17,6 +17,10 @@
>>
>>  .. raw:: html
>>
>> +   function init() {
>> +   WaveDrom.ProcessAll();
>> +   }
>> +   window.onload = init;
>> 
>> 
>>
>> diff --git a/drm-misc-timeline.rst b/drm-misc-timeline.rst
>> index 69724362..a9a80d6a4cfb 100644
>> --- a/drm-misc-timeline.rst
>> +++ b/drm-misc-timeline.rst
>> @@ -8,6 +8,10 @@
>>
>>  .. raw:: html
>>
>> +   function init() {
>> +   WaveDrom.ProcessAll();
>> +   }
>> +   window.onload = init;
>> 
>> 

Re: [Intel-gfx] [maintainer-tools PATCH 3/5] doc: use window.onload to call WaveDrom.ProcessAll()

2017-08-09 Thread Rodrigo Vivi
I don't know WaveDrom much... but it works so

Acked-by: Rodrigo Vivi 

On Wed, Aug 9, 2017 at 2:08 PM, Jani Nikula  wrote:
> Simplify the build by doing the WaveDrom processing from the rst file
> (albeit raw html block) instead of post-processing the output html.
>
> Signed-off-by: Jani Nikula 
> ---
>  Makefile   | 6 --
>  drm-intel-timeline.rst | 4 
>  drm-misc-timeline.rst  | 4 
>  3 files changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index 8bbabae56741..7059eec42720 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -12,15 +12,9 @@ all: drm-intel.html dim.html drm-misc.html
>  %.html: %.rst
> rst2html $< > $@
>
> -# the sed bit here is a hack to make wavedrom process the timeline
>  drm-intel.html: drm-intel.rst drm-intel-flow.svg drm-intel-timeline.rst 
> drm-intel-timeline.json
> -   rst2html $< > $@
> -   sed -i 's/
> -# the sed bit here is a hack to make wavedrom process the timeline
>  drm-misc.html: drm-misc.rst drm-misc-timeline.rst drm-misc-timeline.json 
> drm-misc-commit-flow.svg
> -   rst2html $< > $@
> -   sed -i 's/
>  dim.html: dim.rst
>
> diff --git a/drm-intel-timeline.rst b/drm-intel-timeline.rst
> index e1766a5df98b..3ab39afd5788 100644
> --- a/drm-intel-timeline.rst
> +++ b/drm-intel-timeline.rst
> @@ -17,6 +17,10 @@
>
>  .. raw:: html
>
> +   function init() {
> +   WaveDrom.ProcessAll();
> +   }
> +   window.onload = init;
> 
> 
>
> diff --git a/drm-misc-timeline.rst b/drm-misc-timeline.rst
> index 69724362..a9a80d6a4cfb 100644
> --- a/drm-misc-timeline.rst
> +++ b/drm-misc-timeline.rst
> @@ -8,6 +8,10 @@
>
>  .. raw:: html
>
> +   function init() {
> +   WaveDrom.ProcessAll();
> +   }
> +   window.onload = init;
> 
>