* [PATCH] Revert "arm64: dts: marvell: add CPU Idle power state support on Armada 7K/8K" @ 2018-12-04 15:46 Baruch Siach 2018-12-04 16:21 ` Gregory CLEMENT 0 siblings, 1 reply; 9+ messages in thread From: Baruch Siach @ 2018-12-04 15:46 UTC (permalink / raw) To: linux-arm-kernel, Jason Cooper, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth, orenbh Cc: Baruch Siach, Sergey Matyukevich, linux-pm, Daniel Lezcano, Rafael J. Wysocki, Russell King This reverts commit 8ed46368776b3bc93d74c1f8f2bfb9fd8a9ad805. This commit breaks boot on Armada 8K based systems. Reverting it makes affected systems boot again. Reported-by: Sergey Matyukevich <geomatsi@gmail.com> Signed-off-by: Baruch Siach <baruch@tkos.co.il> --- .../boot/dts/marvell/armada-ap806-quad.dtsi | 4 --- arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 27 ------------------- 2 files changed, 31 deletions(-) diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi index 64632c873888..01ea662afba8 100644 --- a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi +++ b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi @@ -20,28 +20,24 @@ compatible = "arm,cortex-a72", "arm,armv8"; reg = <0x000>; enable-method = "psci"; - cpu-idle-states = <&CPU_SLEEP_0>; }; cpu1: cpu@1 { device_type = "cpu"; compatible = "arm,cortex-a72", "arm,armv8"; reg = <0x001>; enable-method = "psci"; - cpu-idle-states = <&CPU_SLEEP_0>; }; cpu2: cpu@100 { device_type = "cpu"; compatible = "arm,cortex-a72", "arm,armv8"; reg = <0x100>; enable-method = "psci"; - cpu-idle-states = <&CPU_SLEEP_0>; }; cpu3: cpu@101 { device_type = "cpu"; compatible = "arm,cortex-a72", "arm,armv8"; reg = <0x101>; enable-method = "psci"; - cpu-idle-states = <&CPU_SLEEP_0>; }; }; }; diff --git a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi index 073610ac0a53..7d94c1fa592a 100644 --- a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi +++ b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi @@ -28,33 +28,6 @@ method = "smc"; }; - cpus { - #address-cells = <1>; - #size-cells = <0>; - - idle_states { - entry_method = "arm,pcsi"; - - CPU_SLEEP_0: cpu-sleep-0 { - compatible = "arm,idle-state"; - local-timer-stop; - arm,psci-suspend-param = <0x0010000>; - entry-latency-us = <80>; - exit-latency-us = <160>; - min-residency-us = <320>; - }; - - CLUSTER_SLEEP_0: cluster-sleep-0 { - compatible = "arm,idle-state"; - local-timer-stop; - arm,psci-suspend-param = <0x1010000>; - entry-latency-us = <500>; - exit-latency-us = <1000>; - min-residency-us = <2500>; - }; - }; - }; - ap806 { #address-cells = <2>; #size-cells = <2>; -- 2.19.2 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] Revert "arm64: dts: marvell: add CPU Idle power state support on Armada 7K/8K" 2018-12-04 15:46 [PATCH] Revert "arm64: dts: marvell: add CPU Idle power state support on Armada 7K/8K" Baruch Siach @ 2018-12-04 16:21 ` Gregory CLEMENT 2018-12-04 16:48 ` Baruch Siach 0 siblings, 1 reply; 9+ messages in thread From: Gregory CLEMENT @ 2018-12-04 16:21 UTC (permalink / raw) To: Baruch Siach Cc: Andrew Lunn, Jason Cooper, Sergey Matyukevich, linux-pm, Rafael J. Wysocki, Daniel Lezcano, orenbh, Russell King, linux-arm-kernel, Sebastian Hesselbarth Hi Baruch and Sergey, On mar., déc. 04 2018, Baruch Siach <baruch@tkos.co.il> wrote: > This reverts commit 8ed46368776b3bc93d74c1f8f2bfb9fd8a9ad805. > > This commit breaks boot on Armada 8K based systems. Reverting it makes > affected systems boot again. > > Reported-by: Sergey Matyukevich <geomatsi@gmail.com> > Signed-off-by: Baruch Siach <baruch@tkos.co.il> I would like to know more about it because on the McBin I use there is no issue with 4.20-rc1. Do you observe this issue with the arm64 defconfig or do you have some specific configuration option? Thanks, Gregory > --- > .../boot/dts/marvell/armada-ap806-quad.dtsi | 4 --- > arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 27 ------------------- > 2 files changed, 31 deletions(-) > > diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > index 64632c873888..01ea662afba8 100644 > --- a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > @@ -20,28 +20,24 @@ > compatible = "arm,cortex-a72", "arm,armv8"; > reg = <0x000>; > enable-method = "psci"; > - cpu-idle-states = <&CPU_SLEEP_0>; > }; > cpu1: cpu@1 { > device_type = "cpu"; > compatible = "arm,cortex-a72", "arm,armv8"; > reg = <0x001>; > enable-method = "psci"; > - cpu-idle-states = <&CPU_SLEEP_0>; > }; > cpu2: cpu@100 { > device_type = "cpu"; > compatible = "arm,cortex-a72", "arm,armv8"; > reg = <0x100>; > enable-method = "psci"; > - cpu-idle-states = <&CPU_SLEEP_0>; > }; > cpu3: cpu@101 { > device_type = "cpu"; > compatible = "arm,cortex-a72", "arm,armv8"; > reg = <0x101>; > enable-method = "psci"; > - cpu-idle-states = <&CPU_SLEEP_0>; > }; > }; > }; > diff --git a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > index 073610ac0a53..7d94c1fa592a 100644 > --- a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > +++ b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > @@ -28,33 +28,6 @@ > method = "smc"; > }; > > - cpus { > - #address-cells = <1>; > - #size-cells = <0>; > - > - idle_states { > - entry_method = "arm,pcsi"; > - > - CPU_SLEEP_0: cpu-sleep-0 { > - compatible = "arm,idle-state"; > - local-timer-stop; > - arm,psci-suspend-param = <0x0010000>; > - entry-latency-us = <80>; > - exit-latency-us = <160>; > - min-residency-us = <320>; > - }; > - > - CLUSTER_SLEEP_0: cluster-sleep-0 { > - compatible = "arm,idle-state"; > - local-timer-stop; > - arm,psci-suspend-param = <0x1010000>; > - entry-latency-us = <500>; > - exit-latency-us = <1000>; > - min-residency-us = <2500>; > - }; > - }; > - }; > - > ap806 { > #address-cells = <2>; > #size-cells = <2>; > -- > 2.19.2 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Gregory Clement, Bootlin Embedded Linux and Kernel engineering http://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] Revert "arm64: dts: marvell: add CPU Idle power state support on Armada 7K/8K" 2018-12-04 16:21 ` Gregory CLEMENT @ 2018-12-04 16:48 ` Baruch Siach 2018-12-05 6:57 ` Baruch Siach 0 siblings, 1 reply; 9+ messages in thread From: Baruch Siach @ 2018-12-04 16:48 UTC (permalink / raw) To: Gregory CLEMENT Cc: Andrew Lunn, Jason Cooper, Sergey Matyukevich, linux-pm, Rafael J. Wysocki, Daniel Lezcano, orenbh, Russell King, linux-arm-kernel, Sebastian Hesselbarth [-- Attachment #1: Type: text/plain, Size: 3462 bytes --] Hi Gregory, On Tue, Dec 04, 2018 at 05:21:25PM +0100, Gregory CLEMENT wrote: > On mar., déc. 04 2018, Baruch Siach <baruch@tkos.co.il> wrote: > > This reverts commit 8ed46368776b3bc93d74c1f8f2bfb9fd8a9ad805. > > > > This commit breaks boot on Armada 8K based systems. Reverting it makes > > affected systems boot again. > > > > Reported-by: Sergey Matyukevich <geomatsi@gmail.com> > > Signed-off-by: Baruch Siach <baruch@tkos.co.il> > > I would like to know more about it because on the McBin I use there is > no issue with 4.20-rc1. > > Do you observe this issue with the arm64 defconfig or do you have some > specific configuration option? I tested the attached defconfig on Clearfog GT-8K and on another custom Armada 8K based board. Tomorrow I'll test Macchiatobin with this config and the arm64 defconfig. baruch > > --- > > .../boot/dts/marvell/armada-ap806-quad.dtsi | 4 --- > > arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 27 ------------------- > > 2 files changed, 31 deletions(-) > > > > diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > > index 64632c873888..01ea662afba8 100644 > > --- a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > > +++ b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > > @@ -20,28 +20,24 @@ > > compatible = "arm,cortex-a72", "arm,armv8"; > > reg = <0x000>; > > enable-method = "psci"; > > - cpu-idle-states = <&CPU_SLEEP_0>; > > }; > > cpu1: cpu@1 { > > device_type = "cpu"; > > compatible = "arm,cortex-a72", "arm,armv8"; > > reg = <0x001>; > > enable-method = "psci"; > > - cpu-idle-states = <&CPU_SLEEP_0>; > > }; > > cpu2: cpu@100 { > > device_type = "cpu"; > > compatible = "arm,cortex-a72", "arm,armv8"; > > reg = <0x100>; > > enable-method = "psci"; > > - cpu-idle-states = <&CPU_SLEEP_0>; > > }; > > cpu3: cpu@101 { > > device_type = "cpu"; > > compatible = "arm,cortex-a72", "arm,armv8"; > > reg = <0x101>; > > enable-method = "psci"; > > - cpu-idle-states = <&CPU_SLEEP_0>; > > }; > > }; > > }; > > diff --git a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > > index 073610ac0a53..7d94c1fa592a 100644 > > --- a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > > +++ b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > > @@ -28,33 +28,6 @@ > > method = "smc"; > > }; > > > > - cpus { > > - #address-cells = <1>; > > - #size-cells = <0>; > > - > > - idle_states { > > - entry_method = "arm,pcsi"; > > - > > - CPU_SLEEP_0: cpu-sleep-0 { > > - compatible = "arm,idle-state"; > > - local-timer-stop; > > - arm,psci-suspend-param = <0x0010000>; > > - entry-latency-us = <80>; > > - exit-latency-us = <160>; > > - min-residency-us = <320>; > > - }; > > - > > - CLUSTER_SLEEP_0: cluster-sleep-0 { > > - compatible = "arm,idle-state"; > > - local-timer-stop; > > - arm,psci-suspend-param = <0x1010000>; > > - entry-latency-us = <500>; > > - exit-latency-us = <1000>; > > - min-residency-us = <2500>; > > - }; > > - }; > > - }; > > - > > ap806 { > > #address-cells = <2>; > > #size-cells = <2>; -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - [-- Attachment #2: defconfig --] [-- Type: text/plain, Size: 8745 bytes --] CONFIG_SYSVIPC=y CONFIG_POSIX_MQUEUE=y CONFIG_AUDIT=y CONFIG_NO_HZ_IDLE=y CONFIG_HIGH_RES_TIMERS=y CONFIG_PREEMPT=y CONFIG_BSD_PROCESS_ACCT=y CONFIG_BSD_PROCESS_ACCT_V3=y CONFIG_TASKSTATS=y CONFIG_TASK_DELAY_ACCT=y CONFIG_TASK_XACCT=y CONFIG_TASK_IO_ACCOUNTING=y CONFIG_IKCONFIG=y CONFIG_IKCONFIG_PROC=y CONFIG_NUMA_BALANCING=y CONFIG_MEMCG=y CONFIG_MEMCG_SWAP=y CONFIG_BLK_CGROUP=y CONFIG_CGROUP_PIDS=y CONFIG_CGROUP_HUGETLB=y CONFIG_CPUSETS=y CONFIG_CGROUP_DEVICE=y CONFIG_CGROUP_CPUACCT=y CONFIG_CGROUP_PERF=y CONFIG_USER_NS=y CONFIG_SCHED_AUTOGROUP=y CONFIG_BLK_DEV_INITRD=y CONFIG_INITRAMFS_SOURCE="rootfs-armv8.cpio" # CONFIG_RD_BZIP2 is not set # CONFIG_RD_LZMA is not set # CONFIG_RD_XZ is not set # CONFIG_RD_LZO is not set # CONFIG_RD_LZ4 is not set CONFIG_CC_OPTIMIZE_FOR_SIZE=y CONFIG_KALLSYMS_ALL=y # CONFIG_COMPAT_BRK is not set CONFIG_PROFILING=y CONFIG_ARCH_MVEBU=y CONFIG_PCI=y CONFIG_PCI_IOV=y CONFIG_HOTPLUG_PCI=y CONFIG_PCI_HOST_GENERIC=y CONFIG_PCIE_ARMADA_8K=y CONFIG_ARM64_VA_BITS_48=y CONFIG_SCHED_MC=y CONFIG_NUMA=y CONFIG_CRASH_DUMP=y # CONFIG_ARM64_VHE is not set CONFIG_COMPAT=y CONFIG_HIBERNATION=y CONFIG_CPU_IDLE=y CONFIG_ARM_CPUIDLE=y CONFIG_CPU_FREQ=y CONFIG_CPUFREQ_DT=y CONFIG_ARM_SCPI_CPUFREQ=y CONFIG_ARM_SCPI_PROTOCOL=y CONFIG_EFI_CAPSULE_LOADER=y CONFIG_ARM64_CRYPTO=y CONFIG_CRYPTO_SHA1_ARM64_CE=y CONFIG_CRYPTO_SHA2_ARM64_CE=y CONFIG_CRYPTO_GHASH_ARM64_CE=y CONFIG_CRYPTO_AES_ARM64_CE_CCM=y CONFIG_CRYPTO_AES_ARM64_CE_BLK=y CONFIG_JUMP_LABEL=y CONFIG_BLK_DEV_INTEGRITY=y # CONFIG_IOSCHED_DEADLINE is not set # CONFIG_CORE_DUMP_DEFAULT_ELF_HEADERS is not set CONFIG_KSM=y CONFIG_TRANSPARENT_HUGEPAGE=y CONFIG_CMA=y CONFIG_NET=y CONFIG_PACKET=y CONFIG_UNIX=y CONFIG_INET=y CONFIG_IP_MULTICAST=y CONFIG_IP_PNP=y CONFIG_IP_PNP_DHCP=y CONFIG_IP_PNP_BOOTP=y CONFIG_NETFILTER=y CONFIG_NF_CONNTRACK=y CONFIG_NF_CONNTRACK_EVENTS=y CONFIG_NETFILTER_XT_TARGET_CHECKSUM=y CONFIG_NETFILTER_XT_TARGET_LOG=y CONFIG_NETFILTER_XT_MATCH_ADDRTYPE=y CONFIG_NETFILTER_XT_MATCH_CONNTRACK=y CONFIG_IP_NF_IPTABLES=y CONFIG_IP_NF_FILTER=y CONFIG_IP_NF_TARGET_REJECT=y CONFIG_IP_NF_NAT=y CONFIG_IP_NF_TARGET_MASQUERADE=y CONFIG_IP_NF_MANGLE=y CONFIG_IP6_NF_IPTABLES=y CONFIG_IP6_NF_FILTER=y CONFIG_IP6_NF_TARGET_REJECT=y CONFIG_IP6_NF_MANGLE=y CONFIG_IP6_NF_NAT=y CONFIG_IP6_NF_TARGET_MASQUERADE=y CONFIG_BRIDGE=y CONFIG_BRIDGE_VLAN_FILTERING=y CONFIG_VLAN_8021Q=y CONFIG_VLAN_8021Q_GVRP=y CONFIG_VLAN_8021Q_MVRP=y CONFIG_DNS_RESOLVER=y # CONFIG_WIRELESS is not set CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug" CONFIG_DEVTMPFS=y CONFIG_DEVTMPFS_MOUNT=y CONFIG_DMA_CMA=y CONFIG_BRCMSTB_GISB_ARB=y CONFIG_VEXPRESS_CONFIG=y CONFIG_MTD=y # CONFIG_MTD_OF_PARTS is not set CONFIG_MTD_BLOCK=y CONFIG_MTD_M25P80=y CONFIG_MTD_SPI_NOR=y CONFIG_BLK_DEV_LOOP=y CONFIG_BLK_DEV_NBD=y CONFIG_VIRTIO_BLK=y CONFIG_SRAM=y CONFIG_EEPROM_AT25=y # CONFIG_SCSI_PROC_FS is not set CONFIG_BLK_DEV_SD=y CONFIG_SCSI_SAS_LIBSAS=y CONFIG_SCSI_SAS_ATA=y CONFIG_ATA=y CONFIG_SATA_AHCI=y CONFIG_SATA_AHCI_PLATFORM=y CONFIG_AHCI_MVEBU=y # CONFIG_ATA_SFF is not set CONFIG_NETDEVICES=y CONFIG_MACVLAN=y CONFIG_MACVTAP=y CONFIG_TUN=y CONFIG_VETH=y CONFIG_VIRTIO_NET=y # CONFIG_NET_VENDOR_3COM is not set # CONFIG_NET_VENDOR_ADAPTEC is not set # CONFIG_NET_VENDOR_AGERE is not set # CONFIG_NET_VENDOR_ALACRITECH is not set # CONFIG_NET_VENDOR_ALTEON is not set # CONFIG_NET_VENDOR_AMAZON is not set # CONFIG_NET_VENDOR_AMD is not set # CONFIG_NET_VENDOR_AQUANTIA is not set # CONFIG_NET_VENDOR_ARC is not set # CONFIG_NET_VENDOR_ATHEROS is not set # CONFIG_NET_VENDOR_BROADCOM is not set # CONFIG_NET_VENDOR_BROCADE is not set # CONFIG_NET_VENDOR_CAVIUM is not set # CONFIG_NET_VENDOR_CHELSIO is not set # CONFIG_NET_VENDOR_CISCO is not set # CONFIG_NET_VENDOR_DEC is not set # CONFIG_NET_VENDOR_DLINK is not set # CONFIG_NET_VENDOR_EMULEX is not set # CONFIG_NET_VENDOR_EZCHIP is not set # CONFIG_NET_VENDOR_HISILICON is not set # CONFIG_NET_VENDOR_HP is not set # CONFIG_NET_VENDOR_HUAWEI is not set # CONFIG_NET_VENDOR_INTEL is not set CONFIG_MVPP2=y # CONFIG_NET_VENDOR_MELLANOX is not set # CONFIG_NET_VENDOR_MICREL is not set # CONFIG_NET_VENDOR_MICROCHIP is not set # CONFIG_NET_VENDOR_MYRI is not set # CONFIG_NET_VENDOR_NATSEMI is not set # CONFIG_NET_VENDOR_NETRONOME is not set # CONFIG_NET_VENDOR_NVIDIA is not set # CONFIG_NET_VENDOR_OKI is not set # CONFIG_NET_VENDOR_QLOGIC is not set # CONFIG_NET_VENDOR_QUALCOMM is not set # CONFIG_NET_VENDOR_RDC is not set # CONFIG_NET_VENDOR_REALTEK is not set # CONFIG_NET_VENDOR_RENESAS is not set # CONFIG_NET_VENDOR_ROCKER is not set # CONFIG_NET_VENDOR_SAMSUNG is not set # CONFIG_NET_VENDOR_SEEQ is not set # CONFIG_NET_VENDOR_SOLARFLARE is not set # CONFIG_NET_VENDOR_SILAN is not set # CONFIG_NET_VENDOR_SIS is not set # CONFIG_NET_VENDOR_SMSC is not set # CONFIG_NET_VENDOR_STMICRO is not set # CONFIG_NET_VENDOR_SUN is not set # CONFIG_NET_VENDOR_SYNOPSYS is not set # CONFIG_NET_VENDOR_TEHUTI is not set # CONFIG_NET_VENDOR_TI is not set # CONFIG_NET_VENDOR_VIA is not set # CONFIG_NET_VENDOR_WIZNET is not set CONFIG_MDIO_BUS_MUX_MMIOREG=y CONFIG_SFP=y CONFIG_MARVELL_PHY=y CONFIG_MARVELL_10G_PHY=y # CONFIG_USB_NET_DRIVERS is not set # CONFIG_WLAN is not set CONFIG_INPUT_POLLDEV=y CONFIG_INPUT_EVDEV=y CONFIG_KEYBOARD_GPIO=y CONFIG_KEYBOARD_CROS_EC=y CONFIG_INPUT_MISC=y # CONFIG_SERIO_SERPORT is not set CONFIG_SERIO_AMBAKMI=y CONFIG_VT_HW_CONSOLE_BINDING=y CONFIG_LEGACY_PTY_COUNT=16 CONFIG_SERIAL_8250=y CONFIG_SERIAL_8250_CONSOLE=y CONFIG_SERIAL_8250_EXTENDED=y CONFIG_SERIAL_8250_SHARE_IRQ=y CONFIG_SERIAL_8250_DW=y CONFIG_SERIAL_OF_PLATFORM=y CONFIG_SERIAL_AMBA_PL011=y CONFIG_SERIAL_AMBA_PL011_CONSOLE=y CONFIG_SERIAL_MVEBU_UART=y # CONFIG_HW_RANDOM_CAVIUM is not set CONFIG_I2C=y # CONFIG_I2C_COMPAT is not set CONFIG_I2C_CHARDEV=y CONFIG_I2C_MUX=y CONFIG_I2C_MUX_PCA954x=y CONFIG_I2C_MV64XXX=y CONFIG_I2C_PXA=y CONFIG_SPI=y CONFIG_SPI_ORION=y CONFIG_SPI_PL022=y CONFIG_SPI_ROCKCHIP=y CONFIG_SPI_SPIDEV=y CONFIG_PTP_1588_CLOCK=y CONFIG_PINCTRL_SINGLE=y CONFIG_PINCTRL_MAX77620=y CONFIG_GPIO_DWAPB=y CONFIG_GPIO_PL061=y CONFIG_GPIO_XGENE=y CONFIG_GPIO_PCA953X=y CONFIG_GPIO_PCA953X_IRQ=y CONFIG_GPIO_MAX77620=y CONFIG_POWER_RESET_GPIO=y CONFIG_POWER_RESET_GPIO_RESTART=y CONFIG_POWER_RESET_SYSCON=y CONFIG_BATTERY_BQ27XXX=y CONFIG_SENSORS_ARM_SCPI=y CONFIG_SENSORS_LM90=y CONFIG_SENSORS_INA2XX=y CONFIG_THERMAL=y CONFIG_THERMAL_GOV_POWER_ALLOCATOR=y CONFIG_CPU_THERMAL=y CONFIG_THERMAL_EMULATION=y CONFIG_ARMADA_THERMAL=y CONFIG_WATCHDOG=y CONFIG_ARM_SBSA_WATCHDOG=y CONFIG_MFD_CROS_EC=y CONFIG_MFD_MAX77620=y CONFIG_MFD_RK808=y CONFIG_MFD_SEC_CORE=y CONFIG_REGULATOR=y CONFIG_REGULATOR_FIXED_VOLTAGE=y CONFIG_REGULATOR_FAN53555=y CONFIG_REGULATOR_GPIO=y CONFIG_REGULATOR_MAX77620=y CONFIG_BACKLIGHT_LCD_SUPPORT=y # CONFIG_LCD_CLASS_DEVICE is not set # CONFIG_BACKLIGHT_GENERIC is not set CONFIG_USB=y CONFIG_USB_XHCI_HCD=y CONFIG_USB_XHCI_PLATFORM=y CONFIG_USB_STORAGE=y CONFIG_NOP_USB_XCEIV=y CONFIG_MMC=y CONFIG_MMC_BLOCK_MINORS=32 CONFIG_MMC_ARMMMCI=y CONFIG_MMC_SDHCI=y CONFIG_MMC_SDHCI_PLTFM=y CONFIG_MMC_SDHCI_OF_ARASAN=y CONFIG_MMC_SDHCI_CADENCE=y CONFIG_MMC_SPI=y CONFIG_MMC_DW=y CONFIG_MMC_DW_EXYNOS=y CONFIG_MMC_DW_K3=y CONFIG_MMC_SDHCI_XENON=y CONFIG_NEW_LEDS=y CONFIG_LEDS_CLASS=y CONFIG_LEDS_GPIO=y CONFIG_LEDS_SYSCON=y CONFIG_LEDS_TRIGGERS=y CONFIG_LEDS_TRIGGER_HEARTBEAT=y CONFIG_LEDS_TRIGGER_CPU=y CONFIG_LEDS_TRIGGER_DEFAULT_ON=y CONFIG_RTC_CLASS=y CONFIG_RTC_DRV_ARMADA38X=y CONFIG_DMADEVICES=y CONFIG_MV_XOR_V2=y CONFIG_SYNC_FILE=y CONFIG_VFIO=y CONFIG_VFIO_PCI=y CONFIG_VIRTIO_PCI=y CONFIG_VIRTIO_BALLOON=y CONFIG_VIRTIO_MMIO=y CONFIG_COMMON_CLK_VERSATILE=y CONFIG_CLK_SP810=y CONFIG_CLK_VEXPRESS_OSC=y CONFIG_COMMON_CLK_RK808=y CONFIG_COMMON_CLK_SCPI=y CONFIG_COMMON_CLK_CS2000_CP=y CONFIG_COMMON_CLK_S2MPS11=y CONFIG_CLK_QORIQ=y CONFIG_HWSPINLOCK=y CONFIG_ARM_TIMER_SP804=y CONFIG_MAILBOX=y CONFIG_ARM_MHU=y CONFIG_PLATFORM_MHU=y CONFIG_ARM_SMMU=y CONFIG_ARM_SMMU_V3=y CONFIG_EXTCON_USB_GPIO=y CONFIG_RESET_CONTROLLER=y CONFIG_PHY_MVEBU_CP110_COMPHY=y CONFIG_EXT4_FS=y CONFIG_EXT4_FS_POSIX_ACL=y CONFIG_FANOTIFY=y CONFIG_VFAT_FS=y CONFIG_TMPFS=y CONFIG_HUGETLBFS=y CONFIG_CONFIGFS_FS=y # CONFIG_MISC_FILESYSTEMS is not set # CONFIG_NETWORK_FILESYSTEMS is not set CONFIG_NLS_CODEPAGE_437=y CONFIG_NLS_ISO8859_1=y CONFIG_KEYS=y CONFIG_CRYPTO_ECDH=y CONFIG_CRYPTO_CCM=y CONFIG_CRYPTO_GCM=y CONFIG_CRYPTO_ECB=y CONFIG_CRYPTO_CMAC=y CONFIG_CRYPTO_ARC4=y CONFIG_CRYPTO_ANSI_CPRNG=y # CONFIG_CRYPTO_DEV_VIRTIO is not set CONFIG_CRYPTO_DEV_SAFEXCEL=y CONFIG_XZ_DEC=y CONFIG_PRINTK_TIME=y CONFIG_DEBUG_INFO=y CONFIG_DEBUG_FS=y CONFIG_MAGIC_SYSRQ=y CONFIG_DEBUG_KERNEL=y # CONFIG_SCHED_DEBUG is not set # CONFIG_DEBUG_PREEMPT is not set # CONFIG_FTRACE is not set CONFIG_MEMTEST=y [-- Attachment #3: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] Revert "arm64: dts: marvell: add CPU Idle power state support on Armada 7K/8K" 2018-12-04 16:48 ` Baruch Siach @ 2018-12-05 6:57 ` Baruch Siach 2018-12-05 8:27 ` Gregory CLEMENT 0 siblings, 1 reply; 9+ messages in thread From: Baruch Siach @ 2018-12-05 6:57 UTC (permalink / raw) To: Gregory CLEMENT Cc: Andrew Lunn, Jason Cooper, Sergey Matyukevich, linux-pm, orenbh, Daniel Lezcano, Rafael J. Wysocki, Russell King, linux-arm-kernel, Sebastian Hesselbarth Hi Gregory, On Tue, Dec 04, 2018 at 06:48:43PM +0200, Baruch Siach wrote: > On Tue, Dec 04, 2018 at 05:21:25PM +0100, Gregory CLEMENT wrote: > > On mar., déc. 04 2018, Baruch Siach <baruch@tkos.co.il> wrote: > > > This reverts commit 8ed46368776b3bc93d74c1f8f2bfb9fd8a9ad805. > > > > > > This commit breaks boot on Armada 8K based systems. Reverting it makes > > > affected systems boot again. > > > > > > Reported-by: Sergey Matyukevich <geomatsi@gmail.com> > > > Signed-off-by: Baruch Siach <baruch@tkos.co.il> > > > > I would like to know more about it because on the McBin I use there is > > no issue with 4.20-rc1. > > > > Do you observe this issue with the arm64 defconfig or do you have some > > specific configuration option? > > I tested the attached defconfig on Clearfog GT-8K and on another custom Armada > 8K based board. > > Tomorrow I'll test Macchiatobin with this config and the arm64 defconfig. I reproduced same issue on a Macchiatobin board with both this config and the arm64 defconfig. The latter produced this splat: [ 22.868568] rcu: INFO: rcu_preempt detected stalls on CPUs/tasks: [ 22.874694] rcu: 0-...!: (1 GPs behind) idle=408/0/0x0 softirq=251/251 fqs=0 [ 22.881950] rcu: 2-...!: (0 ticks this GP) idle=7b4/0/0x0 softirq=168/168 fqs=0 [ 22.889465] rcu: (detected by 1, t=5252 jiffies, g=-967, q=294) [ 22.895498] Task dump for CPU 0: [ 22.898739] swapper/0 R running task 0 0 0 0x00000028 [ 22.905819] Call trace: [ 22.908285] __switch_to+0x94/0xd0 [ 22.911702] arm_cpuidle_suspend+0x30/0x38 [ 22.915818] arm_enter_idle_state+0x38/0x68 [ 22.920019] cpuidle_enter_state+0x1d0/0x240 [ 22.924306] cpuidle_enter+0x18/0x20 [ 22.927898] call_cpuidle+0x1c/0x38 [ 22.931400] do_idle+0x254/0x280 [ 22.934641] cpu_startup_entry+0x24/0x28 [ 22.938581] rest_init+0xd4/0xe0 [ 22.941824] arch_call_rest_init+0xc/0x14 [ 22.945849] start_kernel+0x3d8/0x400 [ 22.949524] Task dump for CPU 2: [ 22.952764] swapper/2 R running task 0 0 1 0x0000002a [ 22.959843] Call trace: [ 22.962299] __switch_to+0x94/0xd0 [ 22.965715] arm_cpuidle_suspend+0x30/0x38 [ 22.969828] arm_enter_idle_state+0x38/0x68 [ 22.974028] cpuidle_enter_state+0x1d0/0x240 [ 22.978315] cpuidle_enter+0x18/0x20 [ 22.981905] call_cpuidle+0x1c/0x38 [ 22.985408] do_idle+0x254/0x280 [ 22.988648] cpu_startup_entry+0x20/0x28 [ 22.992590] secondary_start_kernel+0x15c/0x170 [ 22.997141] rcu: rcu_preempt kthread starved for 5252 jiffies! g-967 f0x0 RCU_GP_WAIT_FQS(5) ->state=0x402 ->cpu=2 [ 23.007534] rcu: RCU grace-period kthread stack dump: [ 23.012606] rcu_preempt I 0 10 2 0x00000028 [ 23.018116] Call trace: [ 23.020572] __switch_to+0x94/0xd0 [ 23.023987] __schedule+0x1e0/0x5c0 [ 23.027488] schedule+0x38/0xa0 [ 23.030643] schedule_timeout+0x198/0x338 [ 23.034670] rcu_gp_kthread+0x43c/0x7b0 [ 23.038524] kthread+0x124/0x128 [ 23.041766] ret_from_fork+0x10/0x18 baruch > > > --- > > > .../boot/dts/marvell/armada-ap806-quad.dtsi | 4 --- > > > arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 27 ------------------- > > > 2 files changed, 31 deletions(-) > > > > > > diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > > > index 64632c873888..01ea662afba8 100644 > > > --- a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > > > +++ b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > > > @@ -20,28 +20,24 @@ > > > compatible = "arm,cortex-a72", "arm,armv8"; > > > reg = <0x000>; > > > enable-method = "psci"; > > > - cpu-idle-states = <&CPU_SLEEP_0>; > > > }; > > > cpu1: cpu@1 { > > > device_type = "cpu"; > > > compatible = "arm,cortex-a72", "arm,armv8"; > > > reg = <0x001>; > > > enable-method = "psci"; > > > - cpu-idle-states = <&CPU_SLEEP_0>; > > > }; > > > cpu2: cpu@100 { > > > device_type = "cpu"; > > > compatible = "arm,cortex-a72", "arm,armv8"; > > > reg = <0x100>; > > > enable-method = "psci"; > > > - cpu-idle-states = <&CPU_SLEEP_0>; > > > }; > > > cpu3: cpu@101 { > > > device_type = "cpu"; > > > compatible = "arm,cortex-a72", "arm,armv8"; > > > reg = <0x101>; > > > enable-method = "psci"; > > > - cpu-idle-states = <&CPU_SLEEP_0>; > > > }; > > > }; > > > }; > > > diff --git a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > > > index 073610ac0a53..7d94c1fa592a 100644 > > > --- a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > > > +++ b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > > > @@ -28,33 +28,6 @@ > > > method = "smc"; > > > }; > > > > > > - cpus { > > > - #address-cells = <1>; > > > - #size-cells = <0>; > > > - > > > - idle_states { > > > - entry_method = "arm,pcsi"; > > > - > > > - CPU_SLEEP_0: cpu-sleep-0 { > > > - compatible = "arm,idle-state"; > > > - local-timer-stop; > > > - arm,psci-suspend-param = <0x0010000>; > > > - entry-latency-us = <80>; > > > - exit-latency-us = <160>; > > > - min-residency-us = <320>; > > > - }; > > > - > > > - CLUSTER_SLEEP_0: cluster-sleep-0 { > > > - compatible = "arm,idle-state"; > > > - local-timer-stop; > > > - arm,psci-suspend-param = <0x1010000>; > > > - entry-latency-us = <500>; > > > - exit-latency-us = <1000>; > > > - min-residency-us = <2500>; > > > - }; > > > - }; > > > - }; > > > - > > > ap806 { > > > #address-cells = <2>; > > > #size-cells = <2>; > > -- > http://baruch.siach.name/blog/ ~. .~ Tk Open Systems > =}------------------------------------------------ooO--U--Ooo------------{= > - baruch@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - > CONFIG_SYSVIPC=y > CONFIG_POSIX_MQUEUE=y > CONFIG_AUDIT=y > CONFIG_NO_HZ_IDLE=y > CONFIG_HIGH_RES_TIMERS=y > CONFIG_PREEMPT=y > CONFIG_BSD_PROCESS_ACCT=y > CONFIG_BSD_PROCESS_ACCT_V3=y > CONFIG_TASKSTATS=y > CONFIG_TASK_DELAY_ACCT=y > CONFIG_TASK_XACCT=y > CONFIG_TASK_IO_ACCOUNTING=y > CONFIG_IKCONFIG=y > CONFIG_IKCONFIG_PROC=y > CONFIG_NUMA_BALANCING=y > CONFIG_MEMCG=y > CONFIG_MEMCG_SWAP=y > CONFIG_BLK_CGROUP=y > CONFIG_CGROUP_PIDS=y > CONFIG_CGROUP_HUGETLB=y > CONFIG_CPUSETS=y > CONFIG_CGROUP_DEVICE=y > CONFIG_CGROUP_CPUACCT=y > CONFIG_CGROUP_PERF=y > CONFIG_USER_NS=y > CONFIG_SCHED_AUTOGROUP=y > CONFIG_BLK_DEV_INITRD=y > CONFIG_INITRAMFS_SOURCE="rootfs-armv8.cpio" > # CONFIG_RD_BZIP2 is not set > # CONFIG_RD_LZMA is not set > # CONFIG_RD_XZ is not set > # CONFIG_RD_LZO is not set > # CONFIG_RD_LZ4 is not set > CONFIG_CC_OPTIMIZE_FOR_SIZE=y > CONFIG_KALLSYMS_ALL=y > # CONFIG_COMPAT_BRK is not set > CONFIG_PROFILING=y > CONFIG_ARCH_MVEBU=y > CONFIG_PCI=y > CONFIG_PCI_IOV=y > CONFIG_HOTPLUG_PCI=y > CONFIG_PCI_HOST_GENERIC=y > CONFIG_PCIE_ARMADA_8K=y > CONFIG_ARM64_VA_BITS_48=y > CONFIG_SCHED_MC=y > CONFIG_NUMA=y > CONFIG_CRASH_DUMP=y > # CONFIG_ARM64_VHE is not set > CONFIG_COMPAT=y > CONFIG_HIBERNATION=y > CONFIG_CPU_IDLE=y > CONFIG_ARM_CPUIDLE=y > CONFIG_CPU_FREQ=y > CONFIG_CPUFREQ_DT=y > CONFIG_ARM_SCPI_CPUFREQ=y > CONFIG_ARM_SCPI_PROTOCOL=y > CONFIG_EFI_CAPSULE_LOADER=y > CONFIG_ARM64_CRYPTO=y > CONFIG_CRYPTO_SHA1_ARM64_CE=y > CONFIG_CRYPTO_SHA2_ARM64_CE=y > CONFIG_CRYPTO_GHASH_ARM64_CE=y > CONFIG_CRYPTO_AES_ARM64_CE_CCM=y > CONFIG_CRYPTO_AES_ARM64_CE_BLK=y > CONFIG_JUMP_LABEL=y > CONFIG_BLK_DEV_INTEGRITY=y > # CONFIG_IOSCHED_DEADLINE is not set > # CONFIG_CORE_DUMP_DEFAULT_ELF_HEADERS is not set > CONFIG_KSM=y > CONFIG_TRANSPARENT_HUGEPAGE=y > CONFIG_CMA=y > CONFIG_NET=y > CONFIG_PACKET=y > CONFIG_UNIX=y > CONFIG_INET=y > CONFIG_IP_MULTICAST=y > CONFIG_IP_PNP=y > CONFIG_IP_PNP_DHCP=y > CONFIG_IP_PNP_BOOTP=y > CONFIG_NETFILTER=y > CONFIG_NF_CONNTRACK=y > CONFIG_NF_CONNTRACK_EVENTS=y > CONFIG_NETFILTER_XT_TARGET_CHECKSUM=y > CONFIG_NETFILTER_XT_TARGET_LOG=y > CONFIG_NETFILTER_XT_MATCH_ADDRTYPE=y > CONFIG_NETFILTER_XT_MATCH_CONNTRACK=y > CONFIG_IP_NF_IPTABLES=y > CONFIG_IP_NF_FILTER=y > CONFIG_IP_NF_TARGET_REJECT=y > CONFIG_IP_NF_NAT=y > CONFIG_IP_NF_TARGET_MASQUERADE=y > CONFIG_IP_NF_MANGLE=y > CONFIG_IP6_NF_IPTABLES=y > CONFIG_IP6_NF_FILTER=y > CONFIG_IP6_NF_TARGET_REJECT=y > CONFIG_IP6_NF_MANGLE=y > CONFIG_IP6_NF_NAT=y > CONFIG_IP6_NF_TARGET_MASQUERADE=y > CONFIG_BRIDGE=y > CONFIG_BRIDGE_VLAN_FILTERING=y > CONFIG_VLAN_8021Q=y > CONFIG_VLAN_8021Q_GVRP=y > CONFIG_VLAN_8021Q_MVRP=y > CONFIG_DNS_RESOLVER=y > # CONFIG_WIRELESS is not set > CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug" > CONFIG_DEVTMPFS=y > CONFIG_DEVTMPFS_MOUNT=y > CONFIG_DMA_CMA=y > CONFIG_BRCMSTB_GISB_ARB=y > CONFIG_VEXPRESS_CONFIG=y > CONFIG_MTD=y > # CONFIG_MTD_OF_PARTS is not set > CONFIG_MTD_BLOCK=y > CONFIG_MTD_M25P80=y > CONFIG_MTD_SPI_NOR=y > CONFIG_BLK_DEV_LOOP=y > CONFIG_BLK_DEV_NBD=y > CONFIG_VIRTIO_BLK=y > CONFIG_SRAM=y > CONFIG_EEPROM_AT25=y > # CONFIG_SCSI_PROC_FS is not set > CONFIG_BLK_DEV_SD=y > CONFIG_SCSI_SAS_LIBSAS=y > CONFIG_SCSI_SAS_ATA=y > CONFIG_ATA=y > CONFIG_SATA_AHCI=y > CONFIG_SATA_AHCI_PLATFORM=y > CONFIG_AHCI_MVEBU=y > # CONFIG_ATA_SFF is not set > CONFIG_NETDEVICES=y > CONFIG_MACVLAN=y > CONFIG_MACVTAP=y > CONFIG_TUN=y > CONFIG_VETH=y > CONFIG_VIRTIO_NET=y > # CONFIG_NET_VENDOR_3COM is not set > # CONFIG_NET_VENDOR_ADAPTEC is not set > # CONFIG_NET_VENDOR_AGERE is not set > # CONFIG_NET_VENDOR_ALACRITECH is not set > # CONFIG_NET_VENDOR_ALTEON is not set > # CONFIG_NET_VENDOR_AMAZON is not set > # CONFIG_NET_VENDOR_AMD is not set > # CONFIG_NET_VENDOR_AQUANTIA is not set > # CONFIG_NET_VENDOR_ARC is not set > # CONFIG_NET_VENDOR_ATHEROS is not set > # CONFIG_NET_VENDOR_BROADCOM is not set > # CONFIG_NET_VENDOR_BROCADE is not set > # CONFIG_NET_VENDOR_CAVIUM is not set > # CONFIG_NET_VENDOR_CHELSIO is not set > # CONFIG_NET_VENDOR_CISCO is not set > # CONFIG_NET_VENDOR_DEC is not set > # CONFIG_NET_VENDOR_DLINK is not set > # CONFIG_NET_VENDOR_EMULEX is not set > # CONFIG_NET_VENDOR_EZCHIP is not set > # CONFIG_NET_VENDOR_HISILICON is not set > # CONFIG_NET_VENDOR_HP is not set > # CONFIG_NET_VENDOR_HUAWEI is not set > # CONFIG_NET_VENDOR_INTEL is not set > CONFIG_MVPP2=y > # CONFIG_NET_VENDOR_MELLANOX is not set > # CONFIG_NET_VENDOR_MICREL is not set > # CONFIG_NET_VENDOR_MICROCHIP is not set > # CONFIG_NET_VENDOR_MYRI is not set > # CONFIG_NET_VENDOR_NATSEMI is not set > # CONFIG_NET_VENDOR_NETRONOME is not set > # CONFIG_NET_VENDOR_NVIDIA is not set > # CONFIG_NET_VENDOR_OKI is not set > # CONFIG_NET_VENDOR_QLOGIC is not set > # CONFIG_NET_VENDOR_QUALCOMM is not set > # CONFIG_NET_VENDOR_RDC is not set > # CONFIG_NET_VENDOR_REALTEK is not set > # CONFIG_NET_VENDOR_RENESAS is not set > # CONFIG_NET_VENDOR_ROCKER is not set > # CONFIG_NET_VENDOR_SAMSUNG is not set > # CONFIG_NET_VENDOR_SEEQ is not set > # CONFIG_NET_VENDOR_SOLARFLARE is not set > # CONFIG_NET_VENDOR_SILAN is not set > # CONFIG_NET_VENDOR_SIS is not set > # CONFIG_NET_VENDOR_SMSC is not set > # CONFIG_NET_VENDOR_STMICRO is not set > # CONFIG_NET_VENDOR_SUN is not set > # CONFIG_NET_VENDOR_SYNOPSYS is not set > # CONFIG_NET_VENDOR_TEHUTI is not set > # CONFIG_NET_VENDOR_TI is not set > # CONFIG_NET_VENDOR_VIA is not set > # CONFIG_NET_VENDOR_WIZNET is not set > CONFIG_MDIO_BUS_MUX_MMIOREG=y > CONFIG_SFP=y > CONFIG_MARVELL_PHY=y > CONFIG_MARVELL_10G_PHY=y > # CONFIG_USB_NET_DRIVERS is not set > # CONFIG_WLAN is not set > CONFIG_INPUT_POLLDEV=y > CONFIG_INPUT_EVDEV=y > CONFIG_KEYBOARD_GPIO=y > CONFIG_KEYBOARD_CROS_EC=y > CONFIG_INPUT_MISC=y > # CONFIG_SERIO_SERPORT is not set > CONFIG_SERIO_AMBAKMI=y > CONFIG_VT_HW_CONSOLE_BINDING=y > CONFIG_LEGACY_PTY_COUNT=16 > CONFIG_SERIAL_8250=y > CONFIG_SERIAL_8250_CONSOLE=y > CONFIG_SERIAL_8250_EXTENDED=y > CONFIG_SERIAL_8250_SHARE_IRQ=y > CONFIG_SERIAL_8250_DW=y > CONFIG_SERIAL_OF_PLATFORM=y > CONFIG_SERIAL_AMBA_PL011=y > CONFIG_SERIAL_AMBA_PL011_CONSOLE=y > CONFIG_SERIAL_MVEBU_UART=y > # CONFIG_HW_RANDOM_CAVIUM is not set > CONFIG_I2C=y > # CONFIG_I2C_COMPAT is not set > CONFIG_I2C_CHARDEV=y > CONFIG_I2C_MUX=y > CONFIG_I2C_MUX_PCA954x=y > CONFIG_I2C_MV64XXX=y > CONFIG_I2C_PXA=y > CONFIG_SPI=y > CONFIG_SPI_ORION=y > CONFIG_SPI_PL022=y > CONFIG_SPI_ROCKCHIP=y > CONFIG_SPI_SPIDEV=y > CONFIG_PTP_1588_CLOCK=y > CONFIG_PINCTRL_SINGLE=y > CONFIG_PINCTRL_MAX77620=y > CONFIG_GPIO_DWAPB=y > CONFIG_GPIO_PL061=y > CONFIG_GPIO_XGENE=y > CONFIG_GPIO_PCA953X=y > CONFIG_GPIO_PCA953X_IRQ=y > CONFIG_GPIO_MAX77620=y > CONFIG_POWER_RESET_GPIO=y > CONFIG_POWER_RESET_GPIO_RESTART=y > CONFIG_POWER_RESET_SYSCON=y > CONFIG_BATTERY_BQ27XXX=y > CONFIG_SENSORS_ARM_SCPI=y > CONFIG_SENSORS_LM90=y > CONFIG_SENSORS_INA2XX=y > CONFIG_THERMAL=y > CONFIG_THERMAL_GOV_POWER_ALLOCATOR=y > CONFIG_CPU_THERMAL=y > CONFIG_THERMAL_EMULATION=y > CONFIG_ARMADA_THERMAL=y > CONFIG_WATCHDOG=y > CONFIG_ARM_SBSA_WATCHDOG=y > CONFIG_MFD_CROS_EC=y > CONFIG_MFD_MAX77620=y > CONFIG_MFD_RK808=y > CONFIG_MFD_SEC_CORE=y > CONFIG_REGULATOR=y > CONFIG_REGULATOR_FIXED_VOLTAGE=y > CONFIG_REGULATOR_FAN53555=y > CONFIG_REGULATOR_GPIO=y > CONFIG_REGULATOR_MAX77620=y > CONFIG_BACKLIGHT_LCD_SUPPORT=y > # CONFIG_LCD_CLASS_DEVICE is not set > # CONFIG_BACKLIGHT_GENERIC is not set > CONFIG_USB=y > CONFIG_USB_XHCI_HCD=y > CONFIG_USB_XHCI_PLATFORM=y > CONFIG_USB_STORAGE=y > CONFIG_NOP_USB_XCEIV=y > CONFIG_MMC=y > CONFIG_MMC_BLOCK_MINORS=32 > CONFIG_MMC_ARMMMCI=y > CONFIG_MMC_SDHCI=y > CONFIG_MMC_SDHCI_PLTFM=y > CONFIG_MMC_SDHCI_OF_ARASAN=y > CONFIG_MMC_SDHCI_CADENCE=y > CONFIG_MMC_SPI=y > CONFIG_MMC_DW=y > CONFIG_MMC_DW_EXYNOS=y > CONFIG_MMC_DW_K3=y > CONFIG_MMC_SDHCI_XENON=y > CONFIG_NEW_LEDS=y > CONFIG_LEDS_CLASS=y > CONFIG_LEDS_GPIO=y > CONFIG_LEDS_SYSCON=y > CONFIG_LEDS_TRIGGERS=y > CONFIG_LEDS_TRIGGER_HEARTBEAT=y > CONFIG_LEDS_TRIGGER_CPU=y > CONFIG_LEDS_TRIGGER_DEFAULT_ON=y > CONFIG_RTC_CLASS=y > CONFIG_RTC_DRV_ARMADA38X=y > CONFIG_DMADEVICES=y > CONFIG_MV_XOR_V2=y > CONFIG_SYNC_FILE=y > CONFIG_VFIO=y > CONFIG_VFIO_PCI=y > CONFIG_VIRTIO_PCI=y > CONFIG_VIRTIO_BALLOON=y > CONFIG_VIRTIO_MMIO=y > CONFIG_COMMON_CLK_VERSATILE=y > CONFIG_CLK_SP810=y > CONFIG_CLK_VEXPRESS_OSC=y > CONFIG_COMMON_CLK_RK808=y > CONFIG_COMMON_CLK_SCPI=y > CONFIG_COMMON_CLK_CS2000_CP=y > CONFIG_COMMON_CLK_S2MPS11=y > CONFIG_CLK_QORIQ=y > CONFIG_HWSPINLOCK=y > CONFIG_ARM_TIMER_SP804=y > CONFIG_MAILBOX=y > CONFIG_ARM_MHU=y > CONFIG_PLATFORM_MHU=y > CONFIG_ARM_SMMU=y > CONFIG_ARM_SMMU_V3=y > CONFIG_EXTCON_USB_GPIO=y > CONFIG_RESET_CONTROLLER=y > CONFIG_PHY_MVEBU_CP110_COMPHY=y > CONFIG_EXT4_FS=y > CONFIG_EXT4_FS_POSIX_ACL=y > CONFIG_FANOTIFY=y > CONFIG_VFAT_FS=y > CONFIG_TMPFS=y > CONFIG_HUGETLBFS=y > CONFIG_CONFIGFS_FS=y > # CONFIG_MISC_FILESYSTEMS is not set > # CONFIG_NETWORK_FILESYSTEMS is not set > CONFIG_NLS_CODEPAGE_437=y > CONFIG_NLS_ISO8859_1=y > CONFIG_KEYS=y > CONFIG_CRYPTO_ECDH=y > CONFIG_CRYPTO_CCM=y > CONFIG_CRYPTO_GCM=y > CONFIG_CRYPTO_ECB=y > CONFIG_CRYPTO_CMAC=y > CONFIG_CRYPTO_ARC4=y > CONFIG_CRYPTO_ANSI_CPRNG=y > # CONFIG_CRYPTO_DEV_VIRTIO is not set > CONFIG_CRYPTO_DEV_SAFEXCEL=y > CONFIG_XZ_DEC=y > CONFIG_PRINTK_TIME=y > CONFIG_DEBUG_INFO=y > CONFIG_DEBUG_FS=y > CONFIG_MAGIC_SYSRQ=y > CONFIG_DEBUG_KERNEL=y > # CONFIG_SCHED_DEBUG is not set > # CONFIG_DEBUG_PREEMPT is not set > # CONFIG_FTRACE is not set > CONFIG_MEMTEST=y > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] Revert "arm64: dts: marvell: add CPU Idle power state support on Armada 7K/8K" 2018-12-05 6:57 ` Baruch Siach @ 2018-12-05 8:27 ` Gregory CLEMENT 2018-12-05 8:40 ` Baruch Siach 2018-12-05 10:10 ` Sudeep Holla 0 siblings, 2 replies; 9+ messages in thread From: Gregory CLEMENT @ 2018-12-05 8:27 UTC (permalink / raw) To: Baruch Siach Cc: Andrew Lunn, Jason Cooper, Sergey Matyukevich, linux-pm, orenbh, Daniel Lezcano, Rafael J. Wysocki, Russell King, linux-arm-kernel, Sebastian Hesselbarth Hi Baruch, On mer., déc. 05 2018, Baruch Siach <baruch@tkos.co.il> wrote: > Hi Gregory, > > On Tue, Dec 04, 2018 at 06:48:43PM +0200, Baruch Siach wrote: >> On Tue, Dec 04, 2018 at 05:21:25PM +0100, Gregory CLEMENT wrote: >> > On mar., déc. 04 2018, Baruch Siach <baruch@tkos.co.il> wrote: >> > > This reverts commit 8ed46368776b3bc93d74c1f8f2bfb9fd8a9ad805. >> > > >> > > This commit breaks boot on Armada 8K based systems. Reverting it makes >> > > affected systems boot again. >> > > >> > > Reported-by: Sergey Matyukevich <geomatsi@gmail.com> >> > > Signed-off-by: Baruch Siach <baruch@tkos.co.il> >> > >> > I would like to know more about it because on the McBin I use there is >> > no issue with 4.20-rc1. >> > >> > Do you observe this issue with the arm64 defconfig or do you have some >> > specific configuration option? >> >> I tested the attached defconfig on Clearfog GT-8K and on another custom Armada >> 8K based board. >> >> Tomorrow I'll test Macchiatobin with this config and the arm64 defconfig. > > I reproduced same issue on a Macchiatobin board with both this config and the > arm64 defconfig. The latter produced this splat: OK thanks, for this, I also got the confirmation by my colleagues. The only difference I can think of, is the firmware, but the surprising thing, is that I have an old firmware: 17.06.2 and that with a recent one, it doesn't work. So I will apply it in mvebu/fixes. Thanks, Gregory > > [ 22.868568] rcu: INFO: rcu_preempt detected stalls on CPUs/tasks: > [ 22.874694] rcu: 0-...!: (1 GPs behind) idle=408/0/0x0 softirq=251/251 fqs=0 > [ 22.881950] rcu: 2-...!: (0 ticks this GP) idle=7b4/0/0x0 softirq=168/168 fqs=0 > [ 22.889465] rcu: (detected by 1, t=5252 jiffies, g=-967, q=294) > [ 22.895498] Task dump for CPU 0: > [ 22.898739] swapper/0 R running task 0 0 0 0x00000028 > [ 22.905819] Call trace: > [ 22.908285] __switch_to+0x94/0xd0 > [ 22.911702] arm_cpuidle_suspend+0x30/0x38 > [ 22.915818] arm_enter_idle_state+0x38/0x68 > [ 22.920019] cpuidle_enter_state+0x1d0/0x240 > [ 22.924306] cpuidle_enter+0x18/0x20 > [ 22.927898] call_cpuidle+0x1c/0x38 > [ 22.931400] do_idle+0x254/0x280 > [ 22.934641] cpu_startup_entry+0x24/0x28 > [ 22.938581] rest_init+0xd4/0xe0 > [ 22.941824] arch_call_rest_init+0xc/0x14 > [ 22.945849] start_kernel+0x3d8/0x400 > [ 22.949524] Task dump for CPU 2: > [ 22.952764] swapper/2 R running task 0 0 1 0x0000002a > [ 22.959843] Call trace: > [ 22.962299] __switch_to+0x94/0xd0 > [ 22.965715] arm_cpuidle_suspend+0x30/0x38 > [ 22.969828] arm_enter_idle_state+0x38/0x68 > [ 22.974028] cpuidle_enter_state+0x1d0/0x240 > [ 22.978315] cpuidle_enter+0x18/0x20 > [ 22.981905] call_cpuidle+0x1c/0x38 > [ 22.985408] do_idle+0x254/0x280 > [ 22.988648] cpu_startup_entry+0x20/0x28 > [ 22.992590] secondary_start_kernel+0x15c/0x170 > [ 22.997141] rcu: rcu_preempt kthread starved for 5252 jiffies! g-967 f0x0 RCU_GP_WAIT_FQS(5) ->state=0x402 ->cpu=2 > [ 23.007534] rcu: RCU grace-period kthread stack dump: > [ 23.012606] rcu_preempt I 0 10 2 0x00000028 > [ 23.018116] Call trace: > [ 23.020572] __switch_to+0x94/0xd0 > [ 23.023987] __schedule+0x1e0/0x5c0 > [ 23.027488] schedule+0x38/0xa0 > [ 23.030643] schedule_timeout+0x198/0x338 > [ 23.034670] rcu_gp_kthread+0x43c/0x7b0 > [ 23.038524] kthread+0x124/0x128 > [ 23.041766] ret_from_fork+0x10/0x18 > > baruch > >> > > --- >> > > .../boot/dts/marvell/armada-ap806-quad.dtsi | 4 --- >> > > arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 27 ------------------- >> > > 2 files changed, 31 deletions(-) >> > > >> > > diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi >> > > index 64632c873888..01ea662afba8 100644 >> > > --- a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi >> > > +++ b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi >> > > @@ -20,28 +20,24 @@ >> > > compatible = "arm,cortex-a72", "arm,armv8"; >> > > reg = <0x000>; >> > > enable-method = "psci"; >> > > - cpu-idle-states = <&CPU_SLEEP_0>; >> > > }; >> > > cpu1: cpu@1 { >> > > device_type = "cpu"; >> > > compatible = "arm,cortex-a72", "arm,armv8"; >> > > reg = <0x001>; >> > > enable-method = "psci"; >> > > - cpu-idle-states = <&CPU_SLEEP_0>; >> > > }; >> > > cpu2: cpu@100 { >> > > device_type = "cpu"; >> > > compatible = "arm,cortex-a72", "arm,armv8"; >> > > reg = <0x100>; >> > > enable-method = "psci"; >> > > - cpu-idle-states = <&CPU_SLEEP_0>; >> > > }; >> > > cpu3: cpu@101 { >> > > device_type = "cpu"; >> > > compatible = "arm,cortex-a72", "arm,armv8"; >> > > reg = <0x101>; >> > > enable-method = "psci"; >> > > - cpu-idle-states = <&CPU_SLEEP_0>; >> > > }; >> > > }; >> > > }; >> > > diff --git a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi >> > > index 073610ac0a53..7d94c1fa592a 100644 >> > > --- a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi >> > > +++ b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi >> > > @@ -28,33 +28,6 @@ >> > > method = "smc"; >> > > }; >> > > >> > > - cpus { >> > > - #address-cells = <1>; >> > > - #size-cells = <0>; >> > > - >> > > - idle_states { >> > > - entry_method = "arm,pcsi"; >> > > - >> > > - CPU_SLEEP_0: cpu-sleep-0 { >> > > - compatible = "arm,idle-state"; >> > > - local-timer-stop; >> > > - arm,psci-suspend-param = <0x0010000>; >> > > - entry-latency-us = <80>; >> > > - exit-latency-us = <160>; >> > > - min-residency-us = <320>; >> > > - }; >> > > - >> > > - CLUSTER_SLEEP_0: cluster-sleep-0 { >> > > - compatible = "arm,idle-state"; >> > > - local-timer-stop; >> > > - arm,psci-suspend-param = <0x1010000>; >> > > - entry-latency-us = <500>; >> > > - exit-latency-us = <1000>; >> > > - min-residency-us = <2500>; >> > > - }; >> > > - }; >> > > - }; >> > > - >> > > ap806 { >> > > #address-cells = <2>; >> > > #size-cells = <2>; >> >> -- >> http://baruch.siach.name/blog/ ~. .~ Tk Open Systems >> =}------------------------------------------------ooO--U--Ooo------------{= >> - baruch@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - > >> CONFIG_SYSVIPC=y >> CONFIG_POSIX_MQUEUE=y >> CONFIG_AUDIT=y >> CONFIG_NO_HZ_IDLE=y >> CONFIG_HIGH_RES_TIMERS=y >> CONFIG_PREEMPT=y >> CONFIG_BSD_PROCESS_ACCT=y >> CONFIG_BSD_PROCESS_ACCT_V3=y >> CONFIG_TASKSTATS=y >> CONFIG_TASK_DELAY_ACCT=y >> CONFIG_TASK_XACCT=y >> CONFIG_TASK_IO_ACCOUNTING=y >> CONFIG_IKCONFIG=y >> CONFIG_IKCONFIG_PROC=y >> CONFIG_NUMA_BALANCING=y >> CONFIG_MEMCG=y >> CONFIG_MEMCG_SWAP=y >> CONFIG_BLK_CGROUP=y >> CONFIG_CGROUP_PIDS=y >> CONFIG_CGROUP_HUGETLB=y >> CONFIG_CPUSETS=y >> CONFIG_CGROUP_DEVICE=y >> CONFIG_CGROUP_CPUACCT=y >> CONFIG_CGROUP_PERF=y >> CONFIG_USER_NS=y >> CONFIG_SCHED_AUTOGROUP=y >> CONFIG_BLK_DEV_INITRD=y >> CONFIG_INITRAMFS_SOURCE="rootfs-armv8.cpio" >> # CONFIG_RD_BZIP2 is not set >> # CONFIG_RD_LZMA is not set >> # CONFIG_RD_XZ is not set >> # CONFIG_RD_LZO is not set >> # CONFIG_RD_LZ4 is not set >> CONFIG_CC_OPTIMIZE_FOR_SIZE=y >> CONFIG_KALLSYMS_ALL=y >> # CONFIG_COMPAT_BRK is not set >> CONFIG_PROFILING=y >> CONFIG_ARCH_MVEBU=y >> CONFIG_PCI=y >> CONFIG_PCI_IOV=y >> CONFIG_HOTPLUG_PCI=y >> CONFIG_PCI_HOST_GENERIC=y >> CONFIG_PCIE_ARMADA_8K=y >> CONFIG_ARM64_VA_BITS_48=y >> CONFIG_SCHED_MC=y >> CONFIG_NUMA=y >> CONFIG_CRASH_DUMP=y >> # CONFIG_ARM64_VHE is not set >> CONFIG_COMPAT=y >> CONFIG_HIBERNATION=y >> CONFIG_CPU_IDLE=y >> CONFIG_ARM_CPUIDLE=y >> CONFIG_CPU_FREQ=y >> CONFIG_CPUFREQ_DT=y >> CONFIG_ARM_SCPI_CPUFREQ=y >> CONFIG_ARM_SCPI_PROTOCOL=y >> CONFIG_EFI_CAPSULE_LOADER=y >> CONFIG_ARM64_CRYPTO=y >> CONFIG_CRYPTO_SHA1_ARM64_CE=y >> CONFIG_CRYPTO_SHA2_ARM64_CE=y >> CONFIG_CRYPTO_GHASH_ARM64_CE=y >> CONFIG_CRYPTO_AES_ARM64_CE_CCM=y >> CONFIG_CRYPTO_AES_ARM64_CE_BLK=y >> CONFIG_JUMP_LABEL=y >> CONFIG_BLK_DEV_INTEGRITY=y >> # CONFIG_IOSCHED_DEADLINE is not set >> # CONFIG_CORE_DUMP_DEFAULT_ELF_HEADERS is not set >> CONFIG_KSM=y >> CONFIG_TRANSPARENT_HUGEPAGE=y >> CONFIG_CMA=y >> CONFIG_NET=y >> CONFIG_PACKET=y >> CONFIG_UNIX=y >> CONFIG_INET=y >> CONFIG_IP_MULTICAST=y >> CONFIG_IP_PNP=y >> CONFIG_IP_PNP_DHCP=y >> CONFIG_IP_PNP_BOOTP=y >> CONFIG_NETFILTER=y >> CONFIG_NF_CONNTRACK=y >> CONFIG_NF_CONNTRACK_EVENTS=y >> CONFIG_NETFILTER_XT_TARGET_CHECKSUM=y >> CONFIG_NETFILTER_XT_TARGET_LOG=y >> CONFIG_NETFILTER_XT_MATCH_ADDRTYPE=y >> CONFIG_NETFILTER_XT_MATCH_CONNTRACK=y >> CONFIG_IP_NF_IPTABLES=y >> CONFIG_IP_NF_FILTER=y >> CONFIG_IP_NF_TARGET_REJECT=y >> CONFIG_IP_NF_NAT=y >> CONFIG_IP_NF_TARGET_MASQUERADE=y >> CONFIG_IP_NF_MANGLE=y >> CONFIG_IP6_NF_IPTABLES=y >> CONFIG_IP6_NF_FILTER=y >> CONFIG_IP6_NF_TARGET_REJECT=y >> CONFIG_IP6_NF_MANGLE=y >> CONFIG_IP6_NF_NAT=y >> CONFIG_IP6_NF_TARGET_MASQUERADE=y >> CONFIG_BRIDGE=y >> CONFIG_BRIDGE_VLAN_FILTERING=y >> CONFIG_VLAN_8021Q=y >> CONFIG_VLAN_8021Q_GVRP=y >> CONFIG_VLAN_8021Q_MVRP=y >> CONFIG_DNS_RESOLVER=y >> # CONFIG_WIRELESS is not set >> CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug" >> CONFIG_DEVTMPFS=y >> CONFIG_DEVTMPFS_MOUNT=y >> CONFIG_DMA_CMA=y >> CONFIG_BRCMSTB_GISB_ARB=y >> CONFIG_VEXPRESS_CONFIG=y >> CONFIG_MTD=y >> # CONFIG_MTD_OF_PARTS is not set >> CONFIG_MTD_BLOCK=y >> CONFIG_MTD_M25P80=y >> CONFIG_MTD_SPI_NOR=y >> CONFIG_BLK_DEV_LOOP=y >> CONFIG_BLK_DEV_NBD=y >> CONFIG_VIRTIO_BLK=y >> CONFIG_SRAM=y >> CONFIG_EEPROM_AT25=y >> # CONFIG_SCSI_PROC_FS is not set >> CONFIG_BLK_DEV_SD=y >> CONFIG_SCSI_SAS_LIBSAS=y >> CONFIG_SCSI_SAS_ATA=y >> CONFIG_ATA=y >> CONFIG_SATA_AHCI=y >> CONFIG_SATA_AHCI_PLATFORM=y >> CONFIG_AHCI_MVEBU=y >> # CONFIG_ATA_SFF is not set >> CONFIG_NETDEVICES=y >> CONFIG_MACVLAN=y >> CONFIG_MACVTAP=y >> CONFIG_TUN=y >> CONFIG_VETH=y >> CONFIG_VIRTIO_NET=y >> # CONFIG_NET_VENDOR_3COM is not set >> # CONFIG_NET_VENDOR_ADAPTEC is not set >> # CONFIG_NET_VENDOR_AGERE is not set >> # CONFIG_NET_VENDOR_ALACRITECH is not set >> # CONFIG_NET_VENDOR_ALTEON is not set >> # CONFIG_NET_VENDOR_AMAZON is not set >> # CONFIG_NET_VENDOR_AMD is not set >> # CONFIG_NET_VENDOR_AQUANTIA is not set >> # CONFIG_NET_VENDOR_ARC is not set >> # CONFIG_NET_VENDOR_ATHEROS is not set >> # CONFIG_NET_VENDOR_BROADCOM is not set >> # CONFIG_NET_VENDOR_BROCADE is not set >> # CONFIG_NET_VENDOR_CAVIUM is not set >> # CONFIG_NET_VENDOR_CHELSIO is not set >> # CONFIG_NET_VENDOR_CISCO is not set >> # CONFIG_NET_VENDOR_DEC is not set >> # CONFIG_NET_VENDOR_DLINK is not set >> # CONFIG_NET_VENDOR_EMULEX is not set >> # CONFIG_NET_VENDOR_EZCHIP is not set >> # CONFIG_NET_VENDOR_HISILICON is not set >> # CONFIG_NET_VENDOR_HP is not set >> # CONFIG_NET_VENDOR_HUAWEI is not set >> # CONFIG_NET_VENDOR_INTEL is not set >> CONFIG_MVPP2=y >> # CONFIG_NET_VENDOR_MELLANOX is not set >> # CONFIG_NET_VENDOR_MICREL is not set >> # CONFIG_NET_VENDOR_MICROCHIP is not set >> # CONFIG_NET_VENDOR_MYRI is not set >> # CONFIG_NET_VENDOR_NATSEMI is not set >> # CONFIG_NET_VENDOR_NETRONOME is not set >> # CONFIG_NET_VENDOR_NVIDIA is not set >> # CONFIG_NET_VENDOR_OKI is not set >> # CONFIG_NET_VENDOR_QLOGIC is not set >> # CONFIG_NET_VENDOR_QUALCOMM is not set >> # CONFIG_NET_VENDOR_RDC is not set >> # CONFIG_NET_VENDOR_REALTEK is not set >> # CONFIG_NET_VENDOR_RENESAS is not set >> # CONFIG_NET_VENDOR_ROCKER is not set >> # CONFIG_NET_VENDOR_SAMSUNG is not set >> # CONFIG_NET_VENDOR_SEEQ is not set >> # CONFIG_NET_VENDOR_SOLARFLARE is not set >> # CONFIG_NET_VENDOR_SILAN is not set >> # CONFIG_NET_VENDOR_SIS is not set >> # CONFIG_NET_VENDOR_SMSC is not set >> # CONFIG_NET_VENDOR_STMICRO is not set >> # CONFIG_NET_VENDOR_SUN is not set >> # CONFIG_NET_VENDOR_SYNOPSYS is not set >> # CONFIG_NET_VENDOR_TEHUTI is not set >> # CONFIG_NET_VENDOR_TI is not set >> # CONFIG_NET_VENDOR_VIA is not set >> # CONFIG_NET_VENDOR_WIZNET is not set >> CONFIG_MDIO_BUS_MUX_MMIOREG=y >> CONFIG_SFP=y >> CONFIG_MARVELL_PHY=y >> CONFIG_MARVELL_10G_PHY=y >> # CONFIG_USB_NET_DRIVERS is not set >> # CONFIG_WLAN is not set >> CONFIG_INPUT_POLLDEV=y >> CONFIG_INPUT_EVDEV=y >> CONFIG_KEYBOARD_GPIO=y >> CONFIG_KEYBOARD_CROS_EC=y >> CONFIG_INPUT_MISC=y >> # CONFIG_SERIO_SERPORT is not set >> CONFIG_SERIO_AMBAKMI=y >> CONFIG_VT_HW_CONSOLE_BINDING=y >> CONFIG_LEGACY_PTY_COUNT=16 >> CONFIG_SERIAL_8250=y >> CONFIG_SERIAL_8250_CONSOLE=y >> CONFIG_SERIAL_8250_EXTENDED=y >> CONFIG_SERIAL_8250_SHARE_IRQ=y >> CONFIG_SERIAL_8250_DW=y >> CONFIG_SERIAL_OF_PLATFORM=y >> CONFIG_SERIAL_AMBA_PL011=y >> CONFIG_SERIAL_AMBA_PL011_CONSOLE=y >> CONFIG_SERIAL_MVEBU_UART=y >> # CONFIG_HW_RANDOM_CAVIUM is not set >> CONFIG_I2C=y >> # CONFIG_I2C_COMPAT is not set >> CONFIG_I2C_CHARDEV=y >> CONFIG_I2C_MUX=y >> CONFIG_I2C_MUX_PCA954x=y >> CONFIG_I2C_MV64XXX=y >> CONFIG_I2C_PXA=y >> CONFIG_SPI=y >> CONFIG_SPI_ORION=y >> CONFIG_SPI_PL022=y >> CONFIG_SPI_ROCKCHIP=y >> CONFIG_SPI_SPIDEV=y >> CONFIG_PTP_1588_CLOCK=y >> CONFIG_PINCTRL_SINGLE=y >> CONFIG_PINCTRL_MAX77620=y >> CONFIG_GPIO_DWAPB=y >> CONFIG_GPIO_PL061=y >> CONFIG_GPIO_XGENE=y >> CONFIG_GPIO_PCA953X=y >> CONFIG_GPIO_PCA953X_IRQ=y >> CONFIG_GPIO_MAX77620=y >> CONFIG_POWER_RESET_GPIO=y >> CONFIG_POWER_RESET_GPIO_RESTART=y >> CONFIG_POWER_RESET_SYSCON=y >> CONFIG_BATTERY_BQ27XXX=y >> CONFIG_SENSORS_ARM_SCPI=y >> CONFIG_SENSORS_LM90=y >> CONFIG_SENSORS_INA2XX=y >> CONFIG_THERMAL=y >> CONFIG_THERMAL_GOV_POWER_ALLOCATOR=y >> CONFIG_CPU_THERMAL=y >> CONFIG_THERMAL_EMULATION=y >> CONFIG_ARMADA_THERMAL=y >> CONFIG_WATCHDOG=y >> CONFIG_ARM_SBSA_WATCHDOG=y >> CONFIG_MFD_CROS_EC=y >> CONFIG_MFD_MAX77620=y >> CONFIG_MFD_RK808=y >> CONFIG_MFD_SEC_CORE=y >> CONFIG_REGULATOR=y >> CONFIG_REGULATOR_FIXED_VOLTAGE=y >> CONFIG_REGULATOR_FAN53555=y >> CONFIG_REGULATOR_GPIO=y >> CONFIG_REGULATOR_MAX77620=y >> CONFIG_BACKLIGHT_LCD_SUPPORT=y >> # CONFIG_LCD_CLASS_DEVICE is not set >> # CONFIG_BACKLIGHT_GENERIC is not set >> CONFIG_USB=y >> CONFIG_USB_XHCI_HCD=y >> CONFIG_USB_XHCI_PLATFORM=y >> CONFIG_USB_STORAGE=y >> CONFIG_NOP_USB_XCEIV=y >> CONFIG_MMC=y >> CONFIG_MMC_BLOCK_MINORS=32 >> CONFIG_MMC_ARMMMCI=y >> CONFIG_MMC_SDHCI=y >> CONFIG_MMC_SDHCI_PLTFM=y >> CONFIG_MMC_SDHCI_OF_ARASAN=y >> CONFIG_MMC_SDHCI_CADENCE=y >> CONFIG_MMC_SPI=y >> CONFIG_MMC_DW=y >> CONFIG_MMC_DW_EXYNOS=y >> CONFIG_MMC_DW_K3=y >> CONFIG_MMC_SDHCI_XENON=y >> CONFIG_NEW_LEDS=y >> CONFIG_LEDS_CLASS=y >> CONFIG_LEDS_GPIO=y >> CONFIG_LEDS_SYSCON=y >> CONFIG_LEDS_TRIGGERS=y >> CONFIG_LEDS_TRIGGER_HEARTBEAT=y >> CONFIG_LEDS_TRIGGER_CPU=y >> CONFIG_LEDS_TRIGGER_DEFAULT_ON=y >> CONFIG_RTC_CLASS=y >> CONFIG_RTC_DRV_ARMADA38X=y >> CONFIG_DMADEVICES=y >> CONFIG_MV_XOR_V2=y >> CONFIG_SYNC_FILE=y >> CONFIG_VFIO=y >> CONFIG_VFIO_PCI=y >> CONFIG_VIRTIO_PCI=y >> CONFIG_VIRTIO_BALLOON=y >> CONFIG_VIRTIO_MMIO=y >> CONFIG_COMMON_CLK_VERSATILE=y >> CONFIG_CLK_SP810=y >> CONFIG_CLK_VEXPRESS_OSC=y >> CONFIG_COMMON_CLK_RK808=y >> CONFIG_COMMON_CLK_SCPI=y >> CONFIG_COMMON_CLK_CS2000_CP=y >> CONFIG_COMMON_CLK_S2MPS11=y >> CONFIG_CLK_QORIQ=y >> CONFIG_HWSPINLOCK=y >> CONFIG_ARM_TIMER_SP804=y >> CONFIG_MAILBOX=y >> CONFIG_ARM_MHU=y >> CONFIG_PLATFORM_MHU=y >> CONFIG_ARM_SMMU=y >> CONFIG_ARM_SMMU_V3=y >> CONFIG_EXTCON_USB_GPIO=y >> CONFIG_RESET_CONTROLLER=y >> CONFIG_PHY_MVEBU_CP110_COMPHY=y >> CONFIG_EXT4_FS=y >> CONFIG_EXT4_FS_POSIX_ACL=y >> CONFIG_FANOTIFY=y >> CONFIG_VFAT_FS=y >> CONFIG_TMPFS=y >> CONFIG_HUGETLBFS=y >> CONFIG_CONFIGFS_FS=y >> # CONFIG_MISC_FILESYSTEMS is not set >> # CONFIG_NETWORK_FILESYSTEMS is not set >> CONFIG_NLS_CODEPAGE_437=y >> CONFIG_NLS_ISO8859_1=y >> CONFIG_KEYS=y >> CONFIG_CRYPTO_ECDH=y >> CONFIG_CRYPTO_CCM=y >> CONFIG_CRYPTO_GCM=y >> CONFIG_CRYPTO_ECB=y >> CONFIG_CRYPTO_CMAC=y >> CONFIG_CRYPTO_ARC4=y >> CONFIG_CRYPTO_ANSI_CPRNG=y >> # CONFIG_CRYPTO_DEV_VIRTIO is not set >> CONFIG_CRYPTO_DEV_SAFEXCEL=y >> CONFIG_XZ_DEC=y >> CONFIG_PRINTK_TIME=y >> CONFIG_DEBUG_INFO=y >> CONFIG_DEBUG_FS=y >> CONFIG_MAGIC_SYSRQ=y >> CONFIG_DEBUG_KERNEL=y >> # CONFIG_SCHED_DEBUG is not set >> # CONFIG_DEBUG_PREEMPT is not set >> # CONFIG_FTRACE is not set >> CONFIG_MEMTEST=y > >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > > > -- > http://baruch.siach.name/blog/ ~. .~ Tk Open Systems > =}------------------------------------------------ooO--U--Ooo------------{= > - baruch@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - -- Gregory Clement, Bootlin Embedded Linux and Kernel engineering http://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] Revert "arm64: dts: marvell: add CPU Idle power state support on Armada 7K/8K" 2018-12-05 8:27 ` Gregory CLEMENT @ 2018-12-05 8:40 ` Baruch Siach 2018-12-05 10:10 ` Sudeep Holla 1 sibling, 0 replies; 9+ messages in thread From: Baruch Siach @ 2018-12-05 8:40 UTC (permalink / raw) To: Gregory CLEMENT Cc: Andrew Lunn, Jason Cooper, Sergey Matyukevich, linux-pm, orenbh, Daniel Lezcano, Rafael J. Wysocki, Russell King, linux-arm-kernel, Sebastian Hesselbarth Hi Gregory, On Wed, Dec 05, 2018 at 09:27:45AM +0100, Gregory CLEMENT wrote: > On mer., déc. 05 2018, Baruch Siach <baruch@tkos.co.il> wrote: > > On Tue, Dec 04, 2018 at 06:48:43PM +0200, Baruch Siach wrote: > >> On Tue, Dec 04, 2018 at 05:21:25PM +0100, Gregory CLEMENT wrote: > >> > On mar., déc. 04 2018, Baruch Siach <baruch@tkos.co.il> wrote: > >> > > This reverts commit 8ed46368776b3bc93d74c1f8f2bfb9fd8a9ad805. > >> > > > >> > > This commit breaks boot on Armada 8K based systems. Reverting it makes > >> > > affected systems boot again. > >> > > > >> > > Reported-by: Sergey Matyukevich <geomatsi@gmail.com> > >> > > Signed-off-by: Baruch Siach <baruch@tkos.co.il> > >> > > >> > I would like to know more about it because on the McBin I use there is > >> > no issue with 4.20-rc1. > >> > > >> > Do you observe this issue with the arm64 defconfig or do you have some > >> > specific configuration option? > >> > >> I tested the attached defconfig on Clearfog GT-8K and on another custom Armada > >> 8K based board. > >> > >> Tomorrow I'll test Macchiatobin with this config and the arm64 defconfig. > > > > I reproduced same issue on a Macchiatobin board with both this config and the > > arm64 defconfig. The latter produced this splat: > > OK thanks, for this, I also got the confirmation by my colleagues. The > only difference I can think of, is the firmware, but the surprising > thing, is that I have an old firmware: 17.06.2 and that with a recent > one, it doesn't work. This is right. I'm running the latest Marvell ATF release 18.09.4 (based on upstream version 1.5). > So I will apply it in mvebu/fixes. Thanks, baruch > > [ 22.868568] rcu: INFO: rcu_preempt detected stalls on CPUs/tasks: > > [ 22.874694] rcu: 0-...!: (1 GPs behind) idle=408/0/0x0 softirq=251/251 fqs=0 > > [ 22.881950] rcu: 2-...!: (0 ticks this GP) idle=7b4/0/0x0 softirq=168/168 fqs=0 > > [ 22.889465] rcu: (detected by 1, t=5252 jiffies, g=-967, q=294) > > [ 22.895498] Task dump for CPU 0: > > [ 22.898739] swapper/0 R running task 0 0 0 0x00000028 > > [ 22.905819] Call trace: > > [ 22.908285] __switch_to+0x94/0xd0 > > [ 22.911702] arm_cpuidle_suspend+0x30/0x38 > > [ 22.915818] arm_enter_idle_state+0x38/0x68 > > [ 22.920019] cpuidle_enter_state+0x1d0/0x240 > > [ 22.924306] cpuidle_enter+0x18/0x20 > > [ 22.927898] call_cpuidle+0x1c/0x38 > > [ 22.931400] do_idle+0x254/0x280 > > [ 22.934641] cpu_startup_entry+0x24/0x28 > > [ 22.938581] rest_init+0xd4/0xe0 > > [ 22.941824] arch_call_rest_init+0xc/0x14 > > [ 22.945849] start_kernel+0x3d8/0x400 > > [ 22.949524] Task dump for CPU 2: > > [ 22.952764] swapper/2 R running task 0 0 1 0x0000002a > > [ 22.959843] Call trace: > > [ 22.962299] __switch_to+0x94/0xd0 > > [ 22.965715] arm_cpuidle_suspend+0x30/0x38 > > [ 22.969828] arm_enter_idle_state+0x38/0x68 > > [ 22.974028] cpuidle_enter_state+0x1d0/0x240 > > [ 22.978315] cpuidle_enter+0x18/0x20 > > [ 22.981905] call_cpuidle+0x1c/0x38 > > [ 22.985408] do_idle+0x254/0x280 > > [ 22.988648] cpu_startup_entry+0x20/0x28 > > [ 22.992590] secondary_start_kernel+0x15c/0x170 > > [ 22.997141] rcu: rcu_preempt kthread starved for 5252 jiffies! g-967 f0x0 RCU_GP_WAIT_FQS(5) ->state=0x402 ->cpu=2 > > [ 23.007534] rcu: RCU grace-period kthread stack dump: > > [ 23.012606] rcu_preempt I 0 10 2 0x00000028 > > [ 23.018116] Call trace: > > [ 23.020572] __switch_to+0x94/0xd0 > > [ 23.023987] __schedule+0x1e0/0x5c0 > > [ 23.027488] schedule+0x38/0xa0 > > [ 23.030643] schedule_timeout+0x198/0x338 > > [ 23.034670] rcu_gp_kthread+0x43c/0x7b0 > > [ 23.038524] kthread+0x124/0x128 > > [ 23.041766] ret_from_fork+0x10/0x18 > > > > baruch > > > >> > > --- > >> > > .../boot/dts/marvell/armada-ap806-quad.dtsi | 4 --- > >> > > arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 27 ------------------- > >> > > 2 files changed, 31 deletions(-) > >> > > > >> > > diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > >> > > index 64632c873888..01ea662afba8 100644 > >> > > --- a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > >> > > +++ b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi > >> > > @@ -20,28 +20,24 @@ > >> > > compatible = "arm,cortex-a72", "arm,armv8"; > >> > > reg = <0x000>; > >> > > enable-method = "psci"; > >> > > - cpu-idle-states = <&CPU_SLEEP_0>; > >> > > }; > >> > > cpu1: cpu@1 { > >> > > device_type = "cpu"; > >> > > compatible = "arm,cortex-a72", "arm,armv8"; > >> > > reg = <0x001>; > >> > > enable-method = "psci"; > >> > > - cpu-idle-states = <&CPU_SLEEP_0>; > >> > > }; > >> > > cpu2: cpu@100 { > >> > > device_type = "cpu"; > >> > > compatible = "arm,cortex-a72", "arm,armv8"; > >> > > reg = <0x100>; > >> > > enable-method = "psci"; > >> > > - cpu-idle-states = <&CPU_SLEEP_0>; > >> > > }; > >> > > cpu3: cpu@101 { > >> > > device_type = "cpu"; > >> > > compatible = "arm,cortex-a72", "arm,armv8"; > >> > > reg = <0x101>; > >> > > enable-method = "psci"; > >> > > - cpu-idle-states = <&CPU_SLEEP_0>; > >> > > }; > >> > > }; > >> > > }; > >> > > diff --git a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > >> > > index 073610ac0a53..7d94c1fa592a 100644 > >> > > --- a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > >> > > +++ b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi > >> > > @@ -28,33 +28,6 @@ > >> > > method = "smc"; > >> > > }; > >> > > > >> > > - cpus { > >> > > - #address-cells = <1>; > >> > > - #size-cells = <0>; > >> > > - > >> > > - idle_states { > >> > > - entry_method = "arm,pcsi"; > >> > > - > >> > > - CPU_SLEEP_0: cpu-sleep-0 { > >> > > - compatible = "arm,idle-state"; > >> > > - local-timer-stop; > >> > > - arm,psci-suspend-param = <0x0010000>; > >> > > - entry-latency-us = <80>; > >> > > - exit-latency-us = <160>; > >> > > - min-residency-us = <320>; > >> > > - }; > >> > > - > >> > > - CLUSTER_SLEEP_0: cluster-sleep-0 { > >> > > - compatible = "arm,idle-state"; > >> > > - local-timer-stop; > >> > > - arm,psci-suspend-param = <0x1010000>; > >> > > - entry-latency-us = <500>; > >> > > - exit-latency-us = <1000>; > >> > > - min-residency-us = <2500>; > >> > > - }; > >> > > - }; > >> > > - }; > >> > > - > >> > > ap806 { > >> > > #address-cells = <2>; > >> > > #size-cells = <2>; -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] Revert "arm64: dts: marvell: add CPU Idle power state support on Armada 7K/8K" 2018-12-05 8:27 ` Gregory CLEMENT 2018-12-05 8:40 ` Baruch Siach @ 2018-12-05 10:10 ` Sudeep Holla 2018-12-18 16:58 ` Gregory CLEMENT 1 sibling, 1 reply; 9+ messages in thread From: Sudeep Holla @ 2018-12-05 10:10 UTC (permalink / raw) To: Gregory CLEMENT Cc: Andrew Lunn, Baruch Siach, Jason Cooper, Sergey Matyukevich, linux-pm, orenbh, Daniel Lezcano, Rafael J. Wysocki, Russell King, linux-arm-kernel, Sebastian Hesselbarth On Wed, Dec 05, 2018 at 09:27:45AM +0100, Gregory CLEMENT wrote: > Hi Baruch, > > On mer., déc. 05 2018, Baruch Siach <baruch@tkos.co.il> wrote: > > > Hi Gregory, > > > > On Tue, Dec 04, 2018 at 06:48:43PM +0200, Baruch Siach wrote: > >> On Tue, Dec 04, 2018 at 05:21:25PM +0100, Gregory CLEMENT wrote: > >> > On mar., déc. 04 2018, Baruch Siach <baruch@tkos.co.il> wrote: > >> > > This reverts commit 8ed46368776b3bc93d74c1f8f2bfb9fd8a9ad805. > >> > > > >> > > This commit breaks boot on Armada 8K based systems. Reverting it makes > >> > > affected systems boot again. > >> > > > >> > > Reported-by: Sergey Matyukevich <geomatsi@gmail.com> > >> > > Signed-off-by: Baruch Siach <baruch@tkos.co.il> > >> > > >> > I would like to know more about it because on the McBin I use there is > >> > no issue with 4.20-rc1. > >> > > >> > Do you observe this issue with the arm64 defconfig or do you have some > >> > specific configuration option? > >> > >> I tested the attached defconfig on Clearfog GT-8K and on another custom Armada > >> 8K based board. > >> > >> Tomorrow I'll test Macchiatobin with this config and the arm64 defconfig. > > > > I reproduced same issue on a Macchiatobin board with both this config and the > > arm64 defconfig. The latter produced this splat: > > OK thanks, for this, I also got the confirmation by my colleagues. The > only difference I can think of, is the firmware, but the surprising > thing, is that I have an old firmware: 17.06.2 and that with a recent > one, it doesn't work. > > So I will apply it in mvebu/fixes. > Sorry if it's too late. But I would rather disable the states that are broken on old firmware and see if it's possible for bootloader or boot firmware loading DT can modify the status to "enabled" if it can detect the firmware version. -- Regards, Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] Revert "arm64: dts: marvell: add CPU Idle power state support on Armada 7K/8K" 2018-12-05 10:10 ` Sudeep Holla @ 2018-12-18 16:58 ` Gregory CLEMENT 2018-12-18 17:11 ` Sudeep Holla 0 siblings, 1 reply; 9+ messages in thread From: Gregory CLEMENT @ 2018-12-18 16:58 UTC (permalink / raw) To: Sudeep Holla Cc: Andrew Lunn, Baruch Siach, Jason Cooper, Sergey Matyukevich, linux-pm, orenbh, Daniel Lezcano, Rafael J. Wysocki, Russell King, linux-arm-kernel, Sebastian Hesselbarth Hi Sudeep, On mer., déc. 05 2018, Sudeep Holla <sudeep.holla@arm.com> wrote: > On Wed, Dec 05, 2018 at 09:27:45AM +0100, Gregory CLEMENT wrote: >> Hi Baruch, >> >> On mer., déc. 05 2018, Baruch Siach <baruch@tkos.co.il> wrote: >> >> > Hi Gregory, >> > >> > On Tue, Dec 04, 2018 at 06:48:43PM +0200, Baruch Siach wrote: >> >> On Tue, Dec 04, 2018 at 05:21:25PM +0100, Gregory CLEMENT wrote: >> >> > On mar., déc. 04 2018, Baruch Siach <baruch@tkos.co.il> wrote: >> >> > > This reverts commit 8ed46368776b3bc93d74c1f8f2bfb9fd8a9ad805. >> >> > > >> >> > > This commit breaks boot on Armada 8K based systems. Reverting it makes >> >> > > affected systems boot again. >> >> > > >> >> > > Reported-by: Sergey Matyukevich <geomatsi@gmail.com> >> >> > > Signed-off-by: Baruch Siach <baruch@tkos.co.il> >> >> > >> >> > I would like to know more about it because on the McBin I use there is >> >> > no issue with 4.20-rc1. >> >> > >> >> > Do you observe this issue with the arm64 defconfig or do you have some >> >> > specific configuration option? >> >> >> >> I tested the attached defconfig on Clearfog GT-8K and on another custom Armada >> >> 8K based board. >> >> >> >> Tomorrow I'll test Macchiatobin with this config and the arm64 defconfig. >> > >> > I reproduced same issue on a Macchiatobin board with both this config and the >> > arm64 defconfig. The latter produced this splat: >> >> OK thanks, for this, I also got the confirmation by my colleagues. The >> only difference I can think of, is the firmware, but the surprising >> thing, is that I have an old firmware: 17.06.2 and that with a recent >> one, it doesn't work. >> >> So I will apply it in mvebu/fixes. >> > > Sorry if it's too late. But I would rather disable the states that are > broken on old firmware and see if it's possible for bootloader or boot > firmware loading DT can modify the status to "enabled" if it can detect > the firmware version. you mean adding status = "disable" on each idle-state subnode? Gregory > > -- > Regards, > Sudeep -- Gregory Clement, Bootlin Embedded Linux and Kernel engineering http://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] Revert "arm64: dts: marvell: add CPU Idle power state support on Armada 7K/8K" 2018-12-18 16:58 ` Gregory CLEMENT @ 2018-12-18 17:11 ` Sudeep Holla 0 siblings, 0 replies; 9+ messages in thread From: Sudeep Holla @ 2018-12-18 17:11 UTC (permalink / raw) To: Gregory CLEMENT Cc: Andrew Lunn, Baruch Siach, Jason Cooper, Sergey Matyukevich, linux-pm, orenbh, Daniel Lezcano, Rafael J. Wysocki, Russell King, Sudeep Holla, linux-arm-kernel, Sebastian Hesselbarth On Tue, Dec 18, 2018 at 05:58:34PM +0100, Gregory CLEMENT wrote: > Hi Sudeep, > > On mer., déc. 05 2018, Sudeep Holla <sudeep.holla@arm.com> wrote: > > > On Wed, Dec 05, 2018 at 09:27:45AM +0100, Gregory CLEMENT wrote: > >> Hi Baruch, > >> > >> On mer., déc. 05 2018, Baruch Siach <baruch@tkos.co.il> wrote: > >> > >> > Hi Gregory, > >> > > >> > On Tue, Dec 04, 2018 at 06:48:43PM +0200, Baruch Siach wrote: > >> >> On Tue, Dec 04, 2018 at 05:21:25PM +0100, Gregory CLEMENT wrote: > >> >> > On mar., déc. 04 2018, Baruch Siach <baruch@tkos.co.il> wrote: > >> >> > > This reverts commit 8ed46368776b3bc93d74c1f8f2bfb9fd8a9ad805. > >> >> > > > >> >> > > This commit breaks boot on Armada 8K based systems. Reverting it makes > >> >> > > affected systems boot again. > >> >> > > > >> >> > > Reported-by: Sergey Matyukevich <geomatsi@gmail.com> > >> >> > > Signed-off-by: Baruch Siach <baruch@tkos.co.il> > >> >> > > >> >> > I would like to know more about it because on the McBin I use there is > >> >> > no issue with 4.20-rc1. > >> >> > > >> >> > Do you observe this issue with the arm64 defconfig or do you have some > >> >> > specific configuration option? > >> >> > >> >> I tested the attached defconfig on Clearfog GT-8K and on another custom Armada > >> >> 8K based board. > >> >> > >> >> Tomorrow I'll test Macchiatobin with this config and the arm64 defconfig. > >> > > >> > I reproduced same issue on a Macchiatobin board with both this config and the > >> > arm64 defconfig. The latter produced this splat: > >> > >> OK thanks, for this, I also got the confirmation by my colleagues. The > >> only difference I can think of, is the firmware, but the surprising > >> thing, is that I have an old firmware: 17.06.2 and that with a recent > >> one, it doesn't work. > >> > >> So I will apply it in mvebu/fixes. > >> > > > > Sorry if it's too late. But I would rather disable the states that are > > broken on old firmware and see if it's possible for bootloader or boot > > firmware loading DT can modify the status to "enabled" if it can detect > > the firmware version. > > you mean adding status = "disable" on each idle-state subnode? > Yes indeed. Keep it disabled by default and can enable it in bootloader if you can detect it that it should work with particular version of firmware. Just a thought especially if certain set of platforms support it while another not. -- Regards, Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-12-18 17:12 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-12-04 15:46 [PATCH] Revert "arm64: dts: marvell: add CPU Idle power state support on Armada 7K/8K" Baruch Siach 2018-12-04 16:21 ` Gregory CLEMENT 2018-12-04 16:48 ` Baruch Siach 2018-12-05 6:57 ` Baruch Siach 2018-12-05 8:27 ` Gregory CLEMENT 2018-12-05 8:40 ` Baruch Siach 2018-12-05 10:10 ` Sudeep Holla 2018-12-18 16:58 ` Gregory CLEMENT 2018-12-18 17:11 ` Sudeep Holla
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).