On Thu, Sep 06, 2012 at 13:07:29, Cousson, Benoit wrote:
> Hi Vaibhav,
>
> On 09/05/2012 10:05 PM, Vaibhav Hiremath wrote:
> > On 9/5/2012 7:57 PM, Benoit Cousson wrote:
> >> Hi Paul,
> >>
> >> On 08/24/2012 06:20 PM, Peter Ujfalusi wrote:
> >>> Hi Paul,
> >>>
> >>> On 08/24/2012 06:38 PM, Paul Wa
Hi Vaibhav,
On 09/05/2012 10:05 PM, Vaibhav Hiremath wrote:
> On 9/5/2012 7:57 PM, Benoit Cousson wrote:
>> Hi Paul,
>>
>> On 08/24/2012 06:20 PM, Peter Ujfalusi wrote:
>>> Hi Paul,
>>>
>>> On 08/24/2012 06:38 PM, Paul Walmsley wrote:
Do we need both this one and your '[PATCH] driver core: Ch
On 9/5/2012 7:57 PM, Benoit Cousson wrote:
> Hi Paul,
>
> On 08/24/2012 06:20 PM, Peter Ujfalusi wrote:
>> Hi Paul,
>>
>> On 08/24/2012 06:38 PM, Paul Walmsley wrote:
>>> Do we need both this one and your '[PATCH] driver core: Check if r->name
>>> is valid in platform_get_resource_byname()' ?
Peter,
I will just slighty change the subject since it is an omap_device.c patch and
not an omap_hwmod.c one.
ARM: OMAP: omap_device: Fix up resource names when booted with devicetree
Regards,
Benoit
On 09/05/2012 04:27 PM, Benoit Cousson wrote:
> Hi Paul,
>
> On 08/24/2012 06:20 PM, Peter Uj
Peter,
I will just slighty change the subject since it is a omap_device.c patch and
not an omap_hwmod.c one.
ARM: OMAP: omap_device: Fix up resource names when booted with devicetree
Regards,
Benoit
On 09/05/2012 04:27 PM, Benoit Cousson wrote:
> Hi Paul,
>
> On 08/24/2012 06:20 PM, Peter Ujf
Hi Paul,
On 08/24/2012 06:20 PM, Peter Ujfalusi wrote:
> Hi Paul,
>
> On 08/24/2012 06:38 PM, Paul Walmsley wrote:
>> Do we need both this one and your '[PATCH] driver core: Check if r->name
>> is valid in platform_get_resource_byname()' ? Or would that second patch
>> be enough? Is the crash
Hi Paul,
On 08/24/2012 06:38 PM, Paul Walmsley wrote:
> Do we need both this one and your '[PATCH] driver core: Check if r->name
> is valid in platform_get_resource_byname()' ? Or would that second patch
> be enough? Is the crash happening in the platform_get_resource_byname()
> iterator?
Th
Hello Péter,
On Thu, 23 Aug 2012, Peter Ujfalusi wrote:
> When booted with some resource will have their name set to NULL. This can
> cause later kernel crash since this is not expected by the platform code.
>
> When we boot without DT the devices are created with platform_device_add()
> which i
When booted with some resource will have their name set to NULL. This can
cause later kernel crash since this is not expected by the platform code.
When we boot without DT the devices are created with platform_device_add()
which itself fixes up the missing resource names:
if (r->name == NULL)