From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Subject: Re: [PATCH 00/10] drm/misc: Kconfig cleanup Date: Tue, 14 Jul 2015 11:18:02 +0200 Message-ID: <20150714091801.GG12465@ulmo.nvidia.com> References: <1436769393-3836-1-git-send-email-architt@codeaurora.org> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0604170759==" Return-path: In-Reply-To: <1436769393-3836-1-git-send-email-architt@codeaurora.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Archit Taneja Cc: linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org List-Id: linux-arm-msm@vger.kernel.org --===============0604170759== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="+1TulI7fc0PCHNy3" Content-Disposition: inline --+1TulI7fc0PCHNy3 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Jul 13, 2015 at 12:06:23PM +0530, Archit Taneja wrote: > The top level drm Kconfig is a bit messy at the moment. Most of the drm > drivers have their own Kconfig files, but a few have their Kconfig files > in the top level Kconfig option itself. >=20 > Create new Kconfig files for drivers that don't have one. This cleans up > the Kconfig quite a bit >=20 > Archit Taneja (10): > drm/tdfx: clean up: create a separate Kconfig file > drm/r128: clean up: create a separate Kconfig file > drm/radeon: clean up: move config option to driver's own Kconfig file > drm/amdgpu: clean up: move config option to driver's own Kconfig file > drm/i810: clean up: create a separate Kconfig file > drm/mga: clean up: create a separate Kconfig file > drm/sis: clean up: create a separate Kconfig file > drm/via: clean up: create a separate Kconfig file > drm/savage: clean up: create a separate Kconfig file > drm/misc: clean up: group device specific Kconfigs >=20 > drivers/gpu/drm/Kconfig | 116 ++++++-------------------------= ------ > drivers/gpu/drm/amd/amdgpu/Kconfig | 19 ++++++ > drivers/gpu/drm/i810/Kconfig | 8 +++ > drivers/gpu/drm/mga/Kconfig | 8 +++ > drivers/gpu/drm/r128/Kconfig | 8 +++ > drivers/gpu/drm/radeon/Kconfig | 21 +++++++ > drivers/gpu/drm/savage/Kconfig | 7 +++ > drivers/gpu/drm/sis/Kconfig | 8 +++ > drivers/gpu/drm/tdfx/Kconfig | 6 ++ > drivers/gpu/drm/via/Kconfig | 6 ++ > 10 files changed, 108 insertions(+), 99 deletions(-) > create mode 100644 drivers/gpu/drm/i810/Kconfig > create mode 100644 drivers/gpu/drm/mga/Kconfig > create mode 100644 drivers/gpu/drm/r128/Kconfig > create mode 100644 drivers/gpu/drm/savage/Kconfig > create mode 100644 drivers/gpu/drm/sis/Kconfig > create mode 100644 drivers/gpu/drm/tdfx/Kconfig > create mode 100644 drivers/gpu/drm/via/Kconfig Looking at the current state of the top-level Kconfig file, I think it'd be a good idea to sort it in some way once this series has been merged. There are a bunch of options that are interleaved in completely non- intuitive ways. I'm not sure what the best ordering would be, but I'd suggest to at least order them alphabetically. I think it'd also make sense to separate out panel and bridge drivers so that they are listed at the very bottom. Perhaps also splitting legacy drivers and modesetting drivers might be useful, though perhaps a little discriminatory... Thierry --+1TulI7fc0PCHNy3 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABCAAGBQJVpNPFAAoJEN0jrNd/PrOhh8wQAICOQFsJYgxk1V+ucMcS+PrY 3qFT9qq2eqXEMY37+gFPLQjEyTRqBg9WciPyVe+sqMIDJrfyDTocpMr9dlo5RuBv KSKIVPTa61lLX9Ox5TBwI32rYDSFjttfn64xveVRnGm6o7EsGjwrUg4NLx+swbdS nD2YjBJqvMbqUDfWpE2HNb7r62qCJmLNjG+vTyiM+jzTaJjhAXpN8JywZ8SLc5qd DxkW5Ot0sp1F9qJywLI8tlpzKFzGu7uBLYv9RGO0N3bgIaV/WYCopB04znR+IiQq OhGNJFV1NNmnOVUwNE1padsdJGHVRt8NlKZbGozBomiyGhenMc00g2juUIlAI5aa AfhBz0XJcadzbyWmaWGbENCkpuxG8bqiKL2pLeuFp7RZgBuUKcQ96smnSNCvamL2 H7jmj5zauFz/CHERQIiM6uiBd6np0sIEzwXv/Tf8aXlPORp6BJJUThCuDW8OUgRu Xp1Yg59BXtzPS/EpgvO9WiB3vPWTxaqjvlOKz1wGH4cn+RqNd1nmYBjPGxwpCSpx w2XXtW0hOum81rIjNATeXkF3GOoN8PTkzU7kemLqO7u+wwP8r5kDJcZCdetyMgkz 9QXtyIcKvbHu8xNgxN0c57SnrnBkPseU5+w/bUABnaIpOox/ibSmpxr34tZoesxQ Ltilwt+raiNnkWhlg2MG =1+T6 -----END PGP SIGNATURE----- --+1TulI7fc0PCHNy3-- --===============0604170759== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0 cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK --===============0604170759==--