[PATCH 0/5] [SRU][B/master] Reduce boot noise on Dragonboard

classic Classic list List threaded Threaded
12 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH 0/5] [SRU][B/master] Reduce boot noise on Dragonboard

Paolo Pisati-5
BugLink: https://bugs.launchpad.net/bugs/1797154

Impact:

During boot on a Dragonboard410c using a recent Bionic or Cosmic kernel,
there's some noise like:

...
[ 3.665431] msm_dsi_phy 1a98300.dsi-phy: Failed to get supply 'vddio':
-517
[ 3.669243] msm_dsi_phy 1a98300.dsi-phy: dsi_phy_regulator_init: failed to
init regulator, ret=-517
[ 3.676180] msm_dsi_phy 1a98300.dsi-phy: dsi_phy_driver_probe:
failed to init regulator
...

or

...
[ 4.192280] qcom,pm8916-wcd-spmi-codec 200f000.spmi:pm8916@1:codec@f000:
Failed to get supply 'vdd-cdc-io': -517
[ 4.195660] qcom,pm8916-wcd-spmi-codec 200f000.spmi:pm8916@1:codec@f000:
Failed to get regulator supplies -517
...

and similar.

This is mainly due to DRM_MSM* and SND_QCOM_MSM* that were built-in, and try to
initialize before the voltage regulators are ready to work, resulting in a
noisy bootlog.

These patches factor out some code as modules, so when the kmods are loaded and
initialized, regulators are already present and ready to attach - all these
config changes shave around ~3K bytes from dmesg, making the bootlog easir to
parse.

Fix:

Apply the attached patches and recompile

How to test:

Boot a patched kernel and check for the above noise in dmesg

Regression potential:

None, we are just turning some code into kmods, and turning off a couple of not
used feature for this flavour - all the other flavours are untouched.

Paolo Pisati (5):
  UBUNTU: [Config] arm64: snapdragon: DRM_MSM=m
  UBUNTU: [Config] arm64: snapdragon: SND*=m
  UBUNTU: [Config] arm64: snapdragon: disable ARM_SDE_INTERFACE
  UBUNTU: [Config] arm64: snapdragon: disable DRM_I2C_ADV7511_CEC
  UBUNTU: [Config] arm64: snapdragon: disable VIDEO_ADV7511,
    VIDEO_COBALT

 debian.master/config/amd64/config.common.amd64     |  6 ++----
 debian.master/config/arm64/config.common.arm64     |  7 +++++++
 debian.master/config/arm64/config.flavour.generic  | 22 ++++------------------
 .../config/arm64/config.flavour.snapdragon         | 22 ++++------------------
 debian.master/config/armhf/config.common.armhf     | 14 +++-----------
 debian.master/config/config.common.ubuntu          | 16 +++++++++++-----
 debian.master/config/i386/config.common.i386       |  6 ++----
 debian.master/config/ppc64el/config.common.ppc64el |  7 +++----
 8 files changed, 36 insertions(+), 64 deletions(-)

--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[PATCH 1/5] UBUNTU: [Config] arm64: snapdragon: DRM_MSM=m

Paolo Pisati-5
BugLink: https://bugs.launchpad.net/bugs/1797154

Signed-off-by: Paolo Pisati <[hidden email]>
---
 debian.master/config/arm64/config.flavour.generic    | 2 --
 debian.master/config/arm64/config.flavour.snapdragon | 2 --
 debian.master/config/armhf/config.common.armhf       | 2 --
 debian.master/config/config.common.ubuntu            | 2 ++
 4 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/debian.master/config/arm64/config.flavour.generic b/debian.master/config/arm64/config.flavour.generic
index b22966a..c5d0543 100644
--- a/debian.master/config/arm64/config.flavour.generic
+++ b/debian.master/config/arm64/config.flavour.generic
@@ -8,7 +8,6 @@ CONFIG_COMMON_CLK_QCOM=m
 CONFIG_DRM=m
 CONFIG_DRM_I2C_ADV7511=m
 CONFIG_DRM_KMS_HELPER=m
-CONFIG_DRM_MSM=m
 CONFIG_FB_SYS_COPYAREA=m
 CONFIG_FB_SYS_FILLRECT=m
 CONFIG_FB_SYS_FOPS=m
@@ -34,7 +33,6 @@ CONFIG_QCOM_CLK_SMD_RPM=m
 # CONFIG_QCOM_CPR is not set
 CONFIG_QCOM_HIDMA=m
 CONFIG_QCOM_HIDMA_MGMT=m
-CONFIG_QCOM_MDT_LOADER=m
 CONFIG_QCOM_QFPROM=m
 CONFIG_QCOM_SMD_RPM=m
 CONFIG_QCOM_SMEM=m
diff --git a/debian.master/config/arm64/config.flavour.snapdragon b/debian.master/config/arm64/config.flavour.snapdragon
index 28179ca..56b7a6a 100644
--- a/debian.master/config/arm64/config.flavour.snapdragon
+++ b/debian.master/config/arm64/config.flavour.snapdragon
@@ -8,7 +8,6 @@ CONFIG_COMMON_CLK_QCOM=y
 CONFIG_DRM=y
 CONFIG_DRM_I2C_ADV7511=y
 CONFIG_DRM_KMS_HELPER=y
-CONFIG_DRM_MSM=y
 CONFIG_FB_SYS_COPYAREA=y
 CONFIG_FB_SYS_FILLRECT=y
 CONFIG_FB_SYS_FOPS=y
@@ -34,7 +33,6 @@ CONFIG_QCOM_CLK_SMD_RPM=y
 CONFIG_QCOM_CPR=y
 CONFIG_QCOM_HIDMA=y
 CONFIG_QCOM_HIDMA_MGMT=y
-CONFIG_QCOM_MDT_LOADER=y
 CONFIG_QCOM_QFPROM=y
 CONFIG_QCOM_SMD_RPM=y
 CONFIG_QCOM_SMEM=y
diff --git a/debian.master/config/armhf/config.common.armhf b/debian.master/config/armhf/config.common.armhf
index b7530fa..a70ee77 100644
--- a/debian.master/config/armhf/config.common.armhf
+++ b/debian.master/config/armhf/config.common.armhf
@@ -93,7 +93,6 @@ CONFIG_DNET=m
 CONFIG_DRM=m
 CONFIG_DRM_I2C_ADV7511=m
 CONFIG_DRM_KMS_HELPER=m
-CONFIG_DRM_MSM=m
 CONFIG_DUMMY_IRQ=m
 CONFIG_DW_WATCHDOG=m
 CONFIG_ECHO=m
@@ -363,7 +362,6 @@ CONFIG_QCOM_CLK_SMD_RPM=m
 # CONFIG_QCOM_CPR is not set
 CONFIG_QCOM_HIDMA=m
 CONFIG_QCOM_HIDMA_MGMT=m
