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

Groups > linux.kernel > #1529114

Re: [PATCH 5/10] dt: bindings: Add bindings for Marvell Xenon SD Host Controller

Show all headers | View raw

Hi Gregory,

2016-11-24 10:44 GMT+01:00 Gregory CLEMENT <gregory.clement@free-electrons.com>:
> Hi Arnd,
>
>  On jeu., nov. 24 2016, Arnd Bergmann <arnd@arndb.de> wrote:
>
>> On Thursday, November 24, 2016 10:22:31 AM CET Gregory CLEMENT wrote:
>>>
>>> I don't have an option for mmc in general, but using child node do not
>>> fit at all the xenon controller.
>>>
>>> For this controller each slot has its own set of register, so there is
>>> no common ressource to share so no advantage to use it. Using child node
>>> in our case will just make the code more complex for no benefit.
>>
>> If every slot has its own registers, what is it that makes up the
>> 'controller'? It sounds to me that you just have to adjust the terminology
>> and talk about multiple controllers then, with one slot per controller.
>>
>
> I agree and actually there were some words about in at the begining of
> the binding:
>
> "A single Xenon IP can support multiple slots.
> Each slot acts as an independent SDHC. It owns independent resources, such
> as register sets clock and PHY.
> Each slot should have an independent device tree node."
>
> All the confusion came from the fact that we still need to identify a
> slot ID. For an obscure reason the hardware can't guess the slot ID from
> the address register."
>

How about to avoid confusion, by simply renaming this number to
port-id/xenon-id or anything else but slot? I guess this may allow to
avoid some misunderstandings.

Best regards,
Marcin

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

Thread

Re: [PATCH 5/10] dt: bindings: Add bindings for Marvell Xenon SD Host Controller Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-11-22 18:40 +0100
  Re: [PATCH 5/10] dt: bindings: Add bindings for Marvell Xenon SD Host Controller Ulf Hansson <ulf.hansson@linaro.org> - 2016-11-24 10:10 +0100
    Re: [PATCH 5/10] dt: bindings: Add bindings for Marvell Xenon SD Host Controller Arnd Bergmann <arnd@arndb.de> - 2016-11-24 10:20 +0100
      Re: [PATCH 5/10] dt: bindings: Add bindings for Marvell Xenon SD Host Controller Gregory CLEMENT <gregory.clement@free-electrons.com> - 2016-11-24 10:30 +0100
        Re: [PATCH 5/10] dt: bindings: Add bindings for Marvell Xenon SD Host Controller Arnd Bergmann <arnd@arndb.de> - 2016-11-24 10:40 +0100
          Re: [PATCH 5/10] dt: bindings: Add bindings for Marvell Xenon SD  Host Controller Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2016-11-24 10:50 +0100
            Re: [PATCH 5/10] dt: bindings: Add bindings for Marvell Xenon SD Host Controller Arnd Bergmann <arnd@arndb.de> - 2016-11-24 11:10 +0100
          Re: [PATCH 5/10] dt: bindings: Add bindings for Marvell Xenon SD Host Controller Marcin Wojtas <mw@semihalf.com> - 2016-11-24 10:50 +0100
            Re: [PATCH 5/10] dt: bindings: Add bindings for Marvell Xenon SD  Host Controller Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2016-11-24 11:20 +0100
              Re: [PATCH 5/10] dt: bindings: Add bindings for Marvell Xenon SD Host  Controller Ziji Hu <huziji@marvell.com> - 2016-11-24 11:40 +0100

csiph-web