Search Linux Wireless

Re: [Rt2400-devel] [RFC/T] rt2x00: Remove redundant code in rfkill setup

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday 27 November 2007, Larry Finger wrote:
> In rt2x00rfkill.c, routine input_allocate_device() is called even though
> input_allocate_polled_device(), which was just called, includes a call to
> that routine. This patch, which has not been tested, removes the redundant code.

Either I was blind for not noticing that before, or the behavior has changed recently. ;)
Applied to rt2x00.git, I will push it to John in a few minutes along with some other patches.

Thanks.

Ivo

> Signed-off-by: Larry Finger <Larry.Finger@xxxxxxxxxxxx>
> ---
> 
> Index: wireless-2.6/drivers/net/wireless/rt2x00/rt2x00rfkill.c
> ===================================================================
> --- wireless-2.6.orig/drivers/net/wireless/rt2x00/rt2x00rfkill.c
> +++ wireless-2.6/drivers/net/wireless/rt2x00/rt2x00rfkill.c
> @@ -141,12 +141,6 @@ int rt2x00rfkill_allocate(struct rt2x00_
>  	rt2x00dev->poll_dev->poll = rt2x00rfkill_poll;
>  	rt2x00dev->poll_dev->poll_interval = RFKILL_POLL_INTERVAL;
>  
> -	rt2x00dev->poll_dev->input = input_allocate_device();
> -	if (!rt2x00dev->poll_dev->input) {
> -		ERROR(rt2x00dev, "Failed to allocate input device.\n");
> -		goto exit_free_polldev;
> -	}
> -
>  	rt2x00dev->poll_dev->input->name = rt2x00dev->ops->name;
>  	rt2x00dev->poll_dev->input->phys = wiphy_name(rt2x00dev->hw->wiphy);
>  	rt2x00dev->poll_dev->input->id.bustype = BUS_HOST;
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2005.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> _______________________________________________
> Rt2400-devel mailing list
> Rt2400-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.sourceforge.net/lists/listinfo/rt2400-devel
> 


-
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux