lkml.org 
[lkml]   [2012]   [Nov]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [RFC v2 5/8] ARM: tegra: Add auxiliary data for nvhost
    On 11/26/2012 06:19 AM, Terje Bergstrom wrote:
    > Add SoC specific auxiliary data to host1x and gr2d. nvhost uses
    > this data.
    >
    > Signed-off-by: Terje Bergstrom <tbergstrom@nvidia.com>
    > Signed-off-by: Arto Merilainen <amerilainen@nvidia.com>

    Arto's S-o-b really should be first and yours last since you're (Terje)
    the one who touched the patches last.

    > diff --git a/arch/arm/mach-tegra/board-dt-tegra20.c b/arch/arm/mach-tegra/board-dt-tegra20.c

    I think none of the changes the board-dt-tegra*.c should be made.

    AUXDATA is a temporary measure to keep things working during the
    transition to device tree. We want to remove entries from the AUXDATA
    tables rather than add them. The only thing that's stopping us from
    doing so right now is the lack of DT-based clock lookups, which hence
    require devices to have a specific name.

    > +static struct nvhost_device_data tegra_host1x_info = {
    > + .clocks = { {"host1x", UINT_MAX} },

    > +static struct nvhost_device_data tegra_gr2d_info = {
    > + .clocks = { {"gr2d", UINT_MAX, true}, {"epp", UINT_MAX, true} },

    Clock names shouldn't be passed in platform data; instead, clk_get()
    should be passed the device object and device-relative (i.e. not global)
    clock name. I expect if the driver is fixed to make this change, the
    changes to tegra*_clocks_data.c won't be needed either.



    \
     
     \ /
      Last update: 2012-11-27 01:01    [W:4.139 / U:0.172 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site