RE: [PATCH 4/8] of: dma: Split of_configure_dma() into mask and ops configuration

2016-09-09 Thread Sricharan
Hi Magnus, >On Tue, Aug 9, 2016 at 7:49 AM, Sricharan R wrote: >> From: Laurent Pinchart >> >> The of_configure_dma() function configures both the DMA masks and ops. >> Moving DMA ops configuration to probe time would thus also delay >> configuration of the DMA masks, which might not be safe. To

Re: [PATCH 4/8] of: dma: Split of_configure_dma() into mask and ops configuration

2016-09-08 Thread Magnus Damm
On Tue, Aug 9, 2016 at 7:49 AM, Sricharan R wrote: > From: Laurent Pinchart > > The of_configure_dma() function configures both the DMA masks and ops. > Moving DMA ops configuration to probe time would thus also delay > configuration of the DMA masks, which might not be safe. To avoid issues > sp

RE: [PATCH 4/8] of: dma: Split of_configure_dma() into mask and ops configuration

2016-08-12 Thread Sricharan
Hi, >> > The of_configure_dma() function configures both the DMA masks and ops. >> > Moving DMA ops configuration to probe time would thus also delay >> > configuration of the DMA masks, which might not be safe. To avoid issues >> >> Do we know any example cases when it might be unsafe? I think we

Re: [PATCH 4/8] of: dma: Split of_configure_dma() into mask and ops configuration

2016-08-12 Thread Laurent Pinchart
Hi Tomasz, On Friday 12 Aug 2016 16:31:47 Tomasz Figa wrote: > On Tue, Aug 9, 2016 at 7:49 AM, Sricharan R wrote: > > From: Laurent Pinchart > > > > The of_configure_dma() function configures both the DMA masks and ops. > > Moving DMA ops configuration to probe time would thus also delay > > con

Re: [PATCH 4/8] of: dma: Split of_configure_dma() into mask and ops configuration

2016-08-12 Thread Tomasz Figa
Hi, On Tue, Aug 9, 2016 at 7:49 AM, Sricharan R wrote: > From: Laurent Pinchart > > The of_configure_dma() function configures both the DMA masks and ops. > Moving DMA ops configuration to probe time would thus also delay > configuration of the DMA masks, which might not be safe. To avoid issues