From: kernel test robot <lkp@intel.com>
To: Ben Skeggs <bskeggs@redhat.com>
Cc: oe-kbuild-all@lists.linux.dev, linux-kernel@vger.kernel.org,
Lyude Paul <lyude@redhat.com>
Subject: drivers/gpu/drm/nouveau/nvkm/subdev/top/ga100.c:82 ga100_top_parse() warn: inconsistent indenting
Date: Sun, 3 Dec 2023 14:31:38 +0800 [thread overview]
Message-ID: <202312030044.q1SxyCnu-lkp@intel.com> (raw)
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 815fb87b753055df2d9e50f6cd80eb10235fe3e9
commit: eec3f6dfedc0b8c5aef7619667dee61a77a37e35 drm/nouveau/top: parse device topology right after devinit
date: 1 year, 1 month ago
config: i386-randconfig-141-20231105 (https://download.01.org/0day-ci/archive/20231203/202312030044.q1SxyCnu-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce: (https://download.01.org/0day-ci/archive/20231203/202312030044.q1SxyCnu-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202312030044.q1SxyCnu-lkp@intel.com/
smatch warnings:
drivers/gpu/drm/nouveau/nvkm/subdev/top/ga100.c:82 ga100_top_parse() warn: inconsistent indenting
vim +82 drivers/gpu/drm/nouveau/nvkm/subdev/top/ga100.c
f6df392dddbb9e Ben Skeggs 2021-02-08 23
f6df392dddbb9e Ben Skeggs 2021-02-08 24 static int
eec3f6dfedc0b8 Ben Skeggs 2022-06-01 25 ga100_top_parse(struct nvkm_top *top)
f6df392dddbb9e Ben Skeggs 2021-02-08 26 {
f6df392dddbb9e Ben Skeggs 2021-02-08 27 struct nvkm_subdev *subdev = &top->subdev;
f6df392dddbb9e Ben Skeggs 2021-02-08 28 struct nvkm_device *device = subdev->device;
f6df392dddbb9e Ben Skeggs 2021-02-08 29 struct nvkm_top_device *info = NULL;
f6df392dddbb9e Ben Skeggs 2021-02-08 30 u32 data, type, inst;
f6df392dddbb9e Ben Skeggs 2021-02-08 31 int i, n, size = nvkm_rd32(device, 0x0224fc) >> 20;
f6df392dddbb9e Ben Skeggs 2021-02-08 32
f6df392dddbb9e Ben Skeggs 2021-02-08 33 for (i = 0, n = 0; i < size; i++) {
f6df392dddbb9e Ben Skeggs 2021-02-08 34 if (!info) {
f6df392dddbb9e Ben Skeggs 2021-02-08 35 if (!(info = nvkm_top_device_new(top)))
f6df392dddbb9e Ben Skeggs 2021-02-08 36 return -ENOMEM;
f6df392dddbb9e Ben Skeggs 2021-02-08 37 type = ~0;
f6df392dddbb9e Ben Skeggs 2021-02-08 38 inst = 0;
f6df392dddbb9e Ben Skeggs 2021-02-08 39 }
f6df392dddbb9e Ben Skeggs 2021-02-08 40
f6df392dddbb9e Ben Skeggs 2021-02-08 41 data = nvkm_rd32(device, 0x022800 + (i * 0x04));
f6df392dddbb9e Ben Skeggs 2021-02-08 42 nvkm_trace(subdev, "%02x: %08x\n", i, data);
f6df392dddbb9e Ben Skeggs 2021-02-08 43 if (!data && n == 0)
f6df392dddbb9e Ben Skeggs 2021-02-08 44 continue;
f6df392dddbb9e Ben Skeggs 2021-02-08 45
f6df392dddbb9e Ben Skeggs 2021-02-08 46 switch (n++) {
f6df392dddbb9e Ben Skeggs 2021-02-08 47 case 0:
f6df392dddbb9e Ben Skeggs 2021-02-08 48 type = (data & 0x3f000000) >> 24;
f6df392dddbb9e Ben Skeggs 2021-02-08 49 inst = (data & 0x000f0000) >> 16;
f6df392dddbb9e Ben Skeggs 2021-02-08 50 info->fault = (data & 0x0000007f);
f6df392dddbb9e Ben Skeggs 2021-02-08 51 break;
f6df392dddbb9e Ben Skeggs 2021-02-08 52 case 1:
f6df392dddbb9e Ben Skeggs 2021-02-08 53 info->addr = (data & 0x00fff000);
f6df392dddbb9e Ben Skeggs 2021-02-08 54 info->reset = (data & 0x0000001f);
f6df392dddbb9e Ben Skeggs 2021-02-08 55 break;
f6df392dddbb9e Ben Skeggs 2021-02-08 56 case 2:
49b2dfc0818268 Ben Skeggs 2021-09-17 57 info->runlist = (data & 0x00fffc00);
f6df392dddbb9e Ben Skeggs 2021-02-08 58 info->engine = (data & 0x00000003);
f6df392dddbb9e Ben Skeggs 2021-02-08 59 break;
f6df392dddbb9e Ben Skeggs 2021-02-08 60 default:
f6df392dddbb9e Ben Skeggs 2021-02-08 61 break;
f6df392dddbb9e Ben Skeggs 2021-02-08 62 }
f6df392dddbb9e Ben Skeggs 2021-02-08 63
f6df392dddbb9e Ben Skeggs 2021-02-08 64 if (data & 0x80000000)
f6df392dddbb9e Ben Skeggs 2021-02-08 65 continue;
f6df392dddbb9e Ben Skeggs 2021-02-08 66 n = 0;
f6df392dddbb9e Ben Skeggs 2021-02-08 67
f6df392dddbb9e Ben Skeggs 2021-02-08 68 /* Translate engine type to NVKM engine identifier. */
f6df392dddbb9e Ben Skeggs 2021-02-08 69 #define I_(T,I) do { info->type = (T); info->inst = (I); } while(0)
f6df392dddbb9e Ben Skeggs 2021-02-08 70 #define O_(T,I) do { WARN_ON(inst); I_(T, I); } while (0)
f6df392dddbb9e Ben Skeggs 2021-02-08 71 switch (type) {
f6df392dddbb9e Ben Skeggs 2021-02-08 72 case 0x00000000: O_(NVKM_ENGINE_GR , 0); break;
f6df392dddbb9e Ben Skeggs 2021-02-08 73 case 0x0000000d: O_(NVKM_ENGINE_SEC2 , 0); break;
f6df392dddbb9e Ben Skeggs 2021-02-08 74 case 0x0000000e: I_(NVKM_ENGINE_NVENC , inst); break;
f6df392dddbb9e Ben Skeggs 2021-02-08 75 case 0x00000010: I_(NVKM_ENGINE_NVDEC , inst); break;
f6df392dddbb9e Ben Skeggs 2021-02-08 76 case 0x00000012: I_(NVKM_SUBDEV_IOCTRL, inst); break;
f6df392dddbb9e Ben Skeggs 2021-02-08 77 case 0x00000013: I_(NVKM_ENGINE_CE , inst); break;
f6df392dddbb9e Ben Skeggs 2021-02-08 78 case 0x00000014: O_(NVKM_SUBDEV_GSP , 0); break;
f6df392dddbb9e Ben Skeggs 2021-02-08 79 case 0x00000015: O_(NVKM_ENGINE_NVJPG , 0); break;
f6df392dddbb9e Ben Skeggs 2021-02-08 80 case 0x00000016: O_(NVKM_ENGINE_OFA , 0); break;
f6df392dddbb9e Ben Skeggs 2021-02-08 81 case 0x00000017: O_(NVKM_SUBDEV_FLA , 0); break;
f6df392dddbb9e Ben Skeggs 2021-02-08 @82 break;
f6df392dddbb9e Ben Skeggs 2021-02-08 83 default:
f6df392dddbb9e Ben Skeggs 2021-02-08 84 break;
f6df392dddbb9e Ben Skeggs 2021-02-08 85 }
f6df392dddbb9e Ben Skeggs 2021-02-08 86
f6df392dddbb9e Ben Skeggs 2021-02-08 87 nvkm_debug(subdev, "%02x.%d (%8s): addr %06x fault %2d "
49b2dfc0818268 Ben Skeggs 2021-09-17 88 "runlist %6x engine %2d reset %2d\n", type, inst,
f6df392dddbb9e Ben Skeggs 2021-02-08 89 info->type == NVKM_SUBDEV_NR ? "????????" : nvkm_subdev_type[info->type],
49b2dfc0818268 Ben Skeggs 2021-09-17 90 info->addr, info->fault, info->runlist < 0 ? 0 : info->runlist,
49b2dfc0818268 Ben Skeggs 2021-09-17 91 info->engine, info->reset);
f6df392dddbb9e Ben Skeggs 2021-02-08 92 info = NULL;
f6df392dddbb9e Ben Skeggs 2021-02-08 93 }
f6df392dddbb9e Ben Skeggs 2021-02-08 94
f6df392dddbb9e Ben Skeggs 2021-02-08 95 return 0;
f6df392dddbb9e Ben Skeggs 2021-02-08 96 }
f6df392dddbb9e Ben Skeggs 2021-02-08 97
:::::: The code at line 82 was first introduced by commit
:::::: f6df392dddbb9e637b785e7e3d9337a74923dc10 drm/nouveau/top/ga100: initial support
:::::: TO: Ben Skeggs <bskeggs@redhat.com>
:::::: CC: Ben Skeggs <bskeggs@redhat.com>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next reply other threads:[~2023-12-03 6:32 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-12-03 6:31 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2022-12-30 1:42 drivers/gpu/drm/nouveau/nvkm/subdev/top/ga100.c:82 ga100_top_parse() warn: inconsistent indenting kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202312030044.q1SxyCnu-lkp@intel.com \
--to=lkp@intel.com \
--cc=bskeggs@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=lyude@redhat.com \
--cc=oe-kbuild-all@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox