All of lore.kernel.org
 help / color / mirror / Atom feed
From: Roger Quadros <roger.quadros-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
To: ext Ohad Ben-Cohen <ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>
Cc: "linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	"linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org"
	<linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
	Chikkature Rajashekar Madhusudhan
	<madhu.cr-l0cyMroinI0@public.gmane.org>,
	"Coelho Luciano (Nokia-MS/Helsinki)"
	<Luciano.Coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>,
	"akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org"
	<akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>,
	San Mehat <san-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>,
	Ohad Ben-Cohen <ohadb-l0cyMroinI0@public.gmane.org>
Subject: Re: [PATCH 11/15] wireless: wl1271: introduce platform device support
Date: Tue, 06 Jul 2010 11:53:45 +0300	[thread overview]
Message-ID: <4C32EF19.1000604@nokia.com> (raw)
In-Reply-To: <1278376666-3509-12-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>

Hi,

On 07/06/2010 03:37 AM, ext Ohad Ben-Cohen wrote:
> From: Ohad Ben-Cohen<ohadb-l0cyMroinI0@public.gmane.org>
>
> Introduce a platform device support which is decoupled
> from the life cycle of the sdio function.
>
> The platform device objective is to deliver board-specific
> values (like irq, ref_clock, and software card-detect
> emulation control).
>
> The driver can then dynamically create (or destroy)
> the sdio function whenever the wlan interface is
> brought up (or down), as part of the power() operation.

Is this the right approach? Why should you emulate a card disconnect event when 
the wlan interface is brought down?

Physically the card is never disconnected. So I don't see why we should fake it.
Could you please explain why you need to do this?

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

WARNING: multiple messages have this Message-ID (diff)
From: Roger Quadros <roger.quadros@nokia.com>
To: ext Ohad Ben-Cohen <ohad@wizery.com>
Cc: "linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	"linux@arm.linux.org.uk" <linux@arm.linux.org.uk>,
	Chikkature Rajashekar Madhusudhan <madhu.cr@ti.com>,
	"Coelho Luciano (Nokia-MS/Helsinki)" <Luciano.Coelho@nokia.com>,
	"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
	San Mehat <san@google.com>, Ohad Ben-Cohen <ohadb@ti.com>
Subject: Re: [PATCH 11/15] wireless: wl1271: introduce platform device support
Date: Tue, 06 Jul 2010 11:53:45 +0300	[thread overview]
Message-ID: <4C32EF19.1000604@nokia.com> (raw)
In-Reply-To: <1278376666-3509-12-git-send-email-ohad@wizery.com>

Hi,

On 07/06/2010 03:37 AM, ext Ohad Ben-Cohen wrote:
> From: Ohad Ben-Cohen<ohadb@ti.com>
>
> Introduce a platform device support which is decoupled
> from the life cycle of the sdio function.
>
> The platform device objective is to deliver board-specific
> values (like irq, ref_clock, and software card-detect
> emulation control).
>
> The driver can then dynamically create (or destroy)
> the sdio function whenever the wlan interface is
> brought up (or down), as part of the power() operation.

Is this the right approach? Why should you emulate a card disconnect event when 
the wlan interface is brought down?

Physically the card is never disconnected. So I don't see why we should fake it.
Could you please explain why you need to do this?

br,
-roger

WARNING: multiple messages have this Message-ID (diff)
From: roger.quadros@nokia.com (Roger Quadros)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 11/15] wireless: wl1271: introduce platform device support
Date: Tue, 06 Jul 2010 11:53:45 +0300	[thread overview]
Message-ID: <4C32EF19.1000604@nokia.com> (raw)
In-Reply-To: <1278376666-3509-12-git-send-email-ohad@wizery.com>

Hi,

On 07/06/2010 03:37 AM, ext Ohad Ben-Cohen wrote:
> From: Ohad Ben-Cohen<ohadb@ti.com>
>
> Introduce a platform device support which is decoupled
> from the life cycle of the sdio function.
>
> The platform device objective is to deliver board-specific
> values (like irq, ref_clock, and software card-detect
> emulation control).
>
> The driver can then dynamically create (or destroy)
> the sdio function whenever the wlan interface is
> brought up (or down), as part of the power() operation.

Is this the right approach? Why should you emulate a card disconnect event when 
the wlan interface is brought down?

Physically the card is never disconnected. So I don't see why we should fake it.
Could you please explain why you need to do this?

br,
-roger

  parent reply	other threads:[~2010-07-06  8:53 UTC|newest]

Thread overview: 161+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-06  0:37 [PATCH 00/15] wlan+omap+mmc: out-of-the-box WLAN support for ZOOM2/3 Ohad Ben-Cohen
2010-07-06  0:37 ` Ohad Ben-Cohen
2010-07-06  0:37 ` Ohad Ben-Cohen
2010-07-06  0:37 ` [PATCH 02/15] wireless: wl1271: remove SDIO IDs from driver Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  7:04   ` Luciano Coelho
2010-07-06  7:04     ` Luciano Coelho
2010-07-06  0:37 ` [PATCH 03/15] omap: mmc: prepare for software card detect support Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37 ` [PATCH 04/15] mmc: support embedded data field in mmc_host Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06 15:49   ` Grazvydas Ignotas
2010-07-06 15:49     ` Grazvydas Ignotas
2010-07-06 15:49     ` Grazvydas Ignotas
2010-07-06 15:54     ` Ohad Ben-Cohen
2010-07-06 15:54       ` Ohad Ben-Cohen
2010-07-08  3:39   ` Ghorai, Sukumar
2010-07-08  3:39   ` Ghorai, Sukumar
2010-07-08  3:39     ` Ghorai, Sukumar
2010-07-08  3:39     ` Ghorai, Sukumar
2010-07-06  0:37 ` [PATCH 05/15] omap: hsmmc: add virtual card detect support Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  1:45   ` Nicolas Pitre
2010-07-06  1:45     ` Nicolas Pitre
     [not found]     ` <alpine.LFD.2.00.1007052134460.6020-QuJgVwGFrdf/9pzu0YdTqQ@public.gmane.org>
2010-07-06 10:22       ` Ohad Ben-Cohen
2010-07-06 10:22         ` Ohad Ben-Cohen
2010-07-06 10:22         ` Ohad Ben-Cohen
2010-07-06 11:02         ` Roger Quadros
2010-07-06 11:02           ` Roger Quadros
2010-07-06 12:02           ` Ohad Ben-Cohen
2010-07-06 12:02             ` Ohad Ben-Cohen
2010-07-06 11:48         ` Ohad Ben-Cohen
2010-07-06 11:48           ` Ohad Ben-Cohen
2010-07-06 12:39           ` Roger Quadros
2010-07-06 12:39             ` Roger Quadros
2010-07-06 13:44             ` Ohad Ben-Cohen
2010-07-06 13:44               ` Ohad Ben-Cohen
2010-07-06 15:34           ` Madhusudhan
2010-07-06 15:34             ` Madhusudhan
2010-07-06 15:34             ` Madhusudhan
2010-07-06 17:00             ` Nicolas Pitre
2010-07-06 17:00               ` Nicolas Pitre
2010-07-06  0:37 ` [PATCH 06/15] omap zoom2: wlan board muxing Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06 11:43   ` Tony Lindgren
2010-07-06 11:43     ` Tony Lindgren
2010-07-08  3:39   ` Ghorai, Sukumar
2010-07-08  3:39     ` Ghorai, Sukumar
2010-07-08  3:39     ` Ghorai, Sukumar
2010-07-08  9:45     ` Tony Lindgren
2010-07-08  9:45       ` Tony Lindgren
2010-07-08 20:12       ` Ohad Ben-Cohen
2010-07-08 20:12         ` Ohad Ben-Cohen
2010-07-08 20:12         ` Ohad Ben-Cohen
2010-07-08  3:39   ` Ghorai, Sukumar
2010-07-06  0:37 ` [PATCH 08/15] wireless: wl1271: make wl12xx.h common to both spi and sdio Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
     [not found]   ` <1278376666-3509-9-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>
2010-07-06  7:08     ` Luciano Coelho
2010-07-06  7:08       ` Luciano Coelho
2010-07-06  7:08       ` Luciano Coelho
2010-07-06  0:37 ` [PATCH 10/15] wireless: wl1271: support return value for the set power func Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  7:11   ` Luciano Coelho
2010-07-06  7:11     ` Luciano Coelho
2010-07-06  0:37 ` [PATCH 11/15] wireless: wl1271: introduce platform device support Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
     [not found]   ` <1278376666-3509-12-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>
2010-07-06  8:53     ` Roger Quadros [this message]
2010-07-06  8:53       ` Roger Quadros
2010-07-06  8:53       ` Roger Quadros
     [not found]       ` <4C32EF19.1000604-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-07-06  9:30         ` Ohad Ben-Cohen
2010-07-06  9:30           ` Ohad Ben-Cohen
2010-07-06  9:30           ` Ohad Ben-Cohen
2010-07-06 10:35           ` Roger Quadros
2010-07-06 10:35             ` Roger Quadros
2010-07-06 12:53             ` Ohad Ben-Cohen
2010-07-06 12:53               ` Ohad Ben-Cohen
2010-07-06 14:30               ` Roger Quadros
2010-07-06 14:30                 ` Roger Quadros
     [not found]                 ` <4C333E0D.2070601-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-07-06 17:42                   ` Nicolas Pitre
2010-07-06 17:42                     ` Nicolas Pitre
2010-07-06 17:42                     ` Nicolas Pitre
2010-07-06 19:51                     ` Adrian Hunter
2010-07-06 19:51                       ` Adrian Hunter
     [not found]                       ` <4C338927.5020400-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-07-07  8:02                         ` Roger Quadros
2010-07-07  8:02                           ` Roger Quadros
2010-07-07  8:02                           ` Roger Quadros
2010-07-07 14:02                           ` Nicolas Pitre
2010-07-07 14:02                             ` Nicolas Pitre
2010-07-07 14:54                             ` Madhusudhan
2010-07-07 14:54                               ` Madhusudhan
2010-07-07 14:54                               ` Madhusudhan
2010-07-07 15:46                               ` Nicolas Pitre
2010-07-07 15:46                                 ` Nicolas Pitre
2010-07-07 19:59                             ` Adrian Hunter
2010-07-07 19:59                               ` Adrian Hunter
     [not found]                               ` <4C34DC92.80908-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-07-08  4:34                                 ` Nicolas Pitre
