linux-kernel - Re: [PATCH 1/5] Clocklib: add generic framework for managing 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] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 15 Apr 2008 13:34:51 +0400
From:	Dmitry Baryshkov <dbaryshkov@...il.com>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	linux-kernel@...r.kernel.org,
	Haavard Skinnemoen <haavard.skinnemoen@...el.com>,
	Russell King <rmk+lkml@....linux.org.uk>,
	Paul Mundt <lethal@...ux-sh.org>,
	pHilipp Zabel <philipp.zabel@...il.com>,
	Pavel Machek <pavel@....cz>, tony@...mide.com, paul@...an.com,
	David Brownell <david-b@...bell.net>
Subject: Re: [PATCH 1/5] Clocklib: add generic framework for managing
	clocks.

On Mon, Apr 14, 2008 at 11:13:30PM -0700, Andrew Morton wrote:
> On Sun, 13 Apr 2008 18:43:12 +0400 Dmitry Baryshkov <dbaryshkov@...il.com> wrote:
> 
> > +#define FUNC_TO_CLK(func)	&(&(struct clk_function) func)->clk
> 
> scratched my head for a while, was unable to work out what this does.
> It is unused, so I think it can be zapped?

It take clk_function and returns pointer to the contained struct clk.
It's unused in the submitted code, but it's used in the yet-in-progress
drivers for Toshiba companion chips. I used it to be able to write
things like:

static struct clk *clks[] = {
        [TC6387XB_CLK_MMC] = FUNC_TO_CLK(MFD_CLK_FUNC(NULL, "MMCCLK",
				&tc6387xb_cells[TC6387XB_CELL_MMC])),
};

where MFD_CLK_FUNC instantiates some special sort of clk_function.

> 
> All the inlined functions in clklib.h are waaaaaaaaaaaay too big
> to be inlined.  Did I fail to notice that last time around?

OK.

> 
> Anyway, this patchset is basically an arm-only thing, so I'll need
> guidance, please...

They were posted to the LKML with CC to arch maintainers primary to get
their feedback. I have a hope that it can be used not only for various
arms, but for other arches. As you can see posting it here helped to make
the whole patchset better and less memory hungry.

As currently the patchset only contains the generic and ARM-related
things, I think it can and should go in via Russell, if he agrees.

Russell?

-- 
With best wishes
Dmitry

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