linux-kernel - Re: OF files

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:	Sun, 11 May 2008 15:59:20 +0200
From:	Michal Simek <monstr@...nam.cz>
To:	Paul Mackerras <paulus@...ba.org>
CC:	Arnd Bergmann <arnd@...db.de>,
	John Williams <john.williams@...alogix.com>,
	Matthew Wilcox <matthew@....cx>,
	Will Newton <will.newton@...il.com>,
	Linux Kernel list <linux-kernel@...r.kernel.org>,
	linux-arch@...r.kernel.org, git@...inx.com,
	Stephen Neuendorffer <stephen.neuendorffer@...inx.com>,
	John Linn <John.Linn@...inx.com>,
	Ulrich Drepper <drepper@...hat.com>
Subject: Re: OF files

Hi All,

I move prom_parse.c to drivers/of.
of_platform.c was completely moved to drivers/of/platform.c
Functions from of_device.c was moved to drivers/of/device.c except
of_device_make_bus_id.
Prom.c file was cleaned - I remove all PPC specific code.

You can check it in my git tree (git.monstr.eu) or wait for next set of patches.

Michal

> Michal Simek writes:
> 
>> I tested OF files changes. I sent you some patches about.
>>
>> I look at headers files too. Is it possible to move prom.h,
>> of_device.h and of_platform.h to asm-generic or linux folder? 
> 
> We can move stuff to drivers/of and include/linux, but we should only
> move the parts that are truly architecture-independent.  We should not
> have anything like #ifdef CONFIG_PPC_PSERIES in the files in those
> directories.  If it needs an ifdef like that, then that part of the
> file should stay under arch/powerpc.
> 
> In other words, it's OK to split files and end up with more files than
> we have at present.  That's better than adding ifdefs.
> 
> Paul.
> 
--
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/