linux-kernel - Re: [PATCH 1/3] drivers/misc: add support for BMP18x pressure sensors

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: <20120301001143.02f6c8c4@pyramind.ukuu.org.uk>
Date:	Thu, 1 Mar 2012 00:11:43 +0000
From:	Alan Cox <alan@...rguk.ukuu.org.uk>
To:	Eric Andersson <eric.andersson@...xphere.com>
Cc:	linux-kernel@...r.kernel.org, arnd@...db.de,
	gregkh@...uxfoundation.org, christoph.mair@...il.com,
	stefan.nilsson@...xphere.com, zhengguang.guo@...ch-sensortec.com,
	peter.moeller@...bosch.com
Subject: Re: [PATCH 1/3] drivers/misc: add support for BMP18x pressure
 sensors

On Wed, 29 Feb 2012 20:58:07 +0100
Eric Andersson <eric.andersson@...xphere.com> wrote:

> This driver adds support for Bosch Sensortec's pressure sensors
> BMP085 and BMP18x.
> 
> The driver is based on the bmp085 driver by Christoph Mair
> and uses the same sysfs interface.

This is really extending bmp085 to cover the 18x chips. We don't normally
rename everything for that, and you risk breaking existing setups
assuming the module name is bmp085.

So please just "update" the existing driver - ie the code looks fine but
it should still end up being bmp085.

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