lkml.org 
[lkml]   [2017]   [Nov]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH 0/3] Add clock driver for Actions S900 SoC
    From
    Date
    Hi Mani,

    Am 01.11.2017 um 03:54 schrieb Manivannan Sadhasivam:
    > This series adds clock driver for Actions Semi OWL series
    > S900 SoC with relevant clock bindings and device tree data.
    >
    > This series also addresses the review comments from previous
    > submission happened last year.
    >
    > https://patchwork.kernel.org/patch/9254471/

    Interesting - I had searched for any S500/S900 patches before starting
    to work on this but came up empty on LAKML.

    It seems you missed at least one review comment from 2016 about the
    example indentation, so please double-check for anything Stephen already
    pointed out.

    > Driver has been validated on Bubblegum-96 board.
    >
    > Thanks,
    > Mani
    >
    > Manivannan Sadhasivam (3):
    > arm64: dts: actions: add s900 clock controller nodes
    > clk: owl: add clock driver for Actions S900 SoC
    > Documentation: add Actions S900 clock bindings

    Please reverse the patch order.

    Have you checked S500 code whether the S900 vs. common split is correct?

    Many thanks for your work on this,

    Andreas

    --
    SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
    GF: Felix Imendörffer, Jane Smithard, Graham Norton
    HRB 21284 (AG Nürnberg)

    \
     
     \ /
      Last update: 2017-11-04 10:06    [W:4.234 / U:0.060 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site