From mboxrd@z Thu Jan 1 00:00:00 1970 From: Doug Goldstein Subject: Re: [PATCHv3] 00/28] Kconfig conversion Date: Fri, 13 Nov 2015 16:31:05 -0600 Message-ID: <564664A9.6020601@cardoe.com> References: <1447368869-17360-1-git-send-email-cardoe@cardoe.com> <5645CC74.6000509@citrix.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2603579111640145039==" Return-path: In-Reply-To: <5645CC74.6000509@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Julien Grall , xen-devel@lists.xen.org Cc: Ian Campbell , Stefano Stabellini List-Id: xen-devel@lists.xenproject.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --===============2603579111640145039== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="vOhXPap1POV4UGw5HofKKIccT5271UtsR" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --vOhXPap1POV4UGw5HofKKIccT5271UtsR Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 11/13/15 5:41 AM, Julien Grall wrote: > Hi Doug, >=20 > On 12/11/15 22:54, Doug Goldstein wrote: >> The following series is a follow on to the Kconfig conversion patch se= ries. >> There are still more components to convert however this is the bare mi= nimal >> to get everything working and get the options out of the existing make= files. >> >> The CONFIG_HAS_ variables are there to match the behavior of the Linux= >> CONFIG_HAVE_ variables. The purpose is to say that this hardware/profi= le/env >> supports this option while the CONFIG_ variable states that this optio= n was >> requested on/off by user intervention. >> >> Ultimately my goal is to allow for more parts of the hypervisor to be = turned >> off at compile time and potentially make it easier to include more >> experimental features by others which can be turned off by default. Al= so to >> provide the one true location for all possible knobs in the source cod= e. >> >> The patch series can be grabbed at: https://github.com/cardoe/xen.git >> The branch is: kconfig_v3 >> >> Changes since v2: >> - drop x86_32 support (patch 2) >> - fix make defconfig (patch 2) >> - fix 'make -C xen' vs 'cd xen && make' behaving differently (patch 2)= >> - fix for ARM64 builds (added patch 3) >> - At this point all targets are tested on x86_64, arm32, and arm64 wit= h >> fresh clones and rebuilds. >=20 > After this series, the resulting binary won't be the same. For instance= > on ARM64 all the UART drivers are disabled. >=20 > The user/test system should be able to get the same options enabled by > default with and without your series. >=20 > I wasn't able to find any documentation how to use your Kconfig with > Xen, so I've tried different things which don't work as I was expected.= >=20 > 1) If I modify myself xen/.config to remove/add an option, the config > won't be recheck and ignored > 2) make menuconfig doesn't expose any options =3D> No possibility to > select any UART on ARM. >=20 > The latter is because how you define the option in the Kconfig: >=20 > config HAS_EXYNOS4210 > bool >=20 > Without any text, it's not possible for the user to select this option.= >=20 > Regards, >=20 Ok. I'm also now understanding you previous hesitation with using HAS_. For my next update I'll make these available and I'm going to change HAS_ to UART_ to give them a common prefix. --=20 Doug Goldstein --vOhXPap1POV4UGw5HofKKIccT5271UtsR Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0 iQJ8BAEBCgBmBQJWRmSpXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXRBNTM5MEQ2RTNFMTkyNzlCNzVDMzIwOTVB MkJDMDNEQzg3RUQxQkQ0AAoJEKK8A9yH7RvUKv4QAJRE5OZW3B24CqPHt6r0/Yn3 /diFz1q2iyA7acHMZvoNugzS+wCsyQMxV1v8pNjlvMCDZne6M6+izFkWQOiTLbef AEgiswwd4OQLd+9hkpavM9A3ThLLdsUiv62kChAtCcbFPExupN6ssdgkVlR5iJvv bQzA5cAZw+q/Qw/wP91PqU52Y4A22+vqq0XeNNnV9AwxQqpQWTv+JO23EfMqxSmv 1fP/vslhMFb/qKAFcVEGumKq1e0GvkMVIKWPhX0RybuieBASeXfhu/fudzPQYipi PKrVHflogbQWDZ/YM9wyYbcb5U9AS4H5ygUwvCdLKpCVHC7UCMaEqCUEcgBVswjK 2lkTF/sDsN43wc2RsRBC8PKTqDGMyZ2GRb/m5EWidosPJ8ww9H8GkWYp/6kZZttw dYiCZ41c0pWYxU3LxDKgTTvQeXJwTCqoidq0JDzsv4R5tX+7gaCR598BtKRJI6C6 hesEHK2LqRJfOLcMiQcYZqrVq6bKniRgQguca7AhGYIP+RLwvjhpm356yLzM7EKb KnW9/mZqL+HRakdTpvGhwZIxzXfeLa8RZASpdU8I1NaGneqV9Z98wCQ3J8C831Hv lSko2mFWU8wZu5Fbpy4lo+/L2A/1jU1tcgxsyHPjnO4WeouOs9AUQG8VFRy5Qx6X LcUHD1FAoNBLtXOVmg7r =U73m -----END PGP SIGNATURE----- --vOhXPap1POV4UGw5HofKKIccT5271UtsR-- --===============2603579111640145039== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel --===============2603579111640145039==--