<!-- Some styling for better description lists --><style type='text/css'>dt { font-weight: bold;float: left;display:inline;margin-right: 1em} dd { display:block; margin-left: 2em}</style>
mchehab: s/agenda/timeframe/
hverkuil: <u>mchehab</u>: last I heard (yesterday) was that pinchartl was still trying to change his schedule for the meeting.
ezequielg: <u>hverkuil</u>: any chance you include stk1160 frame scaling support in your PR for 4.2?
<br> <u>hverkuil</u>: or was the patch too late?
pinchartl: <u>mchehab</u>: I haven't heard back from my customer yet I'm afraid :-/
sailus: <u>janaszewski</u>: Dobry wieczor!
mchehab: rchehab, shuah: ping
shuah: yes
mchehab: we need to coordinate the MC patches related to au0828
rchehab: pong
shuah: <u>rchehab</u>: when do you plan to send v2 for your patches
<br> you will have to update them to use the new media controller interfaces I am adding in my patch series
rchehab: <u>shuah</u>: I'll send them until Friday
mchehab: <u>shuah</u>: as some of the patches will need to go through the alsa ML, I think that the best...
<br> is to merge first the rafael's patches...
<br> then your patch 1/2 on my tree
<br> and then either you or rchehab would write a patch to use the devres approach on au0828
shuah: hmm. shouldn't au0828 use the work on my 1/2 patch then rafaels' patches that use those interfacesgo in
mchehab: shuah's patch 2/2 will then be merged via alsa
shuah: I see - did Rafael test his patches with ALSA that is media controller unaware - does that work
<br> guess there is no locking - maybe that is just fine
mchehab: afaikt, he didn't test your alsa patches
<br> that's why it sounds simpler to apply his patch first, and then yours
<br> and a third patch merging it, after testing with alsa
<br> we need to be sure that the same struct device will be used on both snd-usb-audio and au0828
<br> while this is not hard, it requires some testing
shuah: what I am asking is Rafael's patches add media controller to just au0828 and ALSA is media controller unaware - the question is does that combination work
mchehab: Rafael's patches add it just to au0828
shuah: struct device isn't an issue. I proved it all works with the media tokens work. It is essentially same thing excpet devres is media_device as opposed to media token
mchehab: <u>rchehab</u>: not sure if you saw shuah patches:
<br> https://patchwork.linuxtv.org/patch/29999/
<br> https://patchwork.linuxtv.org/patch/30000/
shuah: I am fine with what you are proposing.
<br> step 1: Merge Rafael patches that add media controller to au0828
mchehab: <u>shuah</u>: rchehab's patch is this one: https://patchwork.linuxtv.org/patch/29703/
shuah: step 2: My new media controller api interfaces
mchehab: (he should work on version 2, of course, to fulfill hverkuil's requirements)
rchehab: <u>mchehab</u>: I was just reading shuah's patches
mchehab: <u>rchehab</u>: ok
shuah: step 3: alsa changed and au0828 updates to use new media controller api
<br> sounds about right
mchehab: <u>shuah</u>: I would actually break step 3 on two steps
shuah: I can do step 3 alsa as well as au0828 - it will make it easier
mchehab: step 3: au0828 updates
<br> step 4: alsa (to be submitted to alsa, c/c us)
shuah: ok I can handle step 3 and 4 - it will be easier that way
mchehab: good
shuah: ok it is a plan then - thanks
mchehab: <u>rchehab</u>: would that work for you?
shuah: can we work on getting my patch 1/2 as it is not dependent on Rafael's or my alsa work
mchehab: yes, sure
<br> my plan is to apply the remaining patches for 4.2 up to this weekend
<br> so, provided that you and rchehab will be able to finish it during the week, they'll be merged
rchehab: that would work for me
mchehab: I may eventually pick some patches next week
shuah: ok my patch 1/2 good as is for you, pending a separate patch to address the ifdef and stubs - still need pinchartl review
mchehab: it would be really cool if we could get the patch from step 3 done up to Monday
<br> yep, patch 1/2 is OK for me to apply, provided that you send latter a patch to address ifdef's/stubs
rchehab: I'm planning in splitting my patch in 2: one is moving an atribution of dev->boards sooner to ensure media_controller register...
shuah: yes that is the plan. I will send a separate patch to fix the ifdef issue
mchehab: ok, works for me
rchehab: and the other would be adding support to media_controler in au0828
mchehab: let's wait until Friday for pinchartl's review
<br> afaikt, he is in Japan those days, so he'll likely be able to answer to this review only latter
<br> but patch 1 is trivial and shouldn't cause any harm, IMHO
shuah: ok
<br> yeah - that is correct
mchehab: so, I'll wait until Friday for his ack
<br> if we don't hear from pinchartl, I'll apply it
rchehab: As for hverkuil's requirements, It's still necessary that media_controller support for more drivers including pci is done before creating helper functions, as he already stated
shuah: ok sounds good to me
<br> helper functions for?? I probably missed that conversation
rchehab: hverkuil wants that helper functions are added to media_controller to avoid code duplication
shuah: now I remember - yes his comments on your patch
mchehab: pinchartl, hverkuil, sailus: any news about the agenda for our meeting?
<br> s/agenda/timeframe/
hverkuil: <u>mchehab</u>: last I heard (yesterday) was that pinchartl was still trying to change his schedule for the meeting.
ezequielg: <u>hverkuil</u>: any chance you include stk1160 frame scaling support in your PR for 4.2?
<br> <u>hverkuil</u>: or was the patch too late?
pinchartl: <u>mchehab</u>: I haven't heard back from my customer yet I'm afraid :-/
sailus: <u>janaszewski</u>: Dobry wieczor!
mchehab: rchehab, shuah: ping
shuah: yes
mchehab: we need to coordinate the MC patches related to au0828
rchehab: pong
shuah: <u>rchehab</u>: when do you plan to send v2 for your patches
<br> you will have to update them to use the new media controller interfaces I am adding in my patch series
rchehab: <u>shuah</u>: I'll send them until Friday
mchehab: <u>shuah</u>: as some of the patches will need to go through the alsa ML, I think that the best...
<br> is to merge first the rafael's patches...
<br> then your patch 1/2 on my tree
<br> and then either you or rchehab would write a patch to use the devres approach on au0828
shuah: hmm. shouldn't au0828 use the work on my 1/2 patch then rafaels' patches that use those interfacesgo in
mchehab: shuah's patch 2/2 will then be merged via alsa
shuah: I see - did Rafael test his patches with ALSA that is media controller unaware - does that work
<br> guess there is no locking - maybe that is just fine
mchehab: afaikt, he didn't test your alsa patches
<br> that's why it sounds simpler to apply his patch first, and then yours
<br> and a third patch merging it, after testing with alsa
<br> we need to be sure that the same struct device will be used on both snd-usb-audio and au0828
<br> while this is not hard, it requires some testing
shuah: what I am asking is Rafael's patches add media controller to just au0828 and ALSA is media controller unaware - the question is does that combination work
mchehab: Rafael's patches add it just to au0828
shuah: struct device isn't an issue. I proved it all works with the media tokens work. It is essentially same thing excpet devres is media_device as opposed to media token
mchehab: <u>rchehab</u>: not sure if you saw shuah patches:
<br> https://patchwork.linuxtv.org/patch/29999/
<br> https://patchwork.linuxtv.org/patch/30000/
shuah: I am fine with what you are proposing.
<br> step 1: Merge Rafael patches that add media controller to au0828
mchehab: <u>shuah</u>: rchehab's patch is this one: https://patchwork.linuxtv.org/patch/29703/
shuah: step 2: My new media controller api interfaces
mchehab: (he should work on version 2, of course, to fulfill hverkuil's requirements)
rchehab: <u>mchehab</u>: I was just reading shuah's patches
mchehab: <u>rchehab</u>: ok
shuah: step 3: alsa changed and au0828 updates to use new media controller api
<br> sounds about right
mchehab: <u>shuah</u>: I would actually break step 3 on two steps
shuah: I can do step 3 alsa as well as au0828 - it will make it easier
mchehab: step 3: au0828 updates
<br> step 4: alsa (to be submitted to alsa, c/c us)
shuah: ok I can handle step 3 and 4 - it will be easier that way
mchehab: good
shuah: ok it is a plan then - thanks
mchehab: <u>rchehab</u>: would that work for you?
shuah: can we work on getting my patch 1/2 as it is not dependent on Rafael's or my alsa work
mchehab: yes, sure
<br> my plan is to apply the remaining patches for 4.2 up to this weekend
<br> so, provided that you and rchehab will be able to finish it during the week, they'll be merged
rchehab: that would work for me
mchehab: I may eventually pick some patches next week
shuah: ok my patch 1/2 good as is for you, pending a separate patch to address the ifdef and stubs - still need pinchartl review
mchehab: it would be really cool if we could get the patch from step 3 done up to Monday
<br> yep, patch 1/2 is OK for me to apply, provided that you send latter a patch to address ifdef's/stubs
rchehab: I'm planning in splitting my patch in 2: one is moving an atribution of dev->boards sooner to ensure media_controller register...
shuah: yes that is the plan. I will send a separate patch to fix the ifdef issue
mchehab: ok, works for me
rchehab: and the other would be adding support to media_controler in au0828
mchehab: let's wait until Friday for pinchartl's review
<br> afaikt, he is in Japan those days, so he'll likely be able to answer to this review only latter
<br> but patch 1 is trivial and shouldn't cause any harm, IMHO
shuah: ok
<br> yeah - that is correct
mchehab: so, I'll wait until Friday for his ack
<br> if we don't hear from pinchartl, I'll apply it
rchehab: As for hverkuil's requirements, It's still necessary that media_controller support for more drivers including pci is done before creating helper functions, as he already stated
shuah: ok sounds good to me
<br> helper functions for?? I probably missed that conversation
rchehab: hverkuil wants that helper functions are added to media_controller to avoid code duplication
shuah: now I remember - yes his comments on your patch