lkml.org 
[lkml]   [2014]   [Sep]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 0/3 v2] drivers/bus: Freescale Management Complex bus driver patch series
Date


> -----Original Message-----
> From: Kim Phillips [mailto:kim.phillips@freescale.com]
> Sent: Monday, September 22, 2014 11:53 AM
> To: Rivera Jose-B46482
> Cc: gregkh@linuxfoundation.org; arnd@arndb.de; linux-kernel@vger.kernel.org; Yoder Stuart-B08248; Wood Scott-
> B07421; agraf@suse.de; linuxppc-release@linux.freescale.net
> Subject: Re: [PATCH 0/3 v2] drivers/bus: Freescale Management Complex bus driver patch series
>
> On Fri, 19 Sep 2014 17:49:38 -0500
> "J. German Rivera" <German.Rivera@freescale.com> wrote:
>
> > CHANGE HISTORY
> >
> > Issues pending resolution not addressed by v2:
> > - What to do with Doxygen comments in patch 1
>
> It's clear they should be removed.
>
> > - Whether to move or not FSL-specific header files added in include/linux,
> > by this patch series, to another location
>
> there wasn't a valid objection against moving them under fsl/ and
> changing them to use dashes instead of underscores, was there?

There was no objection, but here is the observation. The current
convention seems to be that under include/linux are 'subsystem'
types--
include/linux/mmc
include/linux/spi
include/linux/raid
etc

There is no other "company" that has an include/linux/[company-name] that I can
see. Freescale seems to be the only one. And there is only a single driver
in there. So it looks like a complete anomaly.

Why is that?

I guess we could try moving our stuff to incluce/linux/fsl and see if there is
any negative feedback on it.

Stuart


\
 
 \ /
  Last update: 2014-09-22 20:41    [W:0.112 / U:0.516 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site