Re: [lng-odp] [PATCH] doc: doxygen: use .svg logo

2016-01-28 Thread Maxim Uvarov 

Merged.

I think checkpatch warnings are ok here.

Maxim.

On 01/28/2016 04:36, Bill Fischofer wrote:

Applying this generates the following warnings:

Applying: doc: doxygen: use .svg logo
/home/bill/linaro/mikedoc/.git/rebase-apply/patch:168: indent with spaces.
 rdf:about="">
/home/bill/linaro/mikedoc/.git/rebase-apply/patch:169: indent with spaces.
image/svg+xml
/home/bill/linaro/mikedoc/.git/rebase-apply/patch:170: indent with spaces.
http://purl.org/dc/dcmitype/StillImage; />
/home/bill/linaro/mikedoc/.git/rebase-apply/patch:172: indent with spaces.
OpenDataPlane-branding_ODP Logo
warning: squelched 4 whitespace errors
warning: 9 lines add whitespace errors.

Not sure if these are legit or if this is another example of an 
overzealous checkpatch trying to apply C rules to non-C files. Can 
checkpatch behave differently based on file type?


Other than that, looks good.

On Tue, Jan 26, 2016 at 12:25 PM, Mike Holmes > wrote:


Signed-off-by: Mike Holmes >


Reviewed-and-tested-by: Bill Fischofer >


--- 


 doc/application-api-guide/doxygen.cfg |   2 +-
 doc/images/ODP-Logo-HQ.png| Bin 5736 -> 0 bytes
 doc/images/ODP-Logo-HQ.svg| 209
++
 3 files changed, 210 insertions(+), 1 deletion(-)
 delete mode 100644 doc/images/ODP-Logo-HQ.png
 create mode 100644 doc/images/ODP-Logo-HQ.svg

diff --git a/doc/application-api-guide/doxygen.cfg
b/doc/application-api-guide/doxygen.cfg
index 75006e2..7951e9e 100644
--- a/doc/application-api-guide/doxygen.cfg
+++ b/doc/application-api-guide/doxygen.cfg
@@ -1,6 +1,6 @@
 PROJECT_NAME = "API Reference Manual for $(WITH_PLATFORM)"
 PROJECT_NUMBER = $(VERSION)
-PROJECT_LOGO = $(SRCDIR)/doc/images/ODP-Logo-HQ.png
+PROJECT_LOGO = $(SRCDIR)/doc/images/ODP-Logo-HQ.svg
 QUIET = YES
 OUTPUT_DIRECTORY = $(DOCDIR)
 FULL_PATH_NAMES = NO
diff --git a/doc/images/ODP-Logo-HQ.png b/doc/images/ODP-Logo-HQ.png
deleted file mode 100644
index

721bf40b185f75ee6899b161a35b2e6fdf0c5ab4..
GIT binary patch
literal 0
HcmV?d1

