Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1531648

Re: Adding a .platform_init callback to sdhci_arasan_ops

Show all headers | View raw

Hi,

On Mon, Nov 28, 2016 at 11:32 AM, Mason <slash.tmp@free.fr> wrote:
> On 28/11/2016 18:46, Doug Anderson wrote:
>
>> As argued in my original patch the field "corecfg_baseclkfreq" is
>> documented in the generic Arasan document
>> <https://arasan.com/wp-content/media/eMMC-5-1-Total-Solution_Rev-1-3.pdf>
>> and thus is unlikely to be Rockchip specific.
>
> I downloaded the data sheet, which doesn't mention registers,
> but "pins" and "signals". Does that mean it is up to every
> platform to decide how to group these wires into individual
> registers?

As I understand it: yes.  ...but remember that I'm not arasan nor even
Rockchip and I don't have any access to that they provide their IP
licensees.

That's why I structured my config the way it did.  It means that no
matter how a licensee stuffs these configs into their register map we
can support it.  Every configuration is individually described by a
register offset, width, and mask.  Technically the width _ought_ to be
the same for all users, so I guess you could try to optimize that out.
...but it doesn't hurt the way it is.


> corecfg_baseclkfreq[7:0]
> Base Clock Frequency for SD Clock.
> This is the frequency of the xin_clk.
>
> How can 8 bits encode a frequency?
> Is there an implicit LUT? Is it a MHz count?

Use the source.  :)

 * The corecfg_baseclkfreq is supposed to contain the MHz of clk_xin.

It also turns out to match the SDHCI spec "Base Clock Frequency for SD
Clock".  See below.

---

It actually turns out that both of the "corecfg" bits we're setting
right now for Rockchip are a bit on the silly side.  All they do is
get mirrored out the other end in the "caps" register.  I found that
out from folks at Rockchip much later after I wrote my patch.

So, for instance, setting corecfg_clockmultiplier (I'm told) does
nothing more than control the bits read here:

  (caps[1] & SDHCI_CLOCK_MUL_MASK) >> SDHCI_CLOCK_MUL_SHIFT;

I can certainly see that they are mirrored, but I have to just take it
on faith that it does nothing else.

>>> w(grf + 0xf02c, 0x00ff0011)
>>> hex((r(sdhci + 0x44) & 0x00ff0000) >> 16)
'0x11L'


...and similar for the speed:

>>> w(grf + 0xf000, 0xff009700)
>>> hex((r(sdhci + 0x40) & 0x0000ff00) >> 8)
'0x97L'


-Doug

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

Thread

Adding a .platform_init callback to sdhci_arasan_ops Sebastian Frias <sf84@laposte.net> - 2016-11-25 16:30 +0100
  Re: Adding a .platform_init callback to sdhci_arasan_ops Adrian Hunter <adrian.hunter@intel.com> - 2016-11-28 11:40 +0100
    Re: Adding a .platform_init callback to sdhci_arasan_ops Sebastian Frias <sf84@laposte.net> - 2016-11-28 12:30 +0100
      Re: Adding a .platform_init callback to sdhci_arasan_ops Adrian Hunter <adrian.hunter@intel.com> - 2016-11-28 13:00 +0100
        Re: Adding a .platform_init callback to sdhci_arasan_ops Sebastian Frias <sf84@laposte.net> - 2016-11-28 14:30 +0100
          Re: Adding a .platform_init callback to sdhci_arasan_ops Sebastian Frias <sf84@laposte.net> - 2016-11-28 15:40 +0100
            Re: Adding a .platform_init callback to sdhci_arasan_ops Doug Anderson <dianders@chromium.org> - 2016-11-28 18:50 +0100
              Re: Adding a .platform_init callback to sdhci_arasan_ops Mason <slash.tmp@free.fr> - 2016-11-28 20:40 +0100
                Re: Adding a .platform_init callback to sdhci_arasan_ops Doug Anderson <dianders@chromium.org> - 2016-11-28 21:40 +0100
              Re: Adding a .platform_init callback to sdhci_arasan_ops Sebastian Frias <sf84@laposte.net> - 2016-12-05 13:30 +0100
                Re: Adding a .platform_init callback to sdhci_arasan_ops Doug Anderson <dianders@chromium.org> - 2016-12-05 17:20 +0100
          Re: Adding a .platform_init callback to sdhci_arasan_ops Doug Anderson <dianders@chromium.org> - 2016-11-28 19:10 +0100
            Re: Adding a .platform_init callback to sdhci_arasan_ops Sebastian Frias <sf84@laposte.net> - 2016-12-05 13:30 +0100
              Re: Adding a .platform_init callback to sdhci_arasan_ops Doug Anderson <dianders@chromium.org> - 2016-12-05 17:40 +0100
                Re: Adding a .platform_init callback to sdhci_arasan_ops Sebastian Frias <sf84@laposte.net> - 2016-12-06 14:50 +0100

csiph-web