Re: [PATCHv3 3/3] misc: add support for bmp18x chips to the bmp085driver

From: Eric Andersson
Date: Thu Apr 05 2012 - 02:53:06 EST


Hi,

Thanks for the review guys!

On 21:11 Wed 04 Apr , Mark Brown wrote:
> On Wed, Apr 04, 2012 at 08:00:33PM +0000, Arnd Bergmann wrote:
> > On Wednesday 04 April 2012, Mark Brown wrote:
> > > > +static const struct of_device_id bmp085_of_match[] = {
> > > > + { .compatible = "bosch-sensortec,bmp085", },
>
> > > Traditionally the stock ticker symbol would be used.
>
> > I though about commenting on that as well, but since Bosch is a foundation
> > and not publically traded, that would be hard to accomplish. Is there
> > an official fallback for companies that don't have a stock ticker symbol?
>
> Guess we should ask the DT people... I mostly just noticed that the
> name appeared somewhat verbose - perhaps just plain "bosch" if there's
> not a better abbrevation?

If there are no further objections let's go with "bosch". I'll prepare a
new patch-set ASAP.

--
Best regards,
Eric

http://www.unixphere.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/