[PATCH] dma: mv_xor: remove mv_desc_get_dest_addr()

[Jason Cooper]

The following commit:

54f8d501e842 dmaengine: remove DMA unmap from drivers

removed the last caller to mv_desc_get_dest_addr(), creating the
warning:

drivers/dma/mv_xor.c:57:12: warning: 'mv_desc_get_dest_addr' defined
but not used [-Wunused-function]

Remove it.

Signed-off-by: Jason Cooper <ja...@lakedaemon.net>
---
drivers/dma/mv_xor.c | 6 ------
1 file changed, 6 deletions(-)

diff --git a/drivers/dma/mv_xor.c b/drivers/dma/mv_xor.c
index 7807f0ef4e20..23bcc9158cbc 100644
--- a/drivers/dma/mv_xor.c
+++ b/drivers/dma/mv_xor.c
@@ -54,12 +54,6 @@ static void mv_desc_init(struct mv_xor_desc_slot *desc, unsigned long flags)
hw_desc->desc_command = (1 << 31);
}

-static u32 mv_desc_get_dest_addr(struct mv_xor_desc_slot *desc)
-{
- struct mv_xor_desc *hw_desc = desc->hw_desc;
- return hw_desc->phy_dest_addr;
-}
-
static void mv_desc_set_byte_count(struct mv_xor_desc_slot *desc,
u32 byte_count)
{
--
1.8.4.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majo...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

[Jason Cooper]

On Mon, Nov 25, 2013 at 07:39:25PM +0000, Jason Cooper wrote:
> The following commit:
>
> 54f8d501e842 dmaengine: remove DMA unmap from drivers
>
> removed the last caller to mv_desc_get_dest_addr(), creating the
> warning:
>
> drivers/dma/mv_xor.c:57:12: warning: 'mv_desc_get_dest_addr' defined
> but not used [-Wunused-function]
>
> Remove it.
>
> Signed-off-by: Jason Cooper <ja...@lakedaemon.net>
> ---
> drivers/dma/mv_xor.c | 6 ------
> 1 file changed, 6 deletions(-)

Gah! I sent too fast. I meant to mention that I have this in my
mvebu/fixes branch, which gets merged into -next each night. (The
warning was bugging me :) )

Just let me know when you take it and I'll drop it from my branches.

thx,

Jason.

[Olof Johansson]

On Mon, Nov 25, 2013 at 11:46 AM, Jason Cooper <ja...@lakedaemon.net> wrote:
> On Mon, Nov 25, 2013 at 07:39:25PM +0000, Jason Cooper wrote:
>> The following commit:
>>
>> 54f8d501e842 dmaengine: remove DMA unmap from drivers
>>
>> removed the last caller to mv_desc_get_dest_addr(), creating the
>> warning:
>>
>> drivers/dma/mv_xor.c:57:12: warning: 'mv_desc_get_dest_addr' defined
>> but not used [-Wunused-function]
>>
>> Remove it.
>>
>> Signed-off-by: Jason Cooper <ja...@lakedaemon.net>
>> ---
>> drivers/dma/mv_xor.c | 6 ------
>> 1 file changed, 6 deletions(-)
>
> Gah! I sent too fast. I meant to mention that I have this in my
> mvebu/fixes branch, which gets merged into -next each night. (The
> warning was bugging me :) )
>
> Just let me know when you take it and I'll drop it from my branches.

Hmm. Please don't add patches for other subsystems into your -next
tree, since then there is no indication for those of us who follow
-next that the patch is not yet merged by the appropriate maintainer.


-Olof

[Jason Cooper]

On Wed, Nov 27, 2013 at 11:45:24AM -0800, Olof Johansson wrote:
> On Mon, Nov 25, 2013 at 11:46 AM, Jason Cooper <ja...@lakedaemon.net> wrote:
> > On Mon, Nov 25, 2013 at 07:39:25PM +0000, Jason Cooper wrote:
> >> The following commit:
> >>
> >> 54f8d501e842 dmaengine: remove DMA unmap from drivers
> >>
> >> removed the last caller to mv_desc_get_dest_addr(), creating the
> >> warning:
> >>
> >> drivers/dma/mv_xor.c:57:12: warning: 'mv_desc_get_dest_addr' defined
> >> but not used [-Wunused-function]
> >>
> >> Remove it.
> >>
> >> Signed-off-by: Jason Cooper <ja...@lakedaemon.net>
> >> ---
> >> drivers/dma/mv_xor.c | 6 ------
> >> 1 file changed, 6 deletions(-)
> >
> > Gah! I sent too fast. I meant to mention that I have this in my
> > mvebu/fixes branch, which gets merged into -next each night. (The
> > warning was bugging me :) )
> >
> > Just let me know when you take it and I'll drop it from my branches.
>
> Hmm. Please don't add patches for other subsystems into your -next
> tree, since then there is no indication for those of us who follow
> -next that the patch is not yet merged by the appropriate maintainer.

