Re: [linux-sunxi] [PATCH] spi: dts: sun4i: Add support for inter-word wait cycles using the SPI Wait Clock Register

From	Marcus Weseloh <mweseloh42@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [linux-sunxi] [PATCH] spi: dts: sun4i: Add support for inter-word wait cycles using the SPI Wait Clock Register
Date	2015-11-20 15:00 +0100
Message-ID	<qwUoh-7Vs-5@gated-at.bofh.it> (permalink)
References	<qwzMS-2PJ-17@gated-at.bofh.it> <qwzMT-2PJ-43@gated-at.bofh.it> <qwGlk-7bk-7@gated-at.bofh.it> <qwPyj-4PB-45@gated-at.bofh.it> <qwQXp-5SQ-35@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Julien,

2015-11-20 11:12 GMT+01:00 Julian Calaby <julian.calaby@gmail.com>:
> Having magic numbers is kind-of a drivers' job.

Yes, of course. What I meant was that I didn't feel comfortable to
include this magic number in driver code because I'm not 100% sure if
it is correct across all SPI configurations and SoCs that this driver
supports (A10 / A20).

> (and the wdelay should
> arguably be a core-spi thing, not a sunxi thing, but that's a separate
> discussion)

I've been thinking about that, but it seemed to big a change to
attempt with my limited kernel hacking experience.

Mark, do you think we should rather talk about adding support for
options like this delay to spi-core? Or would it be OK to add it to
the sun4i driver and possibly refactor later?

Cheers,

  Marcus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH] spi: dts: sun4i: Add support for inter-word wait cycles Marcus Weseloh <mweseloh42@gmail.com> - 2015-11-19 17:00 +0100
  [PATCH] spi: dts: sun4i: Add support for inter-word wait cycles using the SPI Wait Clock Register Marcus Weseloh <mweseloh42@gmail.com> - 2015-11-19 17:00 +0100
    Re: [linux-sunxi] [PATCH] spi: dts: sun4i: Add support for inter-word  wait cycles using the SPI Wait Clock Register Julian Calaby <julian.calaby@gmail.com> - 2015-11-20 00:00 +0100
      Re: [linux-sunxi] [PATCH] spi: dts: sun4i: Add support for inter-word  wait cycles using the SPI Wait Clock Register Marcus Weseloh <mweseloh42@gmail.com> - 2015-11-20 09:50 +0100
        Re: [linux-sunxi] [PATCH] spi: dts: sun4i: Add support for inter-word  wait cycles using the SPI Wait Clock Register Julian Calaby <julian.calaby@gmail.com> - 2015-11-20 11:20 +0100
          Re: [linux-sunxi] [PATCH] spi: dts: sun4i: Add support for inter-word  wait cycles using the SPI Wait Clock Register Marcus Weseloh <mweseloh42@gmail.com> - 2015-11-20 15:00 +0100
            Re: [linux-sunxi] [PATCH] spi: dts: sun4i: Add support for  inter-word wait cycles using the SPI Wait Clock Register Rob Herring <robh@kernel.org> - 2015-11-20 17:20 +0100
              Re: [linux-sunxi] [PATCH] spi: dts: sun4i: Add support for inter-word  wait cycles using the SPI Wait Clock Register Marcus Weseloh <mweseloh42@gmail.com> - 2015-11-20 17:50 +0100
                Re: [linux-sunxi] [PATCH] spi: dts: sun4i: Add support for  inter-word wait cycles using the SPI Wait Clock Register Maxime Ripard <maxime.ripard@free-electrons.com> - 2015-11-22 20:50 +0100
                Re: [linux-sunxi] [PATCH] spi: dts: sun4i: Add support for inter-word  wait cycles using the SPI Wait Clock Register Marcus Weseloh <mweseloh42@gmail.com> - 2015-11-23 10:20 +0100
    Re: [PATCH] spi: dts: sun4i: Add support for inter-word wait cycles  using the SPI Wait Clock Register Rob Herring <robh@kernel.org> - 2015-11-20 17:10 +0100

csiph-web