Re: [PATCH v4 3/8] wireless: wl1271: add platform driver to get board data

Hi Vitaly,

On Thu, Aug 12, 2010 at 12:34 AM, Vitaly Wool [off-list ref] wrote:

On Wed, Aug 11, 2010 at 10:10 PM, Ohad Ben-Cohen [off-list ref] wrote:

quoted

Think of several wl1271 devices, each of which is represented by two
devices; an SDIO function, and a platform device. The SDIO function
stands for a specific MMC controller the device is hardwired to, and
the platform device stands for the external irq line that the device
is hardwired to.

my view on this is that you're hunting the false problem here. I
wonder how likely it is that two hardwired WL12xx chips are there on
the same board.

It's theoretically possible that there's one hardwired device and one
hot-pluggable one but the latter one can't have platform_data anyway.

We do have strange-looking setups with two hardwired devices
(especially on new bring-ups), so it would be nice not to limit the
driver to a single device. But I agree, it's far from being important.

Thanks,
Ohad.

So in my opinion, we either should keep things simple and consider 2
hardwired WL12XX's to be a very very unlikely case, or we'll end up
discussing board_info once again.

Thanks,
  Vitaly