Re: [PATCH 0/3] Kconfig symbol cleanup for v5.0-rc1

2019-01-14 Thread Paul Bolle
Laura,

Laura Abbott schreef op ma 14-01-2019 om 13:52 [-0800]:
> Nothing changed after running ./build_configs.sh so I went ahead
> and applied these to rawhide. It should show up in tomorrow's build.

Not sure what you mean here. But please note that my series of course made
sure that the 14 kernel_*.config files in the repo matched what was removed in
configs/*. As a proper series should.

The real test is whether the 14 kernel-5.0.0*.config files generated during
the build do not change. Ie, whether this series actually only drops stuff
that the kernel's build system dropped anyhow.

I'll hope to check tomorrow's build as soon as I can.


Paul Bolle
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org


Re: [PATCH] configs: add arm file for CONFIG_PCI_MESON

2019-01-14 Thread Laura Abbott

On 1/12/19 3:13 AM, Paul Bolle wrote:

Commit c9c254b74995 ("minor Arm config tweaks") set CONFIG_PCI_MESON to
'y' for all shipped arm and aarch64 .config files. It didn't add the
corrresponding change to the configuration generation directory. Do so
now.

Signed-off-by: Paul Bolle 
---
  configs/fedora/generic/arm/CONFIG_PCI_MESON | 1 +
  1 file changed, 1 insertion(+)
  create mode 100644 configs/fedora/generic/arm/CONFIG_PCI_MESON

diff --git a/configs/fedora/generic/arm/CONFIG_PCI_MESON 
b/configs/fedora/generic/arm/CONFIG_PCI_MESON
new file mode 100644
index ..2c5ba5ddf5f8
--- /dev/null
+++ b/configs/fedora/generic/arm/CONFIG_PCI_MESON
@@ -0,0 +1 @@
+CONFIG_PCI_MESON=y



Peter corrected this already.
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org


Re: [PATCH] configs: Correct four non-standard filenames

2019-01-14 Thread Laura Abbott

On 1/12/19 9:37 AM, Paul Bolle wrote:

These non-standard filenames have no effect on the build, but should
still be fixed since they could be confusing (to people and scripts).

Signed-off-by: Paul Bolle 
---
  ...FIG_ALTERA_PR_IP_CORE_PLAT=n => CONFIG_ALTERA_PR_IP_CORE_PLAT} | 0
  configs/fedora/generic/{CONFIG_B53_SERDES=n => CONFIG_B53_SERDES} | 0
  .../{CONFIG_FPGA_MGR_ZYNQ_FPGA=n => CONFIG_FPGA_MGR_ZYNQ_FPGA}| 0
  .../x86/{CONFIG_SENSORS_APPLESMC=m => CONFIG_SENSORS_APPLESMC}| 0
  4 files changed, 0 insertions(+), 0 deletions(-)
  rename configs/fedora/generic/{CONFIG_ALTERA_PR_IP_CORE_PLAT=n => 
CONFIG_ALTERA_PR_IP_CORE_PLAT} (100%)
  rename configs/fedora/generic/{CONFIG_B53_SERDES=n => CONFIG_B53_SERDES} 
(100%)
  rename configs/fedora/generic/{CONFIG_FPGA_MGR_ZYNQ_FPGA=n => 
CONFIG_FPGA_MGR_ZYNQ_FPGA} (100%)
  rename configs/fedora/generic/x86/{CONFIG_SENSORS_APPLESMC=m => 
CONFIG_SENSORS_APPLESMC} (100%)

diff --git a/configs/fedora/generic/CONFIG_ALTERA_PR_IP_CORE_PLAT=n 
b/configs/fedora/generic/CONFIG_ALTERA_PR_IP_CORE_PLAT
similarity index 100%
rename from configs/fedora/generic/CONFIG_ALTERA_PR_IP_CORE_PLAT=n
rename to configs/fedora/generic/CONFIG_ALTERA_PR_IP_CORE_PLAT
diff --git a/configs/fedora/generic/CONFIG_B53_SERDES=n 
b/configs/fedora/generic/CONFIG_B53_SERDES
similarity index 100%
rename from configs/fedora/generic/CONFIG_B53_SERDES=n
rename to configs/fedora/generic/CONFIG_B53_SERDES
diff --git a/configs/fedora/generic/CONFIG_FPGA_MGR_ZYNQ_FPGA=n 
b/configs/fedora/generic/CONFIG_FPGA_MGR_ZYNQ_FPGA
similarity index 100%
rename from configs/fedora/generic/CONFIG_FPGA_MGR_ZYNQ_FPGA=n
rename to configs/fedora/generic/CONFIG_FPGA_MGR_ZYNQ_FPGA
diff --git a/configs/fedora/generic/x86/CONFIG_SENSORS_APPLESMC=m 
b/configs/fedora/generic/x86/CONFIG_SENSORS_APPLESMC
similarity index 100%
rename from configs/fedora/generic/x86/CONFIG_SENSORS_APPLESMC=m
rename to configs/fedora/generic/x86/CONFIG_SENSORS_APPLESMC



Applied. Thanks again for the cleanup!
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org


Re: [PATCH 0/3] Kconfig symbol cleanup for v5.0-rc1

2019-01-14 Thread Laura Abbott

On 1/11/19 6:55 AM, Paul Bolle wrote:

This short series cleans up all references to Kconfig symbols that have
no effect on the build.

This series is way too boring to review. But it can be tested by
comparing the generated .config files (in the BUILD directory) before
and after this series. These generated .config files should not change.
Unless I made a mistake, that is.

Anyhow, review and/or comments are, as always, appreciated. A build
run on Fedora's build farm would be nice too. (I don't have access to
that system.) But if I'm right that build should show no differences
whatsoever.



Nothing changed after running ./build_configs.sh so I went ahead
and applied these to rawhide. It should show up in tomorrow's build.
Thanks for taking the time to clean this up.

Laura
___
kernel mailing list -- kernel@lists.fedoraproject.org
To unsubscribe send an email to kernel-le...@lists.fedoraproject.org
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/kernel@lists.fedoraproject.org