literal 5736
zcmV-u7MJOXP)dwdkt*~h=<%(o^W)U#*NX;%*^%gzyJP!_9url(f}~~tD;aSbVo24{EpTd(=@Sd8^bV888c?gyd67s
z3==}={^F8G8nQns357z-gTdf;Y}-aDh1Q|;+qP@!)Tvp0%_ogCppS)LR8&+Q3Q3ee{lTq$B*qzj%lO;
zz4!pj7yY-AhBxPY$R%@EQS##e@um7#E5LCg`XSo{RB*lvcMl)(;Fw^akHF(m3W>Lmt-c+JLe?}d1E?dpY>cRxk14yaK$WUB&^K>+$*!M;&
z`wl^CG@%;Hw(Y;?<>h_ySox-r1{|aC=g*|l6v}cc5vzC4^v%L$cC4OCx|{y
zKtM*&<+>%)i8`Y>u&7b__CyzfC|6p#~}PrD0F{DM#kkSJ^kF(
ztXfknT0?WILCkKiS_&4XQ7zQ*air;K)ZQV3s!h{o!
zmUkLyz){BdMMXuwwJht>RHDE1@`qVjUC1*p4QD`R=a}Ah1j`rK@=)z*JicKR*;$=$
zii6DtKmXyg49(fjq%UN0pa}*fX7Q*%AaKgoty|wd8vaz~hY`#}N{hDCI03kN2SQv$
zac$(N?DiI_3(mu(2Ize23k2u`Aa123^1uDgt1^Esh@S$TgHeG%>s@HR%eCt^P+3q2
z{2a6mVi%>6Gy0wDVbJ63%up!wfMr>Q38HT{`1;wN=vS@0IH~AkwqW_2KO!-@WaqcV3
ze|+;1+gvey6e4gB5Jd<{N#uea^Aa)#OO8#-ONt4&`WZkV+<{nuu{bU$J+R01^aFYn
z>J$|f{XQ5BCKdhHzEa0!^E^-g^ot`>c=`xJ3N9!)=UkL3As|6%2%1^k
zT`-|XHhUXdXk5Vcha*Rq;p#H9K8UM5o>6B(k8B##bg#V{@;V4Q!H#9UnM@zPr3
zxVPh|JL8-J22|%?iIy?19018DcONs3*yD#np$9F?`erK8&%NSdR#hkW^if-|VsQi^
zG}kZ5XXv0kl>by9#9$ULU5k{nSpCFU1`hD@7leSEY=?!neV*GEN;W^Ug-^}MV%rWF
zl-;R%>ALP~~uYE0{CyQ@10jrf*^?2
z;F5zDAEoU7s{A=1rXp}~)$bU<9$L(aNG(_pc?y7!2^?n#OKx3Dz%1Yowc{AnMf~5t{d3&9Q1I51?{dnF0ldErayk#5!nW;K
zeDS|>1kKSM{^yN&%cc+SRKjT66eI7$lv8MNpt9fq0b>xZ_Q+M{S5UgHU3^sqby

Re: [lng-odp] [PATCH] doc: doxygen: use .svg logo

2016-01-27 Thread Bill Fischofer 
Applying this generates the following warnings:

Applying: doc: doxygen: use .svg logo
/home/bill/linaro/mikedoc/.git/rebase-apply/patch:168: indent with spaces.
 rdf:about="">
/home/bill/linaro/mikedoc/.git/rebase-apply/patch:169: indent with spaces.
image/svg+xml
/home/bill/linaro/mikedoc/.git/rebase-apply/patch:170: indent with spaces.
http://purl.org/dc/dcmitype/StillImage; />
/home/bill/linaro/mikedoc/.git/rebase-apply/patch:172: indent with spaces.
OpenDataPlane-branding_ODP Logo
warning: squelched 4 whitespace errors
warning: 9 lines add whitespace errors.

Not sure if these are legit or if this is another example of an overzealous
checkpatch trying to apply C rules to non-C files. Can checkpatch behave
differently based on file type?

Other than that, looks good.

On Tue, Jan 26, 2016 at 12:25 PM, Mike Holmes 
wrote:

> Signed-off-by: Mike Holmes 
>

Reviewed-and-tested-by: Bill Fischofer 


> ---

 doc/application-api-guide/doxygen.cfg |   2 +-
>  doc/images/ODP-Logo-HQ.png| Bin 5736 -> 0 bytes
>  doc/images/ODP-Logo-HQ.svg| 209
> ++
>  3 files changed, 210 insertions(+), 1 deletion(-)
>  delete mode 100644 doc/images/ODP-Logo-HQ.png
>  create mode 100644 doc/images/ODP-Logo-HQ.svg
>
> diff --git a/doc/application-api-guide/doxygen.cfg
> b/doc/application-api-guide/doxygen.cfg
> index 75006e2..7951e9e 100644
> --- a/doc/application-api-guide/doxygen.cfg
> +++ b/doc/application-api-guide/doxygen.cfg
> @@ -1,6 +1,6 @@
>  PROJECT_NAME = "API Reference Manual for $(WITH_PLATFORM)"
>  PROJECT_NUMBER = $(VERSION)
> -PROJECT_LOGO = $(SRCDIR)/doc/images/ODP-Logo-HQ.png
> +PROJECT_LOGO = $(SRCDIR)/doc/images/ODP-Logo-HQ.svg
>  QUIET = YES
>  OUTPUT_DIRECTORY = $(DOCDIR)
>  FULL_PATH_NAMES = NO
> diff --git a/doc/images/ODP-Logo-HQ.png b/doc/images/ODP-Logo-HQ.png
> deleted file mode 100644
> index
> 721bf40b185f75ee6899b161a35b2e6fdf0c5ab4..
> GIT binary patch
> literal 0
> HcmV?d1
>
> literal 5736
> zcmV-u7MJOXP) zddwdkt*~h=<% zl?^8G^0sej-_{~YkXEhK`YI%WY;Ehu+PAjWid7^_#S3EV{fbhh3P>(o zA2Zopb~g}6;@ftApAVPaIrE&~IrE$6Jm<`FW+9C<(nuqXe-DWMtnBKmubyTY#!Y+n
> z>^W)U#*NX;%*^%gzyJP!_9url(f}~~tD;aSbVo24{EpTd(=@Sd8^bV888c?gyd67s
> z3==}={^F8G8nQns357z-gTdf;Y}-aDh1Q zG8hbAYuk2r#j>|;+qP@!)Tvp0%_ogCppS)LR8&+Q3 z=JX|}G}3@R6n;@r(JITbuIVcJre=fLXCGwWUWwx*oR(EzUtd3E%9Npf#Vd_8ppS$f
> z3WdU!WfgT5{lP;f7tGjBQSlnKzXCgVf}JS-%)NW}?wB-b(uls`mPQ)T2f`19LiYrN
> z!THHVe_{TI6khfStEw+%@ZcddycuQJPKb30|Ai14hYlUuHh%p0QN7JCjWnQ_g
> zeiaM`uSzERZ{%-h+z-7FO1;2iWy`N7{<=GqU_$4T>Q3ee{lTq$B*qzj%lO;
> zz4!pj7yY-AhBxPY$R%@EQS##e@um7#E5LCg`XSo{RB*lvcMl) zCcgLHdmr`|*EG`T72&5Ay}^a~JK~;x<153+%24s+t}AGbN|rBv60I8 zAP_il>(;Fw^akHF(m3W>Lmt-c+JLe?}d1E?dpY>cRxk14yaK$WUB&^K>+$*!M;&
> z`wl^CG@%;Hw(Y;?<>h_ySox-r1{|aC=g*|l6v}cc5vzC4^v%L$cC4OCx|{y
> zKtM*&<+>%)i8`Y>u&7b__CyzfC|6p#~}PrD0F{DM#kkSJ^kF(
> ztXfknT0?WILCkKiS_&4XQ7zQ*air;K)ZQV3s!h{o!
> zmUkLyz){BdMMXuwwJht>RHDE1@`qVjUC1*p4QD`R=a}Ah1j`rK@=)z*JicKR*;$=$
> zii6DtKmXyg49(fjq%UN0pa}*fX7Q*%AaKgoty|wd8vaz~hY`#}N{hDCI03kN2SQv$
> zac$(N?DiI_3(mu(2Ize23k2u`Aa123^1uDgt1^Esh@S$TgHeG%>s@HR%eCt^P+3q2
> z{2a6mVi%>6Gy0wDVbJ63%up!wfMr>Q38HT{`1;wN=vS@0IH~AkwqW_ z*0WjrY;IiqLcqW*#dm)4C6@hQBU_^u$#Yg)gyK@$gVcYhb zD`zgiGUn6jfCfzI-Ep=Hh8T--{{#%^X}|O;w9Y1Ij=}cxI!u3bbX>2KO!-@WaqcV3
> ze|+;1+gvey6e4gB5Jd<{N#uea^Aa)#OO8#-ONt4&`WZkV+<{nuu{bU$J+R01^aFYn
> z>J$|f{XQ5BCKdhHzEa0!^E^-g^ot`>c=`xJ3N9! z9RBN;FVL`i0$ZMICU-a-Je0^?DfRks#~pV{k6P^NbBOr@((V>)=UkL3As|6%2%1^k
> zT`-|XHhUXdXk5Vcha*Rq;p#H9K8UM5o>6B(k8B##bg#V{@;V4Q!H#9UnM@zPr3
> zxVPh|JL8-J22|%?iIy?19018DcONs3*yD#np$9F?`erK8&%NSdR#hkW^if-|VsQi^
> zG}kZ5XXv0kl>by9#9$ULU5k{nSpCFU1`hD@7leSEY=?!neV*GEN;W^Ug-^}MV%rWF
> zl-;R%>ALP~ z)%pL879$X%f#TX_3AU^9uS3W@w9p7G(7G8^4JA+P@QxRJ3R4b2X+d%A(^Sp;4(PE6
> z_D~$TB`IGiV8PlY095ABL&|}ECv_f`s`HD{Vj@yB;Y5E%hV= zxu6ByTQGrEF`5On8{;-AXMPW%bAg?dM1D$D!51;5MCt>~uYE0{CyQ@10jrf*^?2
> z;F5zDAEoU7s{A=1rXp}~)$bU<9$L(aNG(_pc?y7 zuM%y}{1NEEAYE`JLNq45J{@pG!ViT)4+Vq43zLd|dc7z5l@}-V^f5=U{3c)YHy02H
> zXr$B(8wO>!2^?n#OKx3Dz%1Yowc{AnMf~5t{d3&9Q1I51?{dnF0ldErayk#5!nW;K
> zeDS|>1kKSM{^yN&%cc+SRKjT66eI7$lv8MNpt9fq0b>xZ_Q+M{S5UgHU3^sqby
> z_;D?~s)SL-iu? zOTe6j(%y~YTIEDJK!zNIuXGyLG+xQ2Lash9+tt@E9l9_1VMpeNN5n?c=gs6HI
> z_)J$aDS9{chN9|%KVzA*{m=+7Zsd^KNl=dT!=^TyPh$cx>9I3RkF#k&8CL{0}_
> z)vS|o^_v*NJ5>nK(!V|hODK(`OmjbMs8du_^xI^jZ)!I9+Vnc+
> zv<}m7xpv7+Oao+xjaFmEz|B7s+_-QE7hM)%_(9D3IT%$Is9PR8T|c&6M19(
> zAtq0Py$2F`+P3|2uP=3KqC{2Og0+p%4Tn1X zRRfOpJggKq`4*$Z6hswhgl{j3qri{?$N6cGIaU^2LC~B;RC#s9)ONH!h!lf}I-qq+
>

[lng-odp] [PATCH] doc: doxygen: use .svg logo

2016-01-26 Thread Mike Holmes 
Signed-off-by: Mike Holmes 
---
 doc/application-api-guide/doxygen.cfg |   2 +-
 doc/images/ODP-Logo-HQ.png| Bin 5736 -> 0 bytes
 doc/images/ODP-Logo-HQ.svg| 209 ++
 3 files changed, 210 insertions(+), 1 deletion(-)
 delete mode 100644 doc/images/ODP-Logo-HQ.png
 create mode 100644 doc/images/ODP-Logo-HQ.svg

diff --git a/doc/application-api-guide/doxygen.cfg 
b/doc/application-api-guide/doxygen.cfg
index 75006e2..7951e9e 100644
--- a/doc/application-api-guide/doxygen.cfg
+++ b/doc/application-api-guide/doxygen.cfg
@@ -1,6 +1,6 @@
 PROJECT_NAME = "API Reference Manual for $(WITH_PLATFORM)"
 PROJECT_NUMBER = $(VERSION)
-PROJECT_LOGO = $(SRCDIR)/doc/images/ODP-Logo-HQ.png
+PROJECT_LOGO = $(SRCDIR)/doc/images/ODP-Logo-HQ.svg
 QUIET = YES
 OUTPUT_DIRECTORY = $(DOCDIR)
 FULL_PATH_NAMES = NO
diff --git a/doc/images/ODP-Logo-HQ.png b/doc/images/ODP-Logo-HQ.png
deleted file mode 100644
index 
721bf40b185f75ee6899b161a35b2e6fdf0c5ab4..
GIT binary patch
literal 0
HcmV?d1

literal 5736
zcmV-u7MJOXP)dwdkt*~h=<%(o^W)U#*NX;%*^%gzyJP!_9url(f}~~tD;aSbVo24{EpTd(=@Sd8^bV888c?gyd67s
z3==}={^F8G8nQns357z-gTdf;Y}-aDh1Q|;+qP@!)Tvp0%_ogCppS)LR8&+Q3Q3ee{lTq$B*qzj%lO;
zz4!pj7yY-AhBxPY$R%@EQS##e@um7#E5LCg`XSo{RB*lvcMl)(;Fw^akHF(m3W>Lmt-c+JLe?}d1E?dpY>cRxk14yaK$WUB&^K>+$*!M;&
z`wl^CG@%;Hw(Y;?<>h_ySox-r1{|aC=g*|l6v}cc5vzC4^v%L$cC4OCx|{y
zKtM*&<+>%)i8`Y>u&7b__CyzfC|6p#~}PrD0F{DM#kkSJ^kF(
ztXfknT0?WILCkKiS_&4XQ7zQ*air;K)ZQV3s!h{o!
zmUkLyz){BdMMXuwwJht>RHDE1@`qVjUC1*p4QD`R=a}Ah1j`rK@=)z*JicKR*;$=$
zii6DtKmXyg49(fjq%UN0pa}*fX7Q*%AaKgoty|wd8vaz~hY`#}N{hDCI03kN2SQv$
zac$(N?DiI_3(mu(2Ize23k2u`Aa123^1uDgt1^Esh@S$TgHeG%>s@HR%eCt^P+3q2
z{2a6mVi%>6Gy0wDVbJ63%up!wfMr>Q38HT{`1;wN=vS@0IH~AkwqW_2KO!-@WaqcV3
ze|+;1+gvey6e4gB5Jd<{N#uea^Aa)#OO8#-ONt4&`WZkV+<{nuu{bU$J+R01^aFYn
z>J$|f{XQ5BCKdhHzEa0!^E^-g^ot`>c=`xJ3N9!)=UkL3As|6%2%1^k
zT`-|XHhUXdXk5Vcha*Rq;p#H9K8UM5o>6B(k8B##bg#V{@;V4Q!H#9UnM@zPr3
zxVPh|JL8-J22|%?iIy?19018DcONs3*yD#np$9F?`erK8&%NSdR#hkW^if-|VsQi^
zG}kZ5XXv0kl>by9#9$ULU5k{nSpCFU1`hD@7leSEY=?!neV*GEN;W^Ug-^}MV%rWF
zl-;R%>ALP~~uYE0{CyQ@10jrf*^?2
z;F5zDAEoU7s{A=1rXp}~)$bU<9$L(aNG(_pc?y7!2^?n#OKx3Dz%1Yowc{AnMf~5t{d3&9Q1I51?{dnF0ldErayk#5!nW;K
zeDS|>1kKSM{^yN&%cc+SRKjT66eI7$lv8MNpt9fq0b>xZ_Q+M{S5UgHU3^sqby
z_;D?~s)SL-iu?uXGyLG+xQ2Lash9+tt@E9l9_1VMpeNN5n?c=gs6HI
z_)J$aDS9{chN9|%KVzA*{m=+7Zsd^KNl=dT!=^TyPh$cx>9I3RkF#k&8CL{0}_
z)vS|o^_v*NJ5>nK(!V|hODK(`OmjbMs8du_^xI^jZ)!I9+Vnc+
zv<}m7xpv7+Oao+xjaFmEz|B7s+_-QE7hM)%_(9D3IT%$Is9PR8T|c&6M19(
zAtq0Py$2F`+P3|2uP=3KqC{2Og0+p%4Tn1X;XU`spf0C%As%ka!z0nYGbrYqL
zv6Mtc#3#tf5RjnMrzu%C4)ivJ@EwW%l+R}abzljN072@ev~rLtJ_e=vaBKi
z!)Tv;Mq`5W