| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| hverkuil | dafna2: ping | [08:05] |
| ..... (idle for 24mn) | ||
| koike: what is the status of this series? "[PATCH v5 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399"
It looks like there are some Acks missing, correct? | [08:29] | |
| ......... (idle for 40mn) | ||
| mchehab: FYI: vidtv smatch warning in the daily build: https://hverkuil.home.xs4all.nl/logs/Thursday.log (at the end) | [09:10] | |
| ..... (idle for 22mn) | ||
| mchehab | hverkuil: thanks! will address that one
hverkuil: that sounds weird... at least at the code I'm looking (maybe there are some unapplied patches upstream?) there's no way to produce such warnings it is basically complaining about this code:https://pastebin.com/RZHYFTxi with uses dev for debug stuff This was added on this changeset: 9cfb4d36c27c ("media: vidtv: prefer using dev_foo() instead of pr_foo()") perhaps this is due to some issues at the compat code inside the linux build? | [09:32] |
| hverkuil | mchehab: you're looking at the wrong warning: that's a warning from the compat build, I need to look at that myself. Go to the end of the log at 'smatch: WARNINGS' | [09:39] |
| mchehab | (as it is warned appeared only Kernel 3.10.108)
ah! | [09:39] |
| fix patch sent
thanks for letting me know about that! not sure why, but sparse reports different things on different machines here, on another machine, it doesn't report the same warning, but reports a different one, instead: drivers/media/test-drivers/vidtv/vidtv_s302m.c:83:1: warning: 'static' is not at beginning of declaration [-Wold-style-declaration] 83 | const static struct tone_duration beethoven_5th_symphony[] = { | ^~~~~ Hmm... I guess this is from a patch not merged upstream yet fixed. will merge the patch upstream with the change folded at this other machine, sparse (or smatch?) returns lots of: (null): (null):0 (null)() parse error: parsing 'kernel.sizeof_param' (null): (null):0 (null)() parse error: problem parsing 'kernel.sizeof_param' | [09:45] | |
| ok, applied the vidtv patches I had on my queue
hopefully, we should get rid of the currently known bugs on it, including the i386 build and the sparse warning | [09:57] | |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |