[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200125104438.GA5646@onstation.org>
Date: Sat, 25 Jan 2020 05:44:38 -0500
From: Brian Masney <masneyb@...tation.org>
To: Luca Weiss <luca@...tu.xyz>
Cc: linux-arm-msm@...r.kernel.org,
Russell King <linux@...linux.org.uk>,
Bjorn Andersson <bjorn.andersson@...aro.org>,
Linus Walleij <linus.walleij@...aro.org>,
Sudeep Holla <sudeep.holla@....com>,
Krzysztof Kozlowski <krzk@...nel.org>,
Miquel Raynal <miquel.raynal@...tlin.com>,
Jordan Crouse <jcrouse@...eaurora.org>,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] ARM: qcom_defconfig: Regenerate
Hi Luca,
On Mon, Nov 04, 2019 at 10:09:40PM +0100, Luca Weiss wrote:
> Several options were dropped a while ago and the options QCOM_ADSP_PIL
> and QCOM_Q6V5_PIL have been renamed.
>
> Signed-off-by: Luca Weiss <luca@...tu.xyz>
> ---
Thanks for the pointer to these two patches today. I confirmed that I'm
able to get the modem booted on the Nexus 5 with these two patches, plus
these two patches from Bjorn:
https://lore.kernel.org/lkml/20191109004033.1496871-1-bjorn.andersson@linaro.org/
I see this patch series wasn't applied. I suggest making a few minor
changes:
- I would make one patch that drops the options that are no longer
present.
- Make one or two more patches for the QCOM_ADSP_PIL / QCOM_Q6V5_PIL
renames.
- I know that running 'make savedefconfig' causes a big diff when the
generated file is moved to qcom_defconfig. I've been using that as a
guide of where to put the new config option and then dropping all of
the other unrelated changes. Maybe if you do this as a separate patch
it'll make the overall diff much smaller and easier to review. I'd put
it at the very end of the series in case it's rejected.
Brian
> arch/arm/configs/qcom_defconfig | 45 ++++++++++++---------------------
> 1 file changed, 16 insertions(+), 29 deletions(-)
>
> diff --git a/arch/arm/configs/qcom_defconfig b/arch/arm/configs/qcom_defconfig
> index 9792dd0aae0c..94d5e1a8c61a 100644
> --- a/arch/arm/configs/qcom_defconfig
> +++ b/arch/arm/configs/qcom_defconfig
> @@ -1,6 +1,7 @@
> CONFIG_SYSVIPC=y
> CONFIG_NO_HZ=y
> CONFIG_HIGH_RES_TIMERS=y
> +CONFIG_PREEMPT=y
> CONFIG_IKCONFIG=y
> CONFIG_IKCONFIG_PROC=y
> CONFIG_CGROUPS=y
> @@ -11,32 +12,28 @@ CONFIG_EMBEDDED=y
> # CONFIG_SLUB_DEBUG is not set
> # CONFIG_COMPAT_BRK is not set
> CONFIG_PROFILING=y
> -CONFIG_OPROFILE=y
> -CONFIG_KPROBES=y
> -CONFIG_MODULES=y
> -CONFIG_MODULE_UNLOAD=y
> -CONFIG_MODULE_FORCE_UNLOAD=y
> -CONFIG_MODVERSIONS=y
> -CONFIG_PARTITION_ADVANCED=y
> CONFIG_ARCH_QCOM=y
> CONFIG_ARCH_MSM8X60=y
> CONFIG_ARCH_MSM8960=y
> CONFIG_ARCH_MSM8974=y
> CONFIG_ARCH_MDM9615=y
> -CONFIG_PCI=y
> -CONFIG_PCI_MSI=y
> -CONFIG_PCIE_QCOM=y
> CONFIG_SMP=y
> -CONFIG_PREEMPT=y
> CONFIG_HIGHMEM=y
> -CONFIG_CLEANCACHE=y
> CONFIG_ARM_APPENDED_DTB=y
> CONFIG_ARM_ATAG_DTB_COMPAT=y
> CONFIG_CPU_IDLE=y
> CONFIG_ARM_CPUIDLE=y
> CONFIG_VFP=y
> CONFIG_NEON=y
> +CONFIG_OPROFILE=y
> +CONFIG_KPROBES=y
> +CONFIG_MODULES=y
> +CONFIG_MODULE_UNLOAD=y
> +CONFIG_MODULE_FORCE_UNLOAD=y
> +CONFIG_MODVERSIONS=y
> +CONFIG_PARTITION_ADVANCED=y
> # CONFIG_CORE_DUMP_DEFAULT_ELF_HEADERS is not set
> +CONFIG_CLEANCACHE=y
> CONFIG_CMA=y
> CONFIG_NET=y
> CONFIG_PACKET=y
> @@ -47,18 +44,17 @@ CONFIG_IP_MULTIPLE_TABLES=y
> CONFIG_IP_ROUTE_VERBOSE=y
> CONFIG_IP_PNP=y
> CONFIG_IP_PNP_DHCP=y
> -# CONFIG_INET_XFRM_MODE_TRANSPORT is not set
> -# CONFIG_INET_XFRM_MODE_TUNNEL is not set
> -# CONFIG_INET_XFRM_MODE_BEET is not set
> # CONFIG_IPV6 is not set
> CONFIG_CFG80211=m
> CONFIG_MAC80211=m
> CONFIG_RFKILL=y
> +CONFIG_PCI=y
> +CONFIG_PCI_MSI=y
> +CONFIG_PCIE_QCOM=y
> CONFIG_DEVTMPFS=y
> CONFIG_DEVTMPFS_MOUNT=y
> CONFIG_MTD=y
> CONFIG_MTD_BLOCK=y
> -CONFIG_MTD_M25P80=y
> CONFIG_MTD_RAW_NAND=y
> CONFIG_MTD_NAND_QCOM=y
> CONFIG_MTD_SPI_NOR=y
> @@ -113,7 +109,6 @@ CONFIG_SERIO_LIBPS2=y
> CONFIG_SERIAL_MSM=y
> CONFIG_SERIAL_MSM_CONSOLE=y
> CONFIG_HW_RANDOM=y
> -CONFIG_I2C=y
> CONFIG_I2C_CHARDEV=y
> CONFIG_I2C_QUP=y
> CONFIG_SPI=y
> @@ -140,7 +135,6 @@ CONFIG_QCOM_TSENS=y
> CONFIG_MFD_PM8XXX=y
> CONFIG_MFD_QCOM_RPM=y
> CONFIG_MFD_SPMI_PMIC=y
> -CONFIG_REGULATOR=y
> CONFIG_REGULATOR_FIXED_VOLTAGE=y
> CONFIG_REGULATOR_QCOM_RPM=y
> CONFIG_REGULATOR_QCOM_SMD_RPM=y
> @@ -149,13 +143,11 @@ CONFIG_MEDIA_SUPPORT=y
> CONFIG_DRM=y
> CONFIG_DRM_MSM=m
> CONFIG_DRM_PANEL_SIMPLE=y
> -CONFIG_FB=y
> -CONFIG_FRAMEBUFFER_CONSOLE=y
> -# CONFIG_LCD_CLASS_DEVICE is not set
> CONFIG_BACKLIGHT_CLASS_DEVICE=y
> # CONFIG_BACKLIGHT_GENERIC is not set
> CONFIG_BACKLIGHT_LM3630A=y
> CONFIG_BACKLIGHT_LP855X=y
> +CONFIG_FRAMEBUFFER_CONSOLE=y
> CONFIG_SOUND=y
> CONFIG_SND=y
> CONFIG_SND_DYNAMIC_MINORS=y
> @@ -169,15 +161,12 @@ CONFIG_USB_ANNOUNCE_NEW_DEVICES=y
> CONFIG_USB_OTG=y
> CONFIG_USB_MON=y
> CONFIG_USB_EHCI_HCD=y
> -CONFIG_USB_EHCI_MSM=y
> CONFIG_USB_ACM=y
> CONFIG_USB_CHIPIDEA=y
> CONFIG_USB_CHIPIDEA_UDC=y
> CONFIG_USB_CHIPIDEA_HOST=y
> -CONFIG_USB_CHIPIDEA_ULPI=y
> CONFIG_USB_SERIAL=y
> CONFIG_USB_HSIC_USB4604=y
> -CONFIG_USB_MSM_OTG=y
> CONFIG_USB_GADGET=y
> CONFIG_USB_GADGET_DEBUG_FILES=y
> CONFIG_USB_GADGET_VBUS_DRAW=500
> @@ -185,7 +174,6 @@ CONFIG_USB_CONFIGFS=y
> CONFIG_USB_CONFIGFS_NCM=y
> CONFIG_USB_CONFIGFS_ECM=y
> CONFIG_USB_CONFIGFS_F_FS=y
> -CONFIG_USB_ULPI_BUS=y
> CONFIG_USB_ETH=m
> CONFIG_MMC=y
> CONFIG_MMC_BLOCK_MINORS=32
> @@ -215,13 +203,13 @@ CONFIG_MSM_LCC_8960=y
> CONFIG_MDM_LCC_9615=y
> CONFIG_MSM_MMCC_8960=y
> CONFIG_MSM_MMCC_8974=y
> -CONFIG_MSM_IOMMU=y
> CONFIG_HWSPINLOCK=y
> CONFIG_HWSPINLOCK_QCOM=y
> CONFIG_MAILBOX=y
> +CONFIG_MSM_IOMMU=y
> CONFIG_REMOTEPROC=y
> -CONFIG_QCOM_ADSP_PIL=y
> -CONFIG_QCOM_Q6V5_PIL=y
> +CONFIG_QCOM_Q6V5_MSS=y
> +CONFIG_QCOM_Q6V5_PAS=y
> CONFIG_QCOM_WCNSS_PIL=y
> CONFIG_RPMSG_CHAR=y
> CONFIG_RPMSG_QCOM_SMD=y
> @@ -257,7 +245,6 @@ CONFIG_EXT2_FS_XATTR=y
> CONFIG_EXT3_FS=y
> CONFIG_FUSE_FS=y
> CONFIG_VFAT_FS=y
> -CONFIG_TMPFS=y
> CONFIG_JFFS2_FS=y
> CONFIG_NFS_FS=y
> CONFIG_NFS_V3_ACL=y
> --
> 2.23.0
>
Powered by blists - more mailing lists