lkml.org 
[lkml]   [2014]   [Sep]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 3/3] staging: dgap: use dgap_release_remap() in dgap_cleanup_board()
From
Hi,

2014-09-24 18:45 GMT+09:00 Dan Carpenter <dan.carpenter@oracle.com>:
> On Tue, Sep 23, 2014 at 09:22:36AM +0900, Daeseok Youn wrote:
>> static void dgap_release_remap(struct board_t *brd)
>> {
>> - release_mem_region(brd->membase, 0x200000);
>> - release_mem_region(brd->membase + PCI_IO_OFFSET, 0x200000);
>> - iounmap(brd->re_map_membase);
>> - iounmap(brd->re_map_port);
>> + if (brd->re_map_membase) {
>
> I hate this if conditions. Let's not complicate dgap_release_remap(),
> let's fix the callers instead. Especially dgap_init_module() is a
> totally sucky function with bad unwinding.
ok. then,
if (brd->re_map_membase && brd->re_map_port)
dgap_release_remap(brd);

right?
>
> Let's not apply this patch.
oh.. this patch was already applied to staging tree.
I will make different patch for this.

thanks.

regards,
Daeseok Youn.
>
> regards,
> dan carpenter
>


\
 
 \ /
  Last update: 2014-09-25 02:41    [W:0.083 / U:1.000 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site