Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1740384

Re: [PATCH] net: stmmac: Meet alignment requirements for DMA

From Paul Burton <paul.burton@imgtec.com>
Newsgroups linux.kernel
Subject Re: [PATCH] net: stmmac: Meet alignment requirements for DMA
Date 2017-09-27 06:40 +0200
Message-ID <uuciB-5Yf-3@gated-at.bofh.it> (permalink)
References <uu35D-8ph-9@gated-at.bofh.it> <uu5K9-1HA-1@gated-at.bofh.it> <uuaJQ-4QW-17@gated-at.bofh.it>
Organization Imagination Technologies

Show all headers | View raw


[Multipart message — attachments visible in raw view] - view raw

Hi David,

On Tuesday, 26 September 2017 19:52:44 PDT David Miller wrote:
> From: Paul Burton <paul.burton@imgtec.com>
> Date: Tue, 26 Sep 2017 14:30:33 -0700
> 
> > I'd suggest that at a minimum if you're unwilling to obey the API as
> > described in Documentation/DMA-API.txt then it would be beneficial
> > if you could propose a change to it such that it works for you, and
> > perhaps we can extend the API & its documentation to allow your
> > usage whilst also allowing us to catch broken uses.
> 
> The networking driver code works fine as is.
> 
> I also didn't write that ill-advised documentation in the DMA docs,
> nor the non-merged new MIPS assertion.
> 
> So I'm trying to figure out on what basis I am required to do
> anything.
> 
> Thank you.

Nobody said you wrote the documentation, but you do maintain code which 
disobeys the documented DMA API & now you're being an ass about it 
unnecessarily.

Nobody said that you are required to do anything, I suggested that it would be 
beneficial if you were to suggest a change to the documented DMA API such that 
it allows your usage where it currently does not. If you don't want to have 
any input into that, and you actually think that your current approach of 
ignoring the documented API is the best path forwards, then we're probably 
done here & I'll be making a note to avoid yourself & anything under net/ to 
whatever extent is possible...

Thanks,
    Paul

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] net: stmmac: Meet alignment requirements for DMA Matt Redfearn <matt.redfearn@imgtec.com> - 2017-09-22 13:20 +0200
  Re: [PATCH] net: stmmac: Meet alignment requirements for DMA David Miller <davem@davemloft.net> - 2017-09-23 03:30 +0200
    Re: [PATCH] net: stmmac: Meet alignment requirements for DMA Matt Redfearn <matt.redfearn@imgtec.com> - 2017-09-26 16:00 +0200
      RE: [PATCH] net: stmmac: Meet alignment requirements for DMA David Laight <David.Laight@ACULAB.COM> - 2017-09-26 16:30 +0200
      Re: [PATCH] net: stmmac: Meet alignment requirements for DMA David Miller <davem@davemloft.net> - 2017-09-26 19:40 +0200
        Re: [PATCH] net: stmmac: Meet alignment requirements for DMA Paul Burton <paul.burton@imgtec.com> - 2017-09-26 20:50 +0200
          Re: [PATCH] net: stmmac: Meet alignment requirements for DMA David Miller <davem@davemloft.net> - 2017-09-26 22:40 +0200
            Re: [PATCH] net: stmmac: Meet alignment requirements for DMA Paul Burton <paul.burton@imgtec.com> - 2017-09-26 23:40 +0200
              Re: [PATCH] net: stmmac: Meet alignment requirements for DMA David Miller <davem@davemloft.net> - 2017-09-27 05:00 +0200
                Re: [PATCH] net: stmmac: Meet alignment requirements for DMA Paul Burton <paul.burton@imgtec.com> - 2017-09-27 06:40 +0200
                Re: [PATCH] net: stmmac: Meet alignment requirements for DMA David Miller <davem@davemloft.net> - 2017-09-27 07:00 +0200
                Re: [PATCH] net: stmmac: Meet alignment requirements for DMA Paul Burton <paul.burton@imgtec.com> - 2017-09-27 07:30 +0200

csiph-web