-CONFIG_QCOM_MDT_LOADER=m
 CONFIG_QCOM_QFPROM=m
 CONFIG_QCOM_SMD_RPM=m
 CONFIG_QCOM_SMEM=m
diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
index 49e10e4..540a5c5 100644
--- a/debian.master/config/config.common.ubuntu
+++ b/debian.master/config/config.common.ubuntu
@@ -2321,6 +2321,7 @@ CONFIG_DRM_MESON=m
 CONFIG_DRM_MESON_DW_HDMI=m
 CONFIG_DRM_MGAG200=m
 CONFIG_DRM_MIPI_DSI=y
+CONFIG_DRM_MSM=m
 CONFIG_DRM_MSM_DSI=y
 CONFIG_DRM_MSM_DSI_14NM_PHY=y
 CONFIG_DRM_MSM_DSI_20NM_PHY=y
@@ -7026,6 +7027,7 @@ CONFIG_QCOM_IOMMU=y
 CONFIG_QCOM_IRQ_COMBINER=y
 CONFIG_QCOM_L2_PMU=y
 CONFIG_QCOM_L3_PMU=y
+CONFIG_QCOM_MDT_LOADER=m
 CONFIG_QCOM_PM=y
 CONFIG_QCOM_PM8XXX_XOADC=m
 # CONFIG_QCOM_Q6V5_PIL is not set
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[PATCH 2/5] UBUNTU: [Config] arm64: snapdragon: SND*=m

Paolo Pisati-5
In reply to this post by Paolo Pisati-5
BugLink: https://bugs.launchpad.net/bugs/1797154

Signed-off-by: Paolo Pisati <[hidden email]>
---
 debian.master/config/amd64/config.common.amd64       |  3 ---
 debian.master/config/arm64/config.common.arm64       |  7 +++++++
 debian.master/config/arm64/config.flavour.generic    | 15 ---------------
 debian.master/config/arm64/config.flavour.snapdragon | 15 ---------------
 debian.master/config/armhf/config.common.armhf       |  8 --------
 debian.master/config/config.common.ubuntu            |  8 ++++++++
 debian.master/config/i386/config.common.i386         |  3 ---
 debian.master/config/ppc64el/config.common.ppc64el   |  3 ---
 8 files changed, 15 insertions(+), 47 deletions(-)

diff --git a/debian.master/config/amd64/config.common.amd64 b/debian.master/config/amd64/config.common.amd64
index 06bd75a..db1ba8e 100644
--- a/debian.master/config/amd64/config.common.amd64
+++ b/debian.master/config/amd64/config.common.amd64
@@ -455,11 +455,8 @@ CONFIG_SND_EMU10K1_SEQ=m
 CONFIG_SND_PCM=m
 CONFIG_SND_SOC=m
 CONFIG_SND_SOC_FSL_SSI=m
-CONFIG_SND_SOC_HDMI_CODEC=m
 CONFIG_SND_SOC_I2C_AND_SPI=m
 CONFIG_SND_SOC_IMX_AUDMUX=m
-CONFIG_SND_SOC_MSM8916_WCD_ANALOG=m
-CONFIG_SND_SOC_MSM8916_WCD_DIGITAL=m
 CONFIG_SND_SOC_RT5677_SPI=m
 CONFIG_SND_SOC_SGTL5000=m
 CONFIG_SND_TIMER=m
diff --git a/debian.master/config/arm64/config.common.arm64 b/debian.master/config/arm64/config.common.arm64
index 67f8bae..5cd036b 100644
--- a/debian.master/config/arm64/config.common.arm64
+++ b/debian.master/config/arm64/config.common.arm64
@@ -4,6 +4,7 @@
 CONFIG_64BIT=y
 CONFIG_6LOWPAN=m
 CONFIG_ABX500_CORE=y
+CONFIG_AC97_BUS=m
 # CONFIG_ACPI_DEBUG is not set
 # CONFIG_ACPI_DEBUGGER is not set
 CONFIG_ACPI_REDUCED_HARDWARE_ONLY=y
@@ -447,12 +448,18 @@ CONFIG_SGI_PARTITION=y
 # CONFIG_SH_TIMER_TMU is not set
 CONFIG_SLIP=m
 CONFIG_SMC91X=y
+CONFIG_SND=m
+CONFIG_SND_DMAENGINE_PCM=m
 CONFIG_SND_EMU10K1_SEQ=m
 # CONFIG_SND_OPL4_LIB_SEQ is not set
+CONFIG_SND_PCM=m
+CONFIG_SND_SOC=m
 CONFIG_SND_SOC_FSL_SSI=m
+CONFIG_SND_SOC_I2C_AND_SPI=m
 CONFIG_SND_SOC_IMX_AUDMUX=m
 # CONFIG_SND_SOC_RT5677_SPI is not set
 CONFIG_SND_SOC_SGTL5000=m
+CONFIG_SND_TIMER=m
 CONFIG_SOC_BRCMSTB=y
 CONFIG_SOC_TI=y
 CONFIG_SOLARIS_X86_PARTITION=y
diff --git a/debian.master/config/arm64/config.flavour.generic b/debian.master/config/arm64/config.flavour.generic
index c5d0543..20a3fbc 100644
--- a/debian.master/config/arm64/config.flavour.generic
+++ b/debian.master/config/arm64/config.flavour.generic
@@ -1,7 +1,6 @@
 #
 # Config options for config.flavour.generic automatically generated by splitconfig.pl
 #
-CONFIG_AC97_BUS=m
 # CONFIG_ADV7511_SNAPDRAGON_HACKS is not set
 CONFIG_CEC_CORE=m
 CONFIG_COMMON_CLK_QCOM=m
@@ -47,20 +46,6 @@ CONFIG_REGULATOR_QCOM_SMD_RPM=m
 CONFIG_REGULATOR_QCOM_SPMI=m
 CONFIG_RPMSG=m
 CONFIG_RPMSG_QCOM_SMD=m
-CONFIG_SND=m
-CONFIG_SND_DMAENGINE_PCM=m
-CONFIG_SND_PCM=m
-CONFIG_SND_SOC=m
-CONFIG_SND_SOC_APQ8016_SBC=m
-CONFIG_SND_SOC_HDMI_CODEC=m
-CONFIG_SND_SOC_I2C_AND_SPI=m
-CONFIG_SND_SOC_LPASS_APQ8016=m
-CONFIG_SND_SOC_LPASS_CPU=m
-CONFIG_SND_SOC_LPASS_PLATFORM=m
-CONFIG_SND_SOC_MSM8916_WCD_ANALOG=m
-CONFIG_SND_SOC_MSM8916_WCD_DIGITAL=m
-CONFIG_SND_SOC_QCOM=m
-CONFIG_SND_TIMER=m
 CONFIG_SOUND=m
 CONFIG_SPI_QUP=m
 CONFIG_SPMI=m
diff --git a/debian.master/config/arm64/config.flavour.snapdragon b/debian.master/config/arm64/config.flavour.snapdragon
index 56b7a6a..7bb997b 100644
--- a/debian.master/config/arm64/config.flavour.snapdragon
+++ b/debian.master/config/arm64/config.flavour.snapdragon
@@ -1,7 +1,6 @@
 #
 # Config options for config.flavour.snapdragon automatically generated by splitconfig.pl
 #
-CONFIG_AC97_BUS=y
 CONFIG_ADV7511_SNAPDRAGON_HACKS=y
 CONFIG_CEC_CORE=y
 CONFIG_COMMON_CLK_QCOM=y
@@ -47,20 +46,6 @@ CONFIG_REGULATOR_QCOM_SMD_RPM=y
 CONFIG_REGULATOR_QCOM_SPMI=y
 CONFIG_RPMSG=y
 CONFIG_RPMSG_QCOM_SMD=y
-CONFIG_SND=y
-CONFIG_SND_DMAENGINE_PCM=y
-CONFIG_SND_PCM=y
-CONFIG_SND_SOC=y
-CONFIG_SND_SOC_APQ8016_SBC=y
-CONFIG_SND_SOC_HDMI_CODEC=y
-CONFIG_SND_SOC_I2C_AND_SPI=y
-CONFIG_SND_SOC_LPASS_APQ8016=y
-CONFIG_SND_SOC_LPASS_CPU=y
-CONFIG_SND_SOC_LPASS_PLATFORM=y
-CONFIG_SND_SOC_MSM8916_WCD_ANALOG=y
-CONFIG_SND_SOC_MSM8916_WCD_DIGITAL=y
-CONFIG_SND_SOC_QCOM=y
-CONFIG_SND_TIMER=y
 CONFIG_SOUND=y
 CONFIG_SPI_QUP=y
 CONFIG_SPMI=y
diff --git a/debian.master/config/armhf/config.common.armhf b/debian.master/config/armhf/config.common.armhf
index a70ee77..7f8f061 100644
--- a/debian.master/config/armhf/config.common.armhf
+++ b/debian.master/config/armhf/config.common.armhf
@@ -471,15 +471,7 @@ CONFIG_SND_DMAENGINE_PCM=y
 # CONFIG_SND_OPL4_LIB_SEQ is not set
 CONFIG_SND_PCM=y
 CONFIG_SND_SOC=y
-CONFIG_SND_SOC_APQ8016_SBC=m
-CONFIG_SND_SOC_HDMI_CODEC=m
 CONFIG_SND_SOC_I2C_AND_SPI=y
-CONFIG_SND_SOC_LPASS_APQ8016=m
-CONFIG_SND_SOC_LPASS_CPU=m
-CONFIG_SND_SOC_LPASS_PLATFORM=m
-CONFIG_SND_SOC_MSM8916_WCD_ANALOG=m
-CONFIG_SND_SOC_MSM8916_WCD_DIGITAL=m
-CONFIG_SND_SOC_QCOM=m
 CONFIG_SND_TIMER=y
 # CONFIG_SOC_BRCMSTB is not set
 CONFIG_SOC_TI=y
diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
index 540a5c5..d4756e3 100644
--- a/debian.master/config/config.common.ubuntu
+++ b/debian.master/config/config.common.ubuntu
@@ -8409,6 +8409,7 @@ CONFIG_SND_SOC_ALC5623=m
 CONFIG_SND_SOC_ALC5632=m
 CONFIG_SND_SOC_AMD_ACP=m
 CONFIG_SND_SOC_AMD_CZ_RT5645_MACH=m
+CONFIG_SND_SOC_APQ8016_SBC=m
 CONFIG_SND_SOC_ARIZONA=m
 CONFIG_SND_SOC_ARNDALE_RT5631_ALC5631=m
 CONFIG_SND_SOC_BT_SCO=m
@@ -8452,6 +8453,7 @@ CONFIG_SND_SOC_FSL_UTILS=m
 CONFIG_SND_SOC_GENERIC_DMAENGINE_PCM=y
 CONFIG_SND_SOC_GTM601=m
 CONFIG_SND_SOC_HDAC_HDMI=m
+CONFIG_SND_SOC_HDMI_CODEC=m
 CONFIG_SND_SOC_IMG=y
 CONFIG_SND_SOC_IMG_I2S_IN=m
 CONFIG_SND_SOC_IMG_I2S_OUT=m
@@ -8496,7 +8498,10 @@ CONFIG_SND_SOC_INTEL_SST=m
 CONFIG_SND_SOC_INTEL_SST_ACPI=m
 CONFIG_SND_SOC_INTEL_SST_FIRMWARE=m
 CONFIG_SND_SOC_INTEL_SST_TOPLEVEL=m
+CONFIG_SND_SOC_LPASS_APQ8016=m
+CONFIG_SND_SOC_LPASS_CPU=m
 CONFIG_SND_SOC_LPASS_IPQ806X=m
+CONFIG_SND_SOC_LPASS_PLATFORM=m
 CONFIG_SND_SOC_MAX98090=m
 CONFIG_SND_SOC_MAX98095=m
 CONFIG_SND_SOC_MAX98357A=m
@@ -8504,6 +8509,8 @@ CONFIG_SND_SOC_MAX98504=m
 CONFIG_SND_SOC_MAX9860=m
 CONFIG_SND_SOC_MAX98927=m
 CONFIG_SND_SOC_MC13783=m
+CONFIG_SND_SOC_MSM8916_WCD_ANALOG=m
+CONFIG_SND_SOC_MSM8916_WCD_DIGITAL=m
 # CONFIG_SND_SOC_MT2701 is not set
 # CONFIG_SND_SOC_MT8173 is not set
 CONFIG_SND_SOC_NAU8540=m
@@ -8521,6 +8528,7 @@ CONFIG_SND_SOC_PCM3168A_SPI=m
 CONFIG_SND_SOC_PCM512x=m
 CONFIG_SND_SOC_PCM512x_I2C=m
 CONFIG_SND_SOC_PCM512x_SPI=m
+CONFIG_SND_SOC_QCOM=m
 CONFIG_SND_SOC_RCAR=m
 CONFIG_SND_SOC_RK3288_HDMI_ANALOG=m
 CONFIG_SND_SOC_RK3399_GRU_SOUND=m
