Re: [PATCH v5 2/2] QE: remove PPCisms for QE

From	Leo Li <pku.leo@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v5 2/2] QE: remove PPCisms for QE
Date	2016-09-19 22:20 +0200
Message-ID	<sjdcJ-1DF-1@gated-at.bofh.it> (permalink)
References	<sjdcJ-1DF-3@gated-at.bofh.it> <sjdcJ-1DF-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Mon, Jul 25, 2016 at 12:43 AM, Zhao Qiang <qiang.zhao@nxp.com> wrote:
> 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
>
>  drivers/irqchip/qe_ic.c       | 28 +++++++++------
>  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 | 68 ++++++++++++++++++------------------
>  include/soc/fsl/qe/qe.h       |  1 -
>  include/soc/fsl/qe/qe_ic.h    | 12 +++----
>  9 files changed, 133 insertions(+), 118 deletions(-)
>

[snip]

> 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

You make it possible to build QE drivers on ARM, but the UCC_GETH
fails to build on arm64.  Please make sure all these drivers can build
on other architectures.  Or you can simply make them only build for
Power architecture as most of them are not available on ARM.

Regards,
Leo

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH v5 2/2] QE: remove PPCisms for QE Leo Li <pku.leo@gmail.com> - 2016-09-19 22:20 +0200
  RE: [PATCH v5 2/2] QE: remove PPCisms for QE Qiang Zhao <qiang.zhao@nxp.com> - 2016-09-21 07:50 +0200
    Re: [PATCH v5 2/2] QE: remove PPCisms for QE Leo Li <pku.leo@gmail.com> - 2016-09-21 20:20 +0200
      RE: [PATCH v5 2/2] QE: remove PPCisms for QE Qiang Zhao <qiang.zhao@nxp.com> - 2016-09-22 10:20 +0200
        Re: [PATCH v5 2/2] QE: remove PPCisms for QE Leo Li <pku.leo@gmail.com> - 2016-09-22 21:40 +0200
          RE: [PATCH v5 2/2] QE: remove PPCisms for QE Qiang Zhao <qiang.zhao@nxp.com> - 2016-09-23 10:50 +0200

csiph-web