[07:42] *** hverkuil1 has left 
[10:19] <sailus> mchehab: Bom dia!
[10:21] <mchehab> moikka
[10:23] <sailus> mchehab: How are you doing?
[10:24] <mchehab> fine, and you
[10:24] <sailus> Fine, too, thanks!
[10:25] <sailus> I'm back from holidays now. :-)
[10:25] <sailus> About the rest documentation --- which branch should I base my documentation patches? I didn't see them in master yet.
[10:25] <mchehab> it is in master already
[10:26] <mchehab> I guess
[10:26] <sailus> Hmm.
[10:26] <mchehab> (except if I forgot to push from local)
[10:27] <hverkuil1> it's there.
[10:27] <sailus> Hmm.
[10:27] <mchehab> https://git.linuxtv.org/media_tree.git/log/
[10:27] <mchehab> yes,
[10:27] <sailus> I might have a problem with my tree then.
[10:27] <mchehab> 6 daysMerge tag 'media/v4.8-4' of git://git.kernel.org/pub/scm/linux/kernel/git/mch...Linus Torvalds
[10:27] <sailus> I recently ran out of inodes in that fs. :-i
[10:28] <sailus> I fixed it but then I suppose something didn't get updated as it should have.
[10:28] <hverkuil1> mchehab: can you look at patchwork and mark all the doc patches that are merged 'Accepted' or something like that?
[10:28] <mchehab> yeah, maybe
[10:28] <sailus> Do you btw. have scripts to move the documentation from DocBook to rst? Or was there a lot of manual work?
[10:28] <hverkuil1> patchwork is very messy right now due to all those long doc patch series.
[10:29] <sailus> The format patches especially require changes in quite a few places presumably.
[10:29] <mchehab> sailus: you can use pandoc... not perfect, but works fine for simple documents
[10:29] <mchehab> there's another tool that does a better conversion
[10:29] <mchehab> doc2rst or something like that
[10:29] <sailus> Ok.
[10:29] <sailus> Is that what you used for the purpose?
[10:29] <mchehab> it was pointed by Markus Heiser... I guess the email is at the ML
[10:30] <mchehab> no, Markus did the initial conversion
[10:30] <sailus> Ok. He probably did then.
[10:30] <mchehab> he used it, together with a bunch of scripts he developped
[10:30] <sailus> I'll check that. Thanks!
[10:31] <mchehab> I used pandoc when converting some things when syncing patches from hverkuil1 CEC tree and from pinchartl's VSP1
[10:31] <mchehab> and manual work to fix the issues
[10:31] <mchehab> for simpler changes, I just re-typed the changes on ReST
[10:33] <mchehab> overall, it is a way easier to change things on rst...
[10:34] <mchehab> the exception is on tables... Markus script added captions like:
[10:34] <mchehab> "row 10"
[10:34] <mchehab> meaning that adding/removing stuff on tables is painful
[10:35] <mchehab> I intend to later remove such captions
[10:35] <hverkuil1> I understood that those are just comments and can be removed.
[10:35] <mchehab> if you're adding new tables, please don't use
[10:35] <mchehab> hverkuil1: they're actually captions
[10:35] <mchehab> you could use them to reference to a certain line in the table
[10:36] <mchehab> I used that on a few tables I changed manually
[10:36] <mchehab> for MC or RC I guess
[10:41] <mchehab> no. here: https://git.linuxtv.org/media_tree.git/commit/?h=docs-next&id=d2cd627be0f889ef7718eac98f68a822ad235f0a
[10:41] <mchehab> nah
[10:45] <mchehab> basically, instead of using a comment there, you can do things like:
[10:45] <mchehab> +       -  .. _FE-CAN-FEC-4-5:
[10:45] <mchehab> and use it to create a cross-reference for the table row
[13:39] <mchehab> hverkuil1: updating the patch status for the rst stuff
[13:39] <mchehab> also, rejected on patchwork those file attr patchbomb patches
[13:40] <hverkuil1> mchehab: thanks! Yeah, that was an awful patch bomb.
[13:40] <mchehab> and, btw, it is a way uglier to use macros than using 0644, IMHO
[13:42] <Renault> how is your workflow about Linux merge window? You submit one patchset by week or merge window?
[13:45] <mchehab> that very much depends on how I merged the patches I accepted before the merge window... and if something bad was discovered during the merge window
[15:07] <awalls1> Intel needs to give their open source developers real problems to work on, instead of a 1285 part "sed -e 's/happy/glad/g' " change.
[15:12] <larsc> paid by the number of commits
[15:24] <awalls1> http://dilbert.com/strip/1995-11-13
[15:30] <pinchartl> :-)
[17:11] <hverkuil1> Well, Baole demonstrated that we have way too many media drivers! It never ends...
[17:13] <larsc> too many module parameters really
[17:29] <hverkuil1> awalls1: that happened actually during a project in one company I worked for :-)
[17:50] <mchehab> I suspect we'll keep receiving patches and needing to reject them at BZ the entire day :(
[17:50] <mchehab> s/BZ/patchwork/
[17:51] * mchehab just read the dilbert picture... that's thy I typo it as BZ :-p
[17:53] <mchehab> bah... https://patchwork.linuxtv.org/project/linux-media/list/?submitter=6965
[17:53] <mchehab> every time I mark things there are rejected, newer ones appear when it refreshes :(
[18:09] <larsc> no mass update?
[19:31] *** awalls1 has left 
[23:36] *** gnac has left