diff --git a/debian.master/config/i386/config.common.i386 b/debian.master/config/i386/config.common.i386
index 1a87045..1ed8561 100644
--- a/debian.master/config/i386/config.common.i386
+++ b/debian.master/config/i386/config.common.i386
@@ -447,11 +447,8 @@ CONFIG_SND_OPL4_LIB_SEQ=m
 CONFIG_SND_PCM=m
 CONFIG_SND_SOC=m
 CONFIG_SND_SOC_FSL_SSI=m
-CONFIG_SND_SOC_HDMI_CODEC=m
 CONFIG_SND_SOC_I2C_AND_SPI=m
 CONFIG_SND_SOC_IMX_AUDMUX=m
-CONFIG_SND_SOC_MSM8916_WCD_ANALOG=m
-CONFIG_SND_SOC_MSM8916_WCD_DIGITAL=m
 CONFIG_SND_SOC_RT5677_SPI=m
 CONFIG_SND_SOC_SGTL5000=m
 CONFIG_SND_TIMER=m
diff --git a/debian.master/config/ppc64el/config.common.ppc64el b/debian.master/config/ppc64el/config.common.ppc64el
index 4719378..7e50201 100644
--- a/debian.master/config/ppc64el/config.common.ppc64el
+++ b/debian.master/config/ppc64el/config.common.ppc64el
@@ -452,11 +452,8 @@ CONFIG_SND_EMU10K1_SEQ=m
 CONFIG_SND_PCM=m
 CONFIG_SND_SOC=m
 CONFIG_SND_SOC_FSL_SSI=m
-CONFIG_SND_SOC_HDMI_CODEC=m
 CONFIG_SND_SOC_I2C_AND_SPI=m
 CONFIG_SND_SOC_IMX_AUDMUX=m
-CONFIG_SND_SOC_MSM8916_WCD_ANALOG=m
-CONFIG_SND_SOC_MSM8916_WCD_DIGITAL=m
 # CONFIG_SND_SOC_RT5677_SPI is not set
 CONFIG_SND_SOC_SGTL5000=m
 CONFIG_SND_TIMER=m
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[PATCH 3/5] UBUNTU: [Config] arm64: snapdragon: disable ARM_SDE_INTERFACE

Paolo Pisati-5
In reply to this post by Paolo Pisati-5
BugLink: https://bugs.launchpad.net/bugs/1797154

Signed-off-by: Paolo Pisati <[hidden email]>
---
 debian.master/config/arm64/config.flavour.generic    | 1 +
 debian.master/config/arm64/config.flavour.snapdragon | 1 +
 debian.master/config/config.common.ubuntu            | 1 -
 3 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/debian.master/config/arm64/config.flavour.generic b/debian.master/config/arm64/config.flavour.generic
index 20a3fbc..58ab73d 100644
--- a/debian.master/config/arm64/config.flavour.generic
+++ b/debian.master/config/arm64/config.flavour.generic
@@ -2,6 +2,7 @@
 # Config options for config.flavour.generic automatically generated by splitconfig.pl
 #
 # CONFIG_ADV7511_SNAPDRAGON_HACKS is not set
+CONFIG_ARM_SDE_INTERFACE=y
 CONFIG_CEC_CORE=m
 CONFIG_COMMON_CLK_QCOM=m
 CONFIG_DRM=m
diff --git a/debian.master/config/arm64/config.flavour.snapdragon b/debian.master/config/arm64/config.flavour.snapdragon
index 7bb997b..74daeba 100644
--- a/debian.master/config/arm64/config.flavour.snapdragon
+++ b/debian.master/config/arm64/config.flavour.snapdragon
@@ -2,6 +2,7 @@
 # Config options for config.flavour.snapdragon automatically generated by splitconfig.pl
 #
 CONFIG_ADV7511_SNAPDRAGON_HACKS=y
+# CONFIG_ARM_SDE_INTERFACE is not set
 CONFIG_CEC_CORE=y
 CONFIG_COMMON_CLK_QCOM=y
 CONFIG_DRM=y
diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
index d4756e3..87b3667 100644
--- a/debian.master/config/config.common.ubuntu
+++ b/debian.master/config/config.common.ubuntu
@@ -653,7 +653,6 @@ CONFIG_ARM_SBSA_WATCHDOG=m
 CONFIG_ARM_SCPI_CPUFREQ=m
 CONFIG_ARM_SCPI_POWER_DOMAIN=m
 CONFIG_ARM_SCPI_PROTOCOL=m
-CONFIG_ARM_SDE_INTERFACE=y
 CONFIG_ARM_SMMU_V3=y
 CONFIG_ARM_SP805_WATCHDOG=m
 CONFIG_ARM_SPE_PMU=m
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[PATCH 4/5] UBUNTU: [Config] arm64: snapdragon: disable DRM_I2C_ADV7511_CEC

Paolo Pisati-5
In reply to this post by Paolo Pisati-5
BugLink: https://bugs.launchpad.net/bugs/1797154

Signed-off-by: Paolo Pisati <[hidden email]>
---
 debian.master/config/amd64/config.common.amd64       | 1 -
 debian.master/config/arm64/config.flavour.generic    | 2 +-
 debian.master/config/arm64/config.flavour.snapdragon | 2 +-
 debian.master/config/armhf/config.common.armhf       | 2 +-
 debian.master/config/config.common.ubuntu            | 2 +-
 debian.master/config/i386/config.common.i386         | 1 -
 debian.master/config/ppc64el/config.common.ppc64el   | 2 +-
 7 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/debian.master/config/amd64/config.common.amd64 b/debian.master/config/amd64/config.common.amd64
index db1ba8e..173b6f7 100644
--- a/debian.master/config/amd64/config.common.amd64
+++ b/debian.master/config/amd64/config.common.amd64
@@ -73,7 +73,6 @@ CONFIG_CAN=m
 CONFIG_CB710_CORE=m
 CONFIG_CC_STACKPROTECTOR=y
 CONFIG_CDROM_PKTCDVD=m
-CONFIG_CEC_CORE=m
 CONFIG_CHASH=m
 CONFIG_CMDLINE_PARTITION=y
 CONFIG_CRAMFS=m
diff --git a/debian.master/config/arm64/config.flavour.generic b/debian.master/config/arm64/config.flavour.generic
index 58ab73d..c09c6c6 100644
--- a/debian.master/config/arm64/config.flavour.generic
+++ b/debian.master/config/arm64/config.flavour.generic
@@ -3,10 +3,10 @@
 #
 # CONFIG_ADV7511_SNAPDRAGON_HACKS is not set
 CONFIG_ARM_SDE_INTERFACE=y
-CONFIG_CEC_CORE=m
 CONFIG_COMMON_CLK_QCOM=m
 CONFIG_DRM=m
 CONFIG_DRM_I2C_ADV7511=m
