diff mbox

[v9] QE: remove PPCisms for QE

Message ID 1480913809-23817-1-git-send-email-qiang.zhao@nxp.com (mailing list archive)
State Not Applicable
Delegated to: Scott Wood
Headers show

Commit Message

Qiang Zhao Dec. 5, 2016, 4:56 a.m. UTC
QE was supported on PowerPC, and dependent on PPC,
Now it is supported on other platforms. so remove PPCisms.

Signed-off-by: Zhao Qiang <qiang.zhao@nxp.com>
---
Changes for v2:
	- na
Changes for v3:
	- add NO_IRQ
Changes for v4:
	- modify spin_event_timeout to opencoded timeout loop
	- remove NO_IRQ
	- modify virq_to_hw to opencoed code
Changes for v5:
	- modify commit msg
	- modify depends of QUICC_ENGINE
	- add kerneldoc header for qe_issue_cmd
Changes for v6:
	- add dependency on FSL_SOC and PPC32 for drivers
	  depending on QUICC_ENGING but not available on ARM
Changes for v7:
	- split qeic part to another patch
	- rebase
Changes for v8:
	- include <asm/cpm.h> in ucc_uart
Changes for v9:
	- fix cast warning

 drivers/net/ethernet/freescale/Kconfig | 10 ++---
 drivers/soc/fsl/qe/Kconfig             |  2 +-
 drivers/soc/fsl/qe/qe.c                | 80 ++++++++++++++++++++--------------
 drivers/soc/fsl/qe/qe_io.c             | 42 ++++++++----------
 drivers/soc/fsl/qe/qe_tdm.c            |  8 ++--
 drivers/soc/fsl/qe/ucc.c               | 10 ++---
 drivers/soc/fsl/qe/ucc_fast.c          | 74 ++++++++++++++++---------------
 drivers/tty/serial/Kconfig             |  2 +-
 drivers/tty/serial/ucc_uart.c          |  1 +
 drivers/usb/gadget/udc/Kconfig         |  2 +-
 drivers/usb/host/Kconfig               |  2 +-
 include/soc/fsl/qe/qe.h                |  1 -
 12 files changed, 123 insertions(+), 111 deletions(-)

Comments

kernel test robot Dec. 5, 2016, 6:59 a.m. UTC | #1
Hi Zhao,

[auto build test ERROR on linus/master]
[also build test ERROR on v4.9-rc8]
[cannot apply to next-20161202]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Zhao-Qiang/QE-remove-PPCisms-for-QE/20161205-131352
config: i386-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

   In file included from drivers/soc/fsl/qe/qe_ic.c:31:0:
   include/soc/fsl/qe/qe_ic.h: In function 'qe_ic_cascade_low_ipic':
>> include/soc/fsl/qe/qe_ic.h:86:21: error: 'NO_IRQ' undeclared (first use in this function)
     if (cascade_irq != NO_IRQ)
                        ^~~~~~
   include/soc/fsl/qe/qe_ic.h:86:21: note: each undeclared identifier is reported only once for each function it appears in
   include/soc/fsl/qe/qe_ic.h: In function 'qe_ic_cascade_high_ipic':
   include/soc/fsl/qe/qe_ic.h:95:21: error: 'NO_IRQ' undeclared (first use in this function)
     if (cascade_irq != NO_IRQ)
                        ^~~~~~
   include/soc/fsl/qe/qe_ic.h: In function 'qe_ic_cascade_low_mpic':
   include/soc/fsl/qe/qe_ic.h:105:21: error: 'NO_IRQ' undeclared (first use in this function)
     if (cascade_irq != NO_IRQ)
                        ^~~~~~
   include/soc/fsl/qe/qe_ic.h: In function 'qe_ic_cascade_high_mpic':
   include/soc/fsl/qe/qe_ic.h:117:21: error: 'NO_IRQ' undeclared (first use in this function)
     if (cascade_irq != NO_IRQ)
                        ^~~~~~
   include/soc/fsl/qe/qe_ic.h: In function 'qe_ic_cascade_muxed_mpic':
   include/soc/fsl/qe/qe_ic.h:130:21: error: 'NO_IRQ' undeclared (first use in this function)
     if (cascade_irq == NO_IRQ)
                        ^~~~~~
   drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_read':
>> drivers/soc/fsl/qe/qe_ic.c:180:9: error: implicit declaration of function 'in_be32' [-Werror=implicit-function-declaration]
     return in_be32(base + (reg >> 2));
            ^~~~~~~
   drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_write':
>> drivers/soc/fsl/qe/qe_ic.c:186:2: error: implicit declaration of function 'out_be32' [-Werror=implicit-function-declaration]
     out_be32(base + (reg >> 2), value);
     ^~~~~~~~
   drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_get_low_irq':
>> drivers/soc/fsl/qe/qe_ic.c:299:10: error: 'NO_IRQ' undeclared (first use in this function)
      return NO_IRQ;
             ^~~~~~
   drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_get_high_irq':
   drivers/soc/fsl/qe/qe_ic.c:315:10: error: 'NO_IRQ' undeclared (first use in this function)
      return NO_IRQ;
             ^~~~~~
   drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_init':
   drivers/soc/fsl/qe/qe_ic.c:350:25: error: 'NO_IRQ' undeclared (first use in this function)
     if (qe_ic->virq_low == NO_IRQ) {
                            ^~~~~~
   drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_set_highest_priority':
>> drivers/soc/fsl/qe/qe_ic.c:392:21: error: implicit declaration of function 'virq_to_hw' [-Werror=implicit-function-declaration]
     unsigned int src = virq_to_hw(virq);
                        ^~~~~~~~~~
   cc1: some warnings being treated as errors

vim +/in_be32 +180 drivers/soc/fsl/qe/qe_ic.c

98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  174  		.pri_reg = QEIC_CIPYCC,
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  175  		},
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  176  };
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  177  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  178  static inline u32 qe_ic_read(volatile __be32  __iomem * base, unsigned int reg)
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  179  {
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03 @180  	return in_be32(base + (reg >> 2));
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  181  }
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  182  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  183  static inline void qe_ic_write(volatile __be32  __iomem * base, unsigned int reg,
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  184  			       u32 value)
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  185  {
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03 @186  	out_be32(base + (reg >> 2), value);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  187  }
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  188  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  189  static inline struct qe_ic *qe_ic_from_irq(unsigned int virq)
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  190  {
ec775d0e arch/powerpc/sysdev/qe_lib/qe_ic.c Thomas Gleixner     2011-03-25  191  	return irq_get_chip_data(virq);
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  192  }
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  193  
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  194  static inline struct qe_ic *qe_ic_from_irq_data(struct irq_data *d)
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  195  {
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  196  	return irq_data_get_irq_chip_data(d);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  197  }
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  198  
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  199  static void qe_ic_unmask_irq(struct irq_data *d)
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  200  {
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  201  	struct qe_ic *qe_ic = qe_ic_from_irq_data(d);
476eb491 arch/powerpc/sysdev/qe_lib/qe_ic.c Grant Likely        2011-05-04  202  	unsigned int src = irqd_to_hwirq(d);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  203  	unsigned long flags;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  204  	u32 temp;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  205  
43a5a01b arch/powerpc/sysdev/qe_lib/qe_ic.c Anton Vorontsov     2010-02-18  206  	raw_spin_lock_irqsave(&qe_ic_lock, flags);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  207  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  208  	temp = qe_ic_read(qe_ic->regs, qe_ic_info[src].mask_reg);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  209  	qe_ic_write(qe_ic->regs, qe_ic_info[src].mask_reg,
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  210  		    temp | qe_ic_info[src].mask);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  211  
43a5a01b arch/powerpc/sysdev/qe_lib/qe_ic.c Anton Vorontsov     2010-02-18  212  	raw_spin_unlock_irqrestore(&qe_ic_lock, flags);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  213  }
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  214  
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  215  static void qe_ic_mask_irq(struct irq_data *d)
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  216  {
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  217  	struct qe_ic *qe_ic = qe_ic_from_irq_data(d);
476eb491 arch/powerpc/sysdev/qe_lib/qe_ic.c Grant Likely        2011-05-04  218  	unsigned int src = irqd_to_hwirq(d);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  219  	unsigned long flags;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  220  	u32 temp;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  221  
43a5a01b arch/powerpc/sysdev/qe_lib/qe_ic.c Anton Vorontsov     2010-02-18  222  	raw_spin_lock_irqsave(&qe_ic_lock, flags);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  223  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  224  	temp = qe_ic_read(qe_ic->regs, qe_ic_info[src].mask_reg);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  225  	qe_ic_write(qe_ic->regs, qe_ic_info[src].mask_reg,
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  226  		    temp & ~qe_ic_info[src].mask);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  227  
2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  228  	/* Flush the above write before enabling interrupts; otherwise,
2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  229  	 * spurious interrupts will sometimes happen.  To be 100% sure
2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  230  	 * that the write has reached the device before interrupts are
2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  231  	 * enabled, the mask register would have to be read back; however,
2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  232  	 * this is not required for correctness, only to avoid wasting
2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  233  	 * time on a large number of spurious interrupts.  In testing,
2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  234  	 * a sync reduced the observed spurious interrupts to zero.
2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  235  	 */
2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  236  	mb();
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  237  
43a5a01b arch/powerpc/sysdev/qe_lib/qe_ic.c Anton Vorontsov     2010-02-18  238  	raw_spin_unlock_irqrestore(&qe_ic_lock, flags);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  239  }
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  240  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  241  static struct irq_chip qe_ic_irq_chip = {
b27df672 arch/powerpc/sysdev/qe_lib/qe_ic.c Thomas Gleixner     2009-11-18  242  	.name = "QEIC",
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  243  	.irq_unmask = qe_ic_unmask_irq,
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  244  	.irq_mask = qe_ic_mask_irq,
3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08  245  	.irq_mask_ack = qe_ic_mask_irq,
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  246  };
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  247  
ad3aedfb arch/powerpc/sysdev/qe_lib/qe_ic.c Marc Zyngier        2015-07-28  248  static int qe_ic_host_match(struct irq_domain *h, struct device_node *node,
ad3aedfb arch/powerpc/sysdev/qe_lib/qe_ic.c Marc Zyngier        2015-07-28  249  			    enum irq_domain_bus_token bus_token)
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  250  {
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  251  	/* Exact match, unless qe_ic node is NULL */
5d4c9bc7 arch/powerpc/sysdev/qe_lib/qe_ic.c Marc Zyngier        2015-10-13  252  	struct device_node *of_node = irq_domain_get_of_node(h);
5d4c9bc7 arch/powerpc/sysdev/qe_lib/qe_ic.c Marc Zyngier        2015-10-13  253  	return of_node == NULL || of_node == node;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  254  }
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  255  
bae1d8f1 arch/powerpc/sysdev/qe_lib/qe_ic.c Grant Likely        2012-02-14  256  static int qe_ic_host_map(struct irq_domain *h, unsigned int virq,
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  257  			  irq_hw_number_t hw)
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  258  {
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  259  	struct qe_ic *qe_ic = h->host_data;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  260  	struct irq_chip *chip;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  261  
c9ee69c5 drivers/soc/fsl/qe/qe_ic.c         Zhao Qiang          2016-01-21  262  	if (hw >= ARRAY_SIZE(qe_ic_info)) {
c9ee69c5 drivers/soc/fsl/qe/qe_ic.c         Zhao Qiang          2016-01-21  263  		pr_err("%s: Invalid hw irq number for QEIC\n", __func__);
c9ee69c5 drivers/soc/fsl/qe/qe_ic.c         Zhao Qiang          2016-01-21  264  		return -EINVAL;
c9ee69c5 drivers/soc/fsl/qe/qe_ic.c         Zhao Qiang          2016-01-21  265  	}
c9ee69c5 drivers/soc/fsl/qe/qe_ic.c         Zhao Qiang          2016-01-21  266  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  267  	if (qe_ic_info[hw].mask == 0) {
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  268  		printk(KERN_ERR "Can't map reserved IRQ\n");
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  269  		return -EINVAL;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  270  	}
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  271  	/* Default chip */
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  272  	chip = &qe_ic->hc_irq;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  273  
ec775d0e arch/powerpc/sysdev/qe_lib/qe_ic.c Thomas Gleixner     2011-03-25  274  	irq_set_chip_data(virq, qe_ic);
98488db9 arch/powerpc/sysdev/qe_lib/qe_ic.c Thomas Gleixner     2011-03-25  275  	irq_set_status_flags(virq, IRQ_LEVEL);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  276  
ec775d0e arch/powerpc/sysdev/qe_lib/qe_ic.c Thomas Gleixner     2011-03-25  277  	irq_set_chip_and_handler(virq, chip, handle_level_irq);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  278  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  279  	return 0;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  280  }
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  281  
202648a6 arch/powerpc/sysdev/qe_lib/qe_ic.c Krzysztof Kozlowski 2015-04-27  282  static const struct irq_domain_ops qe_ic_host_ops = {
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  283  	.match = qe_ic_host_match,
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  284  	.map = qe_ic_host_map,
ff8c3ab8 arch/powerpc/sysdev/qe_lib/qe_ic.c Grant Likely        2012-01-24  285  	.xlate = irq_domain_xlate_onetwocell,
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  286  };
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  287  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  288  /* Return an interrupt vector or NO_IRQ if no interrupt is pending. */
35a84c2f arch/powerpc/sysdev/qe_lib/qe_ic.c Olaf Hering         2006-10-07  289  unsigned int qe_ic_get_low_irq(struct qe_ic *qe_ic)
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  290  {
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  291  	int irq;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  292  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  293  	BUG_ON(qe_ic == NULL);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  294  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  295  	/* get the interrupt source vector. */
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  296  	irq = qe_ic_read(qe_ic->regs, QEIC_CIVEC) >> 26;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  297  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  298  	if (irq == 0)
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03 @299  		return NO_IRQ;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  300  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  301  	return irq_linear_revmap(qe_ic->irqhost, irq);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  302  }
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  303  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  304  /* Return an interrupt vector or NO_IRQ if no interrupt is pending. */
35a84c2f arch/powerpc/sysdev/qe_lib/qe_ic.c Olaf Hering         2006-10-07  305  unsigned int qe_ic_get_high_irq(struct qe_ic *qe_ic)
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  306  {
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  307  	int irq;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  308  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  309  	BUG_ON(qe_ic == NULL);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  310  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  311  	/* get the interrupt source vector. */
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  312  	irq = qe_ic_read(qe_ic->regs, QEIC_CHIVEC) >> 26;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  313  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  314  	if (irq == 0)
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03 @315  		return NO_IRQ;
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  316  
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  317  	return irq_linear_revmap(qe_ic->irqhost, irq);
98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  318  }

:::::: The code at line 180 was first introduced by commit
:::::: 9865853851313e0d94a4acde42d6f9d8070bb376 [POWERPC] Add QUICC Engine (QE) infrastructure

:::::: TO: Li Yang <leoli@freescale.com>
:::::: CC: Paul Mackerras <paulus@samba.org>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
Qiang Zhao Dec. 5, 2016, 7:10 a.m. UTC | #2
This patch depends on the patchset of QEIC as following links:
	http://patchwork.ozlabs.org/patch/675925/
	http://patchwork.ozlabs.org/patch/675926/
	http://patchwork.ozlabs.org/patch/675927/
	http://patchwork.ozlabs.org/patch/675928/

> -----Original Message-----
> From: kbuild test robot [mailto:lkp@intel.com]
> Sent: Monday, December 05, 2016 3:00 PM
> To: Qiang Zhao <qiang.zhao@nxp.com>
> Cc: kbuild-all@01.org; oss@buserror.net; balbi@kernel.org;
> gregkh@linuxfoundation.org; Xiaobo Xie <xiaobo.xie@nxp.com>; linux-
> kernel@vger.kernel.org; linuxppc-dev@lists.ozlabs.org; Qiang Zhao
> <qiang.zhao@nxp.com>
> Subject: Re: [PATCH v9] QE: remove PPCisms for QE
> 
> Hi Zhao,
> 
> [auto build test ERROR on linus/master]
> [also build test ERROR on v4.9-rc8]
> [cannot apply to next-20161202]
> [if your patch is applied to the wrong git tree, please drop us a note to help
> improve the system]
> 
> url:    https://github.com/0day-ci/linux/commits/Zhao-Qiang/QE-remove-
> PPCisms-for-QE/20161205-131352
> config: i386-allmodconfig (attached as .config)
> compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
> reproduce:
>         # save the attached .config to linux build tree
>         make ARCH=i386
> 
> All errors (new ones prefixed by >>):
> 
>    In file included from drivers/soc/fsl/qe/qe_ic.c:31:0:
>    include/soc/fsl/qe/qe_ic.h: In function 'qe_ic_cascade_low_ipic':
> >> include/soc/fsl/qe/qe_ic.h:86:21: error: 'NO_IRQ' undeclared (first use in
> this function)
>      if (cascade_irq != NO_IRQ)
>                         ^~~~~~
>    include/soc/fsl/qe/qe_ic.h:86:21: note: each undeclared identifier is reported
> only once for each function it appears in
>    include/soc/fsl/qe/qe_ic.h: In function 'qe_ic_cascade_high_ipic':
>    include/soc/fsl/qe/qe_ic.h:95:21: error: 'NO_IRQ' undeclared (first use in this
> function)
>      if (cascade_irq != NO_IRQ)
>                         ^~~~~~
>    include/soc/fsl/qe/qe_ic.h: In function 'qe_ic_cascade_low_mpic':
>    include/soc/fsl/qe/qe_ic.h:105:21: error: 'NO_IRQ' undeclared (first use in
> this function)
>      if (cascade_irq != NO_IRQ)
>                         ^~~~~~
>    include/soc/fsl/qe/qe_ic.h: In function 'qe_ic_cascade_high_mpic':
>    include/soc/fsl/qe/qe_ic.h:117:21: error: 'NO_IRQ' undeclared (first use in
> this function)
>      if (cascade_irq != NO_IRQ)
>                         ^~~~~~
>    include/soc/fsl/qe/qe_ic.h: In function 'qe_ic_cascade_muxed_mpic':
>    include/soc/fsl/qe/qe_ic.h:130:21: error: 'NO_IRQ' undeclared (first use in
> this function)
>      if (cascade_irq == NO_IRQ)
>                         ^~~~~~
>    drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_read':
> >> drivers/soc/fsl/qe/qe_ic.c:180:9: error: implicit declaration of function
> 'in_be32' [-Werror=implicit-function-declaration]
>      return in_be32(base + (reg >> 2));
>             ^~~~~~~
>    drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_write':
> >> drivers/soc/fsl/qe/qe_ic.c:186:2: error: implicit declaration of function
> 'out_be32' [-Werror=implicit-function-declaration]
>      out_be32(base + (reg >> 2), value);
>      ^~~~~~~~
>    drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_get_low_irq':
> >> drivers/soc/fsl/qe/qe_ic.c:299:10: error: 'NO_IRQ' undeclared (first use in
> this function)
>       return NO_IRQ;
>              ^~~~~~
>    drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_get_high_irq':
>    drivers/soc/fsl/qe/qe_ic.c:315:10: error: 'NO_IRQ' undeclared (first use in this
> function)
>       return NO_IRQ;
>              ^~~~~~
>    drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_init':
>    drivers/soc/fsl/qe/qe_ic.c:350:25: error: 'NO_IRQ' undeclared (first use in this
> function)
>      if (qe_ic->virq_low == NO_IRQ) {
>                             ^~~~~~
>    drivers/soc/fsl/qe/qe_ic.c: In function 'qe_ic_set_highest_priority':
> >> drivers/soc/fsl/qe/qe_ic.c:392:21: error: implicit declaration of function
> 'virq_to_hw' [-Werror=implicit-function-declaration]
>      unsigned int src = virq_to_hw(virq);
>                         ^~~~~~~~~~
>    cc1: some warnings being treated as errors
> 
> vim +/in_be32 +180 drivers/soc/fsl/qe/qe_ic.c
> 
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  174
> 	.pri_reg = QEIC_CIPYCC,
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  175
> 	},
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  176  };
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  177
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  178
> static inline u32 qe_ic_read(volatile __be32  __iomem * base, unsigned int reg)
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  179  {
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03 @180
> 	return in_be32(base + (reg >> 2));
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  181  }
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  182
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  183
> static inline void qe_ic_write(volatile __be32  __iomem * base, unsigned int
> reg,
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  184
> 		       u32 value)
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  185  {
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03 @186
> 	out_be32(base + (reg >> 2), value);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  187  }
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  188
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  189
> static inline struct qe_ic *qe_ic_from_irq(unsigned int virq)
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  190  {
> ec775d0e arch/powerpc/sysdev/qe_lib/qe_ic.c Thomas Gleixner     2011-03-25
> 191  	return irq_get_chip_data(virq);
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 192  }
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 193
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 194  static inline struct qe_ic *qe_ic_from_irq_data(struct irq_data *d)
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 195  {
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 196  	return irq_data_get_irq_chip_data(d);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  197  }
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  198
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 199  static void qe_ic_unmask_irq(struct irq_data *d)
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  200  {
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 201  	struct qe_ic *qe_ic = qe_ic_from_irq_data(d);
> 476eb491 arch/powerpc/sysdev/qe_lib/qe_ic.c Grant Likely        2011-05-04  202
> 	unsigned int src = irqd_to_hwirq(d);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  203
> 	unsigned long flags;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  204
> 	u32 temp;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  205
> 43a5a01b arch/powerpc/sysdev/qe_lib/qe_ic.c Anton Vorontsov     2010-02-18
> 206  	raw_spin_lock_irqsave(&qe_ic_lock, flags);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  207
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  208
> 	temp = qe_ic_read(qe_ic->regs, qe_ic_info[src].mask_reg);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  209
> 	qe_ic_write(qe_ic->regs, qe_ic_info[src].mask_reg,
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  210
> 	    temp | qe_ic_info[src].mask);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  211
> 43a5a01b arch/powerpc/sysdev/qe_lib/qe_ic.c Anton Vorontsov     2010-02-18
> 212  	raw_spin_unlock_irqrestore(&qe_ic_lock, flags);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  213  }
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  214
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 215  static void qe_ic_mask_irq(struct irq_data *d)
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  216  {
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 217  	struct qe_ic *qe_ic = qe_ic_from_irq_data(d);
> 476eb491 arch/powerpc/sysdev/qe_lib/qe_ic.c Grant Likely        2011-05-04  218
> 	unsigned int src = irqd_to_hwirq(d);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  219
> 	unsigned long flags;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  220
> 	u32 temp;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  221
> 43a5a01b arch/powerpc/sysdev/qe_lib/qe_ic.c Anton Vorontsov     2010-02-18
> 222  	raw_spin_lock_irqsave(&qe_ic_lock, flags);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  223
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  224
> 	temp = qe_ic_read(qe_ic->regs, qe_ic_info[src].mask_reg);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  225
> 	qe_ic_write(qe_ic->regs, qe_ic_info[src].mask_reg,
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  226
> 	    temp & ~qe_ic_info[src].mask);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  227
> 2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  228
> 	/* Flush the above write before enabling interrupts; otherwise,
> 2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  229
> 	 * spurious interrupts will sometimes happen.  To be 100% sure
> 2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  230
> 	 * that the write has reached the device before interrupts are
> 2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  231
> 	 * enabled, the mask register would have to be read back; however,
> 2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  232
> 	 * this is not required for correctness, only to avoid wasting
> 2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  233
> 	 * time on a large number of spurious interrupts.  In testing,
> 2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  234
> 	 * a sync reduced the observed spurious interrupts to zero.
> 2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  235
> 	 */
> 2c1d2f34 arch/powerpc/sysdev/qe_lib/qe_ic.c Scott Wood          2006-12-06  236
> 	mb();
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  237
> 43a5a01b arch/powerpc/sysdev/qe_lib/qe_ic.c Anton Vorontsov     2010-02-18
> 238  	raw_spin_unlock_irqrestore(&qe_ic_lock, flags);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  239  }
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  240
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  241
> static struct irq_chip qe_ic_irq_chip = {
> b27df672 arch/powerpc/sysdev/qe_lib/qe_ic.c Thomas Gleixner     2009-11-18
> 242  	.name = "QEIC",
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 243  	.irq_unmask = qe_ic_unmask_irq,
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 244  	.irq_mask = qe_ic_mask_irq,
> 3a0adfab arch/powerpc/sysdev/qe_lib/qe_ic.c Lennert Buytenhek   2011-03-08
> 245  	.irq_mask_ack = qe_ic_mask_irq,
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  246  };
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  247
> ad3aedfb arch/powerpc/sysdev/qe_lib/qe_ic.c Marc Zyngier        2015-07-28
> 248  static int qe_ic_host_match(struct irq_domain *h, struct device_node
> *node,
> ad3aedfb arch/powerpc/sysdev/qe_lib/qe_ic.c Marc Zyngier        2015-07-28
> 249  			    enum irq_domain_bus_token bus_token)
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  250  {
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  251
> 	/* Exact match, unless qe_ic node is NULL */
> 5d4c9bc7 arch/powerpc/sysdev/qe_lib/qe_ic.c Marc Zyngier        2015-10-13
> 252  	struct device_node *of_node = irq_domain_get_of_node(h);
> 5d4c9bc7 arch/powerpc/sysdev/qe_lib/qe_ic.c Marc Zyngier        2015-10-13
> 253  	return of_node == NULL || of_node == node;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  254  }
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  255
> bae1d8f1 arch/powerpc/sysdev/qe_lib/qe_ic.c Grant Likely        2012-02-14  256
> static int qe_ic_host_map(struct irq_domain *h, unsigned int virq,
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  257
> 		  irq_hw_number_t hw)
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  258  {
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  259
> 	struct qe_ic *qe_ic = h->host_data;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  260
> 	struct irq_chip *chip;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  261
> c9ee69c5 drivers/soc/fsl/qe/qe_ic.c         Zhao Qiang          2016-01-21  262
> 	if (hw >= ARRAY_SIZE(qe_ic_info)) {
> c9ee69c5 drivers/soc/fsl/qe/qe_ic.c         Zhao Qiang          2016-01-21  263
> 	pr_err("%s: Invalid hw irq number for QEIC\n", __func__);
> c9ee69c5 drivers/soc/fsl/qe/qe_ic.c         Zhao Qiang          2016-01-21  264
> 	return -EINVAL;
> c9ee69c5 drivers/soc/fsl/qe/qe_ic.c         Zhao Qiang          2016-01-21  265  	}
> c9ee69c5 drivers/soc/fsl/qe/qe_ic.c         Zhao Qiang          2016-01-21  266
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  267
> 	if (qe_ic_info[hw].mask == 0) {
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  268
> 	printk(KERN_ERR "Can't map reserved IRQ\n");
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  269
> 	return -EINVAL;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  270  	}
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  271
> 	/* Default chip */
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  272
> 	chip = &qe_ic->hc_irq;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  273
> ec775d0e arch/powerpc/sysdev/qe_lib/qe_ic.c Thomas Gleixner     2011-03-25
> 274  	irq_set_chip_data(virq, qe_ic);
> 98488db9 arch/powerpc/sysdev/qe_lib/qe_ic.c Thomas Gleixner     2011-03-25
> 275  	irq_set_status_flags(virq, IRQ_LEVEL);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  276
> ec775d0e arch/powerpc/sysdev/qe_lib/qe_ic.c Thomas Gleixner     2011-03-25
> 277  	irq_set_chip_and_handler(virq, chip, handle_level_irq);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  278
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  279
> 	return 0;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  280  }
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  281
> 202648a6 arch/powerpc/sysdev/qe_lib/qe_ic.c Krzysztof Kozlowski 2015-04-27
> 282  static const struct irq_domain_ops qe_ic_host_ops = {
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  283
> 	.match = qe_ic_host_match,
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  284
> 	.map = qe_ic_host_map,
> ff8c3ab8 arch/powerpc/sysdev/qe_lib/qe_ic.c Grant Likely        2012-01-24  285
> 	.xlate = irq_domain_xlate_onetwocell,
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  286  };
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  287
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  288  /*
> Return an interrupt vector or NO_IRQ if no interrupt is pending. */
> 35a84c2f arch/powerpc/sysdev/qe_lib/qe_ic.c Olaf Hering         2006-10-07  289
> unsigned int qe_ic_get_low_irq(struct qe_ic *qe_ic)
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  290  {
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  291
> 	int irq;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  292
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  293
> 	BUG_ON(qe_ic == NULL);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  294
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  295
> 	/* get the interrupt source vector. */
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  296
> 	irq = qe_ic_read(qe_ic->regs, QEIC_CIVEC) >> 26;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  297
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  298
> 	if (irq == 0)
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03 @299
> 	return NO_IRQ;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  300
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  301
> 	return irq_linear_revmap(qe_ic->irqhost, irq);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  302  }
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  303
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  304  /*
> Return an interrupt vector or NO_IRQ if no interrupt is pending. */
> 35a84c2f arch/powerpc/sysdev/qe_lib/qe_ic.c Olaf Hering         2006-10-07  305
> unsigned int qe_ic_get_high_irq(struct qe_ic *qe_ic)
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  306  {
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  307
> 	int irq;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  308
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  309
> 	BUG_ON(qe_ic == NULL);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  310
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  311
> 	/* get the interrupt source vector. */
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  312
> 	irq = qe_ic_read(qe_ic->regs, QEIC_CHIVEC) >> 26;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  313
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  314
> 	if (irq == 0)
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03 @315
> 	return NO_IRQ;
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  316
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  317
> 	return irq_linear_revmap(qe_ic->irqhost, irq);
> 98658538 arch/powerpc/sysdev/qe_lib/qe_ic.c Li Yang             2006-10-03  318  }
> 
> :::::: The code at line 180 was first introduced by commit
> :::::: 9865853851313e0d94a4acde42d6f9d8070bb376 [POWERPC] Add QUICC
> Engine (QE) infrastructure
> 
> :::::: TO: Li Yang <leoli@freescale.com>
> :::::: CC: Paul Mackerras <paulus@samba.org>
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
diff mbox

Patch

diff --git a/drivers/net/ethernet/freescale/Kconfig b/drivers/net/ethernet/freescale/Kconfig
index d1ca45f..6677aff 100644
--- a/drivers/net/ethernet/freescale/Kconfig
+++ b/drivers/net/ethernet/freescale/Kconfig
@@ -5,10 +5,10 @@ 
 config NET_VENDOR_FREESCALE
 	bool "Freescale devices"
 	default y
-	depends on FSL_SOC || QUICC_ENGINE || CPM1 || CPM2 || PPC_MPC512x || \
-		   M523x || M527x || M5272 || M528x || M520x || M532x || \
-		   ARCH_MXC || ARCH_MXS || (PPC_MPC52xx && PPC_BESTCOMM) || \
-		   ARCH_LAYERSCAPE
+	depends on FSL_SOC || (QUICC_ENGINE && PPC32) || CPM1 || CPM2 || \
+		   PPC_MPC512x || M523x || M527x || M5272 || M528x || M520x || \
+		   M532x || ARCH_MXC || ARCH_MXS || \
+		   (PPC_MPC52xx && PPC_BESTCOMM) || ARCH_LAYERSCAPE
 	---help---
 	  If you have a network (Ethernet) card belonging to this class, say Y.
 
@@ -72,7 +72,7 @@  config FSL_XGMAC_MDIO
 
 config UCC_GETH
 	tristate "Freescale QE Gigabit Ethernet"
-	depends on QUICC_ENGINE
+	depends on QUICC_ENGINE && FSL_SOC && PPC32
 	select FSL_PQ_MDIO
 	select PHYLIB
 	---help---
diff --git a/drivers/soc/fsl/qe/Kconfig b/drivers/soc/fsl/qe/Kconfig
index 73a2e08..b26b643 100644
--- a/drivers/soc/fsl/qe/Kconfig
+++ b/drivers/soc/fsl/qe/Kconfig
@@ -4,7 +4,7 @@ 
 
 config QUICC_ENGINE
 	bool "Freescale QUICC Engine (QE) Support"
-	depends on FSL_SOC && PPC32
+	depends on OF && HAS_IOMEM
 	select GENERIC_ALLOCATOR
 	select CRC32
 	help
diff --git a/drivers/soc/fsl/qe/qe.c b/drivers/soc/fsl/qe/qe.c
index 2707a82..2b53e85 100644
--- a/drivers/soc/fsl/qe/qe.c
+++ b/drivers/soc/fsl/qe/qe.c
@@ -33,8 +33,6 @@ 
 #include <asm/pgtable.h>
 #include <soc/fsl/qe/immap_qe.h>
 #include <soc/fsl/qe/qe.h>
