On Jul 27, 2007, at 6:45 AM, Anton Vorontsov wrote:
> On Fri, Jul 27, 2007 at 03:14:06AM -0500, Kumar Gala wrote:
>>
>> On Jul 26, 2007, at 8:57 AM, Anton Vorontsov wrote:
> [...]
>>>
>>> @@ -207,20 +222,36 @@
>>>
>>> [EMAIL PROTECTED] {
>>> device_type = "spi";
>>> + device-id = <1>;
>>
>> Can we just use the reg value for bus_num in the kernel.
>
> Sure, technically nothing prevents this. But, QE specs names
> SPIs by these ids. Plus, from the kernel side spi name will be
> not pretty, it will be spi1216.1. Reg value making little sense
> to the userspace (or kernel-side SPI layer). Still want get
> rid of device-id?
>
>>> compatible = "fsl_spi";
>>> reg = <4c0 40>;
>>> interrupts = <2>;
>>> interrupt-parent = <&qeic>;
>>> - mode = "cpu";
>>> + mode = "qe";
>>> + sysclk = <5f5e100>; /* 100000000 Hz */
>>
>> we don't need this in the spi node, its just the system clock
>> frequency
>> which we can get other ways.
>
> Will try, I guess "bus-frequency" property of soc8323 is what I need.
>
>>> + max-chipselect = <1>;
>>
>> I'm not sure how I feel about this in here, I'm thinking it should
>> go.
>
> It's board-specific, i.e. how much chips connected to this SPI bus.
> SPI layer needs this. Otherwise I have to pass it from board file,
> but isn't it fits nicely in the DT?
>
>>> + pio-handle = <&spi1pio>;
>>> +
>>> + [EMAIL PROTECTED] {
>>> + device_type = "mmc";
>>> + compatible = "mmc-spi";
>>> + device-id = <1>;
>>> + max-speed-hz = <bebc20>; /* 12500000 Hz */
>>> + chip-select = <0>;
>>> + pio-handle = <&mmc1pio>;
>>> + };
>>
>> we should do this in board code and not the device tree.
>
> Well, I've done this initially. But Vitaly hinted that this could
> be done in the DT instead, which made sense to me - mmc is the child
> device of SPI bus. Why do you think it shouldn't be in the DT? I'm
> not arguing, just want understand this.
I understand. Look at the dts/device tree from a non-Linux
perspective. From that point of view its not always clear how one
would interpret some of the fields or if they are necessary. I get
very concerned when we start trying to describe any devices connect
to a bus when the bus doesn't have any specific for things that are
just software convention for a given OS.
The "id" concept and number is from Linux and another OS may have
come up w/some other way of describe this.
- k
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