2010-07-08  4:34                                   ` Nicolas Pitre
2010-07-08  4:34                                   ` Nicolas Pitre
2010-07-07  7:48                     ` Roger Quadros
2010-07-07  7:48                       ` Roger Quadros
2010-07-07 13:52                       ` Nicolas Pitre
2010-07-07 13:52                         ` Nicolas Pitre
2010-07-08  8:54                         ` Roger Quadros
2010-07-08  8:54                           ` Roger Quadros
2010-07-08 20:10                     ` Ohad Ben-Cohen
2010-07-08 20:10                       ` Ohad Ben-Cohen
2010-07-09  8:12                       ` Roger Quadros
2010-07-09  8:12                         ` Roger Quadros
2010-07-09  8:32                         ` Ohad Ben-Cohen
2010-07-09  8:32                           ` Ohad Ben-Cohen
2010-07-09  9:24                           ` Grazvydas Ignotas
2010-07-09  9:24                             ` Grazvydas Ignotas
2010-08-10 21:21                     ` Ohad Ben-Cohen
2010-08-10 21:21                       ` Ohad Ben-Cohen
2010-07-06  0:37 ` [PATCH 12/15] wireless: wl1271: take irq info from platform data Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37 ` [PATCH 13/15] wireless: wl1271: make ref_clock configurable by board Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
     [not found] ` <1278376666-3509-1-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>
2010-07-06  0:37   ` [PATCH 01/15] sdio: add TI + wl1271 ids Ohad Ben-Cohen
2010-07-06  0:37     ` Ohad Ben-Cohen
2010-07-06  0:37     ` Ohad Ben-Cohen
2010-07-06  0:37   ` [PATCH 07/15] omap zoom3: wlan board muxing Ohad Ben-Cohen
2010-07-06  0:37     ` Ohad Ben-Cohen
2010-07-06  0:37     ` Ohad Ben-Cohen
2010-07-06  0:37   ` [PATCH 09/15] wireless: wl12xx: support pdata SDIO handlers Ohad Ben-Cohen
2010-07-06  0:37     ` Ohad Ben-Cohen
2010-07-06  0:37     ` Ohad Ben-Cohen
2010-07-06  0:37   ` [PATCH 14/15] omap: zoom: add WLAN device Ohad Ben-Cohen
2010-07-06  0:37     ` Ohad Ben-Cohen
2010-07-06  0:37     ` Ohad Ben-Cohen
2010-07-06 12:33     ` Roger Quadros
2010-07-06 12:33       ` Roger Quadros
     [not found]       ` <4C33228D.8000908-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org>
2010-07-06 13:47         ` Ohad Ben-Cohen
2010-07-06 13:47           ` Ohad Ben-Cohen
2010-07-06 13:47           ` Ohad Ben-Cohen
2010-07-08  3:39     ` Ghorai, Sukumar
2010-07-08  3:39       ` Ghorai, Sukumar
2010-07-08  3:39       ` Ghorai, Sukumar
2010-07-06  0:37 ` [PATCH 15/15] omap: zoom: enable " Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
2010-07-06  0:37   ` Ohad Ben-Cohen
     [not found]   ` <1278376666-3509-16-git-send-email-ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org>
2010-07-08  3:39     ` Ghorai, Sukumar
2010-07-08  3:39       ` Ghorai, Sukumar
2010-07-08  3:39       ` Ghorai, Sukumar
2010-07-08 20:13       ` Ohad Ben-Cohen
2010-07-08 20:13         ` Ohad Ben-Cohen
2010-07-08  3:39   ` Ghorai, Sukumar
2010-07-08  3:39 ` [PATCH 00/15] wlan+omap+mmc: out-of-the-box WLAN support for ZOOM2/3 Ghorai, Sukumar
2010-07-08  3:39   ` Ghorai, Sukumar
2010-07-08  3:39   ` Ghorai, Sukumar
2010-07-08  3:39 ` Ghorai, Sukumar

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4C32EF19.1000604@nokia.com \
    --to=roger.quadros-xnzwkgviw5gavxtiumwx3w@public.gmane.org \
    --cc=Luciano.Coelho-xNZwKgViW5gAvxtiuMwx3w@public.gmane.org \
    --cc=akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org \
    --cc=linux-mmc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=madhu.cr-l0cyMroinI0@public.gmane.org \
    --cc=ohad-Ix1uc/W3ht7QT0dZR+AlfA@public.gmane.org \
    --cc=ohadb-l0cyMroinI0@public.gmane.org \
    --cc=san-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.