From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.1 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DCFC9C47404 for ; Mon, 7 Oct 2019 13:29:24 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 9F60B222C5 for ; Mon, 7 Oct 2019 13:29:24 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="CyyjX6t8" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728752AbfJGN3X (ORCPT ); Mon, 7 Oct 2019 09:29:23 -0400 Received: from mail-pg1-f196.google.com ([209.85.215.196]:46589 "EHLO mail-pg1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727801AbfJGN3W (ORCPT ); Mon, 7 Oct 2019 09:29:22 -0400 Received: by mail-pg1-f196.google.com with SMTP id b8so1873135pgm.13; Mon, 07 Oct 2019 06:29:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=u8Nzm65IfZcCND5MuDnuptkHPq86YqiyKCsKJJv/NSk=; b=CyyjX6t8MqvcCFFxISrJGmPCeaa/nIhjtQqn8vGpXZKTcSET97JKWRwADAPEU967qO MlasrY10gHlY1Ok6n5bN3zR78X5cNxhqqxl0pnxFIpdoQoeVL7udE4Az+MsCnyYaGapz mUoQWrPyefm93fnoGTVIPSJink4uBBogJhHoSBf3QQbc1HXsTvPuK5tF4Seh3p4HtLgz 3wDwZZXvvyj9JuzgRxVvqAuZcgdB5TYNzdf1Yc5wAGoc0DEnsgAa5lshmqvorcas0Rv2 qw/APrLrycL5Ac4rqWAPjMt7P5LY/44c/syZIa4K/KDpUQi3KKS2H3pio0s6pwXZGK+N sMVA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :references:mime-version:content-disposition:in-reply-to:user-agent; bh=u8Nzm65IfZcCND5MuDnuptkHPq86YqiyKCsKJJv/NSk=; b=ArmzUUucAEzBsTjR3wQCXzKRYTnrKm7JHCm5zvTXqBbgl87hy9WYVG98zME7AH0ifz XXwyX75qBSIgXfKufLM+WGt053DYUocPEl2HpgcpMya1+qISYqvvrVyBwR48HSgQ30Af 5zmB7Cw0iB3jkZiW4uIR9ceYbUoMcqO3AZHYjKRIpT/iORg3721TCAi1C0YMJ1GGZbAH /UtsvMS44rL4gbBqNOwnMjuE8nG/R0YSy3dPmPGefF3x+I3DblYRRAaf6MxWjrzy0icS jwKSncXmFZo4mmL8QVzlHNauFIIDJyHRCmW4qVV3ISezOTeGG3blqwKutMzOCKJnwH2y r7CA== X-Gm-Message-State: APjAAAUpLCNFlq7PWTHxMWvKOLl9O/fP5IH/aEmtwIJeI2NmbUEKJ1jO t2MzItnvLaxwn5nxNoyvys0= X-Google-Smtp-Source: APXvYqwffi3XqlYFyxmKhnhJGWSiRqfyVGcvbUuEKidoCGhTKiZUaR+UuytGmy2fNFFpvQzm7d0iNQ== X-Received: by 2002:a65:5802:: with SMTP id g2mr31885589pgr.333.1570454960975; Mon, 07 Oct 2019 06:29:20 -0700 (PDT) Received: from localhost ([2600:1700:e321:62f0:329c:23ff:fee3:9d7c]) by smtp.gmail.com with ESMTPSA id w7sm11107533pjn.1.2019.10.07.06.29.19 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 07 Oct 2019 06:29:19 -0700 (PDT) Date: Mon, 7 Oct 2019 06:29:18 -0700 From: Guenter Roeck To: Philippe Schenker Cc: "linux-kernel@vger.kernel.org" , Mark Brown , Shawn Guo , Sascha Hauer , Liam Girdwood , "devicetree@vger.kernel.org" , Max Krummenacher , Rob Herring , Stefan Agner , Marcel Ziswiler , Mark Rutland , Luka Pivk Subject: Re: [PATCH v2 1/3] regulator: fixed: add possibility to enable by clock Message-ID: <20191007132918.GA580@roeck-us.net> References: <20190910062103.39641-1-philippe.schenker@toradex.com> <20190910062103.39641-2-philippe.schenker@toradex.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190910062103.39641-2-philippe.schenker@toradex.com> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Sep 10, 2019 at 06:21:15AM +0000, Philippe Schenker wrote: > This commit adds the possibility to choose the compatible > "regulator-fixed-clock" in devicetree. > > This is a special regulator-fixed that has to have a clock, from which > the regulator gets switched on and off. > > Signed-off-by: Philippe Schenker This patch causes a crash in one of my qemu tests (kzm machine with imx_v6_v7_defconfig). Reverting this patch fixes the problem. Crash backtrace and bisect log attached below. Guenter --- [ 24.718451] Unable to handle kernel NULL pointer dereference at virtual address 000000c0 [ 24.718774] pgd = (ptrval) [ 24.718946] [000000c0] *pgd=00000000 [ 24.719688] Internal error: Oops: 5 [#1] SMP ARM [ 24.720007] Modules linked in: [ 24.720356] CPU: 0 PID: 1 Comm: swapper/0 Not tainted 5.4.0-rc1-00258-g2d00aee21a5d #1 [ 24.720522] Hardware name: Kyoto Microcomputer Co., Ltd. KZM-ARM11-01 [ 24.721143] PC is at reg_fixed_voltage_probe+0x38/0x2f4 [ 24.721279] LR is at reg_fixed_voltage_probe+0x2c/0x2f4 [ 24.721407] pc : [] lr : [] psr: 60000053 [ 24.721551] sp : c20a9d60 ip : 00000000 fp : c13bf34c [ 24.721677] r10: 00000000 r9 : c1479348 r8 : c30208d0 [ 24.721805] r7 : c1408b08 r6 : c30208e0 r5 : 00000000 r4 : c30208e0 [ 24.721958] r3 : 00000000 r2 : 00000dc0 r1 : 000000e8 r0 : 00000000 [ 24.722145] Flags: nZCv IRQs on FIQs off Mode SVC_32 ISA ARM Segment none [ 24.722273] Control: 00c5387d Table: 80004008 DAC: 00000051 [ 24.722435] Process swapper/0 (pid: 1, stack limit = 0x(ptrval)) [ 24.722616] Stack: (0xc20a9d60 to 0xc20aa000) [ 24.723031] 9d60: c3019d90 c033ff9c 00000000 c301b380 c21c2700 00000000 c21c2700 c301b380 [ 24.723407] 9d80: c3019d90 61093934 00000001 c30208e0 00000000 c1479348 00000000 00000000 [ 24.723767] 9da0: c1479348 00000000 c13bf34c c07ae608 c30208e0 c1b63da0 c1b63da4 00000000 [ 24.724120] 9dc0: 00000000 c07ac34c c30208e0 c1479348 c07ac7a0 c1408b08 00000001 00000000 [ 24.724485] 9de0: c1300584 c07ac5e0 c1479348 c20a9e44 c30208e0 00000000 c20a9e44 c07ac7a0 [ 24.724862] 9e00: c1408b08 00000001 00000000 c1300584 c13bf34c c07aa4fc fffffff3 c20904d4 [ 24.725215] 9e20: c21a1dd4 61093934 c30208e0 c30208e0 c3020968 c1408b08 c1408b08 c07ac0a4 [ 24.725575] 9e40: c209c000 c30208e0 00000001 61093934 c30208e0 c30208e0 c1488db8 c1408b08 [ 24.725963] 9e60: 00000000 c07ab3b4 c30208e0 c1488b00 00000000 c07a7ab8 c1300584 c0dd7b04 [ 24.726347] 9e80: c1408b08 c30208d0 c30208d0 61093934 c20a9ea8 c3020800 00000000 c30208d0 [ 24.726744] 9ea0: c30208e0 c14146d8 c14f93cc c07ae3d4 c3020800 c30208d0 00000000 00000002 [ 24.727127] 9ec0: c14146d8 c14f93cc c1300584 c070a4a8 c14146d8 c1408b28 c1408b08 c1303c60 [ 24.727504] 9ee0: 00000000 c130d094 00000000 c01aa5a4 c14fa7a0 c1303c7c c14fa7a0 c01031d8 [ 24.727894] 9f00: 0000019e 00000000 c7ffccec c7ffccda c12b19ac 0000019e 0000019e c01513b8 [ 24.728277] 9f20: c1408b08 c018ab84 c1408b08 c14fa7a0 c1408b28 c1500700 c139f850 0000019e [ 24.728662] 9f40: c1300584 c01aa5a4 00000000 61093934 c139f874 c1500700 00000008 c1500700 [ 24.729041] 9f60: c139f854 0000019e c1300584 c1301028 00000007 00000007 00000000 c1300584 [ 24.729410] 9f80: 00000000 00000000 c0decf18 00000000 00000000 00000000 00000000 00000000 [ 24.729788] 9fa0: 00000000 c0decf20 00000000 c01010b4 00000000 00000000 00000000 00000000 [ 24.730167] 9fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 [ 24.730544] 9fe0: 00000000 00000000 00000000 00000000 00000013 00000000 00000000 00000000 [ 24.730948] [] (reg_fixed_voltage_probe) from [] (platform_drv_probe+0x48/0x98) [ 24.731194] [] (platform_drv_probe) from [] (really_probe+0x22c/0x34c) [ 24.731404] [] (really_probe) from [] (driver_probe_device+0x5c/0x16c) [ 24.731606] [] (driver_probe_device) from [] (bus_for_each_drv+0x54/0xb8) [ 24.731810] [] (bus_for_each_drv) from [] (__device_attach+0xcc/0x140) [ 24.732010] [] (__device_attach) from [] (bus_probe_device+0x88/0x90) [ 24.732210] [] (bus_probe_device) from [] (device_add+0x388/0x608) [ 24.732406] [] (device_add) from [] (platform_device_add+0x100/0x210) [ 24.732612] [] (platform_device_add) from [] (regulator_register_always_on+0xa0/0xc8) [ 24.732847] [] (regulator_register_always_on) from [] (kzm_late_init+0x48/0x88) [ 24.733069] [] (kzm_late_init) from [] (init_machine_late+0x1c/0x8c) [ 24.733292] [] (init_machine_late) from [] (do_one_initcall+0x80/0x3ac) [ 24.733500] [] (do_one_initcall) from [] (kernel_init_freeable+0x120/0x1e8) [ 24.733710] [] (kernel_init_freeable) from [] (kernel_init+0x8/0x118) [ 24.733909] [] (kernel_init) from [] (ret_from_fork+0x14/0x20) [ 24.734111] Exception stack(0xc20a9fb0 to 0xc20a9ff8) [ 24.734355] 9fa0: 00000000 00000000 00000000 00000000 [ 24.734758] 9fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 [ 24.735101] 9fe0: 00000000 00000000 00000000 00000000 00000013 00000000 [ 24.735650] Code: eb0d74e5 e3a03000 e3a02d37 e3a010e8 (e59090c0) [ 24.737370] ---[ end trace d0f5e68eb88278a8 ]--- [ 24.739335] Kernel panic - not syncing: Attempted to kill init! exitcode=0x0000000b --- # bad: [ad062195731bea1624ce7160e79e0fcdaa25c1b5] Merge tag 'platform-drivers-x86-v5.4-1' of git://git.infradead.org/linux-platform-drivers-x86 # good: [4d856f72c10ecb060868ed10ff1b1453943fc6c8] Linux 5.3 git bisect start 'ad062195731b' '4d856f72c10e' # bad: [d47ebd684229f0048be5def6027bfcfbfe2db0d6] Merge tag 'armsoc-defconfig' of git://git.kernel.org/pub/scm/linux/kernel/git/soc/soc git bisect bad d47ebd684229f0048be5def6027bfcfbfe2db0d6 # bad: [52a5525214d0d612160154d902956eca0558b7c0] Merge tag 'iommu-updates-v5.4' of git://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu git bisect bad 52a5525214d0d612160154d902956eca0558b7c0 # bad: [aa62325dc38de2be8b1c27eb180ad3751b3f58ec] Merge tag 'spi-v5.4' of git://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi git bisect bad aa62325dc38de2be8b1c27eb180ad3751b3f58ec # bad: [c4d11ccb2b5cec6cdef7aeeb0017473d23031d96] Merge tag 'regulator-v5.4' of git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator git bisect bad c4d11ccb2b5cec6cdef7aeeb0017473d23031d96 # good: [6729fb666a3b5a9a5ffa1bb6589127f7e4d35c38] Merge tag 'hwmon-for-v5.4' of git://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging git bisect good 6729fb666a3b5a9a5ffa1bb6589127f7e4d35c38 # good: [92fd0db2cd414c5c78f8f959ed7325e2b24f9445] regulator: sy8824x: add SY20276 support git bisect good 92fd0db2cd414c5c78f8f959ed7325e2b24f9445 # good: [c0b913447b75538c3cf4b8016fd2e06509895020] regulator: slg51000: use devm_gpiod_get_optional() in probe git bisect good c0b913447b75538c3cf4b8016fd2e06509895020 # bad: [c82f27df07573ec7b124efe176d2ac6c038787a5] regulator: core: Fix error return for /sys access git bisect bad c82f27df07573ec7b124efe176d2ac6c038787a5 # bad: [9c86d003d62030e064f5670fb6172b65afd21e86] dt-bindings: regulator: add regulator-fixed-clock binding git bisect bad 9c86d003d62030e064f5670fb6172b65afd21e86 # good: [d57d90f4443bd725b3facdc6130a1940af4560c4] regulator: s2mps11: Consistently use local variable git bisect good d57d90f4443bd725b3facdc6130a1940af4560c4 # bad: [8959e5324485ace9bedc33ce1e760b759d4dd2ac] regulator: fixed: add possibility to enable by clock git bisect bad 8959e5324485ace9bedc33ce1e760b759d4dd2ac # first bad commit: [8959e5324485ace9bedc33ce1e760b759d4dd2ac] regulator: fixed: add possibility to enable by clock