linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/2] DMA: AMBA PL08x DMA driver fixes due to ARCH_DMA_ADDR_T_64BIT
Date: Wed, 14 Aug 2013 14:00:19 +0100	[thread overview]
Message-ID: <20130814130018.GR23006@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <1376484729-11826-1-git-send-email-andre.przywara@linaro.org>

On Wed, Aug 14, 2013 at 02:52:07PM +0200, Andre Przywara wrote:
> Rob's recent pull request (Highbank fixes for 3.12) promoted the
> dma_addr_t to 64bit:
> commit ac072f427a4266b28a948c67a97a8c46f49b662d
> Author: Rob Herring <rob.herring@calxeda.com>
> Date:   Tue Aug 13 15:55:07 2013 -0500
> 
>     ARM: highbank: select ARCH_DMA_ADDR_T_64BIT for LPAE
>     
> Unfortunately this breaks the AMBA PL08x driver.
> Fixes are pretty easy: replacing modulo with AND and fix printk
> format types.

I thought I effectively said no to that patch, and told Rob to go and look
at my patch series.

Your patch series is wrong to fix this.  We're just piling more bad fixes
on top of a bug in the generic parts of the kernel.  Please stop this
madness.

  parent reply	other threads:[~2013-08-14 13:00 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-14 12:52 [PATCH 0/2] DMA: AMBA PL08x DMA driver fixes due to ARCH_DMA_ADDR_T_64BIT Andre Przywara
2013-08-14 12:52 ` [PATCH 1/2] DMA: fix AMBA PL08x driver issue with 64bit DMA address type Andre Przywara
2013-08-14 13:00   ` Russell King - ARM Linux
2013-08-14 19:13     ` Rob Herring
2013-08-15 21:07       ` Andre Przywara
2013-08-15 21:15         ` Russell King - ARM Linux
2013-08-19 10:19           ` [PATCH v2] DMA: fix AMBA PL08x compilation " Andre Przywara
2013-08-19 13:26             ` Rob Herring
2013-08-21  6:17             ` Vinod Koul
2013-08-21 21:01   ` [PATCH 1/2] DMA: fix AMBA PL08x driver " Linus Walleij
2013-08-21 21:22     ` Andre Przywara
2013-08-21 21:49     ` Matt Sealey
2013-08-14 12:52 ` [PATCH 2/2] DMA: fix printk warning in AMBA PL08x DMA driver Andre Przywara
2013-08-14 13:00   ` Russell King - ARM Linux
2013-08-14 13:00 ` Russell King - ARM Linux [this message]
2013-08-14 14:05   ` [PATCH 0/2] DMA: AMBA PL08x DMA driver fixes due to ARCH_DMA_ADDR_T_64BIT Rob Herring

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130814130018.GR23006@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).