+CONFIG_DRM_I2C_ADV7511_CEC=y
 CONFIG_DRM_KMS_HELPER=m
 CONFIG_FB_SYS_COPYAREA=m
 CONFIG_FB_SYS_FILLRECT=m
diff --git a/debian.master/config/arm64/config.flavour.snapdragon b/debian.master/config/arm64/config.flavour.snapdragon
index 74daeba..30c744b 100644
--- a/debian.master/config/arm64/config.flavour.snapdragon
+++ b/debian.master/config/arm64/config.flavour.snapdragon
@@ -3,10 +3,10 @@
 #
 CONFIG_ADV7511_SNAPDRAGON_HACKS=y
 # CONFIG_ARM_SDE_INTERFACE is not set
-CONFIG_CEC_CORE=y
 CONFIG_COMMON_CLK_QCOM=y
 CONFIG_DRM=y
 CONFIG_DRM_I2C_ADV7511=y
+# CONFIG_DRM_I2C_ADV7511_CEC is not set
 CONFIG_DRM_KMS_HELPER=y
 CONFIG_FB_SYS_COPYAREA=y
 CONFIG_FB_SYS_FILLRECT=y
diff --git a/debian.master/config/armhf/config.common.armhf b/debian.master/config/armhf/config.common.armhf
index 7f8f061..7815e0e 100644
--- a/debian.master/config/armhf/config.common.armhf
+++ b/debian.master/config/armhf/config.common.armhf
@@ -71,7 +71,6 @@ CONFIG_CAN=m
 CONFIG_CB710_CORE=m
 CONFIG_CC_STACKPROTECTOR=y
 CONFIG_CDROM_PKTCDVD=m
-CONFIG_CEC_CORE=m
 CONFIG_CHASH=m
 CONFIG_CMDLINE=""
 CONFIG_CMDLINE_PARTITION=y
@@ -92,6 +91,7 @@ CONFIG_DMA_VIRTUAL_CHANNELS=y
 CONFIG_DNET=m
 CONFIG_DRM=m
 CONFIG_DRM_I2C_ADV7511=m
+CONFIG_DRM_I2C_ADV7511_CEC=y
 CONFIG_DRM_KMS_HELPER=m
 CONFIG_DUMMY_IRQ=m
 CONFIG_DW_WATCHDOG=m
diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
index 87b3667..3c6c15df 100644
--- a/debian.master/config/config.common.ubuntu
+++ b/debian.master/config/config.common.ubuntu
@@ -1266,6 +1266,7 @@ CONFIG_CC_STACKPROTECTOR_STRONG=y
 CONFIG_CDROM=y
 CONFIG_CDROM_PKTCDVD_BUFFERS=8
 # CONFIG_CDROM_PKTCDVD_WCACHE is not set
+CONFIG_CEC_CORE=m
 CONFIG_CEC_GPIO=m
 CONFIG_CEC_NOTIFIER=y
 CONFIG_CEC_PIN=y
@@ -2281,7 +2282,6 @@ CONFIG_DRM_HDLCD=m
 CONFIG_DRM_HISI_HIBMC=m
 CONFIG_DRM_HISI_KIRIN=m
 CONFIG_DRM_I2C_ADV7511_AUDIO=y
-CONFIG_DRM_I2C_ADV7511_CEC=y
 CONFIG_DRM_I2C_ADV7533=y
 CONFIG_DRM_I2C_CH7006=m
 CONFIG_DRM_I2C_NXP_TDA998X=m
diff --git a/debian.master/config/i386/config.common.i386 b/debian.master/config/i386/config.common.i386
index 1ed8561..0c9330e 100644
--- a/debian.master/config/i386/config.common.i386
+++ b/debian.master/config/i386/config.common.i386
@@ -70,7 +70,6 @@ CONFIG_CAN=m
 CONFIG_CB710_CORE=m
 CONFIG_CC_STACKPROTECTOR=y
 CONFIG_CDROM_PKTCDVD=m
-CONFIG_CEC_CORE=m
 CONFIG_CHASH=m
 CONFIG_CMDLINE_PARTITION=y
 CONFIG_CRAMFS=m
diff --git a/debian.master/config/ppc64el/config.common.ppc64el b/debian.master/config/ppc64el/config.common.ppc64el
index 7e50201..918f2bc 100644
--- a/debian.master/config/ppc64el/config.common.ppc64el
+++ b/debian.master/config/ppc64el/config.common.ppc64el
@@ -68,7 +68,6 @@ CONFIG_CAN=m
 CONFIG_CB710_CORE=m
 # CONFIG_CC_STACKPROTECTOR is not set
 CONFIG_CDROM_PKTCDVD=m
-CONFIG_CEC_CORE=m
 CONFIG_CHASH=m
 CONFIG_CMDLINE_PARTITION=y
 CONFIG_CMM=m
@@ -89,6 +88,7 @@ CONFIG_DMA_VIRTUAL_CHANNELS=m
 CONFIG_DNET=m
 CONFIG_DRM=m
 CONFIG_DRM_I2C_ADV7511=m
+CONFIG_DRM_I2C_ADV7511_CEC=y
 CONFIG_DRM_KMS_HELPER=m
 CONFIG_DUMMY_IRQ=m
 CONFIG_DW_WATCHDOG=m
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

[PATCH 5/5] UBUNTU: [Config] arm64: snapdragon: disable VIDEO_ADV7511, VIDEO_COBALT

Paolo Pisati-5
In reply to this post by Paolo Pisati-5
BugLink: https://bugs.launchpad.net/bugs/1797154

Signed-off-by: Paolo Pisati <[hidden email]>
---
 debian.master/config/amd64/config.common.amd64       | 2 ++
 debian.master/config/arm64/config.flavour.generic    | 2 ++
 debian.master/config/arm64/config.flavour.snapdragon | 2 ++
 debian.master/config/armhf/config.common.armhf       | 2 ++
 debian.master/config/config.common.ubuntu            | 3 ---
 debian.master/config/i386/config.common.i386         | 2 ++
 debian.master/config/ppc64el/config.common.ppc64el   | 2 ++
 7 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/debian.master/config/amd64/config.common.amd64 b/debian.master/config/amd64/config.common.amd64
index 173b6f7..042d09e 100644
--- a/debian.master/config/amd64/config.common.amd64
+++ b/debian.master/config/amd64/config.common.amd64
@@ -514,6 +514,8 @@ CONFIG_VFIO_IOMMU_TYPE1=m
 CONFIG_VFIO_PCI=m
 CONFIG_VFIO_VIRQFD=m
 CONFIG_VGASTATE=m
+CONFIG_VIDEO_ADV7511=m
+CONFIG_VIDEO_COBALT=m
 CONFIG_VIRTIO_MMIO=y
 # CONFIG_VM86 is not set
 CONFIG_VME_BUS=y
