From patchwork Fri Jun 26 19:11:05 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "kernelci.org bot" X-Patchwork-Id: 11628501 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 0A29B14B7 for ; Fri, 26 Jun 2020 19:12:58 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id CA267206A1 for ; Fri, 26 Jun 2020 19:12:57 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="nNoDQi8w"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=kernelci-org.20150623.gappssmtp.com header.i=@kernelci-org.20150623.gappssmtp.com header.b="ftibM33U" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org CA267206A1 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernelci.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:From:To:Subject:MIME-Version:Date:Message-ID: Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender :Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=Rk98ZgNBTBx2nQWlX2SEqAU1QagA/rz053debILo4og=; b=nNoDQi8wnOUBu7FglsCxqiKG3o DKqW4Uip/w99rV9EYz6kG18REp5XUSudmi0PGKLS8IDDzTUoY1XGRN90Nszl+a/Px72BzU+IeAJpt qIdeFBBd27RAaNZbGJGf/lv08VHs1WbUyZ2cLyUnZ0qnzeTPXn/iITBP+byQWJEmFpv27xuoUhqeA tgNgyA7Uka/7dEyvAkvCISmXyuKHiV0e54vBdsjG1UDieiAnruZwS8J+ODqF5p6t8PV1vOWkNYsyU p5xH9BjRExu+kVDJ/eymYFk65snOKUSLJQnwOoTs1+nGdJjcSfmcpJ6VRRpmOEVXf4T4tyYTyce6u 90HOuBbw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jotkl-0000aK-RZ; Fri, 26 Jun 2020 19:11:11 +0000 Received: from mail-pl1-x644.google.com ([2607:f8b0:4864:20::644]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jotkj-0000Zm-NF for linux-arm-kernel@lists.infradead.org; Fri, 26 Jun 2020 19:11:10 +0000 Received: by mail-pl1-x644.google.com with SMTP id f2so4596025plr.8 for ; Fri, 26 Jun 2020 12:11:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernelci-org.20150623.gappssmtp.com; s=20150623; h=message-id:date:mime-version:content-transfer-encoding:subject:to :from:cc; bh=6UjzTo6kKJLkeVtxkIJDMzomu4dcIQKvoQRVUhtv+x8=; b=ftibM33UckAR0AQRwJCEc0N3RtukE/tcWr7W8C2v18E/QHcM3tbLBTqQSqVZ8e9PDa pTlaH8bhqVGpp6ZD6Jvn1VUJkukqNVhjUsapEOvwCwqSK1DmMbmJxbesGeEwRpZ2aVYB I2895JfEdF7L9L35GDRY0lcMuGP0VpzOfcL2KP0vpTPykQJ85Kf4OKA21q3paEMMuE76 nxL2LZWsDUFNO0d+9JH+3+uKeNGUI46di8fvVAIwpS2pX4z/WmJxf4GEjmYYTWDNLonR /XkfhepSXp1asia6CiiT5hbwvvywp9szOSpbyH2TJ2v9UX9cO9yQEYno+0BemwVFrOeB zJzQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:date:mime-version :content-transfer-encoding:subject:to:from:cc; bh=6UjzTo6kKJLkeVtxkIJDMzomu4dcIQKvoQRVUhtv+x8=; b=lnv3SFmcjf1v+7sLhha2xUIWYHefEKyWXP98k6hQizWp5u+wRgBYVH0jgxfhNxs7PT HD//cRBN9/bPAqwlKzInUQ1YYfr8xReyFAjEXeoEVURkYOmcOELyrN4Eim1acGZSkGAx s3KWdjPGU8ia5tXv53tzEPSqXXdyJmSfrMBXExYN/dJHa6JTv86H++kP1Gtz5iFoSTIJ 9VnYtSOkOGrRwE4Dmx/oxQASm7IUsMdM75SlTZdt4Ag6cmOOlbTcdnrcDNRas7asCBaZ f0bHICAkhQ7Hz597C0IOmC4+cDN8rB5/uZ5g1HF4Nns+rwvkk7RLtwLz0YftpNlKO+pV 8hjQ== X-Gm-Message-State: AOAM532qHwAh59yBpBpWxD5qNQu2FpADio98Sz46NA0tTTcEB0IIUypR IYCeXHB8ADAZ/BMsgooGSI8irg== X-Google-Smtp-Source: ABdhPJyWzCHTf8ufqy3E7/C4fSQ7qgRGKfRfK2NcfHD2I4t2KKzo7b1eCYSiyyc8Dygezj1pctsiUQ== X-Received: by 2002:a17:90b:2350:: with SMTP id ms16mr5162309pjb.127.1593198665787; Fri, 26 Jun 2020 12:11:05 -0700 (PDT) Received: from kernelci-production.internal.cloudapp.net ([52.250.1.28]) by smtp.gmail.com with ESMTPSA id z5sm978327pfn.117.2020.06.26.12.11.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 26 Jun 2020 12:11:05 -0700 (PDT) Message-ID: <5ef64849.1c69fb81.2d891.248a@mx.google.com> Date: Fri, 26 Jun 2020 12:11:05 -0700 (PDT) MIME-Version: 1.0 X-Kernelci-Kernel: next-20200626 X-Kernelci-Report-Type: bisect X-Kernelci-Tree: next X-Kernelci-Branch: master X-Kernelci-Lab-Name: lab-collabora Subject: next/master bisection: baseline.dmesg.crit on qemu_arm-vexpress-a15 To: Sudeep Holla , gtucker@collabora.com, kernelci-results@groups.io, Andre Przywara From: "kernelci.org bot" X-Spam-Note: CRM114 invocation failed X-Spam-Score: 0.0 (/) X-Spam-Report: SpamAssassin version 3.4.4 on merlin.infradead.org summary: Content analysis details: (0.0 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at https://www.dnswl.org/, no trust [2607:f8b0:4864:20:0:0:0:644 listed in] [list.dnswl.org] 0.0 SPF_NONE SPF: sender does not publish an SPF Record 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, Lorenzo Pieralisi , Liviu Dudau , linux-kernel@vger.kernel.org, Rob Herring , linux-arm-kernel@lists.infradead.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * This automated bisection report was sent to you on the basis * * that you may be involved with the breaking commit it has * * found. No manual investigation has been done to verify it, * * and the root cause of the problem may be somewhere else. * * * * If you do send a fix, please include this trailer: * * Reported-by: "kernelci.org bot" * * * * Hope this helps! * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * next/master bisection: baseline.dmesg.crit on qemu_arm-vexpress-a15 Summary: Start: 36e3135df4d4 Add linux-next specific files for 20200626 Plain log: https://storage.kernelci.org/next/master/next-20200626/arm/vexpress_defconfig/gcc-8/lab-collabora/baseline-vexpress-v2p-ca15-tc1.txt HTML log: https://storage.kernelci.org/next/master/next-20200626/arm/vexpress_defconfig/gcc-8/lab-collabora/baseline-vexpress-v2p-ca15-tc1.html Result: 38ac46002d1d arm: dts: vexpress: Move mcc node back into motherboard node Checks: revert: PASS verify: PASS Parameters: Tree: next URL: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git Branch: master Target: qemu_arm-vexpress-a15 CPU arch: arm Lab: lab-collabora Compiler: gcc-8 Config: vexpress_defconfig Test case: baseline.dmesg.crit Breaking commit found: ------------------------------------------------------------------------------- commit 38ac46002d1df5707566a73486452851341028d2 Author: Andre Przywara Date: Wed Jun 3 17:22:37 2020 +0100 arm: dts: vexpress: Move mcc node back into motherboard node Commit d9258898ad49 ("arm64: dts: arm: vexpress: Move fixed devices out of bus node") moved the "mcc" DT node into the root node, because it does not have any children using "reg" properties, so does violate some dtc checks about "simple-bus" nodes. However this broke the vexpress config-bus code, which walks up the device tree to find the first node with an "arm,vexpress,site" property. This gave the wrong result (matching the root node instead of the motherboard node), so broke the clocks and some other devices for VExpress boards. Move the whole node back into its original position. This re-introduces the dtc warning, but is conceptually the right thing to do. The dtc warning seems to be overzealous here, there are discussions on fixing or relaxing this check instead. Link: https://lore.kernel.org/r/20200603162237.16319-1-andre.przywara@arm.com Fixes: d9258898ad49 ("arm64: dts: vexpress: Move fixed devices out of bus node") Reported-and-tested-by: Guenter Roeck Signed-off-by: Andre Przywara Signed-off-by: Sudeep Holla ------------------------------------------------------------------------------- Git bisection log: ------------------------------------------------------------------------------- git bisect start # good: [52366a107bf0600cf366f5ff3ea1f147b285e41f] Merge tag 'fsnotify_for_v5.8-rc3' of git://git.kernel.org/pub/scm/linux/kernel/git/jack/linux-fs git bisect good 52366a107bf0600cf366f5ff3ea1f147b285e41f # bad: [36e3135df4d426612fc77db26a312c2531108603] Add linux-next specific files for 20200626 git bisect bad 36e3135df4d426612fc77db26a312c2531108603 # bad: [11fdea666694c5c8c8a52cb75b2f0e70a2c2c201] Merge remote-tracking branch 'drm/drm-next' git bisect bad 11fdea666694c5c8c8a52cb75b2f0e70a2c2c201 # bad: [39ac1a242d0940dabd9192d99113c2b082ba45bc] Merge remote-tracking branch 'printk/for-next' git bisect bad 39ac1a242d0940dabd9192d99113c2b082ba45bc # good: [4a750150d9fe543e2163998501b3ad947d6dff74] Merge remote-tracking branch 'at91/at91-next' git bisect good 4a750150d9fe543e2163998501b3ad947d6dff74 # bad: [d7645ac1101c5160a05e2dced672a8d63c2a7ec0] Merge remote-tracking branch 'scmi/for-linux-next' git bisect bad d7645ac1101c5160a05e2dced672a8d63c2a7ec0 # good: [2408a915a05c109169ab689dc91ce31315406513] Merge branches 'arm64-defconfig-for-5.9', 'arm64-for-5.9', 'drivers-for-5.9' and 'dts-for-5.9' into for-next git bisect good 2408a915a05c109169ab689dc91ce31315406513 # good: [1679681fb8b2d169ac9d98660d7390620990bf77] Merge remote-tracking branch 'raspberrypi/for-next' git bisect good 1679681fb8b2d169ac9d98660d7390620990bf77 # good: [137233bdcd265762a251dfa24e82ebc5468e0ec2] Merge remote-tracking branch 'reset/reset/next' git bisect good 137233bdcd265762a251dfa24e82ebc5468e0ec2 # good: [99bcf38dd05b76b41f8564b53d9be6b44613fa92] Merge branch 'v5.9-clk/next' into for-next git bisect good 99bcf38dd05b76b41f8564b53d9be6b44613fa92 # good: [dc45e438fac0b5df3c31bb83f3d809cd0f67dcfe] Merge branch 'next/dt' into for-next git bisect good dc45e438fac0b5df3c31bb83f3d809cd0f67dcfe # good: [d6fe116541b73a56110310c39a270c99766cd909] Merge branch 'next/soc' into for-next git bisect good d6fe116541b73a56110310c39a270c99766cd909 # bad: [24077bf8f9e69a3a6a2c714634e6c813566a152f] Merge tag 'juno-fix-5.8' of git://git.kernel.org/pub/scm/linux/kernel/git/sudeep.holla/linux into for-linux-next git bisect bad 24077bf8f9e69a3a6a2c714634e6c813566a152f # bad: [38ac46002d1df5707566a73486452851341028d2] arm: dts: vexpress: Move mcc node back into motherboard node git bisect bad 38ac46002d1df5707566a73486452851341028d2 # first bad commit: [38ac46002d1df5707566a73486452851341028d2] arm: dts: vexpress: Move mcc node back into motherboard node ------------------------------------------------------------------------------- diff --git a/arch/arm/boot/dts/vexpress-v2m-rs1.dtsi b/arch/arm/boot/dts/vexpress-v2m-rs1.dtsi index e6308fb76183..a88ee5294d35 100644 --- a/arch/arm/boot/dts/vexpress-v2m-rs1.dtsi +++ b/arch/arm/boot/dts/vexpress-v2m-rs1.dtsi @@ -100,79 +100,6 @@ }; }; - mcc { - compatible = "arm,vexpress,config-bus"; - arm,vexpress,config-bridge = <&v2m_sysreg>; - - oscclk0 { - /* MCC static memory clock */ - compatible = "arm,vexpress-osc"; - arm,vexpress-sysreg,func = <1 0>; - freq-range = <25000000 60000000>; - #clock-cells = <0>; - clock-output-names = "v2m:oscclk0"; - }; - - v2m_oscclk1: oscclk1 { - /* CLCD clock */ - compatible = "arm,vexpress-osc"; - arm,vexpress-sysreg,func = <1 1>; - freq-range = <23750000 65000000>; - #clock-cells = <0>; - clock-output-names = "v2m:oscclk1"; - }; - - v2m_oscclk2: oscclk2 { - /* IO FPGA peripheral clock */ - compatible = "arm,vexpress-osc"; - arm,vexpress-sysreg,func = <1 2>; - freq-range = <24000000 24000000>; - #clock-cells = <0>; - clock-output-names = "v2m:oscclk2"; - }; - - volt-vio { - /* Logic level voltage */ - compatible = "arm,vexpress-volt"; - arm,vexpress-sysreg,func = <2 0>; - regulator-name = "VIO"; - regulator-always-on; - label = "VIO"; - }; - - temp-mcc { - /* MCC internal operating temperature */ - compatible = "arm,vexpress-temp"; - arm,vexpress-sysreg,func = <4 0>; - label = "MCC"; - }; - - reset { - compatible = "arm,vexpress-reset"; - arm,vexpress-sysreg,func = <5 0>; - }; - - muxfpga { - compatible = "arm,vexpress-muxfpga"; - arm,vexpress-sysreg,func = <7 0>; - }; - - shutdown { - compatible = "arm,vexpress-shutdown"; - arm,vexpress-sysreg,func = <8 0>; - }; - - reboot { - compatible = "arm,vexpress-reboot"; - arm,vexpress-sysreg,func = <9 0>; - }; - - dvimode { - compatible = "arm,vexpress-dvimode"; - arm,vexpress-sysreg,func = <11 0>; - }; - }; - bus@8000000 { motherboard-bus { model = "V2M-P1"; @@ -435,6 +362,79 @@ }; }; }; + + mcc { + compatible = "arm,vexpress,config-bus"; + arm,vexpress,config-bridge = <&v2m_sysreg>; + + oscclk0 { + /* MCC static memory clock */ + compatible = "arm,vexpress-osc"; + arm,vexpress-sysreg,func = <1 0>; + freq-range = <25000000 60000000>; + #clock-cells = <0>; + clock-output-names = "v2m:oscclk0"; + }; + + v2m_oscclk1: oscclk1 { + /* CLCD clock */ + compatible = "arm,vexpress-osc"; + arm,vexpress-sysreg,func = <1 1>; + freq-range = <23750000 65000000>; + #clock-cells = <0>; + clock-output-names = "v2m:oscclk1"; + }; + + v2m_oscclk2: oscclk2 { + /* IO FPGA peripheral clock */ + compatible = "arm,vexpress-osc"; + arm,vexpress-sysreg,func = <1 2>; + freq-range = <24000000 24000000>; + #clock-cells = <0>; + clock-output-names = "v2m:oscclk2"; + }; + + volt-vio { + /* Logic level voltage */ + compatible = "arm,vexpress-volt"; + arm,vexpress-sysreg,func = <2 0>; + regulator-name = "VIO"; + regulator-always-on; + label = "VIO"; + }; + + temp-mcc { + /* MCC internal operating temperature */ + compatible = "arm,vexpress-temp"; + arm,vexpress-sysreg,func = <4 0>; + label = "MCC"; + }; + + reset { + compatible = "arm,vexpress-reset"; + arm,vexpress-sysreg,func = <5 0>; + }; + + muxfpga { + compatible = "arm,vexpress-muxfpga"; + arm,vexpress-sysreg,func = <7 0>; + }; + + shutdown { + compatible = "arm,vexpress-shutdown"; + arm,vexpress-sysreg,func = <8 0>; + }; + + reboot { + compatible = "arm,vexpress-reboot"; + arm,vexpress-sysreg,func = <9 0>; + }; + + dvimode { + compatible = "arm,vexpress-dvimode"; + arm,vexpress-sysreg,func = <11 0>; + }; + }; }; }; };