Re: [PATCH v2] mfd: syscon: Add Spreadtrum physical regmap bus support

From: kbuild test robot
Date: Tue Apr 21 2020 - 06:03:56 EST


Hi Baolin,

I love your patch! Yet something to improve:

[auto build test ERROR on ljones-mfd/for-mfd-next]
[also build test ERROR on arm-soc/for-next linus/master linux/master v5.7-rc2 next-20200420]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url: https://github.com/0day-ci/linux/commits/Baolin-Wang/mfd-syscon-Add-Spreadtrum-physical-regmap-bus-support/20200421-035442
base: https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next
config: x86_64-randconfig-f002-20200421 (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project a9b137f9ffba8cb25dfd7dd1fb613e8aac121b37)
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install x86_64 cross compiling tool for clang build
# apt-get install binutils-x86-64-linux-gnu
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@xxxxxxxxx>

All errors (new ones prefixed by >>):

>> drivers/mfd/syscon.c:182:31: error: use of undeclared identifier 'sprd_syscon_regmap_bus'
regmap = regmap_init(NULL, &sprd_syscon_regmap_bus, base,
^
>> drivers/mfd/syscon.c:182:10: error: assigning to 'struct regmap *' from incompatible type 'void'
regmap = regmap_init(NULL, &sprd_syscon_regmap_bus, base,
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 errors generated.

vim +/sprd_syscon_regmap_bus +182 drivers/mfd/syscon.c

106
107 static struct syscon *of_syscon_register(struct device_node *np, bool check_clk)
108 {
109 struct clk *clk;
110 struct syscon *syscon;
111 struct regmap *regmap;
112 void __iomem *base;
113 u32 reg_io_width;
114 int ret;
115 struct regmap_config syscon_config = syscon_regmap_config;
116 struct resource res;
117 bool use_phy_regmap_bus = false;
118
119 syscon = kzalloc(sizeof(*syscon), GFP_KERNEL);
120 if (!syscon)
121 return ERR_PTR(-ENOMEM);
122
123 if (of_address_to_resource(np, 0, &res)) {
124 ret = -ENOMEM;
125 goto err_map;
126 }
127
128 base = ioremap(res.start, resource_size(&res));
129 if (!base) {
130 ret = -ENOMEM;
131 goto err_map;
132 }
133
134 /* Parse the device's DT node for an endianness specification */
135 if (of_property_read_bool(np, "big-endian"))
136 syscon_config.val_format_endian = REGMAP_ENDIAN_BIG;
137 else if (of_property_read_bool(np, "little-endian"))
138 syscon_config.val_format_endian = REGMAP_ENDIAN_LITTLE;
139 else if (of_property_read_bool(np, "native-endian"))
140 syscon_config.val_format_endian = REGMAP_ENDIAN_NATIVE;
141
142 /*
143 * search for reg-io-width property in DT. If it is not provided,
144 * default to 4 bytes. regmap_init_mmio will return an error if values
145 * are invalid so there is no need to check them here.
146 */
147 ret = of_property_read_u32(np, "reg-io-width", &reg_io_width);
148 if (ret)
149 reg_io_width = 4;
150
151 ret = of_hwspin_lock_get_id(np, 0);
152 if (ret > 0 || (IS_ENABLED(CONFIG_HWSPINLOCK) && ret == 0)) {
153 syscon_config.use_hwlock = true;
154 syscon_config.hwlock_id = ret;
155 syscon_config.hwlock_mode = HWLOCK_IRQSTATE;
156 } else if (ret < 0) {
157 switch (ret) {
158 case -ENOENT:
159 /* Ignore missing hwlock, it's optional. */
160 break;
161 default:
162 pr_err("Failed to retrieve valid hwlock: %d\n", ret);
163 /* fall-through */
164 case -EPROBE_DEFER:
165 goto err_regmap;
166 }
167 }
168
169 syscon_config.name = of_node_full_name(np);
170 syscon_config.reg_stride = reg_io_width;
171 syscon_config.val_bits = reg_io_width * 8;
172 syscon_config.max_register = resource_size(&res) - reg_io_width;
173
174 /*
175 * The Spreadtrum syscon need register a real physical regmap bus
176 * with new atomic bits updating operation instead of using
177 * read-modify-write.
178 */
179 if (IS_ENABLED(CONFIG_ARCH_SPRD) &&
180 of_device_is_compatible(np, "sprd,atomic-syscon")) {
181 use_phy_regmap_bus = true;
> 182 regmap = regmap_init(NULL, &sprd_syscon_regmap_bus, base,
183 &syscon_config);
184 } else {
185 regmap = regmap_init_mmio(NULL, base, &syscon_config);
186 }
187 if (IS_ERR(regmap)) {
188 pr_err("regmap init failed\n");
189 ret = PTR_ERR(regmap);
190 goto err_regmap;
191 }
192
193 if (!use_phy_regmap_bus && check_clk) {
194 clk = of_clk_get(np, 0);
195 if (IS_ERR(clk)) {
196 ret = PTR_ERR(clk);
197 /* clock is optional */
198 if (ret != -ENOENT)
199 goto err_clk;
200 } else {
201 ret = regmap_mmio_attach_clk(regmap, clk);
202 if (ret)
203 goto err_attach;
204 }
205 }
206
207 syscon->regmap = regmap;
208 syscon->np = np;
209
210 spin_lock(&syscon_list_slock);
211 list_add_tail(&syscon->list, &syscon_list);
212 spin_unlock(&syscon_list_slock);
213
214 return syscon;
215
216 err_attach:
217 if (!IS_ERR(clk))
218 clk_put(clk);
219 err_clk:
220 regmap_exit(regmap);
221 err_regmap:
222 iounmap(base);
223 err_map:
224 kfree(syscon);
225 return ERR_PTR(ret);
226 }
227

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

Attachment: .config.gz
Description: application/gzip