linux-kernel - Re: [PATCH 05/11] spi/pxa2xx: make clock rate configurable from platform data

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]

Message-ID: <20130110095803.GJ13897@intel.com>
Date:	Thu, 10 Jan 2013 11:58:03 +0200
From:	Mika Westerberg <mika.westerberg@...ux.intel.com>
To:	"Rafael J. Wysocki" <rjw@...k.pl>
Cc:	Mark Brown <broonie@...nsource.wolfsonmicro.com>,
	linux-kernel@...r.kernel.org, grant.likely@...retlab.ca,
	linus.walleij@...aro.org, eric.y.miao@...il.com,
	linux@....linux.org.uk, haojian.zhuang@...il.com,
	chao.bi@...el.com, "H. Peter Anvin" <hpa@...ux.intel.com>
Subject: Re: [PATCH 05/11] spi/pxa2xx: make clock rate configurable from
 platform data

On Wed, Jan 09, 2013 at 11:07:26PM +0100, Rafael J. Wysocki wrote:
> We hardcode different kinds of information into the kernel anyway, but it
> would be good to be able to use some criteria to decide whether or not we need
> that information on the given system.
> 
> In this particular case, we'll only need the fixed clock rate for the SPI
> which is not present on other x86 systems to date.
> 
> Mika, do you have any idea what checks to apply to figure out whether or not
> the SPI clock will be necessary?

I've tried to figure out them but so far I have no ideas :-/

If we put that behind some config option, it 1) confuses users and 2)
distro makers will select it anyway, so I don't see this as an option.

> Perhaps we can add a check into acpi_create_platform_device()?  Something like
> "if the device ID is present in table X, then run function Y for the device"
> where Y will configure the clock if the SPI is matched?

That would work but then we must also make sure that if the device was
enumerated from PCI, it gets this clock via some other means.

The LPSS devices can be configured to either appear on PCI or ACPI and of
course ACPI has the advantage because then we can enumerate the slave
devices behind the bus as well. However, it might be possible that we need
to support PCI enumeration at some point.

Furthermore, the SPI controller has an existing PCI glue driver
(spi-pxa2xx-pci.c) that needs to pass this clock as well. This is for Intel
CE4100 platform and we must make sure that it still works after our change.
--
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/