All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Rutland <mark.rutland@arm.com>
To: Kalle Valo <kvalo@qca.qualcomm.com>
Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"ath10k@lists.infradead.org" <ath10k@lists.infradead.org>
Subject: Re: ath10k: calibration data through Device Tree?
Date: Thu, 2 Oct 2014 14:29:15 +0100	[thread overview]
Message-ID: <20141002132914.GN5788@leverpostej> (raw)
In-Reply-To: <87tx3mmx4s.fsf@kamboji.qca.qualcomm.com>

Hi,

> ath10k is a wireless driver for Qualcomm Atheros 802.11ac hardware and
> located in drivers/net/wireless/ath/ath10k/. Currently it only supports
> PCI devices.
> 
> Some of the devices store the calibration data to the host flash and the
> bootloader reads the data from the flash. And now we need a method to
> deliver the calibration data from bootloader to ath10k. 

What does this calibration data consist of?

What happens if you don't have the calibration data? Is it a critical
requirement for the use of the device, or does its absence simply result
in degraded performance?

What do you do on non-DT systems? Where does the information come from
in that case?

I'm somewhat puzzled as to why a discoverable PCI device would require
non-discoverable information to use.

> * The calibration data is now 2116 bytes, in the future it might be
>   longer. The data is unique for each radio and is created at the
>   factory.

Why would this change in future? Who is in charge of providing this
information, and deciding upon the format thereof?

> * ath10k must be able to reliably map the PCI device (=radio) to the
>   correct calibration data. Maybe with using PCI bus and slot numbers?

I guess we'd have to do something along those lines.

I'd like to get a better understanding of the problem before we start
figuring out how to pass an arbitrary blob of information around.

Thanks,
Mark.

_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

WARNING: multiple messages have this Message-ID (diff)
From: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>
To: Kalle Valo <kvalo-A+ZNKFmMK5xy9aJCnZT0Uw@public.gmane.org>
Cc: "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"ath10k-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
	<ath10k-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Subject: Re: ath10k: calibration data through Device Tree?
Date: Thu, 2 Oct 2014 14:29:15 +0100	[thread overview]
Message-ID: <20141002132914.GN5788@leverpostej> (raw)
In-Reply-To: <87tx3mmx4s.fsf-HodKDYzPHsUD5k0oWYwrnHL1okKdlPRT@public.gmane.org>

Hi,

> ath10k is a wireless driver for Qualcomm Atheros 802.11ac hardware and
> located in drivers/net/wireless/ath/ath10k/. Currently it only supports
> PCI devices.
> 
> Some of the devices store the calibration data to the host flash and the
> bootloader reads the data from the flash. And now we need a method to
> deliver the calibration data from bootloader to ath10k. 

What does this calibration data consist of?

What happens if you don't have the calibration data? Is it a critical
requirement for the use of the device, or does its absence simply result
in degraded performance?

What do you do on non-DT systems? Where does the information come from
in that case?

I'm somewhat puzzled as to why a discoverable PCI device would require
non-discoverable information to use.

> * The calibration data is now 2116 bytes, in the future it might be
>   longer. The data is unique for each radio and is created at the
>   factory.

Why would this change in future? Who is in charge of providing this
information, and deciding upon the format thereof?

> * ath10k must be able to reliably map the PCI device (=radio) to the
>   correct calibration data. Maybe with using PCI bus and slot numbers?

I guess we'd have to do something along those lines.

I'd like to get a better understanding of the problem before we start
figuring out how to pass an arbitrary blob of information around.

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

  parent reply	other threads:[~2014-10-02 13:29 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-02 13:14 ath10k: calibration data through Device Tree? Kalle Valo
2014-10-02 13:14 ` Kalle Valo
2014-10-02 13:27 ` Arnd Bergmann
2014-10-02 13:27   ` Arnd Bergmann
2014-10-02 13:47   ` Kalle Valo
2014-10-02 13:47     ` Kalle Valo
2014-10-02 14:19     ` Arnd Bergmann
2014-10-02 14:19       ` Arnd Bergmann
2014-10-02 14:55       ` Kalle Valo
2014-10-02 14:55         ` Kalle Valo
2014-10-02 15:58         ` Arnd Bergmann
2014-10-02 15:58           ` Arnd Bergmann
2014-10-02 13:29 ` Mark Rutland [this message]
2014-10-02 13:29   ` Mark Rutland
2014-10-02 13:44   ` Kalle Valo
2014-10-02 13:44     ` Kalle Valo
2014-10-02 15:07     ` Mark Rutland
2014-10-02 15:07       ` Mark Rutland
2014-10-02 19:05     ` Andy Lutomirski
2014-10-02 19:05       ` Andy Lutomirski
2014-10-02 19:28       ` Adrian Chadd
2014-10-02 19:28         ` Adrian Chadd
2014-10-02 19:35         ` Andy Lutomirski
2014-10-02 19:35           ` Andy Lutomirski
2014-10-07 16:44           ` Kalle Valo
2014-10-07 16:44             ` Kalle Valo
2014-10-17 12:25             ` Kumar Gala
2014-10-17 12:25               ` Kumar Gala
2014-10-22 12:02               ` Kalle Valo
2014-10-22 12:02                 ` Kalle Valo
2014-10-03 15:29       ` Arnd Bergmann
2014-10-03 15:29         ` Arnd Bergmann
2014-10-03 16:24         ` Andy Lutomirski
2014-10-03 16:24           ` Andy Lutomirski
2014-10-03 16:25         ` Mark Rutland
2014-10-03 16:25           ` Mark Rutland
2014-10-03 16:42           ` Arnd Bergmann
2014-10-03 16:42             ` Arnd Bergmann
2014-10-03 16:54             ` Andy Lutomirski
2014-10-03 16:54               ` Andy Lutomirski
2014-10-03 17:21               ` Adrian Chadd
2014-10-03 17:21                 ` Adrian Chadd
  -- strict thread matches above, loose matches on Subject: below --
2014-10-02 13:05 Kalle Valo
2014-10-02 13:09 ` Kalle Valo

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=20141002132914.GN5788@leverpostej \
    --to=mark.rutland@arm.com \
    --cc=ath10k@lists.infradead.org \
    --cc=devicetree@vger.kernel.org \
    --cc=kvalo@qca.qualcomm.com \
    /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.