[PATCH 4/5] sh: split arch/sh/mm/consistent.c

2018-07-25 Thread Christoph Hellwig
Half of the file just contains platform device memory setup code which is required for all builds, and half contains helpers for dma coherent allocation, which is only needed if CONFIG_DMA_NONCOHERENT is enabled. Signed-off-by: Christoph Hellwig --- arch/sh/kernel/Makefile | 2 +- arch/sh

RE: [PATCH 4/5] sh: split arch/sh/mm/consistent.c

2018-07-24 Thread Li, Philip
> Subject: Re: [PATCH 4/5] sh: split arch/sh/mm/consistent.c > > On Mon, Jul 23, 2018 at 10:49:39AM +0200, Geert Uytterhoeven wrote: > > > > + *dma_handle = virt_to_phys(ret); > > > > + if (!WARN_ON(!dev)) > > > > + *dma_han

[PATCH 4/5] sh: split arch/sh/mm/consistent.c

2018-07-24 Thread Christoph Hellwig
Half of the file just contains platform device memory setup code which is required for all builds, and half contains helpers for dma coherent allocation, which is only needed if CONFIG_DMA_NONCOHERENT is enabled. Signed-off-by: Christoph Hellwig --- arch/sh/kernel/Makefile | 2 +- arch/sh

Re: [PATCH 4/5] sh: split arch/sh/mm/consistent.c

2018-07-24 Thread Christoph Hellwig
On Mon, Jul 23, 2018 at 10:49:39AM +0200, Geert Uytterhoeven wrote: > > > + *dma_handle = virt_to_phys(ret); > > > + if (!WARN_ON(!dev)) > > > + *dma_handle - PFN_PHYS(dev->dma_pfn_offset); > > vs > > > - *dma_handle = virt_to_phys(ret); > > > - if (!WARN_ON(!dev)) > > >

Re: [PATCH 4/5] sh: split arch/sh/mm/consistent.c

2018-07-23 Thread jacopo mondi
Hi Christoph, I have a single comment on something I noticed which might be an implementation bug. On the overall patch architecture, it's very hard for me to provide a valuable opinion as it's all relatively new for me here :) On Thu, Jul 19, 2018 at 06:05:15AM -0700, Christoph Hellwig wrote: >

Re: [PATCH 4/5] sh: split arch/sh/mm/consistent.c

2018-07-23 Thread Geert Uytterhoeven
Hi Jacopo, On Mon, Jul 23, 2018 at 10:42 AM jacopo mondi wrote: > I have a single comment on something I noticed which might be an > implementation bug. On the overall patch architecture, it's very hard > for me to provide a valuable opinion as it's all relatively new for me > here :) > > On Thu,

[PATCH 4/5] sh: split arch/sh/mm/consistent.c

2018-07-19 Thread Christoph Hellwig
Half of the file just contains platform device memory setup code which is required for all builds, and half contains helpers for dma coherent allocation, which is only needed if CONFIG_DMA_NONCOHERENT is enabled. Signed-off-by: Christoph Hellwig --- arch/sh/kernel/Makefile | 2 +- arch/sh