<!-- 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>

   hverkuil: <u>pinchartl</u>: "implementing input support": I'm not sure what you mean with that.
   <br> <u>mchehab</u>: pong
   mchehab: while looking at this patch: https://patchwork.linuxtv.org/patch/31435/
   <br> I'm wandering why we're using the dvb old API inside ivtv
   hverkuil: it predates VIDIOC_DQEVENT.
   mchehab: I would very much prefer to just get rid of video.h and audio.h than to fix it for 2038-yr
   hverkuil: I have considered removing it, but since it wasn't blocking anything I never bothered.
   <br> So you want to drop video.h and audio.h completely? Isn't it used by another driver as well?
   mchehab: does it implement VIDIOC_DQEVENT? Are any application using the legacy DVB API to talk with ivtv?
   hverkuil: Yes, it has the modern API as well.
   <br> As to applications using the old API: ivtv-ctl in v4l-utils still uses it (I never bothered updating it), but I suspect I can just drop that code from that utility since v4l2-ctl can do the same.
   mchehab: does MythTV (or some other widely-used app) use it?
   hverkuil: Whether there are other apps out there that use it, I have no idea.
   mchehab: (and don't have a fall-back to VIDIOC_DQEVENT)
   <br> Hmm.. let's trace a plan to remove it then...
   <br> what about:
   <br> 1) either drop ivtv-ctl or convert it to use VIDIOC_DQEVENT;
   hverkuil: It would not surprise me if it is still in use: it was a very popular card in the past.
   mchehab: 2) add a printk_once() msg telling that the API is obsolete;
   hverkuil: <u>1</u>: no problem
   mchehab: 3) wait for a few Kernel cycles;
   <br> 4) remove its support from ivtv
   hverkuil: <u>2</u>: ditto
   mchehab: 5) drop support for it from Kernel, together with av7110 driver
   hverkuil: do you have the mythtv source? If so, can you grep for VIDEO_GET_?
   <br> MythTV is the most likely app that support this.
   mchehab`: no, I don't have mythtv source here
   hverkuil: I just downloaded the latest.
   mchehab`: https://github.com/MythTV
   hverkuil: Looks like it isn't used anymore (if ever).
   mchehab: getting it from git clone https://github.com/MythTV/mythtv
   hverkuil: I'm OK with your plan.
   mchehab: $ git grep dvb/video\\.h
   <br> doesn't return anything
   <br> neither:
   <br> $ git grep VIDEO_GET_EVENT
   <br> so, except if it hardcodes the IOCTL number, it should be safe to drop it
   hverkuil: I'm removing the dvb ioctls from ivtv-ctl.
   mchehab: OK
   <br> thanks!
   hverkuil: Done.
   mchehab: thanks!
   hverkuil: Can you make a patch for ivtv adding the printk? I'll review and ack it.
   mchehab: sure
   hverkuil: ivtv_default() in ivtv_ioctl.c
   mchehab: only ivtv-alsa.h seems to be using pr_foo()
   <br> I guess I'll convert it to pr_foo() first
   <br> and use pr_err_once()
   <br> ivtv-driver.h is included everywhere?
   hverkuil: as far as I remember, yes.
   mchehab: ok. I'll add pr_fmt() there and be sure that it is the first file included
   <br> and use pr_foo() instead of printk()
   <br> then, on a next patch, I'll add the pr_err_once()
   <br> <u>hverkuil</u>: just sent the patches
   <br> I opted to also add a kconfig option to disable the DVB API abuse
   ***: prabhakarlad has left
   pinchartl: <u>hverkuil</u>: most drivers have a single input, and still have to implement useless code for g/s_input and enuminput
   <br> if we want to make those ioctls mandatory shouldn't we implement them in the core instead of duplicating useless code in drivers ?
   ***: indy has quit IRC (Ping timeout: 244 seconds)
   <br> ndufresne has left
   <br> benjiG has left
   joebobjoe: <u>hverkuil</u>: are you on?
   <br> and can I ask you a question about proper HDMI quantization levels for RGB?
   <br> I was told you might know
   pinchartl: is there anyone with access to the Synopsys DWC HDMI TX documentation here ?
   <br> (scratch that Synopsys doc request, I've found the information I need)
   ***: joebobjoe has left