[lkml]   [2013]   [Feb]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 1/4] documentation: add palmas dts definition
On Thursday 28 February 2013 12:02 AM, Stephen Warren wrote:
> On 02/17/2013 10:11 PM, J Keerthy wrote:
> +- interrupt-parent : The parent interrupt controller.
> +
> +Optional node:
> +- Child nodes contain in the palmas. The palmas family is made of several
> + variants that support a different number of features.
> + The child nodes will thus depend of the capability of the variant.
> Are there DT bindings for those child nodes anywhere?
> Representing each internal component as a separate DT node feels a
> little like designing the DT bindings to model the Linux-internal MFD
> structure. DT bindings should be driven by the HW design and OS-agnostic.
> From a DT perspective, is there any need at all to create a separate DT
> node for each component? This would only be needed or useful if the
> child IP blocks (and hence DT bindings for those blocks) could be
> re-used in other top-level devices that aren't represented by this
> top-level ti,palmas DT binding. Are the HW IP blocks here re-used
> anywhere, or will they be?

I dont think that child IP block can be used outside of the palma
although other mfd device may have same IP.

The child driver very much used the palma's API for register access and
they can not be separated untill driver is write completely independent
of palmas API. Currently, child driver include the palma header, uses
palma mfd stcruture and plama's api for accessing registers.

>> + interrupt-controller;
>> + #interrupt-cells = <1>;
>> + interrupt-parent = <&gic>;
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> +
>> + ti,mux-pad1 = <0x00>;
>> + ti,mux-pad2 = <0x00>;
>> + ti,power-ctrl = <0x03>;
>> +
>> + palmas_pmic {
> Just "pmic" seems simpler, although I dare say the node name isn't
> really used for anything.

Just curios, why do we require the palma_pmic node at all, We can start
with regulator node directly. Is it not too much nested here?

> +
> + palmas_rtc {
> + compatible = "ti,palmas_rtc";
> + interrupts = <8 9>;
> Are the interrupt outputs of the RTC fed directly to the GIC interrupt
> mentioned in the top-level Palmas node, or do these interrupts feed into
> a top-level IRQ controller in the Palmas device, which then feeds into
> the external IRQ controller?

The interrupt goes to the chip-internal irq, not to external of chip.
We have only one int line from chip which can be connected to processor/GIC.
yes, interrupt parent need to be define here to get the proper interrupt

 \ /
  Last update: 2013-02-28 10:41    [W:0.077 / U:0.076 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site