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

Groups > linux.kernel > #1282359 > unrolled thread

[PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with dma_pool_zalloc

Back to article view | Back to linux.kernel

Contents

  [PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with dma_pool_zalloc "Geyslan G. Bem" <geyslan@gmail.com> - 2015-12-02 21:50 +0100
    Re: [PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with  dma_pool_zalloc Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2015-12-04 17:30 +0100
      Re: [PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with dma_pool_zalloc "Geyslan G. Bem" <geyslan@gmail.com> - 2015-12-04 17:40 +0100
        Re: [PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with  dma_pool_zalloc Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2015-12-04 17:50 +0100
          Re: [PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with  dma_pool_zalloc Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2015-12-04 18:20 +0100
            Re: [PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with dma_pool_zalloc "Geyslan G. Bem" <geyslan@gmail.com> - 2015-12-04 19:10 +0100
          Re: [PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with dma_pool_zalloc "Geyslan G. Bem" <geyslan@gmail.com> - 2015-12-04 18:20 +0100

#1282359 — [PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with dma_pool_zalloc

Replace dma_pool_alloc and memset with a single call to dma_pool_zalloc.

Caught by coccinelle.

Signed-off-by: Geyslan G. Bem <geyslan@gmail.com>
---
 drivers/usb/host/whci/qset.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/usb/host/whci/qset.c b/drivers/usb/host/whci/qset.c
index dc31c42..3297473 100644
--- a/drivers/usb/host/whci/qset.c
+++ b/drivers/usb/host/whci/qset.c
@@ -30,10 +30,9 @@ struct whc_qset *qset_alloc(struct whc *whc, gfp_t mem_flags)
 	struct whc_qset *qset;
 	dma_addr_t dma;
 
-	qset = dma_pool_alloc(whc->qset_pool, mem_flags, &dma);
+	qset = dma_pool_zalloc(whc->qset_pool, mem_flags, &dma);
 	if (qset == NULL)
 		return NULL;
-	memset(qset, 0, sizeof(struct whc_qset));
 
 	qset->qset_dma = dma;
 	qset->whc = whc;
-- 
2.6.2

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

[toc] | [next] | [standalone]

#1283986 — Re: [PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with dma_pool_zalloc

On Wed, Dec 02, 2015 at 05:43:11PM -0300, Geyslan G. Bem wrote:
> Replace dma_pool_alloc and memset with a single call to dma_pool_zalloc.
> 
> Caught by coccinelle.
> 
> Signed-off-by: Geyslan G. Bem <geyslan@gmail.com>
> ---
>  drivers/usb/host/whci/qset.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

This, and patch 3/3 were also already done by someone else :(
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1284005

2015-12-04 13:25 GMT-03:00 Greg Kroah-Hartman <gregkh@linuxfoundation.org>:
> On Wed, Dec 02, 2015 at 05:43:11PM -0300, Geyslan G. Bem wrote:
>> Replace dma_pool_alloc and memset with a single call to dma_pool_zalloc.
>>
>> Caught by coccinelle.
>>
>> Signed-off-by: Geyslan G. Bem <geyslan@gmail.com>
>> ---
>>  drivers/usb/host/whci/qset.c | 3 +--
>>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> This, and patch 3/3 were also already done by someone else :(

I see. No problem at all.

If you wish, you could list me subsystems in need of maintenance.

Thanks.

-- 
Regards,

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

[toc] | [prev] | [next] | [standalone]

#1284008 — Re: [PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with dma_pool_zalloc

On Fri, Dec 04, 2015 at 01:33:18PM -0300, Geyslan G. Bem wrote:
> 
> If you wish, you could list me subsystems in need of maintenance.

I don't understand what you mean by this.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1284052 — Re: [PATCH 2/3] usb: whci: replace dma_pool_alloc and memset with dma_pool_zalloc

On Fri, Dec 04, 2015 at 02:13:35PM -0300, Geyslan G. Bem wrote:
> 2015-12-04 13:44 GMT-03:00 Greg Kroah-Hartman <gregkh@linuxfoundation.org>:
> > On Fri, Dec 04, 2015 at 01:33:18PM -0300, Geyslan G. Bem wrote:
> >>
> >> If you wish, you could list me subsystems in need of maintenance.
> >
> > I don't understand what you mean by this.
> 
> I meant that would be great to know which usb subsystems you do prefer
> that I continue cleaning/refactoring or if any would be ok. :)

Any part of the drivers/usb/* tree that you find needs cleanups and
fixes like these are great to do, please send patches for this, I'll
gladly take them.

thanks,

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

[toc] | [prev] | [next] | [standalone]

#1284130

2015-12-04 14:18 GMT-03:00 Greg Kroah-Hartman <gregkh@linuxfoundation.org>:
> On Fri, Dec 04, 2015 at 02:13:35PM -0300, Geyslan G. Bem wrote:
>> 2015-12-04 13:44 GMT-03:00 Greg Kroah-Hartman <gregkh@linuxfoundation.org>:
>> > On Fri, Dec 04, 2015 at 01:33:18PM -0300, Geyslan G. Bem wrote:
>> >>
>> >> If you wish, you could list me subsystems in need of maintenance.
>> >
>> > I don't understand what you mean by this.
>>
>> I meant that would be great to know which usb subsystems you do prefer
>> that I continue cleaning/refactoring or if any would be ok. :)
>
> Any part of the drivers/usb/* tree that you find needs cleanups and
> fixes like these are great to do, please send patches for this, I'll
> gladly take them.

Nice, I'll indeed.

Thanks.
>
> thanks,
>
> gre k-h



-- 
Regards,

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

[toc] | [prev] | [next] | [standalone]

#1284063

2015-12-04 13:44 GMT-03:00 Greg Kroah-Hartman <gregkh@linuxfoundation.org>:
> On Fri, Dec 04, 2015 at 01:33:18PM -0300, Geyslan G. Bem wrote:
>>
>> If you wish, you could list me subsystems in need of maintenance.
>
> I don't understand what you mean by this.

I meant that would be great to know which usb subsystems you do prefer
that I continue cleaning/refactoring or if any would be ok. :)


-- 
Regards,

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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web