Re: [PATCH 7/7] openrisc: remove the redundant of_platform_populate

From: Rob Herring
Date: Mon Sep 19 2016 - 12:14:31 EST


On Mon, Sep 19, 2016 at 9:32 AM, Jonas Bonn <jonas@xxxxxxxxxxxx> wrote:
> On 09/16/2016 04:43 PM, Stafford Horne wrote:
>>
>> From: Rob Herring <robh@xxxxxxxxxx>
>>
>> The of_platform_populate call in the openrisc arch code is now redundant
>> as the DT core provides a default call. Openrisc has a NULL match table
>> which means only top level nodes with compatible strings will have
>> devices creates. The default version will also descend nodes in the
>> match table such as "simple-bus" which should be fine as openrisc
>> doesn't have any of these (though it is preferred that memory-mapped
>> peripherals be grouped under a bus node(s)).
>>
>> Signed-off-by: Rob Herring <robh@xxxxxxxxxx>
>> Cc: Jonas Bonn <jonas@xxxxxxxxxxxx>
>> Signed-off-by: Stafford Horne <shorne@xxxxxxxxx>
>
>
> Yes, this is fine, but trivial/generic stuff like this doesn't really need
> to go via the OpenRISC repo, anyway.

Then which tree do you think it should go thru? I could have applied
it, but it doesn't depend on anything else. So it should go via the
correct maintainer's tree.

Rob