-#include <asm/prom.h>
-#include <asm/rheap.h>
 
 static void qe_snums_init(void);
 static int qe_sdma_init(void);
@@ -109,15 +107,27 @@  void qe_reset(void)
 		panic("sdma init failed!");
 }
 
+/* issue commands to QE, return 0 on success while -EIO on error
+ *
+ * @cmd: the command code, should be QE_INIT_TX_RX, QE_STOP_TX and so on
+ * @device: which sub-block will run the command, QE_CR_SUBBLOCK_UCCFAST1 - 8
+ * , QE_CR_SUBBLOCK_UCCSLOW1 - 8, QE_CR_SUBBLOCK_MCC1 - 3,
+ * QE_CR_SUBBLOCK_IDMA1 - 4 and such on.
+ * @mcn_protocol: specifies mode for the command for non-MCC, should be
+ * QE_CR_PROTOCOL_HDLC_TRANSPARENT, QE_CR_PROTOCOL_QMC, QE_CR_PROTOCOL_UART
+ * and such on.
+ * @cmd_input: command related data.
+ */
 int qe_issue_cmd(u32 cmd, u32 device, u8 mcn_protocol, u32 cmd_input)
 {
 	unsigned long flags;
 	u8 mcn_shift = 0, dev_shift = 0;
-	u32 ret;
+	int ret;
+	int i;
 
 	spin_lock_irqsave(&qe_lock, flags);
 	if (cmd == QE_RESET) {
-		out_be32(&qe_immr->cp.cecr, (u32) (cmd | QE_CR_FLG));
+		iowrite32be((cmd | QE_CR_FLG), &qe_immr->cp.cecr);
 	} else {
 		if (cmd == QE_ASSIGN_PAGE) {
 			/* Here device is the SNUM, not sub-block */
@@ -134,20 +144,26 @@  int qe_issue_cmd(u32 cmd, u32 device, u8 mcn_protocol, u32 cmd_input)
 				mcn_shift = QE_CR_MCN_NORMAL_SHIFT;
 		}
 
-		out_be32(&qe_immr->cp.cecdr, cmd_input);
-		out_be32(&qe_immr->cp.cecr,
-			 (cmd | QE_CR_FLG | ((u32) device << dev_shift) | (u32)
-			  mcn_protocol << mcn_shift));
+		iowrite32be(cmd_input, &qe_immr->cp.cecdr);
+		iowrite32be((cmd | QE_CR_FLG | ((u32)device << dev_shift) |
+			    (u32)mcn_protocol << mcn_shift), &qe_immr->cp.cecr);
 	}
 
 	/* wait for the QE_CR_FLG to clear */
-	ret = spin_event_timeout((in_be32(&qe_immr->cp.cecr) & QE_CR_FLG) == 0,
-			   100, 0);
+	ret = -EIO;
+	for (i = 0; i < 100; i++) {
+		if ((ioread32be(&qe_immr->cp.cecr) & QE_CR_FLG) == 0) {
+			ret = 0;
+			break;
+		}
+		udelay(1);
+	}
+
 	/* On timeout (e.g. failure), the expression will be false (ret == 0),
 	   otherwise it will be true (ret == 1). */
 	spin_unlock_irqrestore(&qe_lock, flags);
 
-	return ret == 1;
+	return ret;
 }
 EXPORT_SYMBOL(qe_issue_cmd);
 
@@ -166,8 +182,8 @@  static unsigned int brg_clk = 0;
 unsigned int qe_get_brg_clk(void)
 {
 	struct device_node *qe;
-	int size;
-	const u32 *prop;
+	u32 val;
+	int ret;
 
 	if (brg_clk)
 		return brg_clk;
@@ -179,9 +195,9 @@  unsigned int qe_get_brg_clk(void)
 			return brg_clk;
 	}
 
-	prop = of_get_property(qe, "brg-frequency", &size);
-	if (prop && size == sizeof(*prop))
-		brg_clk = *prop;
+	ret = of_property_read_u32(qe, "brg-frequency", &val);
+	if (!ret)
+		brg_clk = val;
 
 	of_node_put(qe);
 
@@ -221,7 +237,7 @@  int qe_setbrg(enum qe_clock brg, unsigned int rate, unsigned int multiplier)
 	tempval = ((divisor - 1) << QE_BRGC_DIVISOR_SHIFT) |
 		QE_BRGC_ENABLE | div16;
 
-	out_be32(&qe_immr->brg.brgc[brg - QE_BRG1], tempval);
+	iowrite32be(tempval, &qe_immr->brg.brgc[brg - QE_BRG1]);
 
 	return 0;
 }
@@ -355,9 +371,9 @@  static int qe_sdma_init(void)
 			return -ENOMEM;
 	}
 
-	out_be32(&sdma->sdebcr, (u32) sdma_buf_offset & QE_SDEBCR_BA_MASK);
- 	out_be32(&sdma->sdmr, (QE_SDMR_GLB_1_MSK |
- 					(0x1 << QE_SDMR_CEN_SHIFT)));
+	iowrite32be((u32)sdma_buf_offset & QE_SDEBCR_BA_MASK, &sdma->sdebcr);
+	iowrite32be((QE_SDMR_GLB_1_MSK | (0x1 << QE_SDMR_CEN_SHIFT)),
+		    &sdma->sdmr);
 
 	return 0;
 }
@@ -395,14 +411,14 @@  static void qe_upload_microcode(const void *base,
 			"uploading microcode '%s'\n", ucode->id);
 
 	/* Use auto-increment */
-	out_be32(&qe_immr->iram.iadd, be32_to_cpu(ucode->iram_offset) |
-		QE_IRAM_IADD_AIE | QE_IRAM_IADD_BADDR);
+	iowrite32be(be32_to_cpu(ucode->iram_offset) | QE_IRAM_IADD_AIE |
+		    QE_IRAM_IADD_BADDR, &qe_immr->iram.iadd);
 
 	for (i = 0; i < be32_to_cpu(ucode->count); i++)
-		out_be32(&qe_immr->iram.idata, be32_to_cpu(code[i]));
+		iowrite32be(be32_to_cpu(code[i]), &qe_immr->iram.idata);
 	
 	/* Set I-RAM Ready Register */
-	out_be32(&qe_immr->iram.iready, be32_to_cpu(QE_IRAM_READY));
+	iowrite32be(be32_to_cpu(QE_IRAM_READY), &qe_immr->iram.iready);
 }
 
 /*
@@ -487,7 +503,7 @@  int qe_upload_firmware(const struct qe_firmware *firmware)
 	 * If the microcode calls for it, split the I-RAM.
 	 */
 	if (!firmware->split)
-		setbits16(&qe_immr->cp.cercr, QE_CP_CERCR_CIR);
+		qe_setbits16(&qe_immr->cp.cercr, QE_CP_CERCR_CIR);
 
 	if (firmware->soc.model)
 		printk(KERN_INFO
@@ -521,11 +537,11 @@  int qe_upload_firmware(const struct qe_firmware *firmware)
 			u32 trap = be32_to_cpu(ucode->traps[j]);
 
 			if (trap)
-				out_be32(&qe_immr->rsp[i].tibcr[j], trap);
+				iowrite32be(trap, &qe_immr->rsp[i].tibcr[j]);
 		}
 
 		/* Enable traps */
-		out_be32(&qe_immr->rsp[i].eccr, be32_to_cpu(ucode->eccr));
+		iowrite32be(be32_to_cpu(ucode->eccr), &qe_immr->rsp[i].eccr);
 	}
 
 	qe_firmware_uploaded = 1;
@@ -644,9 +660,9 @@  EXPORT_SYMBOL(qe_get_num_of_risc);
 unsigned int qe_get_num_of_snums(void)
 {
 	struct device_node *qe;
-	int size;
 	unsigned int num_of_snums;
-	const u32 *prop;
+	u32 val;
+	int ret;
 
 	num_of_snums = 28; /* The default number of snum for threads is 28 */
 	qe = of_find_compatible_node(NULL, NULL, "fsl,qe");
@@ -660,9 +676,9 @@  unsigned int qe_get_num_of_snums(void)
 			return num_of_snums;
 	}
 
-	prop = of_get_property(qe, "fsl,qe-num-snums", &size);
-	if (prop && size == sizeof(*prop)) {
-		num_of_snums = *prop;
+	ret = of_property_read_u32(qe, "fsl,qe-num-snums", &val);
+	if (!ret) {
+		num_of_snums = val;
 		if ((num_of_snums < 28) || (num_of_snums > QE_NUM_OF_SNUM)) {
 			/* No QE ever has fewer than 28 SNUMs */
 			pr_err("QE: number of snum is invalid\n");
diff --git a/drivers/soc/fsl/qe/qe_io.c b/drivers/soc/fsl/qe/qe_io.c
index 7ae59ab..8966e8b 100644
--- a/drivers/soc/fsl/qe/qe_io.c
+++ b/drivers/soc/fsl/qe/qe_io.c
@@ -22,8 +22,6 @@ 
 
 #include <asm/io.h>
 #include <soc/fsl/qe/qe.h>
-#include <asm/prom.h>
-#include <sysdev/fsl_soc.h>
 
 #undef DEBUG
 
@@ -61,16 +59,16 @@  void __par_io_config_pin(struct qe_pio_regs __iomem *par_io, u8 pin, int dir,
 	pin_mask1bit = (u32) (1 << (QE_PIO_PINS - (pin + 1)));
 
 	/* Set open drain, if required */
-	tmp_val = in_be32(&par_io->cpodr);
+	tmp_val = ioread32be(&par_io->cpodr);
 	if (open_drain)
-		out_be32(&par_io->cpodr, pin_mask1bit | tmp_val);
+		iowrite32be(pin_mask1bit | tmp_val, &par_io->cpodr);
 	else
-		out_be32(&par_io->cpodr, ~pin_mask1bit & tmp_val);
+		iowrite32be(~pin_mask1bit & tmp_val, &par_io->cpodr);
 
 	/* define direction */
 	tmp_val = (pin > (QE_PIO_PINS / 2) - 1) ?
-		in_be32(&par_io->cpdir2) :
-		in_be32(&par_io->cpdir1);
+		ioread32be(&par_io->cpdir2) :
+		ioread32be(&par_io->cpdir1);
 
 	/* get all bits mask for 2 bit per port */
 	pin_mask2bits = (u32) (0x3 << (QE_PIO_PINS -
@@ -82,34 +80,30 @@  void __par_io_config_pin(struct qe_pio_regs __iomem *par_io, u8 pin, int dir,
 
 	/* clear and set 2 bits mask */
 	if (pin > (QE_PIO_PINS / 2) - 1) {
-		out_be32(&par_io->cpdir2,
-			 ~pin_mask2bits & tmp_val);
+		iowrite32be(~pin_mask2bits & tmp_val, &par_io->cpdir2);
 		tmp_val &= ~pin_mask2bits;
-		out_be32(&par_io->cpdir2, new_mask2bits | tmp_val);
+		iowrite32be(new_mask2bits | tmp_val, &par_io->cpdir2);
 	} else {
-		out_be32(&par_io->cpdir1,
-			 ~pin_mask2bits & tmp_val);
+		iowrite32be(~pin_mask2bits & tmp_val, &par_io->cpdir1);
 		tmp_val &= ~pin_mask2bits;
-		out_be32(&par_io->cpdir1, new_mask2bits | tmp_val);
+		iowrite32be(new_mask2bits | tmp_val, &par_io->cpdir1);
 	}
 	/* define pin assignment */
 	tmp_val = (pin > (QE_PIO_PINS / 2) - 1) ?
-		in_be32(&par_io->cppar2) :
-		in_be32(&par_io->cppar1);
+		ioread32be(&par_io->cppar2) :
+		ioread32be(&par_io->cppar1);
 
 	new_mask2bits = (u32) (assignment << (QE_PIO_PINS -
 			(pin % (QE_PIO_PINS / 2) + 1) * 2));
 	/* clear and set 2 bits mask */
 	if (pin > (QE_PIO_PINS / 2) - 1) {
-		out_be32(&par_io->cppar2,
-			 ~pin_mask2bits & tmp_val);
+		iowrite32be(~pin_mask2bits & tmp_val, &par_io->cppar2);
 		tmp_val &= ~pin_mask2bits;
-		out_be32(&par_io->cppar2, new_mask2bits | tmp_val);
+		iowrite32be(new_mask2bits | tmp_val, &par_io->cppar2);
 	} else {
-		out_be32(&par_io->cppar1,
-			 ~pin_mask2bits & tmp_val);
+		iowrite32be(~pin_mask2bits & tmp_val, &par_io->cppar1);
 		tmp_val &= ~pin_mask2bits;
-		out_be32(&par_io->cppar1, new_mask2bits | tmp_val);
+		iowrite32be(new_mask2bits | tmp_val, &par_io->cppar1);
 	}
 }
 EXPORT_SYMBOL(__par_io_config_pin);
@@ -137,12 +131,12 @@  int par_io_data_set(u8 port, u8 pin, u8 val)
 	/* calculate pin location */
 	pin_mask = (u32) (1 << (QE_PIO_PINS - 1 - pin));
 
-	tmp_val = in_be32(&par_io[port].cpdata);
+	tmp_val = ioread32be(&par_io[port].cpdata);
 
 	if (val == 0)		/* clear */
-		out_be32(&par_io[port].cpdata, ~pin_mask & tmp_val);
+		iowrite32be(~pin_mask & tmp_val, &par_io[port].cpdata);
 	else			/* set */
-		out_be32(&par_io[port].cpdata, pin_mask | tmp_val);
+		iowrite32be(pin_mask | tmp_val, &par_io[port].cpdata);
 
 	return 0;
 }
diff --git a/drivers/soc/fsl/qe/qe_tdm.c b/drivers/soc/fsl/qe/qe_tdm.c
index a1048b4..818e679 100644
--- a/drivers/soc/fsl/qe/qe_tdm.c
+++ b/drivers/soc/fsl/qe/qe_tdm.c
@@ -227,10 +227,10 @@  void ucc_tdm_init(struct ucc_tdm *utdm, struct ucc_tdm_info *ut_info)
 				    &siram[siram_entry_id * 32 + 0x200 +  i]);
 	}
 
-	setbits16(&siram[(siram_entry_id * 32) + (utdm->num_of_ts - 1)],
-		  SIR_LAST);
-	setbits16(&siram[(siram_entry_id * 32) + 0x200 + (utdm->num_of_ts - 1)],
-		  SIR_LAST);
+	qe_setbits16(&siram[(siram_entry_id * 32) + (utdm->num_of_ts - 1)],
+		     SIR_LAST);
+	qe_setbits16(&siram[(siram_entry_id * 32) + 0x200 +
+		     (utdm->num_of_ts - 1)], SIR_LAST);
 
 	/* Set SIxMR register */
 	sixmr = SIMR_SAD(siram_entry_id);
diff --git a/drivers/soc/fsl/qe/ucc.c b/drivers/soc/fsl/qe/ucc.c
index c646d87..bc64b83 100644
--- a/drivers/soc/fsl/qe/ucc.c
+++ b/drivers/soc/fsl/qe/ucc.c
@@ -39,7 +39,7 @@  int ucc_set_qe_mux_mii_mng(unsigned int ucc_num)
 		return -EINVAL;
 
 	spin_lock_irqsave(&cmxgcr_lock, flags);
-	clrsetbits_be32(&qe_immr->qmx.cmxgcr, QE_CMXGCR_MII_ENET_MNG,
+	qe_clrsetbits32(&qe_immr->qmx.cmxgcr, QE_CMXGCR_MII_ENET_MNG,
 		ucc_num << QE_CMXGCR_MII_ENET_MNG_SHIFT);
 	spin_unlock_irqrestore(&cmxgcr_lock, flags);
 
@@ -84,7 +84,7 @@  int ucc_set_type(unsigned int ucc_num, enum ucc_speed_type speed)
 		return -EINVAL;
 	}
 
-	clrsetbits_8(guemr, UCC_GUEMR_MODE_MASK,
+	qe_clrsetbits8(guemr, UCC_GUEMR_MODE_MASK,
 		UCC_GUEMR_SET_RESERVED3 | speed);
 
 	return 0;
@@ -113,9 +113,9 @@  int ucc_mux_set_grant_tsa_bkpt(unsigned int ucc_num, int set, u32 mask)
 	get_cmxucr_reg(ucc_num, &cmxucr, &reg_num, &shift);
 
 	if (set)
-		setbits32(cmxucr, mask << shift);
+		qe_setbits32(cmxucr, mask << shift);
 	else
-		clrbits32(cmxucr, mask << shift);
+		qe_clrbits32(cmxucr, mask << shift);
 
 	return 0;
 }
@@ -211,7 +211,7 @@  int ucc_set_qe_mux_rxtx(unsigned int ucc_num, enum qe_clock clock,
 	if (mode == COMM_DIR_RX)
 		shift += 4;
 
-	clrsetbits_be32(cmxucr, QE_CMXUCR_TX_CLK_SRC_MASK << shift,
+	qe_clrsetbits32(cmxucr, QE_CMXUCR_TX_CLK_SRC_MASK << shift,
 		clock_bits << shift);
 
 	return 0;
diff --git a/drivers/soc/fsl/qe/ucc_fast.c b/drivers/soc/fsl/qe/ucc_fast.c
index 83d8d16..5115e93 100644
--- a/drivers/soc/fsl/qe/ucc_fast.c
+++ b/drivers/soc/fsl/qe/ucc_fast.c
@@ -33,41 +33,41 @@  void ucc_fast_dump_regs(struct ucc_fast_private * uccf)
 	printk(KERN_INFO "Base address: 0x%p\n", uccf->uf_regs);
 
 	printk(KERN_INFO "gumr  : addr=0x%p, val=0x%08x\n",
-		  &uccf->uf_regs->gumr, in_be32(&uccf->uf_regs->gumr));
+	       &uccf->uf_regs->gumr, ioread32be(&uccf->uf_regs->gumr));
 	printk(KERN_INFO "upsmr : addr=0x%p, val=0x%08x\n",
-		  &uccf->uf_regs->upsmr, in_be32(&uccf->uf_regs->upsmr));
+		  &uccf->uf_regs->upsmr, ioread32be(&uccf->uf_regs->upsmr));
 	printk(KERN_INFO "utodr : addr=0x%p, val=0x%04x\n",
-		  &uccf->uf_regs->utodr, in_be16(&uccf->uf_regs->utodr));
+		  &uccf->uf_regs->utodr, ioread16be(&uccf->uf_regs->utodr));
 	printk(KERN_INFO "udsr  : addr=0x%p, val=0x%04x\n",
-		  &uccf->uf_regs->udsr, in_be16(&uccf->uf_regs->udsr));
+		  &uccf->uf_regs->udsr, ioread16be(&uccf->uf_regs->udsr));
 	printk(KERN_INFO "ucce  : addr=0x%p, val=0x%08x\n",
-		  &uccf->uf_regs->ucce, in_be32(&uccf->uf_regs->ucce));
+		  &uccf->uf_regs->ucce, ioread32be(&uccf->uf_regs->ucce));
 	printk(KERN_INFO "uccm  : addr=0x%p, val=0x%08x\n",
-		  &uccf->uf_regs->uccm, in_be32(&uccf->uf_regs->uccm));
+		  &uccf->uf_regs->uccm, ioread32be(&uccf->uf_regs->uccm));
 	printk(KERN_INFO "uccs  : addr=0x%p, val=0x%02x\n",
-		  &uccf->uf_regs->uccs, in_8(&uccf->uf_regs->uccs));
+		  &uccf->uf_regs->uccs, ioread8(&uccf->uf_regs->uccs));
 	printk(KERN_INFO "urfb  : addr=0x%p, val=0x%08x\n",
-		  &uccf->uf_regs->urfb, in_be32(&uccf->uf_regs->urfb));
+		  &uccf->uf_regs->urfb, ioread32be(&uccf->uf_regs->urfb));
 	printk(KERN_INFO "urfs  : addr=0x%p, val=0x%04x\n",
-		  &uccf->uf_regs->urfs, in_be16(&uccf->uf_regs->urfs));
+		  &uccf->uf_regs->urfs, ioread16be(&uccf->uf_regs->urfs));
 	printk(KERN_INFO "urfet : addr=0x%p, val=0x%04x\n",
-		  &uccf->uf_regs->urfet, in_be16(&uccf->uf_regs->urfet));
+		  &uccf->uf_regs->urfet, ioread16be(&uccf->uf_regs->urfet));
 	printk(KERN_INFO "urfset: addr=0x%p, val=0x%04x\n",
-		  &uccf->uf_regs->urfset, in_be16(&uccf->uf_regs->urfset));
+		  &uccf->uf_regs->urfset, ioread16be(&uccf->uf_regs->urfset));
 	printk(KERN_INFO "utfb  : addr=0x%p, val=0x%08x\n",
-		  &uccf->uf_regs->utfb, in_be32(&uccf->uf_regs->utfb));
+		  &uccf->uf_regs->utfb, ioread32be(&uccf->uf_regs->utfb));
 	printk(KERN_INFO "utfs  : addr=0x%p, val=0x%04x\n",
-		  &uccf->uf_regs->utfs, in_be16(&uccf->uf_regs->utfs));
+		  &uccf->uf_regs->utfs, ioread16be(&uccf->uf_regs->utfs));
 	printk(KERN_INFO "utfet : addr=0x%p, val=0x%04x\n",
-		  &uccf->uf_regs->utfet, in_be16(&uccf->uf_regs->utfet));
+		  &uccf->uf_regs->utfet, ioread16be(&uccf->uf_regs->utfet));
 	printk(KERN_INFO "utftt : addr=0x%p, val=0x%04x\n",
-		  &uccf->uf_regs->utftt, in_be16(&uccf->uf_regs->utftt));
+		  &uccf->uf_regs->utftt, ioread16be(&uccf->uf_regs->utftt));
 	printk(KERN_INFO "utpt  : addr=0x%p, val=0x%04x\n",
-		  &uccf->uf_regs->utpt, in_be16(&uccf->uf_regs->utpt));
+		  &uccf->uf_regs->utpt, ioread16be(&uccf->uf_regs->utpt));
 	printk(KERN_INFO "urtry : addr=0x%p, val=0x%08x\n",
-		  &uccf->uf_regs->urtry, in_be32(&uccf->uf_regs->urtry));
+		  &uccf->uf_regs->urtry, ioread32be(&uccf->uf_regs->urtry));
 	printk(KERN_INFO "guemr : addr=0x%p, val=0x%02x\n",
-		  &uccf->uf_regs->guemr, in_8(&uccf->uf_regs->guemr));
+		  &uccf->uf_regs->guemr, ioread8(&uccf->uf_regs->guemr));
 }
 EXPORT_SYMBOL(ucc_fast_dump_regs);
 
