All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v6 0/3] drm/lsdc: add drm driver for loongson display controller
@ 2022-02-03  8:25 ` Sui Jingfeng
  0 siblings, 0 replies; 83+ messages in thread
From: Sui Jingfeng @ 2022-02-03  8:25 UTC (permalink / raw)
  To: Dan Carpenter, Lucas Stach, Maarten Lankhorst, Maxime Ripard,
	Roland Scheidegger, Zack Rusin, Christian Gmeiner, David Airlie,
	Daniel Vetter, Rob Herring, Thomas Bogendoerfer,
	Krzysztof Kozlowski, Andrey Zhizhikin, Sam Ravnborg, suijingfeng
  Cc: linux-mips, linux-kernel, dri-devel, Randy Dunlap

There is a display controller in loongson's LS2K1000 SoC and LS7A1000
bridge, and the DC in those chip is a PCI device. This patch provide
a minimal support for this display controller which is mainly for
graphic environment bring up.

This display controller has two display pipes but with only one hardware
cursor. Each way has a DVO output interface and the CRTC is able to scanout
from 1920x1080 resolution at 60Hz. The maxmium resolution is 2048x2048@60hz.

LS2K1000 is a SoC, only system memory is available. Therefore scanout from
system memory is the only choice. We prefer the CMA helper base solution
because the gc1000 gpu can use etnaviv driver, in this case etnaviv and
lsdc could made a compatible pair. Even through it is possible to use VRAM
helper base solution on ls2k1000 by carving out part of system memory as
VRAM.

For LS7A1000, there are 4 gpios whos control register is located at the dc
register space which is not the geneal purpose GPIO. The 4 gpios can emulate
two way i2c. One for DVO0, another for DVO1. This is the reason the why we
are not using the drm bridge framework.

LS2K1000 and LS2K0500 SoC don't have such hardware, they use general purpose
GPIO emulated i2c or hardware i2c adapter from other module to serve this
purpose. Drm bridge and drm panel is suitable for those SoC, we have already
implement it on our own downstream kernel. But due to the upstream kernel
don't have gpio, pwm and i2c driver support for LS2K1000. We just can not
upstream our support for the drm bridge subsystem.

The DC in LS7A1000 can scanout from both the system memory and the dedicate
VRAM attached to the ddr3 memory controller. Sadly, only scanout from the
VRAM is proved to be a reliable solution for massive product. Scanout from
the system memory suffer from some hardware deficiency which cause the
screen flickering under RAM pressure. This is the reason why we integrate
two distict helpers into one piece of device driver. But CMA base helper is
still usable on ls7a1000 for normal usage, expecially on ls7a1000+ bridge
chip. We have also implemented demage update on top of CMA helper which
copy the demaged shadow framebuffer region from system RAM to the real
framebuffer in VRAM manually. Using "lsdc.dirty_update=1" in the commmand
line will enable this driver mode.

LS7A1000 have a 32x32 harware cursor, we just let the two CRTC share it
simply with the help of universe plane. LS7A2000 have two 64x64 harware
cursor. Surport for LS7A2000 is on the way.

In short, we have built-in gpio emulated i2c support, we also have hardware
cursor support. The kind of tiny drivers in drm/tiny is not suitable for us,
we are not "tiny".

    +------+  HyperTransport 3.0
    | DDR4 |       |
    +------+       |     +------------------------------------+
       || MC0      |     |   LS7A1000            +------------|
  +----------+     |     |                       |    DDR3    |   +------+
  | LS3A4000 |<--------->| +--------+  +-------+ |   memory   |<->| VRAM |
  |   CPU    |<--------->| | GC1000 |  |  LSDC | | controller |   +------+
  +----------+           | +--------+  +-+---+-+ +------------|
       || MC1            +---------------|---|----------------+
    +------+                             |   |
    | DDR4 |          +-------+    DVO0  |   |  DVO1  +------+
    +------+   VGA <--|ADV7125|<---------+   +------->|TFP410|--> DVI/HDMI
                      +-------+                       +------+

The above picture give a simple usage of LS7A1000, note that the encoder
is not necessary adv7125 or tfp410, it is a choice of the downstream board
manufacturer. Other candicate encoder can be ch7034b, sil9022 and ite66121
etc. Therefore, we need device tree to provide board specific information.
Besides, the dc in both ls2k1000 and ls7k1000 have the vendor:device id of
0x0014:0x7a06, the reverison id is also same. We can't tell it apart simply
(this is the firmware engineer's mistake). But firmware already flushed to
the board and borad already sold out, we choose to resolve those issues by
introduing device tree with board specific device support.

For lsdc, there is only a 1:1 mapping of encoders and connectors.

     +-------------------+                                      _________
     |                   |                                     |         |
     |  CRTC0 --> DVO0 ---------> Encoder0 --> Connector0 ---> | Monitor |
     |                   |           ^            ^            |_________|
     |                   |           |            |
     |                <----- i2c0 ----------------+
     |   LSDC IP CORE    |
     |                <----- i2c1 ----------------+
     |                   |           |            |             _________
     |                   |           |            |            |         |
     |  CRTC1 --> DVO1 ---------> Encoder1 --> Connector1 ---> |  Panel  |
     |                   |                                     |_________|
     +-------------------+

Below is a brief introduction of loongson's CPU, bridge chip and SoC.
LS2K1000 is a double core 1.0Ghz mips64r2 compatible SoC[1]. LS7A1000 is
a bridge chip made by Loongson corporation which act as north and/or south
bridge of loongson's desktop and server level processor. It is equivalent
to AMD RS780E+SB710 or something like that. More details can be read from
its user manual[2].

This bridge chip is typically use with LS3A3000, LS3A4000 and LS3A5000 cpu.
LS3A3000 is 4 core 1.45gHz mips64r2 compatible cpu.
LS3A4000 is 4 core 1.8gHz mips64r5 compatible cpu.
LS3A5000 is 4 core 2.5gHz loongarch cpu[3].

Nearly all mordern loongson CPU's cache coherency is maintained by hardware,
except for early version of ls2k1000. So we using cached coherent memory by
default, not writecombine.

v2: fixup warnings reported by kernel test robot

v3: fix more grammar mistakes in Kconfig reported by Randy Dunlap and give
    more details about lsdc.

v4:
   1) Add dts required and explain why device tree is required.
   2) Give more description about lsdc and vram helper base driver.
   3) Fix warnings reported by kernel test robot.
   4) Introduce stride_alignment member into struct lsdc_chip_desc, the
      stride alignment is 256 bytes for ls7a1000, ls2k1000 and ls2k0500 SoC.
      But ls7a2000 improve it to 32 bytes, We are prepare for extend the
      support for the on coming device.

v5:
   1) using writel and readl replace writeq and readq, to fix kernel test
      robot report build error on other archtecture
   2) set default fb format to XRGB8888 at crtc reset time.
   3) fix typos.

v6:
   1) Explain why we are not switch to drm dridge subsystem.
   2) Explain why tiny drm driver is not suitable for us.
   3) Give a short description of the trival dirty uppdate implement based
      on CMA helper.
   4) code clean up.

[1] https://wiki.debian.org/InstallingDebianOn/Lemote/Loongson2K1000
[2] https://loongson.github.io/LoongArch-Documentation/Loongson-7A1000-usermanual-EN.html
[3] https://loongson.github.io/LoongArch-Documentation/Loongson-3A5000-usermanual-EN.html

Reported-by: Randy Dunlap <rdunlap@infradead.org>
Reported-by: kernel test robot
Signed-off-by: suijingfeng <suijingfeng@loongson.cn>
Signed-off-by: Sui Jingfeng <15330273260@189.cn>

suijingfeng (3):
  drm/lsdc: add drm driver for loongson display controller
  dt-bindings: ls2k1000: add the display controller device node
  dt-bindings: mips: loongson: introduce board specific dts

 arch/mips/boot/dts/loongson/lemote_a1901.dts  |  64 ++
 .../boot/dts/loongson/loongson64-2k1000.dtsi  |  11 +
 .../boot/dts/loongson/ls3a4000_7a1000_evb.dts |  68 ++
 arch/mips/boot/dts/loongson/ls7a-pch.dtsi     |   2 +-
 drivers/gpu/drm/Kconfig                       |   2 +
 drivers/gpu/drm/Makefile                      |   1 +
 drivers/gpu/drm/lsdc/Kconfig                  |  38 +
 drivers/gpu/drm/lsdc/Makefile                 |  15 +
 drivers/gpu/drm/lsdc/lsdc_connector.c         | 443 +++++++++
 drivers/gpu/drm/lsdc/lsdc_connector.h         |  60 ++
 drivers/gpu/drm/lsdc/lsdc_crtc.c              | 440 +++++++++
 drivers/gpu/drm/lsdc/lsdc_drv.c               | 846 ++++++++++++++++++
 drivers/gpu/drm/lsdc/lsdc_drv.h               | 216 +++++
 drivers/gpu/drm/lsdc/lsdc_encoder.c           |  79 ++
 drivers/gpu/drm/lsdc/lsdc_i2c.c               | 220 +++++
 drivers/gpu/drm/lsdc/lsdc_i2c.h               |  61 ++
 drivers/gpu/drm/lsdc/lsdc_irq.c               |  77 ++
 drivers/gpu/drm/lsdc/lsdc_irq.h               |  37 +
 drivers/gpu/drm/lsdc/lsdc_plane.c             | 681 ++++++++++++++
 drivers/gpu/drm/lsdc/lsdc_pll.c               | 657 ++++++++++++++
 drivers/gpu/drm/lsdc/lsdc_pll.h               | 109 +++
 drivers/gpu/drm/lsdc/lsdc_regs.h              | 246 +++++
 22 files changed, 4372 insertions(+), 1 deletion(-)
 create mode 100644 arch/mips/boot/dts/loongson/lemote_a1901.dts
 create mode 100644 arch/mips/boot/dts/loongson/ls3a4000_7a1000_evb.dts
 create mode 100644 drivers/gpu/drm/lsdc/Kconfig
 create mode 100644 drivers/gpu/drm/lsdc/Makefile
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_connector.c
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_connector.h
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_crtc.c
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_drv.c
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_drv.h
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_encoder.c
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_i2c.c
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_i2c.h
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_irq.c
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_irq.h
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_plane.c
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_pll.c
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_pll.h
 create mode 100644 drivers/gpu/drm/lsdc/lsdc_regs.h

-- 
2.20.1


^ permalink raw reply	[flat|nested] 83+ messages in thread
* Re: [PATCH v6 1/3] drm/lsdc: add drm driver for loongson display controller
@ 2022-02-06 19:38 kernel test robot
  0 siblings, 0 replies; 83+ messages in thread
From: kernel test robot @ 2022-02-06 19:38 UTC (permalink / raw)
  To: kbuild

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

CC: llvm(a)lists.linux.dev
CC: kbuild-all(a)lists.01.org
In-Reply-To: <20220203082546.3099-2-15330273260@189.cn>
References: <20220203082546.3099-2-15330273260@189.cn>
TO: Sui Jingfeng <15330273260@189.cn>

Hi Sui,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm/drm-next]
[also build test WARNING on robh/for-next drm-intel/for-linux-next drm-tip/drm-tip v5.17-rc2 next-20220204]
[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/Sui-Jingfeng/drm-lsdc-add-drm-driver-for-loongson-display-controller/20220203-162953
base:   git://anongit.freedesktop.org/drm/drm drm-next
:::::: branch date: 3 days ago
:::::: commit date: 3 days ago
config: riscv-randconfig-c006-20220205 (https://download.01.org/0day-ci/archive/20220207/202202070332.W1dAT59x-lkp(a)intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project dee058c670593b999fec19c458dbbd882ad9de56)
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/983916ec4a8df390a947a7f539dbc99f5d511e26
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Sui-Jingfeng/drm-lsdc-add-drm-driver-for-loongson-display-controller/20220203-162953
        git checkout 983916ec4a8df390a947a7f539dbc99f5d511e26
        # save the config file to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=riscv clang-analyzer 

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


clang-analyzer warnings: (new ones prefixed by >>)
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   2 warnings generated.
   Suppressed 2 warnings (2 in non-user code).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   2 warnings generated.
   Suppressed 2 warnings (2 in non-user code).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   3 warnings generated.
   fs/kernfs/dir.c:672:6: warning: Value stored to 'gen' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
           u32 gen = kernfs_id_gen(id);
               ^~~   ~~~~~~~~~~~~~~~~~
   fs/kernfs/dir.c:672:6: note: Value stored to 'gen' during its initialization is never read
           u32 gen = kernfs_id_gen(id);
               ^~~   ~~~~~~~~~~~~~~~~~
   Suppressed 2 warnings (2 in non-user code).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   10 warnings generated.
   Suppressed 10 warnings (2 in non-user code, 8 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   13 warnings generated.
   drivers/gpu/drm/lsdc/lsdc_crtc.c:123:33: warning: The left operand of '&' is a garbage value [clang-analyzer-core.UndefinedBinaryOperatorResult]
           priv_crtc_state->pix_fmt = val & CFG_PIX_FMT_MASK;
                                      ~~~ ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:97:2: note: 'val' declared without an initial value
           u32 val;
           ^~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:104:6: note: Assuming 'index' is not equal to 0
           if (index == 0) {
               ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:104:2: note: Taking false branch
           if (index == 0) {
           ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:107:13: note: Assuming 'index' is not equal to 1
           } else if (index == 1) {
                      ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:107:9: note: Taking false branch
           } else if (index == 1) {
                  ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:113:6: note: Assuming field 'state' is null
           if (crtc->state) {
               ^~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:113:2: note: Taking false branch
           if (crtc->state) {
           ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:119:20: note: Calling 'kzalloc'
           priv_crtc_state = kzalloc(sizeof(*priv_crtc_state), GFP_KERNEL);
                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:715:9: note: Calling 'kmalloc'
           return kmalloc(size, flags | __GFP_ZERO);
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:569:2: note: Taking false branch
           if (__builtin_constant_p(size)) {
           ^
   include/linux/slab.h:586:2: note: Returning pointer, which participates in a condition later
           return __kmalloc(size, flags);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:715:9: note: Returning from 'kmalloc'
           return kmalloc(size, flags | __GFP_ZERO);
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:715:2: note: Returning pointer, which participates in a condition later
           return kmalloc(size, flags | __GFP_ZERO);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:119:20: note: Returning from 'kzalloc'
           priv_crtc_state = kzalloc(sizeof(*priv_crtc_state), GFP_KERNEL);
                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:120:6: note: Assuming 'priv_crtc_state' is non-null
           if (!priv_crtc_state)
               ^~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:120:2: note: Taking false branch
           if (!priv_crtc_state)
           ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:123:33: note: The left operand of '&' is a garbage value
           priv_crtc_state->pix_fmt = val & CFG_PIX_FMT_MASK;
                                      ~~~ ^
>> drivers/gpu/drm/lsdc/lsdc_crtc.c:146:21: warning: Value stored to 'ddev' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
           struct drm_device *ddev = crtc->dev;
                              ^~~~   ~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:146:21: note: Value stored to 'ddev' during its initialization is never read
           struct drm_device *ddev = crtc->dev;
                              ^~~~   ~~~~~~~~~
   Suppressed 11 warnings (4 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   11 warnings generated.
   Suppressed 11 warnings (4 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   22 warnings generated.
   drivers/gpu/drm/lsdc/lsdc_plane.c:163:13: warning: The left operand of '&' is a garbage value [clang-analyzer-core.UndefinedBinaryOperatorResult]
           return val & CFG_PIX_FMT_MASK;
                      ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:277:6: note: Assuming 'crtc' is non-null
           if (!crtc)
               ^~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:277:2: note: Taking false branch
           if (!crtc)
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:281:14: note: Assuming 'new_crtc_state' is non-null
           if (WARN_ON(!new_crtc_state))
                       ^
   include/asm-generic/bug.h:166:25: note: expanded from macro 'WARN_ON'
           int __ret_warn_on = !!(condition);                              \
                                  ^~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:281:2: note: Taking false branch
           if (WARN_ON(!new_crtc_state))
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:292:6: note: Assuming 'ret' is 0
           if (ret)
               ^~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:292:2: note: Taking false branch
           if (ret)
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:299:8: note: 'new_fb' is non-null
           if ((!new_fb || !old_fb ||
                 ^~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:299:7: note: Left side of '||' is false
           if ((!new_fb || !old_fb ||
                ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:299:18: note: Assuming 'old_fb' is null
           if ((!new_fb || !old_fb ||
                           ^~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:299:26: note: Left side of '||' is true
           if ((!new_fb || !old_fb ||
                                   ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:308:29: note: Calling 'lsdc_primary_get_default_format'
           priv_crtc_state->pix_fmt = lsdc_primary_get_default_format(crtc);
                                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:156:2: note: 'val' declared without an initial value
           u32 val;
           ^~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:158:6: note: Assuming 'index' is not equal to 0
           if (index == 0)
               ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:158:2: note: Taking false branch
           if (index == 0)
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:160:11: note: Assuming 'index' is not equal to 1
           else if (index == 1)
                    ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:160:7: note: Taking false branch
           else if (index == 1)
                ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:163:13: note: The left operand of '&' is a garbage value
           return val & CFG_PIX_FMT_MASK;
                  ~~~ ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:209:2: warning: 2nd function call argument is an uninitialized value [clang-analyzer-core.CallAndMessage]
           lsdc_reg_write32(ldev, addr_reg, paddr);
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:365:6: note: Assuming field 'use_vram_helper' is false
           if (ldev->use_vram_helper) {
               ^~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:365:2: note: Taking false branch
           if (ldev->use_vram_helper) {
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:378:7: note: Assuming field 'dirty_update' is true
                   if (ldev->dirty_update)
                       ^~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:378:3: note: Taking true branch
                   if (ldev->dirty_update)
                   ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:384:2: note: Calling 'lsdc_update_fb_start_addr'
           lsdc_update_fb_start_addr(ldev, crtc, fb_addr);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:172:2: note: 'addr_reg' declared without an initial value
           u32 addr_reg;
           ^~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:181:6: note: Assuming 'index' is not equal to 0
           if (index == 0) {
               ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:181:2: note: Taking false branch
           if (index == 0) {
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:194:13: note: Assuming 'index' is not equal to 1
           } else if (index == 1) {
                      ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:194:9: note: Taking false branch
           } else if (index == 1) {

vim +/ddev +146 drivers/gpu/drm/lsdc/lsdc_crtc.c

983916ec4a8df3 suijingfeng 2022-02-03  140  
983916ec4a8df3 suijingfeng 2022-02-03  141  
983916ec4a8df3 suijingfeng 2022-02-03  142  static struct drm_crtc_state *lsdc_crtc_atomic_duplicate_state(struct drm_crtc *crtc)
983916ec4a8df3 suijingfeng 2022-02-03  143  {
983916ec4a8df3 suijingfeng 2022-02-03  144  	struct lsdc_crtc_state *new_priv_state;
983916ec4a8df3 suijingfeng 2022-02-03  145  	struct lsdc_crtc_state *old_priv_state;
983916ec4a8df3 suijingfeng 2022-02-03 @146  	struct drm_device *ddev = crtc->dev;
983916ec4a8df3 suijingfeng 2022-02-03  147  
983916ec4a8df3 suijingfeng 2022-02-03  148  	if (drm_WARN_ON(ddev, !crtc->state))
983916ec4a8df3 suijingfeng 2022-02-03  149  		return NULL;
983916ec4a8df3 suijingfeng 2022-02-03  150  
983916ec4a8df3 suijingfeng 2022-02-03  151  	new_priv_state = kmalloc(sizeof(*new_priv_state), GFP_KERNEL);
983916ec4a8df3 suijingfeng 2022-02-03  152  	if (!new_priv_state)
983916ec4a8df3 suijingfeng 2022-02-03  153  		return NULL;
983916ec4a8df3 suijingfeng 2022-02-03  154  
983916ec4a8df3 suijingfeng 2022-02-03  155  	__drm_atomic_helper_crtc_duplicate_state(crtc, &new_priv_state->base);
983916ec4a8df3 suijingfeng 2022-02-03  156  
983916ec4a8df3 suijingfeng 2022-02-03  157  	old_priv_state = to_lsdc_crtc_state(crtc->state);
983916ec4a8df3 suijingfeng 2022-02-03  158  
983916ec4a8df3 suijingfeng 2022-02-03  159  	memcpy(&new_priv_state->pparams, &old_priv_state->pparams,
983916ec4a8df3 suijingfeng 2022-02-03  160  		sizeof(new_priv_state->pparams));
983916ec4a8df3 suijingfeng 2022-02-03  161  
983916ec4a8df3 suijingfeng 2022-02-03  162  	new_priv_state->pix_fmt = old_priv_state->pix_fmt;
983916ec4a8df3 suijingfeng 2022-02-03  163  
983916ec4a8df3 suijingfeng 2022-02-03  164  	return &new_priv_state->base;
983916ec4a8df3 suijingfeng 2022-02-03  165  }
983916ec4a8df3 suijingfeng 2022-02-03  166  

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

^ permalink raw reply	[flat|nested] 83+ messages in thread
* Re: [PATCH v6 1/3] drm/lsdc: add drm driver for loongson display controller
@ 2022-02-10 23:10 kernel test robot
  0 siblings, 0 replies; 83+ messages in thread
From: kernel test robot @ 2022-02-10 23:10 UTC (permalink / raw)
  To: kbuild

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

CC: llvm(a)lists.linux.dev
CC: kbuild-all(a)lists.01.org
In-Reply-To: <20220203082546.3099-2-15330273260@189.cn>
References: <20220203082546.3099-2-15330273260@189.cn>
TO: Sui Jingfeng <15330273260@189.cn>

Hi Sui,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm/drm-next]
[also build test WARNING on robh/for-next drm-intel/for-linux-next drm-tip/drm-tip v5.17-rc3]
[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/Sui-Jingfeng/drm-lsdc-add-drm-driver-for-loongson-display-controller/20220203-162953
base:   git://anongit.freedesktop.org/drm/drm drm-next
:::::: branch date: 8 days ago
:::::: commit date: 8 days ago
config: riscv-randconfig-c006-20220205 (https://download.01.org/0day-ci/archive/20220211/202202110707.gB1LATq7-lkp(a)intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project dee058c670593b999fec19c458dbbd882ad9de56)
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/983916ec4a8df390a947a7f539dbc99f5d511e26
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Sui-Jingfeng/drm-lsdc-add-drm-driver-for-loongson-display-controller/20220203-162953
        git checkout 983916ec4a8df390a947a7f539dbc99f5d511e26
        # save the config file to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=riscv clang-analyzer 

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


clang-analyzer warnings: (new ones prefixed by >>)
               ^~~~~~~~~~~~~~~~~~~~
   include/asm-generic/bitops/__ffs.h:23:2: note: Taking true branch
           if ((word & 0xffff) == 0) {
           ^
   include/asm-generic/bitops/__ffs.h:27:6: note: Assuming the condition is true
           if ((word & 0xff) == 0) {
               ^~~~~~~~~~~~~~~~~~
   include/asm-generic/bitops/__ffs.h:27:2: note: Taking true branch
           if ((word & 0xff) == 0) {
           ^
   include/asm-generic/bitops/__ffs.h:31:6: note: Assuming the condition is true
           if ((word & 0xf) == 0) {
               ^~~~~~~~~~~~~~~~~
   include/asm-generic/bitops/__ffs.h:31:2: note: Taking true branch
           if ((word & 0xf) == 0) {
           ^
   include/asm-generic/bitops/__ffs.h:35:6: note: Assuming the condition is true
           if ((word & 0x3) == 0) {
               ^~~~~~~~~~~~~~~~~
   include/asm-generic/bitops/__ffs.h:35:2: note: Taking true branch
           if ((word & 0x3) == 0) {
           ^
   include/asm-generic/bitops/__ffs.h:39:6: note: Assuming the condition is true
           if ((word & 0x1) == 0)
               ^~~~~~~~~~~~~~~~~
   include/asm-generic/bitops/__ffs.h:39:2: note: Taking true branch
           if ((word & 0x1) == 0)
           ^
   include/asm-generic/bitops/__ffs.h:40:3: note: The value 63 is assigned to 'num'
                   num += 1;
                   ^~~~~~~~
   include/asm-generic/bitops/__ffs.h:41:2: note: Returning the value 63 (loaded from 'num')
           return num;
           ^~~~~~~~~~
   drivers/mfd/stmpe.c:1124:14: note: Returning from '__ffs'
                           int bit = __ffs(status);
                                     ^~~~~~~~~~~~~
   drivers/mfd/stmpe.c:1124:4: note: 'bit' initialized to 63
                           int bit = __ffs(status);
                           ^~~~~~~
   drivers/mfd/stmpe.c:1129:18: note: The result of the left shift is undefined due to shifting by '63', which is greater or equal to the width of type 'int'
                           status &= ~(1 << bit);
                                         ^  ~~~
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   13 warnings generated.
   drivers/gpu/drm/lsdc/lsdc_crtc.c:123:33: warning: The left operand of '&' is a garbage value [clang-analyzer-core.UndefinedBinaryOperatorResult]
           priv_crtc_state->pix_fmt = val & CFG_PIX_FMT_MASK;
                                      ~~~ ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:97:2: note: 'val' declared without an initial value
           u32 val;
           ^~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:104:6: note: Assuming 'index' is not equal to 0
           if (index == 0) {
               ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:104:2: note: Taking false branch
           if (index == 0) {
           ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:107:13: note: Assuming 'index' is not equal to 1
           } else if (index == 1) {
                      ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:107:9: note: Taking false branch
           } else if (index == 1) {
                  ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:113:6: note: Assuming field 'state' is null
           if (crtc->state) {
               ^~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:113:2: note: Taking false branch
           if (crtc->state) {
           ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:119:20: note: Calling 'kzalloc'
           priv_crtc_state = kzalloc(sizeof(*priv_crtc_state), GFP_KERNEL);
                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:715:9: note: Calling 'kmalloc'
           return kmalloc(size, flags | __GFP_ZERO);
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:569:2: note: Taking false branch
           if (__builtin_constant_p(size)) {
           ^
   include/linux/slab.h:586:2: note: Returning pointer, which participates in a condition later
           return __kmalloc(size, flags);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:715:9: note: Returning from 'kmalloc'
           return kmalloc(size, flags | __GFP_ZERO);
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:715:2: note: Returning pointer, which participates in a condition later
           return kmalloc(size, flags | __GFP_ZERO);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:119:20: note: Returning from 'kzalloc'
           priv_crtc_state = kzalloc(sizeof(*priv_crtc_state), GFP_KERNEL);
                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:120:6: note: Assuming 'priv_crtc_state' is non-null
           if (!priv_crtc_state)
               ^~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:120:2: note: Taking false branch
           if (!priv_crtc_state)
           ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:123:33: note: The left operand of '&' is a garbage value
           priv_crtc_state->pix_fmt = val & CFG_PIX_FMT_MASK;
                                      ~~~ ^
>> drivers/gpu/drm/lsdc/lsdc_crtc.c:146:21: warning: Value stored to 'ddev' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
           struct drm_device *ddev = crtc->dev;
                              ^~~~   ~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:146:21: note: Value stored to 'ddev' during its initialization is never read
           struct drm_device *ddev = crtc->dev;
                              ^~~~   ~~~~~~~~~
   Suppressed 11 warnings (4 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   11 warnings generated.
   Suppressed 11 warnings (4 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   22 warnings generated.
   drivers/gpu/drm/lsdc/lsdc_plane.c:163:13: warning: The left operand of '&' is a garbage value [clang-analyzer-core.UndefinedBinaryOperatorResult]
           return val & CFG_PIX_FMT_MASK;
                      ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:277:6: note: Assuming 'crtc' is non-null
           if (!crtc)
               ^~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:277:2: note: Taking false branch
           if (!crtc)
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:281:14: note: Assuming 'new_crtc_state' is non-null
           if (WARN_ON(!new_crtc_state))
                       ^
   include/asm-generic/bug.h:166:25: note: expanded from macro 'WARN_ON'
           int __ret_warn_on = !!(condition);                              \
                                  ^~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:281:2: note: Taking false branch
           if (WARN_ON(!new_crtc_state))
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:292:6: note: Assuming 'ret' is 0
           if (ret)
               ^~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:292:2: note: Taking false branch
           if (ret)
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:299:8: note: 'new_fb' is non-null
           if ((!new_fb || !old_fb ||
                 ^~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:299:7: note: Left side of '||' is false
           if ((!new_fb || !old_fb ||
                ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:299:18: note: Assuming 'old_fb' is null
           if ((!new_fb || !old_fb ||
                           ^~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:299:26: note: Left side of '||' is true
           if ((!new_fb || !old_fb ||
                                   ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:308:29: note: Calling 'lsdc_primary_get_default_format'
           priv_crtc_state->pix_fmt = lsdc_primary_get_default_format(crtc);
                                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:156:2: note: 'val' declared without an initial value
           u32 val;
           ^~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:158:6: note: Assuming 'index' is not equal to 0
           if (index == 0)
               ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:158:2: note: Taking false branch
           if (index == 0)
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:160:11: note: Assuming 'index' is not equal to 1
           else if (index == 1)
                    ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:160:7: note: Taking false branch
           else if (index == 1)
                ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:163:13: note: The left operand of '&' is a garbage value
           return val & CFG_PIX_FMT_MASK;
                  ~~~ ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:209:2: warning: 2nd function call argument is an uninitialized value [clang-analyzer-core.CallAndMessage]
           lsdc_reg_write32(ldev, addr_reg, paddr);
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:365:6: note: Assuming field 'use_vram_helper' is false
           if (ldev->use_vram_helper) {
               ^~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:365:2: note: Taking false branch
           if (ldev->use_vram_helper) {
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:378:7: note: Assuming field 'dirty_update' is true
                   if (ldev->dirty_update)
                       ^~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:378:3: note: Taking true branch
                   if (ldev->dirty_update)
                   ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:384:2: note: Calling 'lsdc_update_fb_start_addr'
           lsdc_update_fb_start_addr(ldev, crtc, fb_addr);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:172:2: note: 'addr_reg' declared without an initial value
           u32 addr_reg;
           ^~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:181:6: note: Assuming 'index' is not equal to 0
           if (index == 0) {
               ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:181:2: note: Taking false branch
           if (index == 0) {
           ^
   drivers/gpu/drm/lsdc/lsdc_plane.c:194:13: note: Assuming 'index' is not equal to 1
           } else if (index == 1) {
                      ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_plane.c:194:9: note: Taking false branch
           } else if (index == 1) {

vim +/ddev +146 drivers/gpu/drm/lsdc/lsdc_crtc.c

983916ec4a8df3 suijingfeng 2022-02-03   89  
983916ec4a8df3 suijingfeng 2022-02-03   90  
983916ec4a8df3 suijingfeng 2022-02-03   91  static void lsdc_crtc_reset(struct drm_crtc *crtc)
983916ec4a8df3 suijingfeng 2022-02-03   92  {
983916ec4a8df3 suijingfeng 2022-02-03   93  	struct drm_device *ddev = crtc->dev;
983916ec4a8df3 suijingfeng 2022-02-03   94  	struct lsdc_device *ldev = to_lsdc(ddev);
983916ec4a8df3 suijingfeng 2022-02-03   95  	unsigned int index = drm_crtc_index(crtc);
983916ec4a8df3 suijingfeng 2022-02-03   96  	struct lsdc_crtc_state *priv_crtc_state;
983916ec4a8df3 suijingfeng 2022-02-03   97  	u32 val;
983916ec4a8df3 suijingfeng 2022-02-03   98  
983916ec4a8df3 suijingfeng 2022-02-03   99  	/* The crtc get soft reset if bit 20 of CRTC*_CFG_REG
983916ec4a8df3 suijingfeng 2022-02-03  100  	 * is write with falling edge.
983916ec4a8df3 suijingfeng 2022-02-03  101  	 *
983916ec4a8df3 suijingfeng 2022-02-03  102  	 * Doing this to switch from soft reset state to working state
983916ec4a8df3 suijingfeng 2022-02-03  103  	 */
983916ec4a8df3 suijingfeng 2022-02-03  104  	if (index == 0) {
983916ec4a8df3 suijingfeng 2022-02-03  105  		val = CFG_RESET_BIT | CFG_OUTPUT_EN_BIT | LSDC_PF_XRGB8888;
983916ec4a8df3 suijingfeng 2022-02-03  106  		lsdc_reg_write32(ldev, LSDC_CRTC0_CFG_REG, val);
983916ec4a8df3 suijingfeng 2022-02-03  107  	} else if (index == 1) {
983916ec4a8df3 suijingfeng 2022-02-03  108  		val = CFG_RESET_BIT | CFG_OUTPUT_EN_BIT | LSDC_PF_XRGB8888;
983916ec4a8df3 suijingfeng 2022-02-03  109  		lsdc_reg_write32(ldev, LSDC_CRTC1_CFG_REG, val);
983916ec4a8df3 suijingfeng 2022-02-03  110  	}
983916ec4a8df3 suijingfeng 2022-02-03  111  
983916ec4a8df3 suijingfeng 2022-02-03  112  
983916ec4a8df3 suijingfeng 2022-02-03  113  	if (crtc->state) {
983916ec4a8df3 suijingfeng 2022-02-03  114  		priv_crtc_state = to_lsdc_crtc_state(crtc->state);
983916ec4a8df3 suijingfeng 2022-02-03  115  		__drm_atomic_helper_crtc_destroy_state(&priv_crtc_state->base);
983916ec4a8df3 suijingfeng 2022-02-03  116  		kfree(priv_crtc_state);
983916ec4a8df3 suijingfeng 2022-02-03  117  	}
983916ec4a8df3 suijingfeng 2022-02-03  118  
983916ec4a8df3 suijingfeng 2022-02-03  119  	priv_crtc_state = kzalloc(sizeof(*priv_crtc_state), GFP_KERNEL);
983916ec4a8df3 suijingfeng 2022-02-03 @120  	if (!priv_crtc_state)
983916ec4a8df3 suijingfeng 2022-02-03  121  		return;
983916ec4a8df3 suijingfeng 2022-02-03  122  
983916ec4a8df3 suijingfeng 2022-02-03  123  	priv_crtc_state->pix_fmt = val & CFG_PIX_FMT_MASK;
983916ec4a8df3 suijingfeng 2022-02-03  124  
983916ec4a8df3 suijingfeng 2022-02-03  125  	__drm_atomic_helper_crtc_reset(crtc, &priv_crtc_state->base);
983916ec4a8df3 suijingfeng 2022-02-03  126  
983916ec4a8df3 suijingfeng 2022-02-03  127  	drm_info(ddev, "crtc%u reset\n", index);
983916ec4a8df3 suijingfeng 2022-02-03  128  }
983916ec4a8df3 suijingfeng 2022-02-03  129  
983916ec4a8df3 suijingfeng 2022-02-03  130  
983916ec4a8df3 suijingfeng 2022-02-03  131  static void lsdc_crtc_atomic_destroy_state(struct drm_crtc *crtc,
983916ec4a8df3 suijingfeng 2022-02-03  132  				    struct drm_crtc_state *state)
983916ec4a8df3 suijingfeng 2022-02-03  133  {
983916ec4a8df3 suijingfeng 2022-02-03  134  	struct lsdc_crtc_state *priv_crtc_state = to_lsdc_crtc_state(state);
983916ec4a8df3 suijingfeng 2022-02-03  135  
983916ec4a8df3 suijingfeng 2022-02-03  136  	__drm_atomic_helper_crtc_destroy_state(&priv_crtc_state->base);
983916ec4a8df3 suijingfeng 2022-02-03  137  
983916ec4a8df3 suijingfeng 2022-02-03  138  	kfree(priv_crtc_state);
983916ec4a8df3 suijingfeng 2022-02-03  139  }
983916ec4a8df3 suijingfeng 2022-02-03  140  
983916ec4a8df3 suijingfeng 2022-02-03  141  
983916ec4a8df3 suijingfeng 2022-02-03  142  static struct drm_crtc_state *lsdc_crtc_atomic_duplicate_state(struct drm_crtc *crtc)
983916ec4a8df3 suijingfeng 2022-02-03  143  {
983916ec4a8df3 suijingfeng 2022-02-03  144  	struct lsdc_crtc_state *new_priv_state;
983916ec4a8df3 suijingfeng 2022-02-03  145  	struct lsdc_crtc_state *old_priv_state;
983916ec4a8df3 suijingfeng 2022-02-03 @146  	struct drm_device *ddev = crtc->dev;
983916ec4a8df3 suijingfeng 2022-02-03  147  
983916ec4a8df3 suijingfeng 2022-02-03  148  	if (drm_WARN_ON(ddev, !crtc->state))
983916ec4a8df3 suijingfeng 2022-02-03  149  		return NULL;
983916ec4a8df3 suijingfeng 2022-02-03  150  
983916ec4a8df3 suijingfeng 2022-02-03  151  	new_priv_state = kmalloc(sizeof(*new_priv_state), GFP_KERNEL);
983916ec4a8df3 suijingfeng 2022-02-03  152  	if (!new_priv_state)
983916ec4a8df3 suijingfeng 2022-02-03  153  		return NULL;
983916ec4a8df3 suijingfeng 2022-02-03  154  
983916ec4a8df3 suijingfeng 2022-02-03  155  	__drm_atomic_helper_crtc_duplicate_state(crtc, &new_priv_state->base);
983916ec4a8df3 suijingfeng 2022-02-03  156  
983916ec4a8df3 suijingfeng 2022-02-03  157  	old_priv_state = to_lsdc_crtc_state(crtc->state);
983916ec4a8df3 suijingfeng 2022-02-03  158  
983916ec4a8df3 suijingfeng 2022-02-03  159  	memcpy(&new_priv_state->pparams, &old_priv_state->pparams,
983916ec4a8df3 suijingfeng 2022-02-03  160  		sizeof(new_priv_state->pparams));
983916ec4a8df3 suijingfeng 2022-02-03  161  
983916ec4a8df3 suijingfeng 2022-02-03  162  	new_priv_state->pix_fmt = old_priv_state->pix_fmt;
983916ec4a8df3 suijingfeng 2022-02-03  163  
983916ec4a8df3 suijingfeng 2022-02-03  164  	return &new_priv_state->base;
983916ec4a8df3 suijingfeng 2022-02-03  165  }
983916ec4a8df3 suijingfeng 2022-02-03  166  

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

^ permalink raw reply	[flat|nested] 83+ messages in thread
* Re: [PATCH v6 1/3] drm/lsdc: add drm driver for loongson display controller
@ 2022-02-12  5:04 kernel test robot
  0 siblings, 0 replies; 83+ messages in thread
From: kernel test robot @ 2022-02-12  5:04 UTC (permalink / raw)
  To: kbuild

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

CC: llvm(a)lists.linux.dev
CC: kbuild-all(a)lists.01.org
In-Reply-To: <20220203082546.3099-2-15330273260@189.cn>
References: <20220203082546.3099-2-15330273260@189.cn>
TO: Sui Jingfeng <15330273260@189.cn>

Hi Sui,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on drm/drm-next]
[also build test WARNING on robh/for-next drm-intel/for-linux-next drm-tip/drm-tip v5.17-rc3 next-20220211]
[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/Sui-Jingfeng/drm-lsdc-add-drm-driver-for-loongson-display-controller/20220203-162953
base:   git://anongit.freedesktop.org/drm/drm drm-next
:::::: branch date: 9 days ago
:::::: commit date: 9 days ago
config: riscv-randconfig-c006-20220205 (https://download.01.org/0day-ci/archive/20220212/202202121140.KF8JhdUn-lkp(a)intel.com/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project dee058c670593b999fec19c458dbbd882ad9de56)
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/983916ec4a8df390a947a7f539dbc99f5d511e26
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Sui-Jingfeng/drm-lsdc-add-drm-driver-for-loongson-display-controller/20220203-162953
        git checkout 983916ec4a8df390a947a7f539dbc99f5d511e26
        # save the config file to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=riscv clang-analyzer 

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


clang-analyzer warnings: (new ones prefixed by >>)
   2 warnings generated.
   Suppressed 2 warnings (2 in non-user code).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   10 warnings generated.
   fs/eventfd.c:285:8: warning: Value stored to 'res' is never read [clang-analyzer-deadcode.DeadStores]
                   for (res = 0;;) {
                        ^     ~
   fs/eventfd.c:285:8: note: Value stored to 'res' is never read
                   for (res = 0;;) {
                        ^     ~
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   2 warnings generated.
   Suppressed 2 warnings (2 in non-user code).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   2 warnings generated.
   Suppressed 2 warnings (2 in non-user code).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   10 warnings generated.
   Suppressed 10 warnings (3 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   11 warnings generated.
   Suppressed 11 warnings (4 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   13 warnings generated.
   drivers/gpu/drm/lsdc/lsdc_crtc.c:123:33: warning: The left operand of '&' is a garbage value [clang-analyzer-core.UndefinedBinaryOperatorResult]
           priv_crtc_state->pix_fmt = val & CFG_PIX_FMT_MASK;
                                      ~~~ ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:97:2: note: 'val' declared without an initial value
           u32 val;
           ^~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:104:6: note: Assuming 'index' is not equal to 0
           if (index == 0) {
               ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:104:2: note: Taking false branch
           if (index == 0) {
           ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:107:13: note: Assuming 'index' is not equal to 1
           } else if (index == 1) {
                      ^~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:107:9: note: Taking false branch
           } else if (index == 1) {
                  ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:113:6: note: Assuming field 'state' is null
           if (crtc->state) {
               ^~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:113:2: note: Taking false branch
           if (crtc->state) {
           ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:119:20: note: Calling 'kzalloc'
           priv_crtc_state = kzalloc(sizeof(*priv_crtc_state), GFP_KERNEL);
                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:715:9: note: Calling 'kmalloc'
           return kmalloc(size, flags | __GFP_ZERO);
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:569:2: note: Taking false branch
           if (__builtin_constant_p(size)) {
           ^
   include/linux/slab.h:586:2: note: Returning pointer, which participates in a condition later
           return __kmalloc(size, flags);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:715:9: note: Returning from 'kmalloc'
           return kmalloc(size, flags | __GFP_ZERO);
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:715:2: note: Returning pointer, which participates in a condition later
           return kmalloc(size, flags | __GFP_ZERO);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:119:20: note: Returning from 'kzalloc'
           priv_crtc_state = kzalloc(sizeof(*priv_crtc_state), GFP_KERNEL);
                             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:120:6: note: Assuming 'priv_crtc_state' is non-null
           if (!priv_crtc_state)
               ^~~~~~~~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:120:2: note: Taking false branch
           if (!priv_crtc_state)
           ^
   drivers/gpu/drm/lsdc/lsdc_crtc.c:123:33: note: The left operand of '&' is a garbage value
           priv_crtc_state->pix_fmt = val & CFG_PIX_FMT_MASK;
                                      ~~~ ^
>> drivers/gpu/drm/lsdc/lsdc_crtc.c:146:21: warning: Value stored to 'ddev' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
           struct drm_device *ddev = crtc->dev;
                              ^~~~   ~~~~~~~~~
   drivers/gpu/drm/lsdc/lsdc_crtc.c:146:21: note: Value stored to 'ddev' during its initialization is never read
           struct drm_device *ddev = crtc->dev;
                              ^~~~   ~~~~~~~~~
   Suppressed 11 warnings (4 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   2 warnings generated.
   Suppressed 2 warnings (2 in non-user code).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   9 warnings generated.
   Suppressed 9 warnings (2 in non-user code, 7 with check filters).
   Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
   5 warnings generated.
   drivers/base/cacheinfo.c:38:31: warning: Access to field 'fw_token' results in a dereference of a null pointer (loaded from variable 'this_leaf') [clang-analyzer-core.NullDereference]
           return sib_leaf->fw_token == this_leaf->fw_token;
                                        ^
   drivers/base/cacheinfo.c:225:6: note: Assuming field 'cpu_map_populated' is false
           if (this_cpu_ci->cpu_map_populated)
               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/base/cacheinfo.c:225:2: note: Taking false branch
           if (this_cpu_ci->cpu_map_populated)
           ^
   drivers/base/cacheinfo.c:228:2: note: Taking false branch
           if (of_have_populated_dt())
           ^
   drivers/base/cacheinfo.c:230:7: note: Taking false branch
           else if (!acpi_disabled)
                ^
   drivers/base/cacheinfo.c:233:6: note: 'ret' is 0
           if (ret)
               ^~~
   drivers/base/cacheinfo.c:233:2: note: Taking false branch
           if (ret)
           ^
   drivers/base/cacheinfo.c:236:26: note: Loop condition is false.  Exiting loop
           for (index = 0; index < cache_leaves(cpu); index++) {
                                   ^
   drivers/base/cacheinfo.c:26:28: note: expanded from macro 'cache_leaves'
   #define cache_leaves(cpu)       (ci_cacheinfo(cpu)->num_leaves)
                                    ^
   drivers/base/cacheinfo.c:25:29: note: expanded from macro 'ci_cacheinfo'
   #define ci_cacheinfo(cpu)       (&per_cpu(ci_cpu_cacheinfo, cpu))
                                     ^
   include/linux/percpu-defs.h:269:29: note: expanded from macro 'per_cpu'
   #define per_cpu(var, cpu)       (*per_cpu_ptr(&(var), cpu))
                                     ^
   include/linux/percpu-defs.h:263:47: note: expanded from macro 'per_cpu_ptr'
   #define per_cpu_ptr(ptr, cpu)   ({ (void)(cpu); VERIFY_PERCPU_PTR(ptr); })
                                                   ^
   include/linux/percpu-defs.h:259:2: note: expanded from macro 'VERIFY_PERCPU_PTR'
           __verify_pcpu_ptr(__p);                                         \
           ^
   include/linux/percpu-defs.h:217:37: note: expanded from macro '__verify_pcpu_ptr'
   #define __verify_pcpu_ptr(ptr)                                          \
                                                                           ^
   drivers/base/cacheinfo.c:236:18: note: Assuming 'index' is < field 'num_leaves'
           for (index = 0; index < cache_leaves(cpu); index++) {
                           ^~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/base/cacheinfo.c:236:2: note: Loop condition is true.  Entering loop body
           for (index = 0; index < cache_leaves(cpu); index++) {
           ^
   drivers/base/cacheinfo.c:241:8: note: Calling 'cpumask_empty'
                   if (!cpumask_empty(&this_leaf->shared_cpu_map))
                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/cpumask.h:557:9: note: Calling 'bitmap_empty'
           return bitmap_empty(cpumask_bits(srcp), nr_cpumask_bits);
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/bitmap.h:385:6: note: Left side of '&&' is false
           if (small_const_nbits(nbits))
               ^
   include/asm-generic/bitsperlong.h:36:31: note: expanded from macro 'small_const_nbits'
           (__builtin_constant_p(nbits) && (nbits) <= BITS_PER_LONG && (nbits) > 0)
                                        ^
   include/linux/bitmap.h:388:9: note: Calling 'find_first_bit'
           return find_first_bit(src, nbits) == nbits;
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/find.h:118:6: note: Left side of '&&' is false
           if (small_const_nbits(size)) {
               ^
   include/asm-generic/bitsperlong.h:36:31: note: expanded from macro 'small_const_nbits'
           (__builtin_constant_p(nbits) && (nbits) <= BITS_PER_LONG && (nbits) > 0)
                                        ^
   include/linux/find.h:124:9: note: Value assigned to 'ci_cpu_cacheinfo.info_list'
           return _find_first_bit(addr, size);
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/bitmap.h:388:9: note: Returning from 'find_first_bit'
           return find_first_bit(src, nbits) == nbits;
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/bitmap.h:388:9: note: Assuming the condition is true
           return find_first_bit(src, nbits) == nbits;
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/cpumask.h:557:9: note: Returning from 'bitmap_empty'
           return bitmap_empty(cpumask_bits(srcp), nr_cpumask_bits);

vim +/ddev +146 drivers/gpu/drm/lsdc/lsdc_crtc.c

983916ec4a8df3 suijingfeng 2022-02-03  140  
983916ec4a8df3 suijingfeng 2022-02-03  141  
983916ec4a8df3 suijingfeng 2022-02-03  142  static struct drm_crtc_state *lsdc_crtc_atomic_duplicate_state(struct drm_crtc *crtc)
983916ec4a8df3 suijingfeng 2022-02-03  143  {
983916ec4a8df3 suijingfeng 2022-02-03  144  	struct lsdc_crtc_state *new_priv_state;
983916ec4a8df3 suijingfeng 2022-02-03  145  	struct lsdc_crtc_state *old_priv_state;
983916ec4a8df3 suijingfeng 2022-02-03 @146  	struct drm_device *ddev = crtc->dev;
983916ec4a8df3 suijingfeng 2022-02-03  147  
983916ec4a8df3 suijingfeng 2022-02-03  148  	if (drm_WARN_ON(ddev, !crtc->state))
983916ec4a8df3 suijingfeng 2022-02-03  149  		return NULL;
983916ec4a8df3 suijingfeng 2022-02-03  150  
983916ec4a8df3 suijingfeng 2022-02-03  151  	new_priv_state = kmalloc(sizeof(*new_priv_state), GFP_KERNEL);
983916ec4a8df3 suijingfeng 2022-02-03  152  	if (!new_priv_state)
983916ec4a8df3 suijingfeng 2022-02-03  153  		return NULL;
983916ec4a8df3 suijingfeng 2022-02-03  154  
983916ec4a8df3 suijingfeng 2022-02-03  155  	__drm_atomic_helper_crtc_duplicate_state(crtc, &new_priv_state->base);
983916ec4a8df3 suijingfeng 2022-02-03  156  
983916ec4a8df3 suijingfeng 2022-02-03  157  	old_priv_state = to_lsdc_crtc_state(crtc->state);
983916ec4a8df3 suijingfeng 2022-02-03  158  
983916ec4a8df3 suijingfeng 2022-02-03  159  	memcpy(&new_priv_state->pparams, &old_priv_state->pparams,
983916ec4a8df3 suijingfeng 2022-02-03  160  		sizeof(new_priv_state->pparams));
983916ec4a8df3 suijingfeng 2022-02-03  161  
983916ec4a8df3 suijingfeng 2022-02-03  162  	new_priv_state->pix_fmt = old_priv_state->pix_fmt;
983916ec4a8df3 suijingfeng 2022-02-03  163  
983916ec4a8df3 suijingfeng 2022-02-03  164  	return &new_priv_state->base;
983916ec4a8df3 suijingfeng 2022-02-03  165  }
983916ec4a8df3 suijingfeng 2022-02-03  166  

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

^ permalink raw reply	[flat|nested] 83+ messages in thread

end of thread, other threads:[~2023-02-03  3:25 UTC | newest]

Thread overview: 83+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-03  8:25 [PATCH v6 0/3] drm/lsdc: add drm driver for loongson display controller Sui Jingfeng
2022-02-03  8:25 ` Sui Jingfeng
2022-02-03  8:25 ` [PATCH v6 1/3] " Sui Jingfeng
2022-02-03  8:25   ` Sui Jingfeng
2022-02-03  8:53   ` Krzysztof Kozlowski
2022-02-03  8:53     ` Krzysztof Kozlowski
2022-02-03 11:07     ` Dan Carpenter
2022-02-03 11:07       ` Dan Carpenter
2022-02-03 11:29       ` Krzysztof Kozlowski
2022-02-03 11:29         ` Krzysztof Kozlowski
2022-02-03 13:34         ` Dan Carpenter
2022-02-03 13:34           ` Dan Carpenter
2022-02-03  8:58   ` Maxime Ripard
2022-02-03  8:58     ` Maxime Ripard
2022-02-03 14:14     ` Sui Jingfeng
2022-02-03 15:27     ` Sui Jingfeng
2022-02-03 15:47     ` Sui Jingfeng
2022-02-03 15:47       ` Sui Jingfeng
2022-02-09  8:52       ` Maxime Ripard
2022-02-09  8:52         ` Maxime Ripard
2022-02-09 11:56         ` Jiaxun Yang
2022-02-09 11:56           ` Jiaxun Yang
2022-02-09 14:04           ` Maxime Ripard
2022-02-09 14:04             ` Maxime Ripard
2022-02-09 14:14             ` Jiaxun Yang
2022-02-09 14:14               ` Jiaxun Yang
2023-02-03  3:25             ` suijingfeng
2022-02-03 16:04     ` Sui Jingfeng
2022-02-03 16:04       ` Sui Jingfeng
2022-02-09  8:49       ` Maxime Ripard
2022-02-09  8:49         ` Maxime Ripard
2022-02-09 14:38         ` Sui Jingfeng
2022-02-09 14:38           ` Sui Jingfeng
2022-02-09 16:16           ` Maxime Ripard
2022-02-09 16:16             ` Maxime Ripard
2022-02-12 18:11             ` Sui Jingfeng
2022-02-12 18:11               ` Sui Jingfeng
2022-02-16 14:08               ` Maxime Ripard
2022-02-16 14:08                 ` Maxime Ripard
2022-02-12 20:52             ` Ilia Mirkin
2022-02-12 20:52               ` Ilia Mirkin
2022-02-16 13:34             ` Sui Jingfeng
2022-02-16 13:34               ` Sui Jingfeng
2022-02-16 14:11               ` Maxime Ripard
2022-02-16 14:11                 ` Maxime Ripard
2022-02-03 16:29     ` Sui Jingfeng
2022-02-03 16:29       ` Sui Jingfeng
2022-02-09  8:43       ` Maxime Ripard
2022-02-09  8:43         ` Maxime Ripard
2022-02-09 15:41         ` Sui Jingfeng
2022-02-09 15:41           ` Sui Jingfeng
2022-02-09 16:19           ` Maxime Ripard
2022-02-09 16:19             ` Maxime Ripard
2022-02-16 13:46           ` Daniel Stone
2022-02-16 13:46             ` Daniel Stone
2022-02-16 14:13             ` Sui Jingfeng
2022-02-16 14:13               ` Sui Jingfeng
2022-02-16 14:43               ` Daniel Stone
2022-02-16 14:43                 ` Daniel Stone
2022-02-03 16:41     ` Sui Jingfeng
2022-02-03 16:41       ` Sui Jingfeng
2022-02-09  8:36       ` Maxime Ripard
2022-02-09  8:36         ` Maxime Ripard
2023-02-02  2:58         ` suijingfeng
2022-02-03  8:25 ` [PATCH v6 2/3] dt-bindings: ls2k1000: add the display controller device node Sui Jingfeng
2022-02-03  8:25   ` Sui Jingfeng
2022-02-03  8:50   ` Krzysztof Kozlowski
2022-02-03  8:50     ` Krzysztof Kozlowski
2022-02-16 14:01     ` Sui Jingfeng
2022-02-16 14:01       ` Sui Jingfeng
2022-02-17  8:38       ` Krzysztof Kozlowski
2022-02-17  8:38         ` Krzysztof Kozlowski
2022-02-03  8:25 ` [PATCH v6 3/3] dt-bindings: mips: loongson: introduce board specific dts Sui Jingfeng
2022-02-03  8:25   ` Sui Jingfeng
2022-02-03  8:49   ` Krzysztof Kozlowski
2022-02-03  8:49     ` Krzysztof Kozlowski
2022-02-09 12:00 ` [PATCH v6 0/3] drm/lsdc: add drm driver for loongson display controller Jiaxun Yang
2022-02-09 12:00   ` Jiaxun Yang
2022-02-10  4:04   ` Sui Jingfeng
2022-02-10  4:04     ` Sui Jingfeng
2022-02-06 19:38 [PATCH v6 1/3] " kernel test robot
2022-02-10 23:10 kernel test robot
2022-02-12  5:04 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.