Hi Neil,

On 2018/12/13 16:53, Neil Armstrong wrote:
Hi Sunny,

On 13/12/2018 09:39, Sunny Luo wrote:
The SPICC controller in Meson-AXG is capable of driving the CLK/MOSI/SS
signal lines through the idle state (between two transmission operation),
which avoid the signals floating in unexpected state.

This is welcome, because it's really missing on GX...
I tried implementing it with pinctrl at [1], but it's complex.

Can you provide more info on how we should implement in on GX to be on par ?

[1] 
https://github.com/superna9999/linux/commit/9c3a95659dd532d186556c1570c54d79ea5a4d45

GX is incapable of OEN. To be on par with it ,we have to pullup/down clk pin as
you did at[1].

  static const struct meson_spicc_data meson_spicc_gx_data = {
-       .max_speed_hz   = 30000000,
+       .max_speed_hz           = 30000000,

Nitpick, but I would have kept the indentation here ...

  };
static const struct meson_spicc_data meson_spicc_axg_data = {
-       .max_speed_hz   = 80000000,
+       .max_speed_hz           = 80000000,
+       .has_oen                = true,

same here

Anywy it's nitpick,
ok, i will revert it next patch.

Reply via email to