All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/4] Config and device tree updates for the Librem 5 devkit
@ 2020-12-17 15:13 ` Guido Günther
  0 siblings, 0 replies; 29+ messages in thread
From: Guido Günther @ 2020-12-17 15:13 UTC (permalink / raw)
  To: Rob Herring, Shawn Guo, Sascha Hauer, Pengutronix Kernel Team,
	Fabio Estevam, NXP Linux Team, Catalin Marinas, Will Deacon,
	Martin Kepplinger, Angus Ainslie, Krzysztof Kozlowski,
	Bjorn Andersson, Li Yang, Geert Uytterhoeven, Vinod Koul,
	Anson Huang, Michael Walle, devicetree, linux-arm-kernel,
	linux-kernel, phone-devel

This enables more components to get a working display, panel, audio and
sensors. It also updates some device tree bits to make mainline boot.

I've skipped wifi (which needs some more driver work) and devfreq (since Martin
is working on that).

The config changes don't depend on the DT parts so could be applied
independently. The series was tested against next-20201214 and i made sure the
defconfig changes also apply on top of shawnguo/imx/defconfig.

changes from v1:
- as per review comments by Krzysztof Kozlowski <krzk@kernel.org>
  https://lore.kernel.org/linux-arm-kernel/20201215091729.GC29321@kozik-lap/
  - describe hardware parts not config options
  - rework commit messages to give more details
- don't keep buck3 always on but rather make sure the board
  doesn't hang when resuming the gpu power domain.
  There's a generic fix pending for that helps all boards
  https://lore.kernel.org/lkml/beba25e85db20649aa040fc0ae549895c9265f6f.camel@fi.rohmeurope.com/

To: Rob Herring <robh+dt@kernel.org>,Shawn Guo <shawnguo@kernel.org>,Sascha Hauer <s.hauer@pengutronix.de>,Pengutronix Kernel Team <kernel@pengutronix.de>,Fabio Estevam <festevam@gmail.com>,NXP Linux Team <linux-imx@nxp.com>,Catalin Marinas <catalin.marinas@arm.com>,Will Deacon <will@kernel.org>,Martin Kepplinger <martink@posteo.de>,Angus Ainslie <angus@akkea.ca>,Krzysztof Kozlowski <krzk@kernel.org>,Bjorn Andersson <bjorn.andersson@linaro.org>,Li Yang <leoyang.li@nxp.com>,Geert Uytterhoeven <geert+renesas@glider.be>,Vinod Koul <vkoul@kernel.org>,Anson Huang <Anson.Huang@nxp.com>,Michael Walle <michael@walle.cc>,devicetree@vger.kernel.org,linux-arm-kernel@lists.infradead.org,linux-kernel@vger.kernel.org,phone-devel@vger.kernel.org

Guido Günther (4):
  arm64: defconfig: Enable Librem 5 devkit components
  arm64: dts: imx8mq-librem5-devkit: Tweak pmic regulators
  arm64: dts: imx8mq-librem5-devkit: Disable snvs_rtc
  arm64: dts: imx8mq-librem5-devkit: Drop custom clock settings

 .../dts/freescale/imx8mq-librem5-devkit.dts   | 20 ++++++++++++++-----
 arch/arm64/configs/defconfig                  |  8 ++++++++
 2 files changed, 23 insertions(+), 5 deletions(-)

-- 
2.29.2


^ permalink raw reply	[flat|nested] 29+ messages in thread
* Re: [PATCH v2 1/4] arm64: defconfig: Enable Librem 5 devkit components
@ 2020-12-21  0:56 kernel test robot
  0 siblings, 0 replies; 29+ messages in thread
From: kernel test robot @ 2020-12-21  0:56 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 11275 bytes --]

In-Reply-To: <2f23f8a4fd7ccfbb4cacf933dcaebe7d38facdfe.1608216796.git.agx(a)sigxcpu.org>
References: <2f23f8a4fd7ccfbb4cacf933dcaebe7d38facdfe.1608216796.git.agx@sigxcpu.org>
TO: "Guido Günther" <agx@sigxcpu.org>
TO: Rob Herring <robh+dt@kernel.org>
TO: Shawn Guo <shawnguo@kernel.org>
TO: Sascha Hauer <s.hauer@pengutronix.de>
TO: Pengutronix Kernel Team <kernel@pengutronix.de>
TO: Fabio Estevam <festevam@gmail.com>
TO: NXP Linux Team <linux-imx@nxp.com>
TO: Catalin Marinas <catalin.marinas@arm.com>
TO: Will Deacon <will@kernel.org>
TO: Martin Kepplinger <martink@posteo.de>
TO: Angus Ainslie <angus@akkea.ca>

Hi "Guido,

I love your patch! Yet something to improve:

[auto build test ERROR on next-20201217]
[cannot apply to robh/for-next arm64/for-next/core shawnguo/for-next v5.10 v5.10-rc7 v5.10-rc6 v5.10]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Guido-G-nther/Config-and-device-tree-updates-for-the-Librem-5-devkit/20201217-231839
base:    90cc8cf2d1ab87d708ebc311ac104ccbbefad9fc
config: riscv-randconfig-r014-20201217 (attached as .config)
compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project cee1e7d14f4628d6174b33640d502bff3b54ae45)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install riscv cross compiling tool for clang build
        # apt-get install binutils-riscv64-linux-gnu
        # https://github.com/0day-ci/linux/commit/ac0db7178a552f39f5286499b7857a9d7e261051
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Guido-G-nther/Config-and-device-tree-updates-for-the-Librem-5-devkit/20201217-231839
        git checkout ac0db7178a552f39f5286499b7857a9d7e261051
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=riscv 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

Note: the linux-review/Guido-G-nther/Config-and-device-tree-updates-for-the-Librem-5-devkit/20201217-231839 HEAD 63dd5d489692c588221597924b6fbb9fdb16451a builds fine.
      It only hurts bisectibility.

All errors (new ones prefixed by >>):

   include/asm-generic/io.h:564:9: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           return inw(addr);
                  ^~~~~~~~~
   arch/riscv/include/asm/io.h:56:76: note: expanded from macro 'inw'
   #define inw(c)          ({ u16 __v; __io_pbr(); __v = readw_cpu((void*)(PCI_IOBASE + (c))); __io_par(__v); __v; })
                                                                           ~~~~~~~~~~ ^
   arch/riscv/include/asm/mmio.h:88:76: note: expanded from macro 'readw_cpu'
   #define readw_cpu(c)            ({ u16 __r = le16_to_cpu((__force __le16)__raw_readw(c)); __r; })
                                                                                        ^
   include/uapi/linux/byteorder/little_endian.h:36:51: note: expanded from macro '__le16_to_cpu'
   #define __le16_to_cpu(x) ((__force __u16)(__le16)(x))
                                                     ^
   In file included from drivers/mfd/tps65010.c:13:
   In file included from include/linux/interrupt.h:11:
   In file included from include/linux/hardirq.h:10:
   In file included from ./arch/riscv/include/generated/asm/hardirq.h:1:
   In file included from include/asm-generic/hardirq.h:17:
   In file included from include/linux/irq.h:20:
   In file included from include/linux/io.h:13:
   In file included from arch/riscv/include/asm/io.h:149:
   include/asm-generic/io.h:572:9: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           return inl(addr);
                  ^~~~~~~~~
   arch/riscv/include/asm/io.h:57:76: note: expanded from macro 'inl'
   #define inl(c)          ({ u32 __v; __io_pbr(); __v = readl_cpu((void*)(PCI_IOBASE + (c))); __io_par(__v); __v; })
                                                                           ~~~~~~~~~~ ^
   arch/riscv/include/asm/mmio.h:89:76: note: expanded from macro 'readl_cpu'
   #define readl_cpu(c)            ({ u32 __r = le32_to_cpu((__force __le32)__raw_readl(c)); __r; })
                                                                                        ^
   include/uapi/linux/byteorder/little_endian.h:34:51: note: expanded from macro '__le32_to_cpu'
   #define __le32_to_cpu(x) ((__force __u32)(__le32)(x))
                                                     ^
   In file included from drivers/mfd/tps65010.c:13:
   In file included from include/linux/interrupt.h:11:
   In file included from include/linux/hardirq.h:10:
   In file included from ./arch/riscv/include/generated/asm/hardirq.h:1:
   In file included from include/asm-generic/hardirq.h:17:
   In file included from include/linux/irq.h:20:
   In file included from include/linux/io.h:13:
   In file included from arch/riscv/include/asm/io.h:149:
   include/asm-generic/io.h:580:2: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           outb(value, addr);
           ^~~~~~~~~~~~~~~~~
   arch/riscv/include/asm/io.h:59:68: note: expanded from macro 'outb'
   #define outb(v,c)       ({ __io_pbw(); writeb_cpu((v),(void*)(PCI_IOBASE + (c))); __io_paw(); })
                                                                 ~~~~~~~~~~ ^
   arch/riscv/include/asm/mmio.h:91:52: note: expanded from macro 'writeb_cpu'
   #define writeb_cpu(v, c)        ((void)__raw_writeb((v), (c)))
                                                             ^
   In file included from drivers/mfd/tps65010.c:13:
   In file included from include/linux/interrupt.h:11:
   In file included from include/linux/hardirq.h:10:
   In file included from ./arch/riscv/include/generated/asm/hardirq.h:1:
   In file included from include/asm-generic/hardirq.h:17:
   In file included from include/linux/irq.h:20:
   In file included from include/linux/io.h:13:
   In file included from arch/riscv/include/asm/io.h:149:
   include/asm-generic/io.h:588:2: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           outw(value, addr);
           ^~~~~~~~~~~~~~~~~
   arch/riscv/include/asm/io.h:60:68: note: expanded from macro 'outw'
   #define outw(v,c)       ({ __io_pbw(); writew_cpu((v),(void*)(PCI_IOBASE + (c))); __io_paw(); })
                                                                 ~~~~~~~~~~ ^
   arch/riscv/include/asm/mmio.h:92:76: note: expanded from macro 'writew_cpu'
   #define writew_cpu(v, c)        ((void)__raw_writew((__force u16)cpu_to_le16(v), (c)))
                                                                                     ^
   In file included from drivers/mfd/tps65010.c:13:
   In file included from include/linux/interrupt.h:11:
   In file included from include/linux/hardirq.h:10:
   In file included from ./arch/riscv/include/generated/asm/hardirq.h:1:
   In file included from include/asm-generic/hardirq.h:17:
   In file included from include/linux/irq.h:20:
   In file included from include/linux/io.h:13:
   In file included from arch/riscv/include/asm/io.h:149:
   include/asm-generic/io.h:596:2: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           outl(value, addr);
           ^~~~~~~~~~~~~~~~~
   arch/riscv/include/asm/io.h:61:68: note: expanded from macro 'outl'
   #define outl(v,c)       ({ __io_pbw(); writel_cpu((v),(void*)(PCI_IOBASE + (c))); __io_paw(); })
                                                                 ~~~~~~~~~~ ^
   arch/riscv/include/asm/mmio.h:93:76: note: expanded from macro 'writel_cpu'
   #define writel_cpu(v, c)        ((void)__raw_writel((__force u32)cpu_to_le32(v), (c)))
                                                                                     ^
   In file included from drivers/mfd/tps65010.c:13:
   In file included from include/linux/interrupt.h:11:
   In file included from include/linux/hardirq.h:10:
   In file included from ./arch/riscv/include/generated/asm/hardirq.h:1:
   In file included from include/asm-generic/hardirq.h:17:
   In file included from include/linux/irq.h:20:
   In file included from include/linux/io.h:13:
   In file included from arch/riscv/include/asm/io.h:149:
   include/asm-generic/io.h:1005:55: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
           return (port > MMIO_UPPER_LIMIT) ? NULL : PCI_IOBASE + port;
                                                     ~~~~~~~~~~ ^
   7 warnings generated.
   /tmp/tps65010-703803.s: Assembler messages:
   /tmp/tps65010-703803.s:187: Error: unrecognized opcode `zext.b a2,a1'
   /tmp/tps65010-703803.s:247: Error: unrecognized opcode `zext.b a2,a2'
   /tmp/tps65010-703803.s:256: Error: unrecognized opcode `zext.b a2,a2'
   /tmp/tps65010-703803.s:420: Error: unrecognized opcode `zext.b a2,s2'
>> /tmp/tps65010-703803.s:466: Error: unrecognized opcode `zext.b a2,s3'
   /tmp/tps65010-703803.s:758: Error: unrecognized opcode `zext.b a0,a0'
   /tmp/tps65010-703803.s:800: Error: unrecognized opcode `zext.b a2,a0'
   /tmp/tps65010-703803.s:963: Error: unrecognized opcode `zext.b a1,a1'
   /tmp/tps65010-703803.s:985: Error: unrecognized opcode `zext.b a2,a1'
   /tmp/tps65010-703803.s:1073: Error: unrecognized opcode `zext.b a2,a1'
   /tmp/tps65010-703803.s:1380: Error: unrecognized opcode `zext.b a1,a0'
   /tmp/tps65010-703803.s:1509: Error: unrecognized opcode `zext.b a3,a0'
   /tmp/tps65010-703803.s:1625: Error: unrecognized opcode `zext.b a3,a0'
   /tmp/tps65010-703803.s:1699: Error: unrecognized opcode `zext.b a3,a0'
   /tmp/tps65010-703803.s:1795: Error: unrecognized opcode `zext.b a3,a0'
   /tmp/tps65010-703803.s:1862: Error: unrecognized opcode `zext.b a3,a0'
   /tmp/tps65010-703803.s:1886: Error: unrecognized opcode `zext.b a2,a0'
   /tmp/tps65010-703803.s:1895: Error: unrecognized opcode `zext.b a2,a0'
   /tmp/tps65010-703803.s:1904: Error: unrecognized opcode `zext.b a2,a0'
   /tmp/tps65010-703803.s:1948: Error: unrecognized opcode `zext.b a3,s1'
   /tmp/tps65010-703803.s:1949: Error: unrecognized opcode `zext.b a4,a0'
   /tmp/tps65010-703803.s:1977: Error: unrecognized opcode `zext.b a3,s1'
   /tmp/tps65010-703803.s:1978: Error: unrecognized opcode `zext.b a4,a0'
   /tmp/tps65010-703803.s:1994: Error: unrecognized opcode `zext.b a2,s1'
   /tmp/tps65010-703803.s:1995: Error: unrecognized opcode `zext.b a3,a0'
   clang-12: error: assembler command failed with exit code 1 (use -v to see invocation)

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 19417 bytes --]

^ permalink raw reply	[flat|nested] 29+ messages in thread
end of thread, other threads:[~2021-01-10 15:09 UTC | newest]

Thread overview: 29+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-12-17 15:13 [PATCH v2 0/4] Config and device tree updates for the Librem 5 devkit Guido Günther
2020-12-17 15:13 ` Guido Günther
2020-12-17 15:13 ` [PATCH v2 1/4] arm64: defconfig: Enable Librem 5 devkit components Guido Günther
2020-12-17 15:13   ` Guido Günther
2020-12-19 12:23   ` Krzysztof Kozlowski
2020-12-19 12:23     ` Krzysztof Kozlowski
2020-12-17 15:13 ` [PATCH v2 2/4] arm64: dts: imx8mq-librem5-devkit: Tweak pmic regulators Guido Günther
2020-12-17 15:13   ` Guido Günther
2020-12-19 12:25   ` Krzysztof Kozlowski
2020-12-19 12:25     ` Krzysztof Kozlowski
2021-01-10 12:24   ` Shawn Guo
2021-01-10 12:24     ` Shawn Guo
2020-12-17 15:13 ` [PATCH v2 3/4] arm64: dts: imx8mq-librem5-devkit: Disable snvs_rtc Guido Günther
2020-12-17 15:13   ` Guido Günther
2020-12-19 12:25   ` Krzysztof Kozlowski
2020-12-19 12:25     ` Krzysztof Kozlowski
2021-01-10 12:25   ` Shawn Guo
2021-01-10 12:25     ` Shawn Guo
2020-12-17 15:13 ` [PATCH v2 4/4] arm64: dts: imx8mq-librem5-devkit: Drop custom clock settings Guido Günther
2020-12-17 15:13   ` Guido Günther
2020-12-19 12:24   ` Krzysztof Kozlowski
2020-12-19 12:24     ` Krzysztof Kozlowski
2021-01-10 12:25   ` Shawn Guo
2021-01-10 12:25     ` Shawn Guo
2021-01-10 12:26 ` [PATCH v2 0/4] Config and device tree updates for the Librem 5 devkit Shawn Guo
2021-01-10 12:26   ` Shawn Guo
2021-01-10 15:06   ` Guido Günther
2021-01-10 15:06     ` Guido Günther
  -- strict thread matches above, loose matches on Subject: below --
2020-12-21  0:56 [PATCH v2 1/4] arm64: defconfig: Enable Librem 5 devkit components kernel test robot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.