diff --git a/debian.master/config/arm64/config.flavour.generic b/debian.master/config/arm64/config.flavour.generic
index c09c6c6..4f77482 100644
--- a/debian.master/config/arm64/config.flavour.generic
+++ b/debian.master/config/arm64/config.flavour.generic
@@ -52,4 +52,6 @@ CONFIG_SPI_QUP=m
 CONFIG_SPMI=m
 CONFIG_SPMI_MSM_PMIC_ARB=m
 CONFIG_USB_ULPI_BUS=m
+CONFIG_VIDEO_ADV7511=m
+CONFIG_VIDEO_COBALT=m
 # CONFIG_WCN36XX_SNAPDRAGON_HACKS is not set
diff --git a/debian.master/config/arm64/config.flavour.snapdragon b/debian.master/config/arm64/config.flavour.snapdragon
index 30c744b..dfb04bd 100644
--- a/debian.master/config/arm64/config.flavour.snapdragon
+++ b/debian.master/config/arm64/config.flavour.snapdragon
@@ -52,4 +52,6 @@ CONFIG_SPI_QUP=y
 CONFIG_SPMI=y
 CONFIG_SPMI_MSM_PMIC_ARB=y
 CONFIG_USB_ULPI_BUS=y
+# CONFIG_VIDEO_ADV7511 is not set
+# CONFIG_VIDEO_COBALT is not set
 CONFIG_WCN36XX_SNAPDRAGON_HACKS=y
diff --git a/debian.master/config/armhf/config.common.armhf b/debian.master/config/armhf/config.common.armhf
index 7815e0e..30abd24 100644
--- a/debian.master/config/armhf/config.common.armhf
+++ b/debian.master/config/armhf/config.common.armhf
@@ -531,6 +531,8 @@ CONFIG_VFIO=m
 CONFIG_VFIO_PCI=m
 CONFIG_VFIO_VIRQFD=m
 CONFIG_VGASTATE=m
+CONFIG_VIDEO_ADV7511=m
+CONFIG_VIDEO_COBALT=m
 CONFIG_VIRTIO_MMIO=y
 CONFIG_VME_BUS=y
 CONFIG_VMXNET3=m
diff --git a/debian.master/config/config.common.ubuntu b/debian.master/config/config.common.ubuntu
index 3c6c15df..50eeef55 100644
--- a/debian.master/config/config.common.ubuntu
+++ b/debian.master/config/config.common.ubuntu
@@ -9943,8 +9943,6 @@ CONFIG_VIDEO_ADV7175=m
 # CONFIG_VIDEO_ADV7343 is not set
 # CONFIG_VIDEO_ADV7393 is not set
 # CONFIG_VIDEO_ADV748X is not set
-CONFIG_VIDEO_ADV7511=m
-# CONFIG_VIDEO_ADV7511_CEC is not set
 CONFIG_VIDEO_ADV7604=m
 # CONFIG_VIDEO_ADV7604_CEC is not set
 CONFIG_VIDEO_ADV7842=m
@@ -9961,7 +9959,6 @@ CONFIG_VIDEO_BT848=m
 CONFIG_VIDEO_BT856=m
 CONFIG_VIDEO_BT866=m
 CONFIG_VIDEO_CAFE_CCIC=m
-CONFIG_VIDEO_COBALT=m
 CONFIG_VIDEO_CODA=m
 CONFIG_VIDEO_CPIA2=m
 CONFIG_VIDEO_CS3308=m
diff --git a/debian.master/config/i386/config.common.i386 b/debian.master/config/i386/config.common.i386
index 0c9330e..6029c21 100644
--- a/debian.master/config/i386/config.common.i386
+++ b/debian.master/config/i386/config.common.i386
@@ -506,6 +506,8 @@ CONFIG_VFIO_IOMMU_TYPE1=m
 CONFIG_VFIO_PCI=m
 CONFIG_VFIO_VIRQFD=m
 CONFIG_VGASTATE=m
+CONFIG_VIDEO_ADV7511=m
+CONFIG_VIDEO_COBALT=m
 CONFIG_VIRTIO_MMIO=y
 CONFIG_VM86=y
 CONFIG_VME_BUS=y
diff --git a/debian.master/config/ppc64el/config.common.ppc64el b/debian.master/config/ppc64el/config.common.ppc64el
index 918f2bc..21bdd55 100644
--- a/debian.master/config/ppc64el/config.common.ppc64el
+++ b/debian.master/config/ppc64el/config.common.ppc64el
@@ -512,6 +512,8 @@ CONFIG_VFIO=y
 CONFIG_VFIO_PCI=y
 CONFIG_VFIO_VIRQFD=y
 CONFIG_VGASTATE=m
+CONFIG_VIDEO_ADV7511=m
+CONFIG_VIDEO_COBALT=m
 CONFIG_VIRTIO_MMIO=y
 # CONFIG_VIRT_CPU_ACCOUNTING_NATIVE is not set
 CONFIG_VME_BUS=y
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH 0/5] [SRU][B/master] Reduce boot noise on Dragonboard

Stefan Bader-2
In reply to this post by Paolo Pisati-5
On 10.10.2018 17:32, Paolo Pisati wrote:

