<!-- 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> sailus: There are a few compile fixes (hopefully). <br> Nothing you'll need now though. <br> Just FYI. neg: <u>sailus</u>: thx for the update, I will do some work on the rcar-vin and rcar-csi2 drivers and once that is out I will circle back to the vc work and rebase it ontop of your new vc branch :-) <br> <u>sailus</u>: do we have a plan on how to start upstreaming stuff from your vc branch? sailus: Well, it's been waiting for a complete, or complete enough, solution to the problem area. <br> I think with your patchset that should be there. <br> Pending review, of course. <br> I can re-post the patches to the list at some point, too. They'll need more review comments. neg: ofc, I was just wondering if we knew about something blocking it, other then the proper upstreaming prosses :-) hverkuil: <u>mchehab</u>: ping mchehab: pong hverkuil: <u>mchehab</u>: when should include/media/drv-intf be used for platform driver include files? <br> I still see some headers like vsp1.h, rcar-fcp.h and blackfin/davinci directories in include/media. <br> Shouldn't they be move to drv-intf? (eventually) mchehab: the idea is that the stuff under include/media are the headers that are sharing some interface with other drivers <br> it could make sense to move vsp1 header, for instance, if DRM drivers use it <br> on a quick grep, it seems that this is not the case of vsp1.h hverkuil: OK, thanks. I didn't quite understand the purpose of drv-intf, thanks for clarifying this. mchehab: rcar-fcp.h, on the other hand, seems to be used by both rcar-fcp and vsp1 <br> it was created when we moved out of include/media/*.h all files that aren't part of media core kAPI <br> in other words, right now, only core kAPI are at include/media/*.h <br> everything else that used to be there were moved to a sub-directory <br> that's said, it probably makes sense to move a lot of other files to an include/media subdir <br> like the dvb-frontend header files <br> as those are similar to include/media/i2c/*.h <br> they're used by multiple drivers funman: <u>hverkuil</u>: what does the "Delegated to" you mean for https://patchwork.linuxtv.org/patch/45426/ (and 45427) ? hverkuil: <u>funman</u>: that it is in my todo list. funman: ok pinchartl: so include/media/drv-intf/ contains headers that are used by a single driver ? ***: prabhakarlad has left funman: looks like i'm already celbrating one month of contributions to linux-media ;) ***: benjiG has left mchehab: <u>pinchartl</u>: it shouldn't ndufresne: pinchartl, do you know why the request API is related to the media controller ? ***: awalls has left headless: <u>ndufresne</u>: he's /away