Add Sunplus SP7021 SoC Support

Hi Qin,

I love your patch! Yet something to improve:

[auto build test ERROR on robh/for-next]
[also build test ERROR on pza/reset/next clk/clk-next tip/irq/core linus/master v5.15-rc7 next-20211029] [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/Qin-Jian/Add-Sunplus-SP7021-SoC-Support/20211029-171054
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arc-allyesconfig (attached as .config)
compiler: arceb-elf-gcc (GCC) 11.2.0

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
# https://github.com/0day-ci/linux/commit/45fff7c04fb4d0ccfbd1ed11f73277d1f8742499
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Qin-Jian/Add-Sunplus-SP7021-SoC-Support/20211029-171054 git checkout 45fff7c04fb4d0ccfbd1ed11f73277d1f8742499
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arc SHELL=/bin/bash

If you fix the issue, kindly add following tag as appropriate

All errors (new ones prefixed by >>):

drivers/reset/reset-sunplus.c: In function 'sp_restart':

>> drivers/reset/reset-sunplus.c:98:42: error: 'RST_SYSTEM' undeclared (first use in this function)
      98 |         sp_reset_assert(&sp_reset.rcdev, RST_SYSTEM);
         |                                          ^~~~~~~~~~
   drivers/reset/reset-sunplus.c:98:42: note: each undeclared identifier is reported only once for each function it appears in
   drivers/reset/reset-sunplus.c: In function 'sp_reset_probe':
>> drivers/reset/reset-sunplus.c:135:33: error: 'RST_MAX' undeclared (first use in this function); did you mean 'S8_MAX'?
     135 |         data->rcdev.nr_resets = RST_MAX;
         |                                 ^~~~~~~
         |                                 S8_MAX

vim +/RST_SYSTEM +98 drivers/reset/reset-sunplus.c

    94	
    95	static int sp_restart(struct notifier_block *this, unsigned long mode,
    96					void *cmd)
    97	{
  > 98		sp_reset_assert(&sp_reset.rcdev, RST_SYSTEM);
    99		sp_reset_deassert(&sp_reset.rcdev, RST_SYSTEM);
   100	
   101		return NOTIFY_DONE;
   102	}
   103	
   104	static struct notifier_block sp_restart_nb = {
   105		.notifier_call = sp_restart,
   106		.priority = 192,
   107	};
   108	
   109	static const struct reset_control_ops sp_reset_ops = {
   110		.assert		= sp_reset_assert,
   111		.deassert	= sp_reset_deassert,
   112		.status		= sp_reset_status,
   113	};
   114	
   115	static const struct of_device_id sp_reset_dt_ids[] = {
   116		{ .compatible = "sunplus,sp7021-reset", },
   117		{ .compatible = "sunplus,q645-reset", },
   118		{ /* sentinel */ },
   119	};
   120	
   121	static int sp_reset_probe(struct platform_device *pdev)
   122	{
   123		struct device *dev = &pdev->dev;
   124		struct sp_reset_data *data = &sp_reset;
   125		void __iomem *membase;
   126		struct resource *res;
   127	
   128		res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
   129		membase = devm_ioremap(dev, res->start, resource_size(res));
   130		if (IS_ERR(membase))
   131			return PTR_ERR(membase);
   132	
   133		data->membase = membase;
   134		data->rcdev.owner = THIS_MODULE;
 > 135		data->rcdev.nr_resets = RST_MAX;

136 data->rcdev.ops = &sp_reset_ops;
137 data->rcdev.of_node = dev->of_node;
138 register_restart_handler(&sp_restart_nb);
139
140 return devm_reset_controller_register(dev, &data->rcdev);
141 }
142

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

Hi Qin,

I love your patch! Perhaps something to improve:

[auto build test WARNING on pza/reset/next]
[also build test WARNING on robh/for-next clk/clk-next tip/irq/core linus/master v5.15 next-20211029] [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/Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211101-140155
base:   https://git.pengutronix.de/git/pza/linux reset/next
config: openrisc-randconfig-r015-20211101 (attached as .config)
compiler: or1k-linux-gcc (GCC) 11.2.0

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
# https://github.com/0day-ci/linux/commit/9767714cf5c22192f99adb0d8e344cb5e38a2e33
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211101-140155 git checkout 9767714cf5c22192f99adb0d8e344cb5e38a2e33
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=openrisc

If you fix the issue, kindly add following tag as appropriate

All warnings (new ones prefixed by >>):

>> drivers/clk/clk-sp7021.c:663:13: warning: no previous prototype for 'clk_register_sp_pll' [-Wmissing-prototypes]

663 | struct clk *clk_register_sp_pll(const char *name, const char *parent,
| ^~~~~~~~~~~~~~~~~~~

vim +/clk_register_sp_pll +663 drivers/clk/clk-sp7021.c

662

 > 663	struct clk *clk_register_sp_pll(const char *name, const char *parent,

664 void __iomem *reg, int pd_bit, int bp_bit,
665 unsigned long brate, int shift, int width,
666 spinlock_t *lock)
667 {
668 struct sp_pll *pll;
669 struct clk *clk;
670 //unsigned long flags = 0;
671 struct clk_init_data initd = {
672 .name = name,
673 .parent_names = &parent,
674 .ops = (bp_bit >= 0) ? &sp_pll_ops : &sp_pll_sub_ops,
675 .num_parents = 1,
676 .flags = CLK_IGNORE_UNUSED
677 };
678
679 pll = kzalloc(sizeof(*pll), GFP_KERNEL);
680 if (!pll)
681 return ERR_PTR(-ENOMEM);
682
683 if (reg == PLLSYS_CTL)
684 initd.flags |= CLK_IS_CRITICAL;
685
686 pll->hw.init = &initd;
687 pll->reg = reg;
688 pll->pd_bit = pd_bit;
689 pll->bp_bit = bp_bit;
690 pll->brate = brate;
691 pll->div_shift = shift;
692 pll->div_width = width;
693 pll->lock = lock;
694
695 clk = clk_register(NULL, &pll->hw);
696 if (WARN_ON(IS_ERR(clk))) {
697 kfree(pll);
698 } else {
699 pr_info("%-20s%lu\n", name, clk_get_rate(clk));
700 clk_register_clkdev(clk, NULL, name);
701 }
702
703 return clk;
704 }
705

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

Hi Qin,

I love your patch! Perhaps something to improve:

[auto build test WARNING on pza/reset/next]
[also build test WARNING on robh/for-next clk/clk-next linus/master v5.15] [cannot apply to tip/irq/core next-20211029]
[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/Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211101-140155
base:   https://git.pengutronix.de/git/pza/linux reset/next
config: sparc-randconfig-r011-20211101 (attached as .config)
compiler: sparc64-linux-gcc (GCC) 11.2.0

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
# https://github.com/0day-ci/linux/commit/ca53c40693bc90c44e3ec4c95c7758d7342229e2
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211101-140155 git checkout ca53c40693bc90c44e3ec4c95c7758d7342229e2
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=sparc

If you fix the issue, kindly add following tag as appropriate

All warnings (new ones prefixed by >>):

drivers/irqchip/irq-sp7021-intc.c: In function 'sp_intc_handle_ext_cascaded':

>> drivers/irqchip/irq-sp7021-intc.c:154:23: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
     154 |         int ext_num = (int)irq_desc_get_handler_data(desc);
         |                       ^
   drivers/irqchip/irq-sp7021-intc.c: In function 'sp_intc_irq_map':
>> drivers/irqchip/irq-sp7021-intc.c:212:76: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
     212 |         irq_set_chained_handler_and_data(irq, sp_intc_handle_ext_cascaded, (void *)i);
         |                                                                            ^
   drivers/irqchip/irq-sp7021-intc.c: At top level:
   drivers/irqchip/irq-sp7021-intc.c:217:6: warning: no previous prototype for 'sp_intc_set_ext' [-Wmissing-prototypes]
     217 | void sp_intc_set_ext(u32 hwirq, int ext_num)
         |      ^~~~~~~~~~~~~~~

drivers/irqchip/irq-sp7021-intc.c:223:12: warning: no previous prototype for 'sp_intc_init_dt' [-Wmissing-prototypes]
223 | int __init sp_intc_init_dt(struct device_node *node, struct device_node *parent)
| ^~~~~~~~~~~~~~~

vim +154 drivers/irqchip/irq-sp7021-intc.c

150
151 static void sp_intc_handle_ext_cascaded(struct irq_desc *desc)
152 {
153 struct irq_chip *chip = irq_desc_get_chip(desc);

 > 154		int ext_num = (int)irq_desc_get_handler_data(desc);
   155		int hwirq;
   156	
   157		chained_irq_enter(chip, desc);
   158	
   159		while ((hwirq = sp_intc_get_ext_irq(ext_num)) >= 0) {
   160			if (unlikely(IS_GPIO_INT(hwirq) && TEST_STATE(hwirq, _IS_ACTIVE))) { // WORKAROUND
   161				ASSIGN_STATE(hwirq, _IS_ACTIVE, false);
   162				sp_intc_assign_bit(hwirq, REG_INTR_POLARITY, TEST_STATE(hwirq, _IS_LOW));
   163			} else {
   164				generic_handle_domain_irq(sp_intc.domain, hwirq);
   165			}
   166		}
   167	
   168		chained_irq_exit(chip, desc);
   169	}
   170	
   171	#ifdef CONFIG_SMP
   172	static int sp_intc_set_affinity(struct irq_data *d, const struct cpumask *mask, bool force)
   173	{
   174		return -EINVAL;
   175	}
   176	#endif
   177	
   178	static struct irq_chip sp_intc_chip = {
   179		.name = "sp_intc",
   180		.irq_ack = sp_intc_ack_irq,
   181		.irq_mask = sp_intc_mask_irq,
   182		.irq_unmask = sp_intc_unmask_irq,
   183		.irq_set_type = sp_intc_set_type,
   184	#ifdef CONFIG_SMP
   185		.irq_set_affinity = sp_intc_set_affinity,
   186	#endif
   187	};
   188	
   189	static int sp_intc_irq_domain_map(struct irq_domain *domain,
   190		unsigned int irq, irq_hw_number_t hwirq)
   191	{
   192		irq_set_chip_and_handler(irq, &sp_intc_chip, handle_level_irq);
   193		irq_set_chip_data(irq, &sp_intc_chip);
   194		irq_set_noprobe(irq);
   195	
   196		return 0;
   197	}
   198	
   199	static const struct irq_domain_ops sp_intc_dm_ops = {
   200		.xlate = irq_domain_xlate_twocell,
   201		.map = sp_intc_irq_domain_map,
   202	};
   203	
   204	static int sp_intc_irq_map(struct device_node *node, int i)
   205	{
   206		unsigned int irq;
   207	
   208		irq = irq_of_parse_and_map(node, i);
   209		if (!irq)
   210			return -ENOENT;
   211	
 > 212		irq_set_chained_handler_and_data(irq, sp_intc_handle_ext_cascaded, (void *)i);

213
214 return 0;
215 }
216

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

Hi Qin,

I love your patch! Perhaps something to improve:

[auto build test WARNING on pza/reset/next]
[also build test WARNING on robh/for-next clk/clk-next linus/master v5.15] [cannot apply to tip/irq/core next-20211029]
[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/Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211101-140155
base:   https://git.pengutronix.de/git/pza/linux reset/next
config: arc-randconfig-r016-20211101 (attached as .config)
compiler: arceb-elf-gcc (GCC) 11.2.0

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
# https://github.com/0day-ci/linux/commit/ca53c40693bc90c44e3ec4c95c7758d7342229e2
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211101-140155 git checkout ca53c40693bc90c44e3ec4c95c7758d7342229e2
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=arc

If you fix the issue, kindly add following tag as appropriate

All warnings (new ones prefixed by >>):

>> drivers/irqchip/irq-sp7021-intc.c:217:6: warning: no previous prototype for 'sp_intc_set_ext' [-Wmissing-prototypes]
     217 | void sp_intc_set_ext(u32 hwirq, int ext_num)
         |      ^~~~~~~~~~~~~~~
>> drivers/irqchip/irq-sp7021-intc.c:223:12: warning: no previous prototype for 'sp_intc_init_dt' [-Wmissing-prototypes]

223 | int __init sp_intc_init_dt(struct device_node *node, struct device_node *parent)
| ^~~~~~~~~~~~~~~

vim +/sp_intc_set_ext +217 drivers/irqchip/irq-sp7021-intc.c

216

 > 217	void sp_intc_set_ext(u32 hwirq, int ext_num)
   218	{
   219		sp_intc_assign_bit(hwirq, REG_INTR_PRIORITY, !ext_num);
   220	}
   221	EXPORT_SYMBOL_GPL(sp_intc_set_ext);
   222	
 > 223	int __init sp_intc_init_dt(struct device_node *node, struct device_node *parent)

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

Hi Qin,

I love your patch! Yet something to improve:

[auto build test ERROR on robh/for-next]
[also build test ERROR on pza/reset/next clk/clk-next linus/master v5.15-rc7] [cannot apply to tip/irq/core next-20211029]
[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/Qin-Jian/Add-Sunplus-SP7021-SoC-Support/20211029-171054
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arc-allyesconfig (attached as .config)
compiler: arceb-elf-gcc (GCC) 11.2.0

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
# https://github.com/0day-ci/linux/commit/91ab876ddb6d1a596c50c43a79d3e06b9695dee7
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Qin-Jian/Add-Sunplus-SP7021-SoC-Support/20211029-171054 git checkout 91ab876ddb6d1a596c50c43a79d3e06b9695dee7
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arc SHELL=/bin/bash

If you fix the issue, kindly add following tag as appropriate

All errors (new ones prefixed by >>):

>> drivers/irqchip/irq-sp7021-intc.c:14:10: fatal error: asm/exception.h: No such file or directory
      14 | #include <asm/exception.h>
         |          ^~~~~~~~~~~~~~~~~

compilation terminated.

vim +14 drivers/irqchip/irq-sp7021-intc.c

    13	
  > 14	#include <asm/exception.h>
    15	#include <dt-bindings/interrupt-controller/sp7021-intc.h>
    16	

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

Hi Qin,

I love your patch! Yet something to improve:

[auto build test ERROR on robh/for-next]
[also build test ERROR on pza/reset/next clk/clk-next linus/master v5.15-rc7] [cannot apply to tip/irq/core next-20211029]
[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/Qin-Jian/Add-Sunplus-SP7021-SoC-Support/20211029-171054
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arm-allyesconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 11.2.0

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
# https://github.com/0day-ci/linux/commit/91ab876ddb6d1a596c50c43a79d3e06b9695dee7
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Qin-Jian/Add-Sunplus-SP7021-SoC-Support/20211029-171054 git checkout 91ab876ddb6d1a596c50c43a79d3e06b9695dee7
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=arm

If you fix the issue, kindly add following tag as appropriate

All errors (new ones prefixed by >>):

>> drivers/irqchip/irq-sp7021-intc.c:211:5: error: no previous prototype for 'sp_intc_xlate_of' [-Werror=missing-prototypes]
     211 | int sp_intc_xlate_of(struct irq_domain *d, struct device_node *node,
         |     ^~~~~~~~~~~~~~~~
>> drivers/irqchip/irq-sp7021-intc.c:274:12: error: no previous prototype for 'sp_intc_init_dt' [-Werror=missing-prototypes]
     274 | int __init sp_intc_init_dt(
         |            ^~~~~~~~~~~~~~~

cc1: all warnings being treated as errors

vim +/sp_intc_xlate_of +211 drivers/irqchip/irq-sp7021-intc.c

210

 > 211	int sp_intc_xlate_of(struct irq_domain *d, struct device_node *node,
   212				  const u32 *intspec, unsigned int intsize,
   213				  irq_hw_number_t *out_hwirq, unsigned int *out_type)
   214	{
   215		int ret;
   216	
   217		ret = irq_domain_xlate_twocell(d, node,
   218			intspec, intsize, out_hwirq, out_type);
   219		if (!ret) {
   220			/* intspec[1]: IRQ_TYPE | SP_INTC_EXT_INT
   221			 * SP_INTC_EXT_INT: 0-1,
   222			 *   to indicate route to which parent irq: EXT_INT0/EXT_INT1
   223			 */
   224			u32 ext_int = (intspec[1] & SP_INTC_EXT_INT_MASK) >> SP_INTC_EXT_INT_SHFIT;
   225	
   226			/* priority = 0, route to EXT_INT1
   227			 *    otherwise, route to EXT_INT0
   228			 */
   229			sp_intc_set_priority(*out_hwirq, 1 - ext_int);
   230		}
   231	
   232		return ret;
   233	}
   234	
   235	static struct irq_chip sp_intc_chip = {
   236		.name = "sp_intc",
   237		.irq_ack = sp_intc_ack_irq,
   238		.irq_mask = sp_intc_mask_irq,
   239		.irq_unmask = sp_intc_unmask_irq,
   240		.irq_set_type = sp_intc_set_type,
   241	};
   242	
   243	static int sp_intc_irq_domain_map(struct irq_domain *domain,
   244		unsigned int irq, irq_hw_number_t hwirq)
   245	{
   246		irq_set_chip_and_handler(irq, &sp_intc_chip, handle_level_irq);
   247		irq_set_chip_data(irq, &sp_intc_chip);
   248		irq_set_noprobe(irq);
   249	
   250		return 0;
   251	}
   252	
   253	static const struct irq_domain_ops sp_intc_dm_ops = {
   254		.xlate = sp_intc_xlate_of,
   255		.map = sp_intc_irq_domain_map,
   256	};
   257	
   258	#ifdef CONFIG_OF
   259	static int sp_intc_irq_map(struct device_node *node, int i)
   260	{
   261		sp_intc.virq[i] = irq_of_parse_and_map(node, i);
   262		if (!sp_intc.virq[i]) {
   263			pr_err("%s: missed EXT_INT%d in DT\n", __func__, i);
   264			return -ENOENT;
   265		}
   266	
   267		pr_info("%s: EXT_INT%d = %d\n", __func__, i, sp_intc.virq[i]);
   268		irq_set_chained_handler_and_data(sp_intc.virq[i],
   269			sp_intc_handle_ext_cascaded, (void *)i);
   270	
   271		return 0;
   272	}
   273	
 > 274	int __init sp_intc_init_dt(

275 struct device_node *node, struct device_node *parent)
276 {
277 void __iomem *base0, *base1;
278
279 base0 = of_iomap(node, 0);
280 if (!base0) {
281 pr_err("unable to map sp-intc base 0\n");
282 return -EINVAL;
283 }
284
285 base1 = of_iomap(node, 1);
286 if (!base1) {
287 pr_err("unable to map sp-intc base 1\n");
288 return -EINVAL;
289 }
290
291 sp_intc.node = node;
292
293 sp_intc_chip_init(base0, base1);
294
295 sp_intc.domain = irq_domain_add_linear(node,
296 SP_INTC_HWIRQ_MAX - SP_INTC_HWIRQ_MIN,
297 &sp_intc_dm_ops, &sp_intc);
298 if (!sp_intc.domain) {
299 pr_err("%s: unable to create linear domain\n", __func__);
300 return -EINVAL;
301 }
302
303 raw_spin_lock_init(&sp_intc.lock);
304
305 if (parent) {
306 /* secondary chained controller */
307 if (sp_intc_irq_map(node, 0)) // EXT_INT0
308 return -ENOENT;
309
310 if (sp_intc_irq_map(node, 1)) // EXT_INT1
311 return -ENOENT;
312 } else {
313 /* primary controller */
314 set_handle_irq(sp_intc_handle_irq);
315 }
316
317 return 0;
318 }
319 IRQCHIP_DECLARE(sp_intc, "sunplus,sp7021-intc", sp_intc_init_dt);
320 #endif
321

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

Hi Qin,

I love your patch! Perhaps something to improve:

[auto build test WARNING on pza/reset/next]
[also build test WARNING on robh/for-next clk/clk-next tip/irq/core linus/master v5.15 next-20211104] [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/Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211104-115746
base:   https://git.pengutronix.de/git/pza/linux reset/next
config: xtensa-randconfig-r012-20211104 (attached as .config)
compiler: xtensa-linux-gcc (GCC) 11.2.0

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
# https://github.com/0day-ci/linux/commit/604141a9aad536dff720a6337561171d72d63d74
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211104-115746 git checkout 604141a9aad536dff720a6337561171d72d63d74
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=xtensa

If you fix the issue, kindly add following tag as appropriate

All warnings (new ones prefixed by >>):

drivers/clk/clk-sp7021.c: In function 'plltv_fractional_div':

>> drivers/clk/clk-sp7021.c:292:13: warning: variable 'diff_min_sign' set but not used [-Wunused-but-set-variable]
     292 |         u32 diff_min_sign = 0;
         |             ^~~~~~~~~~~~~

drivers/clk/clk-sp7021.c: At top level:
drivers/clk/clk-sp7021.c:625:13: warning: no previous prototype for 'clk_register_sp_pll' [-Wmissing-prototypes]
625 | struct clk *clk_register_sp_pll(const char *name, const char *parent,
| ^~~~~~~~~~~~~~~~~~~

vim +/diff_min_sign +292 drivers/clk/clk-sp7021.c

286
287 static long plltv_fractional_div(struct sp_pll *clk, unsigned long freq)
288 {
289 u32 m, r;
290 u32 nint, nfra;
291 u32 diff_min_quotient = 210000000, diff_min_remainder = 0;

 > 292		u32 diff_min_sign = 0;
   293		unsigned long fvco, nf, f, fout = 0;
   294		int sdm, ph;
   295	
   296		/* check freq */
   297		if (freq < F_MIN) {
   298			pr_warn("%s: %s freq:%lu < F_MIN:%lu, round up\n",
   299				__func__, clk_hw_get_name(&clk->hw), freq, F_MIN);
   300			freq = F_MIN;
   301		} else if (freq > F_MAX) {
   302			pr_warn("%s: %s freq:%lu > F_MAX:%lu, round down\n",
   303				__func__, clk_hw_get_name(&clk->hw), freq, F_MAX);
   304			freq = F_MAX;
   305		}
   306	
   307		/* DIVR 0~3 */
   308		for (r = 0; r <= 3; r++) {
   309			fvco = freq << r;
   310			if (fvco <= FVCO_MAX)
   311				break;
   312		}
   313		f = F_27M >> r;
   314	
   315		/* PH_SEL 1/0 */
   316		for (ph = 1; ph >= 0; ph--) {
   317			const u32 *pp = pt[ph];
   318			u32 ms = 1;
   319	
   320			/* SDM_MOD 0/1 */
   321			for (sdm = 0; sdm <= 1; sdm++) {
   322				u32 mod = mods[sdm];
   323	
   324				/* DIVM 1~32 */
   325				for (m = ms; m <= 32; m++) {
   326					u32 diff_freq;
   327					u32 diff_freq_quotient = 0, diff_freq_remainder = 0;
   328					u32 diff_freq_sign = 0; /* 0:Positive number, 1:Negative number */
   329	
   330					nf = fvco * m;
   331					nint = nf / pp[3];
   332	
   333					if (nint < pp[1])
   334						continue;
   335					if (nint > pp[1])
   336						break;
   337	
   338					nfra = (((nf % pp[3]) * mod * pp[4]) + (F_27M / 2)) / F_27M;
   339					if (nfra)
   340						diff_freq = (f * (nint + pp[2]) / pp[0]) -
   341									(f * (mod - nfra) / mod / pp[4]);
   342					else
   343						diff_freq = (f * (nint) / pp[0]);
   344	
   345					diff_freq_quotient  = diff_freq / m;
   346					diff_freq_remainder = ((diff_freq % m) * 1000) / m;
   347	
   348					if (freq > diff_freq_quotient) {
   349						diff_freq_quotient  = freq - diff_freq_quotient - 1;
   350						diff_freq_remainder = 1000 - diff_freq_remainder;
   351						diff_freq_sign = 1;
   352					} else {
   353						diff_freq_quotient = diff_freq_quotient - freq;
   354						diff_freq_sign = 0;
   355					}
   356	
   357					if ((diff_min_quotient > diff_freq_quotient) ||
   358						((diff_min_quotient == diff_freq_quotient) &&
   359						(diff_min_remainder > diff_freq_remainder))) {
   360	
   361						/* found a closer freq, save parameters */
   362						clk->p[SEL_FRA] = 1;
   363						clk->p[SDM_MOD] = sdm;
   364						clk->p[PH_SEL]  = ph;
   365						clk->p[NFRA]    = nfra;
   366						clk->p[DIVR]    = r;
   367						clk->p[DIVM]    = m;

368
369 fout = diff_freq / m;
370 diff_min_quotient = diff_freq_quotient;
371 diff_min_remainder = diff_freq_remainder;
372 diff_min_sign = diff_freq_sign;
373 }
374 }
375 }
376 }
377
378 if (!fout) {
379 pr_err("%s: %s freq:%lu not found a valid setting\n",
380 __func__, clk_hw_get_name(&clk->hw), freq);
381 return -EINVAL;
382 }
383
384 return fout;
385 }
386

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

Hi Qin,

I love your patch! Yet something to improve:

[auto build test ERROR on pza/reset/next]
[also build test ERROR on robh/for-next clk/clk-next tip/irq/core linus/master v5.15 next-20211104] [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/Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211104-115746
base:   https://git.pengutronix.de/git/pza/linux reset/next
config: arc-allyesconfig (attached as .config)
compiler: arceb-elf-gcc (GCC) 11.2.0

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
# https://github.com/0day-ci/linux/commit/604141a9aad536dff720a6337561171d72d63d74
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211104-115746 git checkout 604141a9aad536dff720a6337561171d72d63d74
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=arc

If you fix the issue, kindly add following tag as appropriate

All errors (new ones prefixed by >>):

drivers/clk/clk-sp7021.c: In function 'plltv_fractional_div':

>> drivers/clk/clk-sp7021.c:292:13: error: variable 'diff_min_sign' set but not used [-Werror=unused-but-set-variable]
     292 |         u32 diff_min_sign = 0;
         |             ^~~~~~~~~~~~~

drivers/clk/clk-sp7021.c: At top level:
drivers/clk/clk-sp7021.c:625:13: error: no previous prototype for 'clk_register_sp_pll' [-Werror=missing-prototypes]
625 | struct clk *clk_register_sp_pll(const char *name, const char *parent,
| ^~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

vim +/diff_min_sign +292 drivers/clk/clk-sp7021.c

286
287 static long plltv_fractional_div(struct sp_pll *clk, unsigned long freq)
288 {
289 u32 m, r;
290 u32 nint, nfra;
291 u32 diff_min_quotient = 210000000, diff_min_remainder = 0;

 > 292		u32 diff_min_sign = 0;
   293		unsigned long fvco, nf, f, fout = 0;
   294		int sdm, ph;
   295	
   296		/* check freq */
   297		if (freq < F_MIN) {
   298			pr_warn("%s: %s freq:%lu < F_MIN:%lu, round up\n",
   299				__func__, clk_hw_get_name(&clk->hw), freq, F_MIN);
   300			freq = F_MIN;
   301		} else if (freq > F_MAX) {
   302			pr_warn("%s: %s freq:%lu > F_MAX:%lu, round down\n",
   303				__func__, clk_hw_get_name(&clk->hw), freq, F_MAX);
   304			freq = F_MAX;
   305		}
   306	
   307		/* DIVR 0~3 */
   308		for (r = 0; r <= 3; r++) {
   309			fvco = freq << r;
   310			if (fvco <= FVCO_MAX)
   311				break;
   312		}
   313		f = F_27M >> r;
   314	
   315		/* PH_SEL 1/0 */
   316		for (ph = 1; ph >= 0; ph--) {
   317			const u32 *pp = pt[ph];
   318			u32 ms = 1;
   319	
   320			/* SDM_MOD 0/1 */
   321			for (sdm = 0; sdm <= 1; sdm++) {
   322				u32 mod = mods[sdm];
   323	
   324				/* DIVM 1~32 */
   325				for (m = ms; m <= 32; m++) {
   326					u32 diff_freq;
   327					u32 diff_freq_quotient = 0, diff_freq_remainder = 0;
   328					u32 diff_freq_sign = 0; /* 0:Positive number, 1:Negative number */
   329	
   330					nf = fvco * m;
   331					nint = nf / pp[3];
   332	
   333					if (nint < pp[1])
   334						continue;
   335					if (nint > pp[1])
   336						break;
   337	
   338					nfra = (((nf % pp[3]) * mod * pp[4]) + (F_27M / 2)) / F_27M;
   339					if (nfra)
   340						diff_freq = (f * (nint + pp[2]) / pp[0]) -
   341									(f * (mod - nfra) / mod / pp[4]);
   342					else
   343						diff_freq = (f * (nint) / pp[0]);
   344	
   345					diff_freq_quotient  = diff_freq / m;
   346					diff_freq_remainder = ((diff_freq % m) * 1000) / m;
   347	
   348					if (freq > diff_freq_quotient) {
   349						diff_freq_quotient  = freq - diff_freq_quotient - 1;
   350						diff_freq_remainder = 1000 - diff_freq_remainder;
   351						diff_freq_sign = 1;
   352					} else {
   353						diff_freq_quotient = diff_freq_quotient - freq;
   354						diff_freq_sign = 0;
   355					}
   356	
   357					if ((diff_min_quotient > diff_freq_quotient) ||
   358						((diff_min_quotient == diff_freq_quotient) &&
   359						(diff_min_remainder > diff_freq_remainder))) {
   360	
   361						/* found a closer freq, save parameters */
   362						clk->p[SEL_FRA] = 1;
   363						clk->p[SDM_MOD] = sdm;
   364						clk->p[PH_SEL]  = ph;
   365						clk->p[NFRA]    = nfra;
   366						clk->p[DIVR]    = r;
   367						clk->p[DIVM]    = m;

368
369 fout = diff_freq / m;
370 diff_min_quotient = diff_freq_quotient;
371 diff_min_remainder = diff_freq_remainder;
372 diff_min_sign = diff_freq_sign;
373 }
374 }
375 }
376 }
377
378 if (!fout) {
379 pr_err("%s: %s freq:%lu not found a valid setting\n",
380 __func__, clk_hw_get_name(&clk->hw), freq);
381 return -EINVAL;
382 }
383
384 return fout;
385 }
386

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

Hi Qin,

I love your patch! Yet something to improve:

[auto build test ERROR on pza/reset/next]
[also build test ERROR on robh/for-next clk/clk-next v5.15] [cannot apply to tip/irq/core linus/master next-20211104] [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/Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211104-115746
base:   https://git.pengutronix.de/git/pza/linux reset/next
config: arm-randconfig-c002-20211104 (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 11.2.0

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
# https://github.com/0day-ci/linux/commit/a0450f4dfa75b60af973bceea07b1be864ae81ba
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Qin-Jian/dt-bindings-vendor-prefixes-Add-Sunplus/20211104-115746
git checkout a0450f4dfa75b60af973bceea07b1be864ae81ba
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arm SHELL=/bin/bash arch/arm/kernel/ arch/arm/mm/

If you fix the issue, kindly add following tag as appropriate

All errors (new ones prefixed by >>):

arch/arm/kernel/setup.c: In function 'early_print':
arch/arm/kernel/setup.c:372:9: warning: function 'early_print' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format]

     372 |         vsnprintf(buf, sizeof(buf), str, ap);
         |         ^~~~~~~~~
   In file included from arch/arm/kernel/setup.c:36:
   arch/arm/kernel/setup.c: In function 'cpuid_init_hwcaps':
>> arch/arm/include/asm/cputype.h:22:25: error: expected ':' or ')' before numeric constant
      22 | #define CPUID_EXT_ISAR0 0x60
         |                         ^~~~
   arch/arm/include/asm/cputype.h:135:51: note: in definition of macro 'read_cpuid_ext'
     135 |                 asm("mrc        p15, 0, %0, c0, " ext_reg               \
         |                                                   ^~~~~~~
   arch/arm/kernel/setup.c:457:17: note: in expansion of macro 'cpuid_feature_extract'
     457 |         block = cpuid_feature_extract(CPUID_EXT_ISAR0, 24);
         |                 ^~~~~~~~~~~~~~~~~~~~~
   arch/arm/kernel/setup.c:457:39: note: in expansion of macro 'CPUID_EXT_ISAR0'
     457 |         block = cpuid_feature_extract(CPUID_EXT_ISAR0, 24);
         |                                       ^~~~~~~~~~~~~~~
   arch/arm/include/asm/cputype.h:18:25: error: expected ':' or ')' before numeric constant
      18 | #define CPUID_EXT_MMFR0 0x50
         |                         ^~~~
   arch/arm/include/asm/cputype.h:135:51: note: in definition of macro 'read_cpuid_ext'
     135 |                 asm("mrc        p15, 0, %0, c0, " ext_reg               \
         |                                                   ^~~~~~~
   arch/arm/kernel/setup.c:464:17: note: in expansion of macro 'cpuid_feature_extract'
     464 |         block = cpuid_feature_extract(CPUID_EXT_MMFR0, 0);
         |                 ^~~~~~~~~~~~~~~~~~~~~
   arch/arm/kernel/setup.c:464:39: note: in expansion of macro 'CPUID_EXT_MMFR0'
     464 |         block = cpuid_feature_extract(CPUID_EXT_MMFR0, 0);
         |                                       ^~~~~~~~~~~~~~~
   arch/arm/include/asm/cputype.h:27:25: error: expected ':' or ')' before numeric constant
      27 | #define CPUID_EXT_ISAR5 0x74
         |                         ^~~~
   arch/arm/include/asm/cputype.h:135:51: note: in definition of macro 'read_cpuid_ext'
     135 |                 asm("mrc        p15, 0, %0, c0, " ext_reg               \
         |                                                   ^~~~~~~
   arch/arm/kernel/setup.c:469:32: note: in expansion of macro 'CPUID_EXT_ISAR5'
     469 |         isar5 = read_cpuid_ext(CPUID_EXT_ISAR5);
         |                                ^~~~~~~~~~~~~~~
   arch/arm/kernel/setup.c: In function 'elf_hwcap_fixup':
   arch/arm/include/asm/cputype.h:25:25: error: expected ':' or ')' before numeric constant
      25 | #define CPUID_EXT_ISAR3 0x6c
         |                         ^~~~
   arch/arm/include/asm/cputype.h:135:51: note: in definition of macro 'read_cpuid_ext'
     135 |                 asm("mrc        p15, 0, %0, c0, " ext_reg               \
         |                                                   ^~~~~~~
   arch/arm/kernel/setup.c:513:13: note: in expansion of macro 'cpuid_feature_extract'
     513 |         if (cpuid_feature_extract(CPUID_EXT_ISAR3, 12) > 1 ||
         |             ^~~~~~~~~~~~~~~~~~~~~
   arch/arm/kernel/setup.c:513:35: note: in expansion of macro 'CPUID_EXT_ISAR3'
     513 |         if (cpuid_feature_extract(CPUID_EXT_ISAR3, 12) > 1 ||
         |                                   ^~~~~~~~~~~~~~~
   arch/arm/include/asm/cputype.h:25:25: error: expected ':' or ')' before numeric constant
      25 | #define CPUID_EXT_ISAR3 0x6c
         |                         ^~~~
   arch/arm/include/asm/cputype.h:135:51: note: in definition of macro 'read_cpuid_ext'
     135 |                 asm("mrc        p15, 0, %0, c0, " ext_reg               \
         |                                                   ^~~~~~~
   arch/arm/kernel/setup.c:514:14: note: in expansion of macro 'cpuid_feature_extract'
     514 |             (cpuid_feature_extract(CPUID_EXT_ISAR3, 12) == 1 &&
         |              ^~~~~~~~~~~~~~~~~~~~~
   arch/arm/kernel/setup.c:514:36: note: in expansion of macro 'CPUID_EXT_ISAR3'
     514 |             (cpuid_feature_extract(CPUID_EXT_ISAR3, 12) == 1 &&
         |                                    ^~~~~~~~~~~~~~~
   arch/arm/include/asm/cputype.h:26:25: error: expected ':' or ')' before numeric constant
      26 | #define CPUID_EXT_ISAR4 0x70
         |                         ^~~~
   arch/arm/include/asm/cputype.h:135:51: note: in definition of macro 'read_cpuid_ext'
     135 |                 asm("mrc        p15, 0, %0, c0, " ext_reg               \
         |                                                   ^~~~~~~
   arch/arm/kernel/setup.c:515:14: note: in expansion of macro 'cpuid_feature_extract'
     515 |              cpuid_feature_extract(CPUID_EXT_ISAR4, 20) >= 3))
         |              ^~~~~~~~~~~~~~~~~~~~~
   arch/arm/kernel/setup.c:515:36: note: in expansion of macro 'CPUID_EXT_ISAR4'
     515 |              cpuid_feature_extract(CPUID_EXT_ISAR4, 20) >= 3))
         |                                    ^~~~~~~~~~~~~~~

In file included from arch/arm/include/asm/efi.h:12,
from arch/arm/kernel/setup.c:37:
At top level:
arch/arm/include/asm/fixmap.h:39:35: warning: '__end_of_fixed_addresses' defined but not used [-Wunused-const-variable=]
39 | static const enum fixed_addresses __end_of_fixed_addresses =
| ^~~~~~~~~~~~~~~~~~~~~~~~
--
arch/arm/mm/pmsa-v8.c: In function 'prlar_read':

>> arch/arm/mm/pmsa-v8.c:53:16: error: implicit declaration of function 'readl_relaxed' [-Werror=implicit-function-declaration]
      53 |         return readl_relaxed(BASEADDR_V7M_SCB + PMSAv8_RLAR);
         |                ^~~~~~~~~~~~~
>> arch/arm/mm/pmsa-v8.c:53:30: error: 'BASEADDR_V7M_SCB' undeclared (first use in this function)
      53 |         return readl_relaxed(BASEADDR_V7M_SCB + PMSAv8_RLAR);
         |                              ^~~~~~~~~~~~~~~~
   arch/arm/mm/pmsa-v8.c:53:30: note: each undeclared identifier is reported only once for each function it appears in
>> arch/arm/mm/pmsa-v8.c:53:49: error: 'PMSAv8_RLAR' undeclared (first use in this function); did you mean 'PMSAv8_MAIR'?
      53 |         return readl_relaxed(BASEADDR_V7M_SCB + PMSAv8_RLAR);
         |                                                 ^~~~~~~~~~~
         |                                                 PMSAv8_MAIR
   arch/arm/mm/pmsa-v8.c: In function 'prbar_read':
   arch/arm/mm/pmsa-v8.c:58:30: error: 'BASEADDR_V7M_SCB' undeclared (first use in this function)
      58 |         return readl_relaxed(BASEADDR_V7M_SCB + PMSAv8_RBAR);
         |                              ^~~~~~~~~~~~~~~~
>> arch/arm/mm/pmsa-v8.c:58:49: error: 'PMSAv8_RBAR' undeclared (first use in this function); did you mean 'PMSAv8_MAIR'?
      58 |         return readl_relaxed(BASEADDR_V7M_SCB + PMSAv8_RBAR);
         |                                                 ^~~~~~~~~~~
         |                                                 PMSAv8_MAIR
   arch/arm/mm/pmsa-v8.c: In function 'prsel_write':
>> arch/arm/mm/pmsa-v8.c:63:9: error: implicit declaration of function 'writel_relaxed' [-Werror=implicit-function-declaration]
      63 |         writel_relaxed(v, BASEADDR_V7M_SCB + PMSAv8_RNR);
         |         ^~~~~~~~~~~~~~
   arch/arm/mm/pmsa-v8.c:63:27: error: 'BASEADDR_V7M_SCB' undeclared (first use in this function)
      63 |         writel_relaxed(v, BASEADDR_V7M_SCB + PMSAv8_RNR);
         |                           ^~~~~~~~~~~~~~~~
>> arch/arm/mm/pmsa-v8.c:63:46: error: 'PMSAv8_RNR' undeclared (first use in this function); did you mean 'PMSAv8_MAIR'?
      63 |         writel_relaxed(v, BASEADDR_V7M_SCB + PMSAv8_RNR);
         |                                              ^~~~~~~~~~
         |                                              PMSAv8_MAIR
   arch/arm/mm/pmsa-v8.c: In function 'prbar_write':
   arch/arm/mm/pmsa-v8.c:68:27: error: 'BASEADDR_V7M_SCB' undeclared (first use in this function)
      68 |         writel_relaxed(v, BASEADDR_V7M_SCB + PMSAv8_RBAR);
         |                           ^~~~~~~~~~~~~~~~
   arch/arm/mm/pmsa-v8.c:68:46: error: 'PMSAv8_RBAR' undeclared (first use in this function); did you mean 'PMSAv8_MAIR'?
      68 |         writel_relaxed(v, BASEADDR_V7M_SCB + PMSAv8_RBAR);
         |                                              ^~~~~~~~~~~
         |                                              PMSAv8_MAIR
   arch/arm/mm/pmsa-v8.c: In function 'prlar_write':
   arch/arm/mm/pmsa-v8.c:73:27: error: 'BASEADDR_V7M_SCB' undeclared (first use in this function)
      73 |         writel_relaxed(v, BASEADDR_V7M_SCB + PMSAv8_RLAR);
         |                           ^~~~~~~~~~~~~~~~
   arch/arm/mm/pmsa-v8.c:73:46: error: 'PMSAv8_RLAR' undeclared (first use in this function); did you mean 'PMSAv8_MAIR'?
      73 |         writel_relaxed(v, BASEADDR_V7M_SCB + PMSAv8_RLAR);
         |                                              ^~~~~~~~~~~
         |                                              PMSAv8_MAIR
   arch/arm/mm/pmsa-v8.c: In function 'prbar_read':
   arch/arm/mm/pmsa-v8.c:59:1: error: control reaches end of non-void function [-Werror=return-type]
      59 | }
         | ^
   arch/arm/mm/pmsa-v8.c: In function 'prlar_read':
   arch/arm/mm/pmsa-v8.c:54:1: error: control reaches end of non-void function [-Werror=return-type]
      54 | }
         | ^

cc1: some warnings being treated as errors

vim +22 arch/arm/include/asm/cputype.h

0ba8b9b273c45d Russell King 2008-08-10 12

6fae9cdafc92ae Uwe Kleine-König 2013-05-06  13  #ifdef CONFIG_CPU_V7M
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  14  #define CPUID_EXT_PFR0	0x40
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  15  #define CPUID_EXT_PFR1	0x44
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  16  #define CPUID_EXT_DFR0	0x48
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  17  #define CPUID_EXT_AFR0	0x4c
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  18  #define CPUID_EXT_MMFR0	0x50
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  19  #define CPUID_EXT_MMFR1	0x54
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  20  #define CPUID_EXT_MMFR2	0x58
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  21  #define CPUID_EXT_MMFR3	0x5c
6fae9cdafc92ae Uwe Kleine-König 2013-05-06 @22  #define CPUID_EXT_ISAR0	0x60
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  23  #define CPUID_EXT_ISAR1	0x64
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  24  #define CPUID_EXT_ISAR2	0x68
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  25  #define CPUID_EXT_ISAR3	0x6c
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  26  #define CPUID_EXT_ISAR4	0x70
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  27  #define CPUID_EXT_ISAR5	0x74
6fae9cdafc92ae Uwe Kleine-König 2013-05-06  28  #else
faa7bc51c11d5b Catalin Marinas  2009-05-30  29  #define CPUID_EXT_PFR0	"c1, 0"
faa7bc51c11d5b Catalin Marinas  2009-05-30  30  #define CPUID_EXT_PFR1	"c1, 1"
faa7bc51c11d5b Catalin Marinas  2009-05-30  31  #define CPUID_EXT_DFR0	"c1, 2"
faa7bc51c11d5b Catalin Marinas  2009-05-30  32  #define CPUID_EXT_AFR0	"c1, 3"
faa7bc51c11d5b Catalin Marinas  2009-05-30  33  #define CPUID_EXT_MMFR0	"c1, 4"
faa7bc51c11d5b Catalin Marinas  2009-05-30  34  #define CPUID_EXT_MMFR1	"c1, 5"
faa7bc51c11d5b Catalin Marinas  2009-05-30  35  #define CPUID_EXT_MMFR2	"c1, 6"
faa7bc51c11d5b Catalin Marinas  2009-05-30  36  #define CPUID_EXT_MMFR3	"c1, 7"
faa7bc51c11d5b Catalin Marinas  2009-05-30  37  #define CPUID_EXT_ISAR0	"c2, 0"
faa7bc51c11d5b Catalin Marinas  2009-05-30  38  #define CPUID_EXT_ISAR1	"c2, 1"
faa7bc51c11d5b Catalin Marinas  2009-05-30  39  #define CPUID_EXT_ISAR2	"c2, 2"
faa7bc51c11d5b Catalin Marinas  2009-05-30  40  #define CPUID_EXT_ISAR3	"c2, 3"
faa7bc51c11d5b Catalin Marinas  2009-05-30  41  #define CPUID_EXT_ISAR4	"c2, 4"
faa7bc51c11d5b Catalin Marinas  2009-05-30  42  #define CPUID_EXT_ISAR5	"c2, 5"

6fae9cdafc92ae Uwe Kleine-König 2013-05-06 43 #endif faa7bc51c11d5b Catalin Marinas 2009-05-30 44

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