> BugLink: https://bugs.launchpad.net/bugs/1797154
>
> Impact:
>
> During boot on a Dragonboard410c using a recent Bionic or Cosmic kernel,
> there's some noise like:
>
> ...
> [ 3.665431] msm_dsi_phy 1a98300.dsi-phy: Failed to get supply 'vddio':
> -517
> [ 3.669243] msm_dsi_phy 1a98300.dsi-phy: dsi_phy_regulator_init: failed to
> init regulator, ret=-517
> [ 3.676180] msm_dsi_phy 1a98300.dsi-phy: dsi_phy_driver_probe:
> failed to init regulator
> ...
>
> or
>
> ...
> [ 4.192280] qcom,pm8916-wcd-spmi-codec 200f000.spmi:pm8916@1:codec@f000:
> Failed to get supply 'vdd-cdc-io': -517
> [ 4.195660] qcom,pm8916-wcd-spmi-codec 200f000.spmi:pm8916@1:codec@f000:
> Failed to get regulator supplies -517
> ...
>
> and similar.
>
> This is mainly due to DRM_MSM* and SND_QCOM_MSM* that were built-in, and try to
> initialize before the voltage regulators are ready to work, resulting in a
> noisy bootlog.
>
> These patches factor out some code as modules, so when the kmods are loaded and
> initialized, regulators are already present and ready to attach - all these
> config changes shave around ~3K bytes from dmesg, making the bootlog easir to
> parse.
>
> Fix:
>
> Apply the attached patches and recompile
>
> How to test:
>
> Boot a patched kernel and check for the above noise in dmesg
>
> Regression potential:
>
> None, we are just turning some code into kmods, and turning off a couple of not
> used feature for this flavour - all the other flavours are untouched.
>
> Paolo Pisati (5):
>   UBUNTU: [Config] arm64: snapdragon: DRM_MSM=m
>   UBUNTU: [Config] arm64: snapdragon: SND*=m
>   UBUNTU: [Config] arm64: snapdragon: disable ARM_SDE_INTERFACE
>   UBUNTU: [Config] arm64: snapdragon: disable DRM_I2C_ADV7511_CEC
>   UBUNTU: [Config] arm64: snapdragon: disable VIDEO_ADV7511,
>     VIDEO_COBALT
>
>  debian.master/config/amd64/config.common.amd64     |  6 ++----
>  debian.master/config/arm64/config.common.arm64     |  7 +++++++
>  debian.master/config/arm64/config.flavour.generic  | 22 ++++------------------
>  .../config/arm64/config.flavour.snapdragon         | 22 ++++------------------
>  debian.master/config/armhf/config.common.armhf     | 14 +++-----------
>  debian.master/config/config.common.ubuntu          | 16 +++++++++++-----
>  debian.master/config/i386/config.common.i386       |  6 ++----
>  debian.master/config/ppc64el/config.common.ppc64el |  7 +++----
>  8 files changed, 36 insertions(+), 64 deletions(-)
>
This is rather confusing: you submitted the same (at least appearing so from the
titles) set twice once for Bionic and again for Bionic *and* Cosmic. Please put
some self-NACK on whichever part can be ignore.

-Stefan


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH 0/5] [SRU][B/master] Reduce boot noise on Dragonboard

Stefan Bader-2
On 11.10.2018 09:53, Stefan Bader wrote:

> On 10.10.2018 17:32, Paolo Pisati wrote:
>> BugLink: https://bugs.launchpad.net/bugs/1797154
>>
>> Impact:
>>
>> During boot on a Dragonboard410c using a recent Bionic or Cosmic kernel,
>> there's some noise like:
>>
>> ...
>> [ 3.665431] msm_dsi_phy 1a98300.dsi-phy: Failed to get supply 'vddio':
>> -517
>> [ 3.669243] msm_dsi_phy 1a98300.dsi-phy: dsi_phy_regulator_init: failed to
>> init regulator, ret=-517
>> [ 3.676180] msm_dsi_phy 1a98300.dsi-phy: dsi_phy_driver_probe:
>> failed to init regulator
>> ...
>>
>> or
>>
>> ...
>> [ 4.192280] qcom,pm8916-wcd-spmi-codec 200f000.spmi:pm8916@1:codec@f000:
>> Failed to get supply 'vdd-cdc-io': -517
>> [ 4.195660] qcom,pm8916-wcd-spmi-codec 200f000.spmi:pm8916@1:codec@f000:
>> Failed to get regulator supplies -517
>> ...
>>
>> and similar.
>>
>> This is mainly due to DRM_MSM* and SND_QCOM_MSM* that were built-in, and try to
>> initialize before the voltage regulators are ready to work, resulting in a
>> noisy bootlog.
>>
>> These patches factor out some code as modules, so when the kmods are loaded and
>> initialized, regulators are already present and ready to attach - all these
>> config changes shave around ~3K bytes from dmesg, making the bootlog easir to
>> parse.
>>
>> Fix:
>>
>> Apply the attached patches and recompile
>>
>> How to test:
>>
>> Boot a patched kernel and check for the above noise in dmesg
>>
>> Regression potential:
>>
>> None, we are just turning some code into kmods, and turning off a couple of not
>> used feature for this flavour - all the other flavours are untouched.
>>
>> Paolo Pisati (5):
>>   UBUNTU: [Config] arm64: snapdragon: DRM_MSM=m
>>   UBUNTU: [Config] arm64: snapdragon: SND*=m
>>   UBUNTU: [Config] arm64: snapdragon: disable ARM_SDE_INTERFACE
>>   UBUNTU: [Config] arm64: snapdragon: disable DRM_I2C_ADV7511_CEC
>>   UBUNTU: [Config] arm64: snapdragon: disable VIDEO_ADV7511,
>>     VIDEO_COBALT
>>
>>  debian.master/config/amd64/config.common.amd64     |  6 ++----
>>  debian.master/config/arm64/config.common.arm64     |  7 +++++++
>>  debian.master/config/arm64/config.flavour.generic  | 22 ++++------------------
>>  .../config/arm64/config.flavour.snapdragon         | 22 ++++------------------
>>  debian.master/config/armhf/config.common.armhf     | 14 +++-----------
>>  debian.master/config/config.common.ubuntu          | 16 +++++++++++-----
>>  debian.master/config/i386/config.common.i386       |  6 ++----
>>  debian.master/config/ppc64el/config.common.ppc64el |  7 +++----
>>  8 files changed, 36 insertions(+), 64 deletions(-)
>>
>
> This is rather confusing: you submitted the same (at least appearing so from the
> titles) set twice once for Bionic and again for Bionic *and* Cosmic. Please put
> some self-NACK on whichever part can be ignore.
Hm, I think I need more coffee. I was somehow sure one of these two sets had B
and C in them but I probably was skidding between lines ... So this is the set
for bionic and the other one is for cosmic...

>
> -Stefan
>
>
>



--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH 0/5] [SRU][B/master] Reduce boot noise on Dragonboard

Paolo Pisati-5
In reply to this post by Stefan Bader-2
On Thu, Oct 11, 2018 at 09:53:57AM +0200, Stefan Bader wrote:

> This is rather confusing: you submitted the same (at least appearing so from the
> titles) set twice once for Bionic and again for Bionic *and* Cosmic. Please put
> some self-NACK on whichever part can be ignore.

In Mutt i see them as separated threads: this one is the Bionic set (see subject
- B/master), the other one is the Cosmic set (C/master)- the config changes are
the same, though due to the surrounding config lines, the patches are slightly
different, AKA the Bionic set of config changes won't apply to Cosmic and
viceversa.
--
bye,
p.

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

ACK/Cmnt: [PATCH 0/5] [SRU][B/master] Reduce boot noise on Dragonboard

Stefan Bader-2
In reply to this post by Paolo Pisati-5
On 10.10.2018 17:32, Paolo Pisati wrote:

> BugLink: https://bugs.launchpad.net/bugs/1797154
>
> Impact:
>
> During boot on a Dragonboard410c using a recent Bionic or Cosmic kernel,
> there's some noise like:
>
> ...
> [ 3.665431] msm_dsi_phy 1a98300.dsi-phy: Failed to get supply 'vddio':
> -517
> [ 3.669243] msm_dsi_phy 1a98300.dsi-phy: dsi_phy_regulator_init: failed to
> init regulator, ret=-517
> [ 3.676180] msm_dsi_phy 1a98300.dsi-phy: dsi_phy_driver_probe:
> failed to init regulator
> ...
>
> or
>
> ...
> [ 4.192280] qcom,pm8916-wcd-spmi-codec 200f000.spmi:pm8916@1:codec@f000:
> Failed to get supply 'vdd-cdc-io': -517
> [ 4.195660] qcom,pm8916-wcd-spmi-codec 200f000.spmi:pm8916@1:codec@f000:
> Failed to get regulator supplies -517
> ...
>
> and similar.
>
> This is mainly due to DRM_MSM* and SND_QCOM_MSM* that were built-in, and try to
> initialize before the voltage regulators are ready to work, resulting in a
> noisy bootlog.
>
> These patches factor out some code as modules, so when the kmods are loaded and
> initialized, regulators are already present and ready to attach - all these
> config changes shave around ~3K bytes from dmesg, making the bootlog easir to
> parse.
>
> Fix:
>
> Apply the attached patches and recompile
>
> How to test:
>
> Boot a patched kernel and check for the above noise in dmesg
>
> Regression potential:
>
> None, we are just turning some code into kmods, and turning off a couple of not
> used feature for this flavour - all the other flavours are untouched.
>
> Paolo Pisati (5):
>   UBUNTU: [Config] arm64: snapdragon: DRM_MSM=m
>   UBUNTU: [Config] arm64: snapdragon: SND*=m
>   UBUNTU: [Config] arm64: snapdragon: disable ARM_SDE_INTERFACE
>   UBUNTU: [Config] arm64: snapdragon: disable DRM_I2C_ADV7511_CEC
>   UBUNTU: [Config] arm64: snapdragon: disable VIDEO_ADV7511,
>     VIDEO_COBALT
>
>  debian.master/config/amd64/config.common.amd64     |  6 ++----
>  debian.master/config/arm64/config.common.arm64     |  7 +++++++
>  debian.master/config/arm64/config.flavour.generic  | 22 ++++------------------
>  .../config/arm64/config.flavour.snapdragon         | 22 ++++------------------
>  debian.master/config/armhf/config.common.armhf     | 14 +++-----------
>  debian.master/config/config.common.ubuntu          | 16 +++++++++++-----
>  debian.master/config/i386/config.common.i386       |  6 ++----
>  debian.master/config/ppc64el/config.common.ppc64el |  7 +++----
>  8 files changed, 36 insertions(+), 64 deletions(-)
>
I wonder whether all those changes to config should not be reflected (and
possibly enforced) in the annotations file? The changes themselves look okaish...

Acked-by: Stefan Bader <[hidden email]>


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (836 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH 0/5] [SRU][B/master] Reduce boot noise on Dragonboard

Colin Ian King-2
In reply to this post by Paolo Pisati-5
On 10/10/18 16:32, Paolo Pisati wrote:

> BugLink: https://bugs.launchpad.net/bugs/1797154
>
> Impact:
>
> During boot on a Dragonboard410c using a recent Bionic or Cosmic kernel,
> there's some noise like:
>
> ...
> [ 3.665431] msm_dsi_phy 1a98300.dsi-phy: Failed to get supply 'vddio':
> -517
> [ 3.669243] msm_dsi_phy 1a98300.dsi-phy: dsi_phy_regulator_init: failed to
> init regulator, ret=-517
> [ 3.676180] msm_dsi_phy 1a98300.dsi-phy: dsi_phy_driver_probe:
> failed to init regulator
> ...
>
> or
>
> ...
> [ 4.192280] qcom,pm8916-wcd-spmi-codec 200f000.spmi:pm8916@1:codec@f000:
> Failed to get supply 'vdd-cdc-io': -517
> [ 4.195660] qcom,pm8916-wcd-spmi-codec 200f000.spmi:pm8916@1:codec@f000:
> Failed to get regulator supplies -517
> ...
>
> and similar.
>
> This is mainly due to DRM_MSM* and SND_QCOM_MSM* that were built-in, and try to
> initialize before the voltage regulators are ready to work, resulting in a
> noisy bootlog.
>
> These patches factor out some code as modules, so when the kmods are loaded and
> initialized, regulators are already present and ready to attach - all these
> config changes shave around ~3K bytes from dmesg, making the bootlog easir to
> parse.
>
> Fix:
>
> Apply the attached patches and recompile
>
> How to test:
>
> Boot a patched kernel and check for the above noise in dmesg
>
> Regression potential:
>
> None, we are just turning some code into kmods, and turning off a couple of not
> used feature for this flavour - all the other flavours are untouched.
>
> Paolo Pisati (5):
>   UBUNTU: [Config] arm64: snapdragon: DRM_MSM=m
>   UBUNTU: [Config] arm64: snapdragon: SND*=m
>   UBUNTU: [Config] arm64: snapdragon: disable ARM_SDE_INTERFACE
>   UBUNTU: [Config] arm64: snapdragon: disable DRM_I2C_ADV7511_CEC
>   UBUNTU: [Config] arm64: snapdragon: disable VIDEO_ADV7511,
>     VIDEO_COBALT
>
>  debian.master/config/amd64/config.common.amd64     |  6 ++----
>  debian.master/config/arm64/config.common.arm64     |  7 +++++++
>  debian.master/config/arm64/config.flavour.generic  | 22 ++++------------------
>  .../config/arm64/config.flavour.snapdragon         | 22 ++++------------------
>  debian.master/config/armhf/config.common.armhf     | 14 +++-----------
>  debian.master/config/config.common.ubuntu          | 16 +++++++++++-----
>  debian.master/config/i386/config.common.i386       |  6 ++----
>  debian.master/config/ppc64el/config.common.ppc64el |  7 +++----
>  8 files changed, 36 insertions(+), 64 deletions(-)
>
Disabling drivers that are not required is a good idea anyhow, plus it
reduces kernel noise, which is a good thing. So...

Acked-by: Colin Ian King <[hidden email]>

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|

Re: ACK/Cmnt: [PATCH 0/5] [SRU][B/master] Reduce boot noise on Dragonboard

Paolo Pisati-5
In reply to this post by Stefan Bader-2
On Thu, Oct 11, 2018 at 11:18:43AM +0200, Stefan Bader wrote:

> I wonder whether all those changes to config should not be reflected (and
> possibly enforced) in the annotations file? The changes themselves look okaish...

I didn't modify annotations because none of the changes here are mandatory to
boot the board or cripple / fix any feature beyond repair.
--
bye,
p.

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team