* [RFT][PATCH] ssb: Look for SPROM at different offset on higher rev CC
@ 2010-03-31 19:59 Rafał Miłecki
2010-04-01 15:36 ` Michael Buesch
0 siblings, 1 reply; 3+ messages in thread
From: Rafał Miłecki @ 2010-03-31 19:59 UTC (permalink / raw)
To: linux-wireless, John W. Linville; +Cc: b43-dev, Rafał Miłecki
Our offset handling becomes even a little more hackish now. For some reason I
do not understand all offsets as inrelative. It assumes base offset is 0x1000
but it will work for now as we make offsets relative anyway by removing base
0x1000. Should be cleaner however.
Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
---
drivers/ssb/pci.c | 9 ++++++---
include/linux/ssb/ssb.h | 1 +
include/linux/ssb/ssb_regs.h | 3 ++-
3 files changed, 9 insertions(+), 4 deletions(-)
diff --git a/drivers/ssb/pci.c b/drivers/ssb/pci.c
index a4b2b99..5bb1278 100644
--- a/drivers/ssb/pci.c
+++ b/drivers/ssb/pci.c
@@ -167,7 +167,7 @@ err_pci:
}
/* Get the word-offset for a SSB_SPROM_XXX define. */
-#define SPOFF(offset) (((offset) - SSB_SPROM_BASE) / sizeof(u16))
+#define SPOFF(offset) (((offset) - SSB_SPROM_BASE1) / sizeof(u16))
/* Helper to extract some _offset, which is one of the SSB_SPROM_XXX defines. */
#define SPEX16(_outvar, _offset, _mask, _shift) \
out->_outvar = ((in[SPOFF(_offset)] & (_mask)) >> (_shift))
@@ -253,7 +253,7 @@ static int sprom_do_read(struct ssb_bus *bus, u16 *sprom)
int i;
for (i = 0; i < bus->sprom_size; i++)
- sprom[i] = ioread16(bus->mmio + SSB_SPROM_BASE + (i * 2));
+ sprom[i] = ioread16(bus->mmio + bus->sprom_offset + (i * 2));
return 0;
}
@@ -284,7 +284,7 @@ static int sprom_do_write(struct ssb_bus *bus, const u16 *sprom)
ssb_printk("75%%");
else if (i % 2)
ssb_printk(".");
- writew(sprom[i], bus->mmio + SSB_SPROM_BASE + (i * 2));
+ writew(sprom[i], bus->mmio + bus->sprom_offset + (i * 2));
mmiowb();
msleep(20);
}
@@ -625,6 +625,9 @@ static int ssb_pci_sprom_get(struct ssb_bus *bus,
return -ENODEV;
}
+ bus->sprom_offset = (bus->chipco.dev->id.revision < 31) ?
+ SSB_SPROM_BASE1 : SSB_SPROM_BASE31;
+
buf = kcalloc(SSB_SPROMSIZE_WORDS_R123, sizeof(u16), GFP_KERNEL);
if (!buf)
goto out;
diff --git a/include/linux/ssb/ssb.h b/include/linux/ssb/ssb.h
index 3b4da23..a2608bf 100644
--- a/include/linux/ssb/ssb.h
+++ b/include/linux/ssb/ssb.h
@@ -305,6 +305,7 @@ struct ssb_bus {
/* ID information about the Chip. */
u16 chip_id;
u16 chip_rev;
+ u16 sprom_offset;
u16 sprom_size; /* number of words in sprom */
u8 chip_package;
diff --git a/include/linux/ssb/ssb_regs.h b/include/linux/ssb/ssb_regs.h
index 9ae9082..b8be23c 100644
--- a/include/linux/ssb/ssb_regs.h
+++ b/include/linux/ssb/ssb_regs.h
@@ -170,7 +170,8 @@
#define SSB_SPROMSIZE_WORDS_R4 220
#define SSB_SPROMSIZE_BYTES_R123 (SSB_SPROMSIZE_WORDS_R123 * sizeof(u16))
#define SSB_SPROMSIZE_BYTES_R4 (SSB_SPROMSIZE_WORDS_R4 * sizeof(u16))
-#define SSB_SPROM_BASE 0x1000
+#define SSB_SPROM_BASE1 0x1000
+#define SSB_SPROM_BASE31 0x0800
#define SSB_SPROM_REVISION 0x107E
#define SSB_SPROM_REVISION_REV 0x00FF /* SPROM Revision number */
#define SSB_SPROM_REVISION_CRC 0xFF00 /* SPROM CRC8 value */
--
1.6.4.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [RFT][PATCH] ssb: Look for SPROM at different offset on higher rev CC
2010-03-31 19:59 [RFT][PATCH] ssb: Look for SPROM at different offset on higher rev CC Rafał Miłecki
@ 2010-04-01 15:36 ` Michael Buesch
2010-04-01 17:11 ` Rafał Miłecki
0 siblings, 1 reply; 3+ messages in thread
From: Michael Buesch @ 2010-04-01 15:36 UTC (permalink / raw)
To: b43-dev; +Cc: Rafał Miłecki, linux-wireless, John W. Linville
On Wednesday 31 March 2010 21:59:21 Rafa? Mi?ecki wrote:
> Our offset handling becomes even a little more hackish now. For some reason I
> do not understand all offsets as inrelative. It assumes base offset is 0x1000
> but it will work for now as we make offsets relative anyway by removing base
> 0x1000. Should be cleaner however.
>
What about fixing it correctly instead adding a hack?
> Signed-off-by: Rafa? Mi?ecki <zajec5@gmail.com>
> ---
> drivers/ssb/pci.c | 9 ++++++---
> include/linux/ssb/ssb.h | 1 +
> include/linux/ssb/ssb_regs.h | 3 ++-
> 3 files changed, 9 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/ssb/pci.c b/drivers/ssb/pci.c
> index a4b2b99..5bb1278 100644
> --- a/drivers/ssb/pci.c
> +++ b/drivers/ssb/pci.c
> @@ -167,7 +167,7 @@ err_pci:
> }
>
> /* Get the word-offset for a SSB_SPROM_XXX define. */
> -#define SPOFF(offset) (((offset) - SSB_SPROM_BASE) / sizeof(u16))
> +#define SPOFF(offset) (((offset) - SSB_SPROM_BASE1) / sizeof(u16))
> /* Helper to extract some _offset, which is one of the SSB_SPROM_XXX defines. */
> #define SPEX16(_outvar, _offset, _mask, _shift) \
> out->_outvar = ((in[SPOFF(_offset)] & (_mask)) >> (_shift))
> @@ -253,7 +253,7 @@ static int sprom_do_read(struct ssb_bus *bus, u16 *sprom)
> int i;
>
> for (i = 0; i < bus->sprom_size; i++)
> - sprom[i] = ioread16(bus->mmio + SSB_SPROM_BASE + (i * 2));
> + sprom[i] = ioread16(bus->mmio + bus->sprom_offset + (i * 2));
>
> return 0;
> }
> @@ -284,7 +284,7 @@ static int sprom_do_write(struct ssb_bus *bus, const u16 *sprom)
> ssb_printk("75%%");
> else if (i % 2)
> ssb_printk(".");
> - writew(sprom[i], bus->mmio + SSB_SPROM_BASE + (i * 2));
> + writew(sprom[i], bus->mmio + bus->sprom_offset + (i * 2));
> mmiowb();
> msleep(20);
> }
> @@ -625,6 +625,9 @@ static int ssb_pci_sprom_get(struct ssb_bus *bus,
> return -ENODEV;
> }
>
> + bus->sprom_offset = (bus->chipco.dev->id.revision < 31) ?
> + SSB_SPROM_BASE1 : SSB_SPROM_BASE31;
> +
> buf = kcalloc(SSB_SPROMSIZE_WORDS_R123, sizeof(u16), GFP_KERNEL);
> if (!buf)
> goto out;
> diff --git a/include/linux/ssb/ssb.h b/include/linux/ssb/ssb.h
> index 3b4da23..a2608bf 100644
> --- a/include/linux/ssb/ssb.h
> +++ b/include/linux/ssb/ssb.h
> @@ -305,6 +305,7 @@ struct ssb_bus {
> /* ID information about the Chip. */
> u16 chip_id;
> u16 chip_rev;
> + u16 sprom_offset;
> u16 sprom_size; /* number of words in sprom */
> u8 chip_package;
>
> diff --git a/include/linux/ssb/ssb_regs.h b/include/linux/ssb/ssb_regs.h
> index 9ae9082..b8be23c 100644
> --- a/include/linux/ssb/ssb_regs.h
> +++ b/include/linux/ssb/ssb_regs.h
> @@ -170,7 +170,8 @@
> #define SSB_SPROMSIZE_WORDS_R4 220
> #define SSB_SPROMSIZE_BYTES_R123 (SSB_SPROMSIZE_WORDS_R123 * sizeof(u16))
> #define SSB_SPROMSIZE_BYTES_R4 (SSB_SPROMSIZE_WORDS_R4 * sizeof(u16))
> -#define SSB_SPROM_BASE 0x1000
> +#define SSB_SPROM_BASE1 0x1000
> +#define SSB_SPROM_BASE31 0x0800
> #define SSB_SPROM_REVISION 0x107E
> #define SSB_SPROM_REVISION_REV 0x00FF /* SPROM Revision number */
> #define SSB_SPROM_REVISION_CRC 0xFF00 /* SPROM CRC8 value */
>
--
Greetings, Michael.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [RFT][PATCH] ssb: Look for SPROM at different offset on higher rev CC
2010-04-01 15:36 ` Michael Buesch
@ 2010-04-01 17:11 ` Rafał Miłecki
0 siblings, 0 replies; 3+ messages in thread
From: Rafał Miłecki @ 2010-04-01 17:11 UTC (permalink / raw)
To: Michael Buesch; +Cc: b43-dev, linux-wireless, John W. Linville
2010/4/1 Michael Buesch <mb@bu3sch.de>:
> On Wednesday 31 March 2010 21:59:21 Rafa? Mi?ecki wrote:
>> Our offset handling becomes even a little more hackish now. For some reason I
>> do not understand all offsets as inrelative. It assumes base offset is 0x1000
>> but it will work for now as we make offsets relative anyway by removing base
>> 0x1000. Should be cleaner however.
>>
>
> What about fixing it correctly instead adding a hack?
See next 2 patches :)
I believe it does not add any hackish code, just exposes weirdness of
current code to bigger degree. Anyway as I said, I fixed this in next
patch I sent already yesterday.
--
Rafa?
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-04-01 17:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-31 19:59 [RFT][PATCH] ssb: Look for SPROM at different offset on higher rev CC Rafał Miłecki
2010-04-01 15:36 ` Michael Buesch
2010-04-01 17:11 ` Rafał Miłecki
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).