lkml.org 
[lkml]   [2015]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH 2/2] of/platform: Mark all device tree resources as SHARED
    Date
    On Fri, 5 Jun 2015 18:51:36 +0200
    , Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>
    wrote:
    > Hello Rob,
    >
    > Thanks for your feedback!
    >
    > On Fri, Jun 5, 2015 at 6:45 PM, Rob Herring <robherring2@gmail.com> wrote:
    > > On Fri, Jun 5, 2015 at 5:51 AM, Ricardo Ribalda Delgado
    > > <ricardo.ribalda@gmail.com> wrote:
    > >> Some device tree platform do not define their resources properly. i.e.
    > >> overlapping or repeated resources.
    > >>
    > >> This patch mark all device tree resources as shareable.
    > >>
    > >> In the future this should only be set for the platforms that have
    > >> problems.
    > >
    > > I don't think we want to do this globally. This should be very rare
    > > and we want to discourage any new cases.
    >
    > I just wanted to mimic the original behaviour. Unfortunately I have no
    > idea of what platform is broken. Grant needs to help us here :)

    I know of powerpc platforms that split the ethernet and mdio controllers
    into separate nodes, even though they share a register block. Those ones
    are broken (ie. mpc5200). I know there are ARM platforms that exhibited
    the same behaviour, but I can't remember specifics at the moment.

    > What do you think about the new flag? Does it make any sense for you?

    I think I have another solution to the whole problem. IIUC, the problem
    is the kernel crashes on unregistering resources. I've got a patch that
    doesn't try to unregister resources that weren't registered in the first
    place. It doesn't fix the problem of DT not registering the resources in
    the first place, but it makes the unregister path safe.

    I'll cc: you on the patch series.

    g.



    \
     
     \ /
      Last update: 2015-06-07 19:21    [W:2.421 / U:0.232 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site