Re: [PATCH v2 1/2] DMA: PL330: Merge PL330 driver into drivers/dma/

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Feb 08, 2012 at 10:10:30AM +0900, Boojin Kim wrote:
> Russell King - ARM Linux wrote:
> >
> > Now that the PL330 code is entirely out of arch/arm, the header should no
> > longer be in asm/hardware/pl330.h.
> >
> > Definitions private to the driver should be in drivers/dma/pl330.c or a
> > header file co-located.  Other definitions for interfaces to that driver
> > (eg, platform data) should be in include/linux/amba.
> >
> > That can be the subject of a follow-up patch if this is already finalized.
> >
> Sorry to late response. I believed my reply was sent. But, It maybe be not.
> Anyway.. get to the point.
> Pl08x also has asm/hardware/pl080.h to include hw specific information.

History basically.

> It seems to be same purpose with asm/hardware/pl330.h
> Do you mean both asm/hardware/pl080.h and asm/hardware/pl330.h should be
> moved to include/linux/amba?

No.  asm/hardware/pl330.h Should go into drivers/dma if nothing but the
PL330 DMA driver in there needs it.

Same for pl080.h but as your patches isn't touching that, then that's not
something that should be done separately.
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux