linux-kernel - Re: [PATCH v4] clk: rockchip: disable unused clocks

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Mon, 3 Nov 2014 09:07:36 -0800
From:	Doug Anderson <dianders@...omium.org>
To:	Kever Yang <kever.yang@...k-chips.com>
Cc:	Mike Turquette <mturquette@...aro.org>,
	Heiko Stuebner <heiko@...ech.de>,
	Sonny Rao <sonnyrao@...omium.org>,
	Addy Ke <addy.ke@...k-chips.com>,
	Eddie Cai <cf@...k-chips.com>, Jianqun Xu <xjq@...k-chips.com>,
	han jiang <hj@...k-chips.com>,
	戴克霖 (Jack) <dkl@...k-chips.com>,
	Tao Huang <huangtao@...k-chips.com>,
	"open list:ARM/Rockchip SoC..." <linux-rockchip@...ts.infradead.org>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v4] clk: rockchip: disable unused clocks

Kever,

On Sat, Nov 1, 2014 at 5:44 AM, Kever Yang <kever.yang@...k-chips.com> wrote:
> -       COMPOSITE(SCLK_EMMC, "sclk_emmc", mux_mmc_src_p, 0,
> +       COMPOSITE(SCLK_EMMC, "sclk_emmc", mux_mmc_src_p, CLK_IGNORE_UNUSED,

Is there a reason why this is needed?  I took it out (AKA let eMMC get
disabled when unused) and my system booted w/ eMMC as the root
filesystem.

> -       GATE(PCLK_UART2, "pclk_uart2", "pclk_cpu", 0, RK3288_CLKGATE_CON(11), 9, GFLAGS),
> +       GATE(PCLK_UART2, "pclk_uart2", "pclk_cpu", CLK_IGNORE_UNUSED, RK3288_CLKGATE_CON(11), 9, GFLAGS),

Is there a reason why this is needed?  I took it out (AKA let UART2
get disabled) and my system booted.

The console-related bits in my cmdline are:
  earlyprintk=ttyS2,115200n8 console=tty1 console=ttyS2,115200n8 kgdboc=ttyS2

Do you have a different cmdline where it's failing?

Other than the two above this looks sane to me (only looked at
rk3288).  At some point soon we should definitely fix the PWM and the
USB ones and then start looking at the others and see if there are any
others that can be fixed.  ...but this seems like it's moving us in
the right direction and I'd like to see it landed.

So with the above two fixes, you can add my Reviewed-by and Tested-by.
I tested this in the chromeos-3.14 tree (which has tons of backports)
on rk3288-pinky.  I tested:
* System continues to work normally (including graphics).
* WIP Suspend / Resume patches work better with this patch than
without (both normal and deep modes).

-Doug
--
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/