@@ -89,7 +89,7 @@  EXPORT_SYMBOL(ucc_fast_get_qe_cr_subblock);
 
 void ucc_fast_transmit_on_demand(struct ucc_fast_private * uccf)
 {
-	out_be16(&uccf->uf_regs->utodr, UCC_FAST_TOD);
+	iowrite16be(UCC_FAST_TOD, &uccf->uf_regs->utodr);
 }
 EXPORT_SYMBOL(ucc_fast_transmit_on_demand);
 
@@ -101,7 +101,7 @@  void ucc_fast_enable(struct ucc_fast_private * uccf, enum comm_dir mode)
 	uf_regs = uccf->uf_regs;
 
 	/* Enable reception and/or transmission on this UCC. */
-	gumr = in_be32(&uf_regs->gumr);
+	gumr = ioread32be(&uf_regs->gumr);
 	if (mode & COMM_DIR_TX) {
 		gumr |= UCC_FAST_GUMR_ENT;
 		uccf->enabled_tx = 1;
@@ -110,7 +110,7 @@  void ucc_fast_enable(struct ucc_fast_private * uccf, enum comm_dir mode)
 		gumr |= UCC_FAST_GUMR_ENR;
 		uccf->enabled_rx = 1;
 	}
-	out_be32(&uf_regs->gumr, gumr);
+	iowrite32be(gumr, &uf_regs->gumr);
 }
 EXPORT_SYMBOL(ucc_fast_enable);
 
