From mboxrd@z Thu Jan 1 00:00:00 1970 From: Doug Goldstein Subject: Re: [PATCHv6] 00/28] Kconfig conversion Date: Mon, 30 Nov 2015 11:05:11 -0600 Message-ID: <565C81C7.5030100@cardoe.com> References: <1448387538-12208-1-git-send-email-cardoe@cardoe.com> <565C7071.5040206@citrix.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============3292703404655695668==" Return-path: In-Reply-To: <565C7071.5040206@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) --===============3292703404655695668== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="AVVcKOp7ELPNm9lX79l2nNpoHqPGKnnrt" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --AVVcKOp7ELPNm9lX79l2nNpoHqPGKnnrt Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 11/30/15 9:51 AM, Julien Grall wrote: > Hi Doug, >=20 > On 24/11/15 17:51, 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. >> >> The UARTs are now uniformly prefixed as CONFIG_UART_ and dropping most= of the >> CONFIG_HAS_ labeling for them. This means they are now user selectable= as >> requested by Julien Grall in the prior review. The question I've got i= s >> the old config was just for selecting defaults. Users could enable the= OMAP >> UART for arm64 for example but I'm not sure if that's valid. Currently= this >> patchset makes those UARTs not user selectable if they were not previo= usly >> defaulted on. But I would like some feedback on this if possible. >=20 > While you know allow the user to enable a specific UART drivers, I stil= l > have the same concern, i.e the default config used for ARM32 and ARM64 > will have all the UARTs drivers disabled. Have you actually tested this? I have built it multiple times locally and that's not the case at all. The same UART drivers that are enabled in master today and enabled by default everytime. >=20 > This means that when your series will be pushed, all the actual users o= f > ARM (included automated test suite such as osstest) will have to write > down a config in order to get Xen boot again on their board. >=20 >> >> 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/tree/kconfig_v6 >=20 > I can't find the kconfig_v6 branch on your github. >=20 Sorry about that. Pushed. It was the same as kconfig_v5 just rebased up to a more recent version of staging. --=20 Doug Goldstein --AVVcKOp7ELPNm9lX79l2nNpoHqPGKnnrt 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 iQJ8BAEBCgBmBQJWXIHHXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXRBNTM5MEQ2RTNFMTkyNzlCNzVDMzIwOTVB MkJDMDNEQzg3RUQxQkQ0AAoJEKK8A9yH7RvU/YcP/0tYcgl+VmFeLtad0JFgfS1f 76mtef7ZV8QYg4Q1YnudrK6Oqg7LQB9mKgqMy7JNnmf34n9hLHOv0zanYnMnMrPE 3ao1ZAQNMxJzoq6RxfTck9W0nErbZonzx6d3QkighrDhKIV4tpXfqcWUnpTx15iP sFOpVrTLBcRl4qbUob/Em4QkdSM3XUuyFFjQ8k2Er8i3ggDsZRLLU4JKnkhxY3JT /ng1p1bkPKmo0dxJyhVB9bEH3ptwumy5icdC8R8ERO7noOTm7atewSoDYd/JhNrj pgRVZt69YPRi4tVBYE30LtlqHgS/njZWL8V968N3akw+ein4vxDRP6UVMNMtiizz MJsjclk2PdcHZU4wk9B6tbr2cZNhT+7baews/NOa+jvchKK1+ql3+ytQ9siwzE8P h+81IRF4J/UPc0rtsEx7n3SevzVDpmCIJfycB4wrLTHocYjVavDHrGlWGvut1NKu IOs5qfPcuiLnkBgtuXgtcs8xn1c2hU94v0lACUItZYN+jYbmnd0wZMac7GRoieZG Xwk5Fd7LGwxKONw3HatY4jolsBA+O0fqcG3hKiIFmb/FptXotrbsa/jOu6CfuOUv XScWg495QAhi7fbD6GMTnGIfQmro9nsnFEMkmsKRHQrlt2VnN1qNI/FD8aXSwKVY Q9jEShUOXQOfjb1iIuol =R5hO -----END PGP SIGNATURE----- --AVVcKOp7ELPNm9lX79l2nNpoHqPGKnnrt-- --===============3292703404655695668== 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 --===============3292703404655695668==--