[08:05] *** ChanServ sets mode: +v hverkuil
[11:51] <hverkuil> hi all
[11:55] <mchehab> hi all
[11:55] <hverkuil> (I'm in a parallel work meeting, so I might be slow in replying)
[11:56] <mchehab> my queue has curently 3 pending PRs:
[11:56] <mchehab> lmml_66870_git_pull_for_v5_10_various_fixes_and_enhancements_csc_uapi.patch
[11:56] <mchehab> lmml_67062_git_pull_for_v5_10_various_fixes_mostly_rcar_related.patch
[11:56] <mchehab> lmml_67063_git_pull_for_v5_10_pm_ref_count_fixes.patch
[11:56] <mchehab> the last 2 were sent today
[11:57] <mchehab> the first one has a patch from Greg KH, that pinchartl wants to discuss with Greg
[11:57] <hverkuil> mchehab: can you reconsider this PR https://patchwork.linuxtv.org/project/linux-media/patch/22861790-3ae1-2009-a4db-c124067771b2@xs4all.nl/ given the email from Alexandre (Re: [PATCH RESEND RESEND] remoteproc: scp: add COMPILE_TEST dependency) from September 15th?
[11:57] <mchehab> pinchartl: did you ping Greg again? What did he explained?
[11:59] <hverkuil> I plan to post one more PR tomorrow (rkisp1 patches)
[12:00] <mchehab> I'll probably handle the remaining PRs on Monday
[12:00] <mchehab> hverkuil: yeah, now that the fix was applied, we can merge it, provided that we merge first the patch with the fix
[12:01] <hverkuil> Do you want me to do anything, or do you have enough info to proceed?
[12:02] <mchehab> just re-tag the PR as new
[12:02] <hverkuil> done
[12:03] <mchehab> I guess I'll cherry-pick the patch from https://git.kernel.org/pub/scm/linux/kernel/git/andersson/remoteproc.git/commit/?id=5185e3a9dc2d68bb52e3e12400428aa060b87733
[12:03] <mchehab> instead of asking for a stable branch
[12:03] <mchehab> this is a trivial patch, so it shouldn't hurt if this gets merged from two different trees
[12:03] <hverkuil> I agree
[12:07] <mchehab> ok, I ended sending an e-mail to Bjorn and Alexandre to double-check if no other changes will cause merge conflicts at linux-next
[12:08] <mchehab> worse case, I'll need to create a separate branch and apply on it after pulling from a stable branch from remoteproc tree
[12:08] <mchehab> anything else for today's discussion?
[12:09] <hverkuil> Was the PR removing usbvision merged? I don't think so.
[12:09] <mchehab> I guess I merged it
[12:10] <hverkuil> was it pushed?
[12:10] <mchehab> commit 50044aa715177b336acca01d711bf8edc19162c7
[12:10] <mchehab> Author:     Hans Verkuil <hverkuil-cisco@xs4all.nl>
[12:10] <mchehab> AuthorDate: Tue Sep 1 08:22:47 2020 +0200
[12:10] <mchehab> Commit:     Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
[12:10] <mchehab> CommitDate: Thu Sep 10 14:06:27 2020 +0200
[12:10] <mchehab>     media: usbvision: remove deprecated driver
[12:10] <mchehab>     
[12:10] <mchehab> I think so
[12:12] <mchehab> https://git.linuxtv.org/media_tree.git/log/?ofs=50
[12:12] <mchehab> yep, it is there
[12:12] <hverkuil> You're right, it's removed.
[12:12] <mchehab> anything else?
[12:12] <hverkuil> not from me.
[12:15] <hverkuil> It would be nice if you could pick up this patch: https://patchwork.linuxtv.org/project/linux-media/patch/8493f131-1b0f-37b5-a9d3-66dabf76bf70@xs4all.nl/
[12:15] <hverkuil> this to fix a daily build warning about the use of strlcpy.
[12:20] <pinchartl> mchehab: I've just did
[12:32] <mchehab> hverkuil: ok, I''ll pick it
[12:34] <mchehab> pinchartl: thanks! btw, Greg already answered it
[12:34] <mchehab> btw, I agree with him: if a debug is needed to warn that debugfs creation failed, the best would be to implement it at core level
[12:36] <mchehab> anyway, I'll wait for the discussions about it to reach some results before merging it
[12:37] <pinchartl> I can also include it in my next pull request
[12:37] <hverkuil> pinchartl: can I delegate the patch to you?
[12:39] <mchehab> pinchartl, hverkuil: I'm ok on merging it the way it works best for you
[12:39] <mchehab> Yet, as Greg is the maintainer of configfs, whatever we do with regards to it should be in sync with other drivers/subsystems
[12:41] <pinchartl> hverkuil: sure
[12:42] <hverkuil> pinchartl: all yours!
[13:51] <sailus> mchehab: Is it still possible to merge fixes to 5.9?
[13:51] <sailus> I'd have a patch for DT bindings, so there's no functional changes.
[14:50] <pinchartl> mchehab: I'm afraid I'll annoy you again. we've changed one of the libcamera configuration options
[14:50] <pinchartl> -Dandroid=true should become -Dandroid=auto
[14:51] <pinchartl> could you update the build script accordingly ?