linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
To: Linux Media Mailing List <linux-media@vger.kernel.org>
Cc: Andy Walls <awalls@md.metrocast.net>, Mike Isely <isely@pobox.com>
Subject: Re: [PATCH v3 22/22] media: Kconfig: better support hybrid TV devices
Date: Wed, 25 Mar 2020 11:05:24 +0100	[thread overview]
Message-ID: <20200325110524.433ef745@coco.lan> (raw)
In-Reply-To: <dc2d32af44bc03620bde14fc3c3ea5f9bde5530d.1585129041.git.mchehab+huawei@kernel.org>

Em Wed, 25 Mar 2020 10:49:36 +0100
Mauro Carvalho Chehab <mchehab+huawei@kernel.org> escreveu:

> Right now, if one has an hybrid TV card, it has to select
> both analog and digital TV support, as otherwise the needed
> core support won't be selected.
> 
> Change the logic to auto-select the core support for those
> drivers, as this is a way more intuitive.
> 
> It should be noticed that, as now both DVB_CORE and VIDEO_DEV
> defaults depends on selecting a hybrid cards, we had to remove
> the explicit dependencies there, in order to avoid circular
> dependencies. In order to get it right, we used a little trick,
> to ensure that the core will be built as "y" if any driver
> was built with "y".
> 
> Note: while here, moved two pure V4L2 PCI drivers out of the
> "hybrid" part of config and consider pvrusb2 as an hybrid
> device.

Found some issues on this patch. Please ignore it.

I'm trying to see if are there any other way of doing that
without incurring into circular dependencies nor allowing
errors at build time or warnings at make menuconfig time.

I remember I tried to do something like that in the past,
but the dependency chain is too complex.

Suggestions are welcomed.

Regards,
Mauro

  reply	other threads:[~2020-03-25 10:05 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-25  9:49 [PATCH v3 00/22] Reorganize media Kconfig Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 01/22] media: dvb-usb: auto-select CYPRESS_FIRMWARE Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 02/22] media: Kconfig: add an option to filter in/out platform drivers Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 03/22] media: Kconfig: not all V4L2 platform drivers are for camera Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 04/22] media: pci: move VIDEO_PCI_SKELETON to a different Kconfig Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 05/22] media: reorganize the drivers menu options Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 06/22] media: Kconfig: update the MEDIA_SUPPORT help message Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 07/22] media: Kconfig: use a sub-menu to select supported devices Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 08/22] media: split test drivers from platform directory Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 09/22] media: Kconfig: make filtering devices optional Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 10/22] media: Kconfig: move CEC-specific options to cec/Kconfig Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 11/22] media: Kconfig: move comment to siano include Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 12/22] media: Kconfig: move DVB-specific options to dvb-core/Kconfig Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 13/22] media: Kconfig: move V4L2 subdev API to v4l2-core/Kconfig Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 14/22] media: Kconfig: move drivers-specific TTPCI_EEPROM Kconfig var Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 15/22] media: Kconfig: move media controller core select to main Kconfig Mauro Carvalho Chehab
2020-03-25 10:00   ` Sakari Ailus
2020-03-25 10:08     ` Mauro Carvalho Chehab
2020-03-25 10:21       ` Mauro Carvalho Chehab
2020-03-30  7:46         ` Sakari Ailus
2020-03-25  9:49 ` [PATCH v3 16/22] media: Kconfig: better organize menu items Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 17/22] media: Kconfig: mode firewire comment to firewire/Kconfig Mauro Carvalho Chehab
2020-03-26 12:09   ` Stefan Richter
2020-03-25  9:49 ` [PATCH v3 18/22] media: Kconfig: place all options under a sub-menu Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 19/22] media: Kconfig: move the position of sub-driver autoselection Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 20/22] media: Kconfig: on !EMBEDDED && !EXPERT, enable driver filtering Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 21/22] media: Kconfig: simplify some dependencies Mauro Carvalho Chehab
2020-03-25  9:49 ` [PATCH v3 22/22] media: Kconfig: better support hybrid TV devices Mauro Carvalho Chehab
2020-03-25 10:05   ` Mauro Carvalho Chehab [this message]
2020-03-25 11:25     ` [PATCH v3.1 " Mauro Carvalho Chehab

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200325110524.433ef745@coco.lan \
    --to=mchehab+huawei@kernel.org \
    --cc=awalls@md.metrocast.net \
    --cc=isely@pobox.com \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).