Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1402000 > unrolled thread
| Started by | Florian Fainelli <f.fainelli@gmail.com> |
|---|---|
| First post | 2016-05-17 04:00 +0200 |
| Last post | 2016-05-17 04:00 +0200 |
| Articles | 1 — 1 participant |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH v2 2/2] pci: host: Add Broadcom STB PCIE RC controller Florian Fainelli <f.fainelli@gmail.com> - 2016-05-17 04:00 +0200
| From | Florian Fainelli <f.fainelli@gmail.com> |
|---|---|
| Date | 2016-05-17 04:00 +0200 |
| Subject | Re: [PATCH v2 2/2] pci: host: Add Broadcom STB PCIE RC controller |
| Message-ID | <rzCsG-6tJ-31@gated-at.bofh.it> |
On 05/11/2016 07:47 AM, Bharat Kumar Gogada wrote:
[snip]
>> + /* Get the value for the log2 of the scb sizes. Subtract 15 from
>> + * each because the target register field has 0==disabled and 1==6KB.
>> + */
>> + log2_scb_sizes = of_get_property(dn, "brcm,log2-scb-sizes", &rlen);
>> + if (log2_scb_sizes) {
>> + for (i = 0; i < rlen / sizeof(u32); i++) {
>> + pcie->scb_size_vals[i]
>> + = (int)of_read_number(log2_scb_sizes + i, 1)
>> + - 15;
>> + pcie->num_memc++;
>> + }
>> + }
> In your device tree documentation this is required property, what if this property
> is missing ?
If you look at the driver, you will see that it assumes a 1GB SCB window
size if this property is absent.
PS: Do you mind trimming your replies in the future just to quote the
relevant part? Pretty much like I just did, this greatly improves the
ability to jump right to where the comments are. Thanks
--
Florian
Back to top | Article view | linux.kernel
csiph-web