lkml.org 
[lkml]   [2015]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [Cocci] [PATCH v13 3/6] clk: Make clk API return per-user struct clk instances
    On 03/12/15 10:20, Sebastian Andrzej Siewior wrote:
    > On 2015-02-17 14:01:04 [-0800], Stephen Boyd wrote:
    >> diff =
    >> --- arch/arm/mach-imx/mach-imx6q.c
    >> +++ /tmp/cocci-output-11792-b62223-mach-imx6q.c
    >> @@ -211,7 +211,6 @@ static void __init imx6q_1588_init(void)
    >> * set bit IOMUXC_GPR1[21]. Or the PTP clock must be from pad
    >> * (external OSC), and we need to clear the bit.
    >> */
    >> - clksel = ptp_clk == enet_ref ? IMX6Q_GPR1_ENET_CLK_SEL_ANATOP :
    >> IMX6Q_GPR1_ENET_CLK_SEL_PAD;
    >> gpr = syscon_regmap_lookup_by_compatible("fsl,imx6q-iomuxc-gpr");
    >> if (!IS_ERR(gpr))
    > Any idea how to do the comparison here? Or should we rely that the bootloader
    > sets this properly (it managed already to select a frequency)? The phy has no
    > clock node in current DT's so we can check this.
    >

    This has been fixed by adding a clk_is_match() helper and using that to
    compare instead of comparing raw pointers. It would be nice if we could
    replace the patch with something else that doesn't require this helper
    though. It looks like this is static board configuration, so I wonder
    why we didn't just have a DT property that indicates how the gpr should
    be configured for this particular board.

    --
    Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
    a Linux Foundation Collaborative Project



    \
     
     \ /
      Last update: 2015-03-12 21:01    [W:10.310 / U:0.116 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site