@@ -122,7 +122,7 @@  void ucc_fast_disable(struct ucc_fast_private * uccf, enum comm_dir mode)
 	uf_regs = uccf->uf_regs;
 
 	/* Disable reception and/or transmission on this UCC. */
-	gumr = in_be32(&uf_regs->gumr);
+	gumr = ioread32be(&uf_regs->gumr);
 	if (mode & COMM_DIR_TX) {
 		gumr &= ~UCC_FAST_GUMR_ENT;
 		uccf->enabled_tx = 0;
@@ -131,7 +131,7 @@  void ucc_fast_disable(struct ucc_fast_private * uccf, enum comm_dir mode)
 		gumr &= ~UCC_FAST_GUMR_ENR;
 		uccf->enabled_rx = 0;
 	}
-	out_be32(&uf_regs->gumr, gumr);
+	iowrite32be(gumr, &uf_regs->gumr);
 }
 EXPORT_SYMBOL(ucc_fast_disable);
 
@@ -263,12 +263,13 @@  int ucc_fast_init(struct ucc_fast_info * uf_info, struct ucc_fast_private ** ucc
 	gumr |= uf_info->tenc;
 	gumr |= uf_info->tcrc;
 	gumr |= uf_info->mode;
-	out_be32(&uf_regs->gumr, gumr);
+	iowrite32be(gumr, &uf_regs->gumr);
 
 	/* Allocate memory for Tx Virtual Fifo */
 	uccf->ucc_fast_tx_virtual_fifo_base_offset =
 	    qe_muram_alloc(uf_info->utfs, UCC_FAST_VIRT_FIFO_REGS_ALIGNMENT);
-	if (IS_ERR_VALUE(uccf->ucc_fast_tx_virtual_fifo_base_offset)) {
+	if (IS_ERR_VALUE((unsigned long)uccf->
+			 ucc_fast_tx_virtual_fifo_base_offset)) {
 		printk(KERN_ERR "%s: cannot allocate MURAM for TX FIFO\n",
 			__func__);
 		uccf->ucc_fast_tx_virtual_fifo_base_offset = 0;
@@ -281,7 +282,8 @@  int ucc_fast_init(struct ucc_fast_info * uf_info, struct ucc_fast_private ** ucc
 		qe_muram_alloc(uf_info->urfs +
 			   UCC_FAST_RECEIVE_VIRTUAL_FIFO_SIZE_FUDGE_FACTOR,
 			   UCC_FAST_VIRT_FIFO_REGS_ALIGNMENT);
-	if (IS_ERR_VALUE(uccf->ucc_fast_rx_virtual_fifo_base_offset)) {
+	if (IS_ERR_VALUE((unsigned long)uccf->
+			 ucc_fast_rx_virtual_fifo_base_offset)) {
 		printk(KERN_ERR "%s: cannot allocate MURAM for RX FIFO\n",
 			__func__);
 		uccf->ucc_fast_rx_virtual_fifo_base_offset = 0;
@@ -290,15 +292,15 @@  int ucc_fast_init(struct ucc_fast_info * uf_info, struct ucc_fast_private ** ucc
 	}
 
 	/* Set Virtual Fifo registers */
-	out_be16(&uf_regs->urfs, uf_info->urfs);
-	out_be16(&uf_regs->urfet, uf_info->urfet);
-	out_be16(&uf_regs->urfset, uf_info->urfset);
-	out_be16(&uf_regs->utfs, uf_info->utfs);
-	out_be16(&uf_regs->utfet, uf_info->utfet);
-	out_be16(&uf_regs->utftt, uf_info->utftt);
+	iowrite16be(uf_info->urfs, &uf_regs->urfs);
+	iowrite16be(uf_info->urfet, &uf_regs->urfet);
+	iowrite16be(uf_info->urfset, &uf_regs->urfset);
+	iowrite16be(uf_info->utfs, &uf_regs->utfs);
+	iowrite16be(uf_info->utfet, &uf_regs->utfet);
+	iowrite16be(uf_info->utftt, &uf_regs->utftt);
 	/* utfb, urfb are offsets from MURAM base */
-	out_be32(&uf_regs->utfb, uccf->ucc_fast_tx_virtual_fifo_base_offset);
-	out_be32(&uf_regs->urfb, uccf->ucc_fast_rx_virtual_fifo_base_offset);
+	iowrite32be(uccf->ucc_fast_tx_virtual_fifo_base_offset, &uf_regs->utfb);
+	iowrite32be(uccf->ucc_fast_rx_virtual_fifo_base_offset, &uf_regs->urfb);
 
 	/* Mux clocking */
 	/* Grant Support */
@@ -366,14 +368,14 @@  int ucc_fast_init(struct ucc_fast_info * uf_info, struct ucc_fast_private ** ucc
 	}
 
 	/* Set interrupt mask register at UCC level. */
-	out_be32(&uf_regs->uccm, uf_info->uccm_mask);
+	iowrite32be(uf_info->uccm_mask, &uf_regs->uccm);
 
 	/* First, clear anything pending at UCC level,
 	 * otherwise, old garbage may come through
 	 * as soon as the dam is opened. */
 
 	/* Writing '1' clears */
-	out_be32(&uf_regs->ucce, 0xffffffff);
+	iowrite32be(0xffffffff, &uf_regs->ucce);
 
 	*uccf_ret = uccf;
 	return 0;
diff --git a/drivers/tty/serial/Kconfig b/drivers/tty/serial/Kconfig
index 518db24a..d436c5d 100644
--- a/drivers/tty/serial/Kconfig
+++ b/drivers/tty/serial/Kconfig
@@ -1153,7 +1153,7 @@  config SERIAL_LANTIQ
 
 config SERIAL_QE
 	tristate "Freescale QUICC Engine serial port support"
-	depends on QUICC_ENGINE
+	depends on QUICC_ENGINE && FSL_SOC && PPC32
 	select SERIAL_CORE
 	select FW_LOADER
 	default n
diff --git a/drivers/tty/serial/ucc_uart.c b/drivers/tty/serial/ucc_uart.c
index 481eb29..ee409fd 100644
--- a/drivers/tty/serial/ucc_uart.c
+++ b/drivers/tty/serial/ucc_uart.c
@@ -34,6 +34,7 @@ 
 #include <soc/fsl/qe/ucc_slow.h>
 
 #include <linux/firmware.h>
+#include <asm/cpm.h>
 #include <asm/reg.h>
 
 /*
diff --git a/drivers/usb/gadget/udc/Kconfig b/drivers/usb/gadget/udc/Kconfig
index 658b8da..e18168c 100644
--- a/drivers/usb/gadget/udc/Kconfig
+++ b/drivers/usb/gadget/udc/Kconfig
@@ -277,7 +277,7 @@  config USB_AMD5536UDC
 
 config USB_FSL_QE
 	tristate "Freescale QE/CPM USB Device Controller"
-	depends on FSL_SOC && (QUICC_ENGINE || CPM)
+	depends on (FSL_SOC && QUICC_ENGINE && PPC32) || (FSL_SOC && CPM)
 	help
 	   Some of Freescale PowerPC processors have a Full Speed
 	   QE/CPM2 USB controller, which support device mode with 4
diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
index 2e710a4..24019cc 100644
--- a/drivers/usb/host/Kconfig
+++ b/drivers/usb/host/Kconfig
@@ -638,7 +638,7 @@  config USB_UHCI_BIG_ENDIAN_DESC
 
 config USB_FHCI_HCD
 	tristate "Freescale QE USB Host Controller support"
-	depends on OF_GPIO && QE_GPIO && QUICC_ENGINE
+	depends on OF_GPIO && QE_GPIO && QUICC_ENGINE && FSL_SOC && PPC32
 	select FSL_GTM
 	select QE_USB
 	help
diff --git a/include/soc/fsl/qe/qe.h b/include/soc/fsl/qe/qe.h
index 70339d7..f7a14f2 100644
--- a/include/soc/fsl/qe/qe.h
+++ b/include/soc/fsl/qe/qe.h
@@ -21,7 +21,6 @@ 
 #include <linux/spinlock.h>
 #include <linux/errno.h>
 #include <linux/err.h>
-#include <asm/cpm.h>
 #include <soc/fsl/qe/immap_qe.h>
 #include <linux/of.h>
 #include <linux/of_address.h>