Discussion:
[GIT PULL 01/02] DaVinci EDMA clean-up for v3.16
Sekhar Nori
2014-05-24 09:43:32 UTC
Permalink
Hi Olof, Kevin, Arnd,

These patches introduce a change to EDMA bindings. They
deprecate some bindings but new kernel will continue
to work with older DTBs since the information is now
read from hardware instead. I have not been able to
get an Ack/response from DT folks on this. Since it
has been close to two weeks since the patches were first
posted, I am asking for the patches to be merged so
we dont miss the merge window.

This pull request is on top of Vinod's topic/edma branch
which he has guaranteed will not be rebased before going
to Linus. This was needed because of dependency with some
patches he has queued.

This series introduces a trivial merge conflict with
Linux-next because of a bug fix which went in after
Vinod's branch was built.

Thanks,
Sekhar

The following changes since commit b0cce4ca3e740b5224d75634aa9d9abe9dfceabb:

dmaengine: edma: update DMA memcpy to use new param element (2014-04-30 10:36:57 +0530)

are available in the git repository at:

git://git.kernel.org/pub/scm/linux/kernel/git/nsekhar/linux-davinci.git tags/davinci-for-v3.16/edma

for you to fetch changes up to 903ed4913c7fe78d2746445564634264291c7493:

ARM: edma: Remove redundant/unused parameters from edma_soc_info (2014-05-22 14:55:25 +0530)

----------------------------------------------------------------
This series makes edma use configuration
information available within the IP instead
of reading it from platform data or DT. Some
other useful clean-ups are included too.

----------------------------------------------------------------
Peter Ujfalusi (14):
ARM: edma: Clean up and simplify the code around irq request
ARM: edma: No need to clean the pdata in edma_of_parse_dt()
ARM: edma: Take the number of tc from edma_soc_info (pdata)
ARM: edma: Do not change TC -> Queue mapping, leave it to default.
ARM: davinci: Remove eDMA3 queue_tc_mapping data from edma_soc_info
ARM: edma: Remove queue_tc_mapping data from edma_soc_info
ARM: edma: Remove num_cc member from struct edma
ARM: edma: Save number of regions from pdata to struct edma
ARM: edma: Get IP configuration from HW (number of channels, tc, etc)
dt/bindings: ti,edma: Remove redundant properties from documentation
ARM: dts: am33xx: Remove obsolete properties from edma node
ARM: dts: am4372: Remove obsolete properties from edma node
ARM: davinci: Remove redundant/unused parameters for edma
ARM: edma: Remove redundant/unused parameters from edma_soc_info

Documentation/devicetree/bindings/dma/ti-edma.txt | 13 +-
arch/arm/boot/dts/am33xx.dtsi | 3 -
arch/arm/boot/dts/am4372.dtsi | 3 -
arch/arm/common/edma.c | 175 ++++++++++-----------
arch/arm/mach-davinci/devices-da8xx.c | 31 ----
arch/arm/mach-davinci/dm355.c | 14 --
arch/arm/mach-davinci/dm365.c | 16 --
arch/arm/mach-davinci/dm644x.c | 14 --
arch/arm/mach-davinci/dm646x.c | 16 --
include/linux/platform_data/edma.h | 8 -
10 files changed, 93 insertions(+), 200 deletions(-)
Sekhar Nori
2014-05-24 09:43:33 UTC
Permalink
The following changes since commit c9eaa447e77efe77b7fa4c953bd62de8297fd6c5:

Linux 3.15-rc1 (2014-04-13 14:18:35 -0700)

are available in the git repository at:

git://git.kernel.org/pub/scm/linux/kernel/git/nsekhar/linux-davinci.git tags/davinci-for-v3.16/board

for you to fetch changes up to ea56785629494394b9e567c0a43690d6c972e137:

ARM: davinci: remove checks for CONFIG_USB_MUSB_PERIPHERAL (2014-05-22 16:29:08 +0530)

----------------------------------------------------------------
This patch clean-up an unused #define
from code.

----------------------------------------------------------------
Paul Bolle (1):
ARM: davinci: remove checks for CONFIG_USB_MUSB_PERIPHERAL

arch/arm/configs/davinci_all_defconfig | 1 -
arch/arm/mach-davinci/board-dm355-evm.c | 4 ----
arch/arm/mach-davinci/board-dm355-leopard.c | 4 ----
3 files changed, 9 deletions(-)
Olof Johansson
2014-05-26 21:44:09 UTC
Permalink
Post by Sekhar Nori
Linux 3.15-rc1 (2014-04-13 14:18:35 -0700)
git://git.kernel.org/pub/scm/linux/kernel/git/nsekhar/linux-davinci.git tags/davinci-for-v3.16/board
ARM: davinci: remove checks for CONFIG_USB_MUSB_PERIPHERAL (2014-05-22 16:29:08 +0530)
Merged into next/boards.


Thanks,

-Olof
Olof Johansson
2014-05-26 21:43:07 UTC
Permalink
Hi,
Post by Sekhar Nori
Hi Olof, Kevin, Arnd,
These patches introduce a change to EDMA bindings. They
deprecate some bindings but new kernel will continue
to work with older DTBs since the information is now
read from hardware instead. I have not been able to
get an Ack/response from DT folks on this. Since it
has been close to two weeks since the patches were first
posted, I am asking for the patches to be merged so
we dont miss the merge window.
That should be fine. I also noticed that the bindings file still describes the
properties but under a deprecated section, that's good practice.
Post by Sekhar Nori
This pull request is on top of Vinod's topic/edma branch
which he has guaranteed will not be rebased before going
to Linus. This was needed because of dependency with some
patches he has queued.
Ok, I've created a depends/vinod-edma to track the dependency, starting at
SHA b0cce4ca3e74. I've also cc:d him on this reply so he's aware of this.
Post by Sekhar Nori
This series introduces a trivial merge conflict with
Linux-next because of a bug fix which went in after
Vinod's branch was built.
Thanks,
Sekhar
dmaengine: edma: update DMA memcpy to use new param element (2014-04-30 10:36:57 +0530)
git://git.kernel.org/pub/scm/linux/kernel/git/nsekhar/linux-davinci.git tags/davinci-for-v3.16/edma
ARM: edma: Remove redundant/unused parameters from edma_soc_info (2014-05-22 14:55:25 +0530)
Merged into next/drivers.


-Olof
Olof Johansson
2014-05-26 21:58:58 UTC
Permalink
Post by Olof Johansson
Hi,
Post by Sekhar Nori
Hi Olof, Kevin, Arnd,
These patches introduce a change to EDMA bindings. They
deprecate some bindings but new kernel will continue
to work with older DTBs since the information is now
read from hardware instead. I have not been able to
get an Ack/response from DT folks on this. Since it
has been close to two weeks since the patches were first
posted, I am asking for the patches to be merged so
we dont miss the merge window.
That should be fine. I also noticed that the bindings file still describes the
properties but under a deprecated section, that's good practice.
Post by Sekhar Nori
This pull request is on top of Vinod's topic/edma branch
which he has guaranteed will not be rebased before going
to Linus. This was needed because of dependency with some
patches he has queued.
Ok, I've created a depends/vinod-edma to track the dependency, starting at
SHA b0cce4ca3e74. I've also cc:d him on this reply so he's aware of this.
Post by Sekhar Nori
This series introduces a trivial merge conflict with
Linux-next because of a bug fix which went in after
Vinod's branch was built.
Thanks,
Sekhar
dmaengine: edma: update DMA memcpy to use new param element (2014-04-30 10:36:57 +0530)
git://git.kernel.org/pub/scm/linux/kernel/git/nsekhar/linux-davinci.git tags/davinci-for-v3.16/edma
ARM: edma: Remove redundant/unused parameters from edma_soc_info (2014-05-22 14:55:25 +0530)
Merged into next/drivers.
This had a conflict with the fix from Thomas that fixes the binding. To
avoid this, you could have merged Vinod's branch in on top of the fix branch
you had, then build your new contents on top.

Anyway, not a huge deal, but something to tweak in the future.


-Olof
Sekhar Nori
2014-05-27 08:10:18 UTC
Permalink
Post by Olof Johansson
This had a conflict with the fix from Thomas that fixes the binding. To
avoid this, you could have merged Vinod's branch in on top of the fix branch
you had, then build your new contents on top.
I did notice the conflict, but did not think about merging my fixes
branch in.
Post by Olof Johansson
Anyway, not a huge deal, but something to tweak in the future.
Thanks. Will keep in mind.

Regards,
Sekhar

Loading...