lkml.org 
[lkml]   [2013]   [Sep]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH V2 1/6] clk: exynos-audss: convert to platform device
    Date
    Hi Andrew,

    I'd like to ack this series, but there is one more thing that I think
    should be fixed. Please see my comment inline.

    On Monday 23 of September 2013 17:21:13 Andrew Bresticker wrote:
    > @@ -128,8 +135,53 @@ static void __init exynos_audss_clk_init(struct device_node *np)
    > #endif
    >
    > pr_info("Exynos: Audss: clock setup completed\n");

    nit (not the thing I mentioned above): This (and possibly other uses of
    pr_*() could be replaced with dev_*().

    > +
    > + return 0;
    > +}
    > +
    > +static int exynos_audss_clk_remove(struct platform_device *pdev)
    > +{
    > + int i;
    > +
    > + for (i = 0; i < EXYNOS_AUDSS_MAX_CLKS; i++) {
    > + if (clk_table[i])

    I believe clk_register_* functions return ERR_PTR() in case of failure,
    not NULL, so this should be accounted for either here or at probe time.
    Possibly checking for registration error in probe() would be the best
    solution, although bloating the code a bit (but what error path isn't?).

    Best regards,
    Tomasz



    \
     
     \ /
      Last update: 2013-09-24 11:41    [W:4.528 / U:0.152 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site