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:19:23 +0200 Message-ID: <20150714091922.GH12465@ulmo.nvidia.com> References: <1436769393-3836-1-git-send-email-architt@codeaurora.org> <20150714091801.GG12465@ulmo.nvidia.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="gTtJ75FAzB1T2CN6" Return-path: Received: from mail-pa0-f46.google.com ([209.85.220.46]:36094 "EHLO mail-pa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751209AbbGNJTa (ORCPT ); Tue, 14 Jul 2015 05:19:30 -0400 Received: by pachj5 with SMTP id hj5so2739290pac.3 for ; Tue, 14 Jul 2015 02:19:29 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20150714091801.GG12465@ulmo.nvidia.com> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Archit Taneja Cc: dri-devel@lists.freedesktop.org, daniel@ffwll.ch, linux-arm-msm@vger.kernel.org --gTtJ75FAzB1T2CN6 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jul 14, 2015 at 11:18:02AM +0200, Thierry Reding wrote: > 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 >=20 > 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. >=20 > 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. >=20 > Perhaps also splitting legacy drivers and modesetting drivers might be > useful, though perhaps a little discriminatory... I ended up patching things while writing this, so I'll just wait until these patches have landed, rebase on top of them and send out a proposal. Thierry --gTtJ75FAzB1T2CN6 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABCAAGBQJVpNQaAAoJEN0jrNd/PrOhswsP/ja3uyJ3d7N+3zuFuZ4d09JR PV0WIKVG+FUWM0qRwmThTv2LFo9aOxiEimwFGU+DWnu/ylu5wFAa0zW9Mih2yKoV DWU/hPxpPL+LrpR2jGU2z8yhN/3iATlzXguH5z4JDsmjxpqxeGZRVw6Qxvm4ASBD BUUHH7bi1rj8+dmcSGE38815aO39Jut5FUiy24hRvA+okVFXi318/vKQcawFkk67 0KkC2nGF6IVdMUX1h0sKvUJ2OjvrgGADNyCa+cNGiBMDSGUWwGe5mThWkD3JFXmy kHR5nvtfi3DQwmre7G52eYG6sqvwXfW4YQke44zZapnB+CB6/IK//PEiFTo1gy+s GiEY4fCQGhAISDC6KwfFpRy3Ts8lcGKYW2S2LA7xFkIvgMtGTMk2qJkhm8F9XbtQ skrMWCYaPJoCzB1/jzCNMujEWilOWg8TTnPC++xWVI7CKpN4oiwKcHBde3VoRAET 0NBZ0eQtP7tUc6VJ70ZUXbi+HhSyb4oiBdUqewpuSwhzppKBO6ubSIoyUfCJnxTX NThrvfDFUs9CZNIGokJeFDGDe+X2alVdbp48Gs7FChxommkbrxYN4F1BcZ3CAJyK FVU8xMAa7WANJ065uSE+91TZ1Ohg5ILDAq1ckdk8wJsP35gXEx2e4FOARm6ZOuB6 NDptO5o24/hS9LYibmIx =Ceyn -----END PGP SIGNATURE----- --gTtJ75FAzB1T2CN6--