| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| *** | hverkuil1 has left | [07:14] |
| ............................................... (idle for 3h50mn) | ||
| mchehab | sailus, did you have time to check those warning cleanups I did for atomisp? | [11:04] |
| hverkuil | mchehab: please note https://patchwork.linuxtv.org/patch/48143/. The driver fixes definitely should go to 4.17, the doc fixes can slip to 4.18, but I feel most/all of those should also go to 4.17 since the docs are simply wrong.
Your call, though. | [11:14] |
| mchehab | hverkuil: I'm sending right now what we already have for 4.17...
after having it applied, I'll take a look on it I'd like to also send the atomisp patches I made too, if sailus is ok... as it does lots of changes at atomisp... not applying it would very likely mean that we'll receive atomisp patches that won't apply cleanly after we merge it on our tree so, better apply it for -rc1 | [11:15] |
| hverkuil | No problem, as long as it will be applied :-) | [11:17] |
| mchehab | we should likely take a look at patchwork to double check if aren't there any build fix patches
there is one from Dan, afaikt... but it is from something that I received another fix from Arnd not sure if they were applied upstream | [11:19] |
| hverkuil | This one from Arnd is in my pull request: https://git.linuxtv.org/hverkuil/media_tree.git/commit/?h=for-v4.17f&id=a193b0bc5407f7f30f789997785032434c821e19 | [11:20] |
| mchehab | I applied on a tree here locally together with other patches for 4.18 I'm developing
no, it is related to I2C new binding way | [11:20] |
| hverkuil | BTW: Jasmin has been doing a great job on media_build: we now have an OK. Still some sparse warnings, though. | [11:22] |
| mchehab | great
well, with the atomisp patchset, we'll get rid of lots of sparse/smatch warnings | [11:22] |
| hverkuil | I'm not building atomisp for the daily build at the moment. | [11:22] |
| mchehab | ah
I wander if atomisp driver works :-) | [11:23] |
| pinchartl | mchehab: I wouldn't bet on that :-) | [11:23] |
| mchehab | the code looks so crappy/scarry that I doubt it works | [11:24] |
| pinchartl | as I said before I doubt it will ever get out of staging | [11:24] |
| mchehab | well, it will... most likely going to /dev/null
that will be a shame, as I bet people put a lot of effort writing it | [11:24] |
| pinchartl | time, yes. effort, yes. quality... :-) | [11:25] |
| mchehab | heh, true | [11:26] |
| hans, this patch: Subject: [PATCH] media: dvb-core: Check for allocation failure in dvb_module_probe()
it should go to 4.17 too, but I'm pretty sure that either I or someone else wrote the same fix Subject: [PATCH 01/30] media: dvbdev: handle ENOMEM error at dvb_module_probe() I did and it was merged already | [11:33] | |
| hverkuil | Sorry, what does that have to do with me? | [11:36] |
| mchehab | https://patchwork.linuxtv.org/patch/48231/
(08:19:00) mchehab: we should likely take a look at patchwork to double check if aren't there any build fix patches (08:19:18) mchehab: there is one from Dan, afaikt... not related to anything you should do... it is just related to the above commens I made ^ | [11:36] |
| hverkuil | Uh, okay. | [11:38] |
| mchehab | on this particular case, all it needs is to mark https://patchwork.linuxtv.org/patch/48231/ as superseeded. I'll do that
done anyway, similar patches (if any) may need to be applied for 4.17 as well so, what I meant to say is that we should take a quick look if are there any patches fixing issues for 4.17 submitted by the usual jenitors just to be sure that we're not missing anything important I'll probably add https://patchwork.linuxtv.org/patch/48346/ for 4.17 as well | [11:38] |
| hverkuil | How about this: https://patchwork.linuxtv.org/patch/47966/ | [11:47] |
| mchehab | i prefer apply it after -rc1
no need to hush on it also, it has potential of causing some harm as it removes depends on from kconfigs if one of dependencies chain there is wrong, it could cause build issues | [11:49] |
| .................................................................................... (idle for 6h55mn) | ||
| *** | ChanServ sets mode: +v mchehab` | [18:46] |
| ................................... (idle for 2h51mn) | ||
| ChanServ sets mode: +v mchehab | [21:37] | |
| ............ (idle for 58mn) | ||
| ChanServ sets mode: +v mchehab | [22:35] | |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |