linux-kernel - Re: [PATCH v3 1/2] mxs-dma : move the mxs dma.h to a more common place

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: <CAMiH66EP6UFDYkguzfSFgysDzVREPZ8P1x29TkYvQ+Cuvz-4hg@mail.gmail.com>
Date:	Sat, 11 Feb 2012 07:36:50 +0800
From:	Huang Shijie <shijie8@...il.com>
To:	Shawn Guo <shawn.guo@...aro.org>
Cc:	Huang Shijie <b32955@...escale.com>, vinod.koul@...el.com,
	artem.bityutskiy@...el.com, linux-arm-kernel@...ts.infradead.org,
	linux-mtd@...ts.infradead.org, linux-mmc@...r.kernel.org,
	linux-kernel@...r.kernel.org, w.sang@...gutronix.de,
	LW@...o-electronics.de, alsa-devel@...a-project.org,
	b29396@...escale.com
Subject: Re: [PATCH v3 1/2] mxs-dma : move the mxs dma.h to a more common place

hi,

On Sat, Feb 11, 2012 at 12:21 AM, Shawn Guo <shawn.guo@...aro.org> wrote:
> On Tue, Feb 07, 2012 at 06:54:34PM +0800, Huang Shijie wrote:
>> Move the header to a more common place.
>> The mxs dma engine is not only used in mx23/mx28, but also used
>> in mx50/mx6q.  It will also be used in the future chips.
>>
>> Rename it to mxs-dma.h
>>
>> change mxs-dma driver, mxs-mmc driver, gpmi-nand driver, mxs-saif driver
>> to the new header file.
>>
>> Signed-off-by: Huang Shijie <b32955@...escale.com>
>> ---
>>  drivers/dma/mxs-dma.c                              |    2 +-
>>  drivers/mmc/host/mxs-mmc.c                         |    2 +-
>>  drivers/mtd/nand/gpmi-nand/gpmi-nand.h             |    2 +-
>>  .../include/mach/dma.h => include/linux/mxs-dma.h  |    0
>>  sound/soc/mxs/mxs-pcm.c                            |    2 +-
>>  sound/soc/mxs/mxs-pcm.h                            |    2 +-
>>  sound/soc/mxs/mxs-saif.c                           |    2 +-
>>  7 files changed, 6 insertions(+), 6 deletions(-)
>>  rename arch/arm/mach-mxs/include/mach/dma.h => include/linux/mxs-dma.h (100%)
>
> I suggested you rename it include/linux/fsl/mxs-dma.h [1].  It seems
> you are ignoring the comment.  Can you explain why it's not a
> reasonable suggestion?

For gpmi-nand.h is placed in include/linux/mtd/ directory, so i think
it's better to place the mxs-dma.h
to similar include/linux/dma/ directory, but the directory does not exit.

If i place it at include/linux/fsl/mxs-dma.h , i have to create a new
folder, I am not sure i have the right
to do so. But i can try it in the next version.

thanks a lot.



>
> And Wolfram and Mark told you that you need to Cc subsystem maintainers
> to collect their acks [2].  But you are still not Ccing Mark Brown and
> Chris Ball for sound/soc/ and drivers/mmc changes.  So you are not
> listening.

ok, i will CC them in next version.  I　really did not know to whom I　should cc .

BR
Huang Shijie

>
> Regards,
> Shawn
>
> [1] http://article.gmane.org/gmane.linux.kernel.mmc/12301
> [2] http://article.gmane.org/gmane.linux.alsa.devel/93751
--
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/