lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 1 Nov 2015 14:11:12 +0100 From: Jens Kuske <jenskuske@...il.com> To: Chen-Yu Tsai <wens@...e.org> Cc: Maxime Ripard <maxime.ripard@...e-electrons.com>, Michael Turquette <mturquette@...libre.com>, Linus Walleij <linus.walleij@...aro.org>, Rob Herring <robh+dt@...nel.org>, Philipp Zabel <p.zabel@...gutronix.de>, Emilio López <emilio@...pez.com.ar>, Vishnu Patekar <vishnupatekar0510@...il.com>, Hans de Goede <hdegoede@...hat.com>, devicetree <devicetree@...r.kernel.org>, linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>, linux-kernel <linux-kernel@...r.kernel.org>, linux-sunxi <linux-sunxi@...glegroups.com> Subject: Re: [PATCH v4 1/6] clk: sunxi: Let divs clocks read the base factor clock name from devicetree On 30/10/15 08:46, Chen-Yu Tsai wrote: > On Wed, Oct 28, 2015 at 12:50 AM, Jens Kuske <jenskuske@...il.com> wrote: [..] >> @@ -991,8 +985,30 @@ static void __init sunxi_divs_clk_setup(struct device_node *node, >> if (data->ndivs) >> ndivs = data->ndivs; >> >> + /* Try to find a name for base factor clock */ >> + for (i = 0; i < ndivs; i++) { >> + if (data->div[i].self) { >> + of_property_read_string_index(node, "clock-output-names", >> + i, &factors.name); >> + break; >> + } >> + } >> + /* If we don't have a .self clk use the first output-name up to '_' */ > > This actually only works for the sun4i divs clocks. sun6i-a31-pll6 uses a > different naming convention. sun4i-a10-pll2 uses yet another, though that's > not a divs clock. Maybe we should work on unifying them. Maxime? All the divs clocks I could find either have a .self (handled by the first case above) or follow this naming convention, so it should work. But if we find a better way to clean up the divs without breaking devicetree bindings (as far as I understood we must not break them, am I right there?) I'd be happy with that too. Jens > > Other than these 2 bits of information, this patch looks good. > > > Regards > ChenYu > >> + if (factors.name == NULL) { >> + of_property_read_string_index(node, "clock-output-names", >> + 0, &clk_name); >> + >> + for (i = 0; i < SUNXI_DIVS_BASE_NAME_MAX_LEN - 1 && >> + clk_name[i] != '_' && >> + clk_name[i] != '\0'; i++) >> + base_name[i] = clk_name[i]; >> + >> + base_name[i] = '\0'; >> + factors.name = base_name; >> + } >> + >> /* Set up factor clock that we will be dividing */ >> - pclk = sunxi_factors_clk_setup(node, data->factors); >> + pclk = sunxi_factors_clk_setup(node, &factors); >> parent = __clk_get_name(pclk); >> >> reg = of_iomap(node, 0); >> -- >> 2.6.2 >> > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists