lkml.org 
[lkml]   [2014]   [Jul]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2] irqchip: add keystone irq controller ip driver
On 07/24/2014 03:58 AM, Varka Bhadram wrote:
>
> On Wednesday 23 July 2014 11:31 PM, Grygorii Strashko wrote:
>> Hi,
>>
>> On 07/23/2014 06:32 PM, Varka Bhadram wrote:
>>> On Wednesday 23 July 2014 08:10 PM, Grygorii Strashko wrote:
>>>> On Keystone SOCs, DSP cores can send interrupts to ARM
>>>> host using the IRQ controller IP. It provides 28 IRQ
>>>> signals to ARM. The IRQ handler running on HOST OS can
>>>> identify DSP signal source by analyzing SRCCx bits in
>>>> IPCARx registers. This is one of the component used by
>>>> the IPC mechanism used on Keystone SOCs.
>>> (...)

[...]

>>>> +static inline void
>>>> +keystone_irq_writel(struct keystone_irq_device *kirq, u32 value)
>>>> +{
>>>> + int ret;
>>>> +
>>>> + ret = regmap_write(kirq->devctrl_regs, kirq->devctrl_offset, value);
>>>> + if (ret < 0)
>>>> + dev_dbg(kirq->dev, "irq write failed ret(%d)\n", ret);
>>> It can be like
>>>
>>> if (!regmap_write(kirq->devctrl_regs, kirq->devctrl_offset, value))
>>> dev_dbg(kirq->dev, "irq write failed \n");
>>>
>>>> +}
>>>> +
>>>> +
>> Pls, Pay attention that I'd like to see ret code here in case of failure.
>
> What we have to do with ret code... ?

Print it :)

> In case of failure only this debug message will be printed.

Yep. And that exactly what I need.

>
>>> (...)
>>>
>>>> +
>>>> +static struct irq_domain_ops keystone_irq_ops = {
>>>> + .map = keystone_irq_map,
>>>> + .xlate = irq_domain_xlate_onecell,
>>>> +};
>>>> +
>>>> +static int keystone_irq_probe(struct platform_device *pdev)
>>>> +{
>>>> + struct device *dev = &pdev->dev;
>>>> + struct device_node *np = dev->of_node;
>>>> + struct keystone_irq_device *kirq;
>>>> + int ret;
>>>> +
>>>> + if (np == NULL)
>>>> + return -EINVAL;
>>> return -ENODEV??????
>> If probe is executed - the dev is present, but it was created in a
>> wrong/unsupported way
>> or dev structure contains wrong data.
>
> Here we are trying to get the device tree node , but that is not present
> we may return the
> error code saying that NO DEVICE is present....

1) Even in case of DT boot device can be creating using platform_device_register()
(by mistake, multiplatform build)

2) I've checked current Kernel code and found that
- if drivers are DT compatible only then they return
-EINVAL -or- -ENOENT
See, for example:
- irq-imgpdc.c
- gpio-tb10x.c

>
>>> (...)

Regards,
-grygorii




\
 
 \ /
  Last update: 2014-07-24 18:22    [W:0.076 / U:3.676 seconds]
©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site