Ack. Dropped. Sorry for the confusion and thanks for setting me
straight.

thx,

Jason.

[Vinod Koul]

+ Dan

On Mon, Nov 25, 2013 at 07:39:25PM +0000, Jason Cooper wrote:

> The following commit:
>
> 54f8d501e842 dmaengine: remove DMA unmap from drivers
>
> removed the last caller to mv_desc_get_dest_addr(), creating the
> warning:
>
> drivers/dma/mv_xor.c:57:12: warning: 'mv_desc_get_dest_addr' defined
> but not used [-Wunused-function]
>
> Remove it.
>
> Signed-off-by: Jason Cooper <ja...@lakedaemon.net>

Acked-by: Vinod Koul <vinod...@intel.com>

This should go thru Dan's tree

[Russell King - ARM Linux]

On Thu, Nov 28, 2013 at 11:27:06AM +0530, Vinod Koul wrote:
> + Dan
>
> On Mon, Nov 25, 2013 at 07:39:25PM +0000, Jason Cooper wrote:
> > The following commit:
> >
> > 54f8d501e842 dmaengine: remove DMA unmap from drivers
> >
> > removed the last caller to mv_desc_get_dest_addr(), creating the
> > warning:
> >
> > drivers/dma/mv_xor.c:57:12: warning: 'mv_desc_get_dest_addr' defined
> > but not used [-Wunused-function]
> >
> > Remove it.
> >
> > Signed-off-by: Jason Cooper <ja...@lakedaemon.net>
> Acked-by: Vinod Koul <vinod...@intel.com>
>
> This should go thru Dan's tree

So what's happening with this patch? I don't see it in -rc yet.

[Vinod Koul]

On Mon, Dec 09, 2013 at 11:50:35PM +0000, Russell King - ARM Linux wrote:
> On Thu, Nov 28, 2013 at 11:27:06AM +0530, Vinod Koul wrote:
> > + Dan
> >
> > On Mon, Nov 25, 2013 at 07:39:25PM +0000, Jason Cooper wrote:
> > > The following commit:
> > >
> > > 54f8d501e842 dmaengine: remove DMA unmap from drivers
> > >
> > > removed the last caller to mv_desc_get_dest_addr(), creating the
> > > warning:
> > >
> > > drivers/dma/mv_xor.c:57:12: warning: 'mv_desc_get_dest_addr' defined
> > > but not used [-Wunused-function]
> > >
> > > Remove it.
> > >
> > > Signed-off-by: Jason Cooper <ja...@lakedaemon.net>
> > Acked-by: Vinod Koul <vinod...@intel.com>
> >
> > This should go thru Dan's tree
>
> So what's happening with this patch? I don't see it in -rc yet.

Dan has applied this to his next. So this should show us in next rc1

--
~Vinod

[Russell King - ARM Linux]

On Tue, Dec 10, 2013 at 03:43:31PM +0530, Vinod Koul wrote:
> On Mon, Dec 09, 2013 at 11:50:35PM +0000, Russell King - ARM Linux wrote:
> > On Thu, Nov 28, 2013 at 11:27:06AM +0530, Vinod Koul wrote:
> > > + Dan
> > >
> > > On Mon, Nov 25, 2013 at 07:39:25PM +0000, Jason Cooper wrote:
> > > > The following commit:
> > > >
> > > > 54f8d501e842 dmaengine: remove DMA unmap from drivers
> > > >
> > > > removed the last caller to mv_desc_get_dest_addr(), creating the
> > > > warning:
> > > >
> > > > drivers/dma/mv_xor.c:57:12: warning: 'mv_desc_get_dest_addr' defined
> > > > but not used [-Wunused-function]
> > > >
> > > > Remove it.
> > > >
> > > > Signed-off-by: Jason Cooper <ja...@lakedaemon.net>
> > > Acked-by: Vinod Koul <vinod...@intel.com>
> > >
> > > This should go thru Dan's tree
> >
> > So what's happening with this patch? I don't see it in -rc yet.
> Dan has applied this to his next. So this should show us in next rc1

Wrong answer. This is a compile regression introduced in the last merge
window, it should be going into an -rc kernel.

[Dan Williams]

On Tue, Dec 10, 2013 at 3:14 AM, Russell King - ARM Linux
<li...@arm.linux.org.uk> wrote:
> On Tue, Dec 10, 2013 at 03:43:31PM +0530, Vinod Koul wrote:
>> On Mon, Dec 09, 2013 at 11:50:35PM +0000, Russell King - ARM Linux wrote:
>> > On Thu, Nov 28, 2013 at 11:27:06AM +0530, Vinod Koul wrote:
>> > > + Dan
>> > >
>> > > On Mon, Nov 25, 2013 at 07:39:25PM +0000, Jason Cooper wrote:
>> > > > The following commit:
>> > > >
>> > > > 54f8d501e842 dmaengine: remove DMA unmap from drivers
>> > > >
>> > > > removed the last caller to mv_desc_get_dest_addr(), creating the
>> > > > warning:
>> > > >
>> > > > drivers/dma/mv_xor.c:57:12: warning: 'mv_desc_get_dest_addr' defined
>> > > > but not used [-Wunused-function]
>> > > >
>> > > > Remove it.
>> > > >
>> > > > Signed-off-by: Jason Cooper <ja...@lakedaemon.net>
>> > > Acked-by: Vinod Koul <vinod...@intel.com>
>> > >
>> > > This should go thru Dan's tree
>> >
>> > So what's happening with this patch? I don't see it in -rc yet.
>> Dan has applied this to his next. So this should show us in next rc1
>
> Wrong answer. This is a compile regression introduced in the last merge
> window, it should be going into an -rc kernel.

Vinod, if your fixes branch is ready I'll take it along with the rest
of the regression fixes to Linus.

--
Dan

[Dan Williams]

On Wed, Dec 11, 2013 at 1:10 AM, Dan Williams <dan.j.w...@intel.com> wrote:
> On Tue, Dec 10, 2013 at 3:14 AM, Russell King - ARM Linux
> <li...@arm.linux.org.uk> wrote:
>> On Tue, Dec 10, 2013 at 03:43:31PM +0530, Vinod Koul wrote:
>>> On Mon, Dec 09, 2013 at 11:50:35PM +0000, Russell King - ARM Linux wrote:
>>> > On Thu, Nov 28, 2013 at 11:27:06AM +0530, Vinod Koul wrote:
>>> > > + Dan
>>> > >
>>> > > On Mon, Nov 25, 2013 at 07:39:25PM +0000, Jason Cooper wrote:
>>> > > > The following commit:
>>> > > >
>>> > > > 54f8d501e842 dmaengine: remove DMA unmap from drivers
>>> > > >
>>> > > > removed the last caller to mv_desc_get_dest_addr(), creating the
>>> > > > warning:
>>> > > >
>>> > > > drivers/dma/mv_xor.c:57:12: warning: 'mv_desc_get_dest_addr' defined
>>> > > > but not used [-Wunused-function]
>>> > > >
>>> > > > Remove it.
>>> > > >
>>> > > > Signed-off-by: Jason Cooper <ja...@lakedaemon.net>
>>> > > Acked-by: Vinod Koul <vinod...@intel.com>
>>> > >
>>> > > This should go thru Dan's tree
>>> >
>>> > So what's happening with this patch? I don't see it in -rc yet.
>>> Dan has applied this to his next. So this should show us in next rc1
>>
>> Wrong answer. This is a compile regression introduced in the last merge
>> window, it should be going into an -rc kernel.
>
> Vinod, if your fixes branch is ready I'll take it along with the rest
> of the regression fixes to Linus.

Ok, I see you sent dma-slave/fixes up, I'll follow up with the rest:

Dan Williams (5):
dma: fix build warnings in ppc4xx
dma: fix fsldma build warnings
dmatest: fix build warning on mips
dma: fix build warnings in txx9
dmaengine: fix enable for high order unmap pools

Ezequiel Garcia (1):
dma: mv_xor: Use dmaengine_unmap_data for the self-tests

Jason Cooper (1):
dma: mv_xor: remove mv_desc_get_dest_addr()

Olof Johansson (1):
dmaengine: at_hdmac: remove unused function

Russell King (1):
dmaengine: mv_xor: fix oops when channels fail to initialise

[Vinod Koul]

Sorry I wasnt well last week, so I just managed to send the PULL on friday
morning. Wasnt sure if Linus will cut rc on friday or later, so sent it!

>
> Ok, I see you sent dma-slave/fixes up, I'll follow up with the rest:

Okay, did you manage to send it? If not I may have few (i think at least 1 more)

--
~Vinod

[Dan Williams]

Ok, glad you are feeling better.

>>
>> Ok, I see you sent dma-slave/fixes up, I'll follow up with the rest:
> Okay, did you manage to send it? If not I may have few (i think at least 1 more)
>

Patchwork id(s)? I want to send it it today.

I was busy tracking down a data corruption report with net_dma.

--
Dan

[Vinod Koul]

On Wed, Dec 18, 2013 at 08:55:23AM -0800, Dan Williams wrote:
> >> > Vinod, if your fixes branch is ready I'll take it along with the rest
> >> > of the regression fixes to Linus.
> > Sorry I wasnt well last week, so I just managed to send the PULL on friday
> > morning. Wasnt sure if Linus will cut rc on friday or later, so sent it!
>
> Ok, glad you are feeling better.

Thanks

>
> >>
> >> Ok, I see you sent dma-slave/fixes up, I'll follow up with the rest:
> > Okay, did you manage to send it? If not I may have few (i think at least 1 more)
> >
>
> Patchwork id(s)? I want to send it it today.

3266451, I have acked it!

--
~Vinod

--