target/linux/brcm47xx/patches-4.0/030-12-MIPS-BCM47xx-Add-generic-function-filling-SPROM-entr.patch

   1 From d55a52ccf8f80cdf51af2c5c6e56c825f98c4f85 Mon Sep 17 00:00:00 2001
   2 From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
   3 Date: Thu, 2 Apr 2015 09:13:49 +0200
   4 Subject: [PATCH] MIPS: BCM47xx: Add generic function filling SPROM entries
   5 MIME-Version: 1.0
   6 Content-Type: text/plain; charset=UTF-8
   7 Content-Transfer-Encoding: 8bit
   8
   9 Handling many SPROM revisions became messy, we have tons of functions
  10 specific to various revision groups which are quite hard to track.
  11 For years there is yet another revision 11 asking for support, but
  12 adding it in current the form would make things even worse.
  13 To resolve this problem let's add new function with table-like entries
  14 that will contain revision bitmask for every SPROM variable.
  15
  16 Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
  17 Cc: linux-mips@linux-mips.org
  18 Cc: Hauke Mehrtens <hauke@hauke-m.de>
  19 Cc: Jonas Gorski <jonas.gorski@gmail.com>
  20 Patchwork: https://patchwork.linux-mips.org/patch/9659/
  21 Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
  22 ---
  23  arch/mips/bcm47xx/sprom.c | 32 +++++++++++++++++++++++++++++---
  24  1 file changed, 29 insertions(+), 3 deletions(-)
  25
  26 --- a/arch/mips/bcm47xx/sprom.c
  27 +++ b/arch/mips/bcm47xx/sprom.c
  28 @@ -180,6 +180,33 @@ static void nvram_read_alpha2(const char
  29         memcpy(val, buf, 2);
  30  }
  31
  32 +/* This is one-function-only macro, it uses local "sprom" variable! */
  33 +#define ENTRY(_revmask, _type, _prefix, _name, _val, _allset, _fallback) \
  34 +       if (_revmask & BIT(sprom->revision)) \
  35 +               nvram_read_ ## _type(_prefix, NULL, _name, &sprom->_val, \
  36 +                                    _allset, _fallback)
  37 +/*
  38 + * Special version of filling function that can be safely called for any SPROM
  39 + * revision. For every NVRAM to SPROM mapping it contains bitmask of revisions
  40 + * for which the mapping is valid.
  41 + * It obviously requires some hexadecimal/bitmasks knowledge, but allows
  42 + * writing cleaner code (easy revisions handling).
  43 + * Note that while SPROM revision 0 was never used, we still keep BIT(0)
  44 + * reserved for it, just to keep numbering sane.
  45 + */
  46 +static void bcm47xx_sprom_fill_auto(struct ssb_sprom *sprom,
  47 +                                   const char *prefix, bool fallback)
  48 +{
  49 +       const char *pre = prefix;
  50 +       bool fb = fallback;
  51 +
  52 +       ENTRY(0xfffffffe, u16, pre, "boardrev", board_rev, 0, true);
  53 +       ENTRY(0xfffffffe, u16, pre, "boardnum", board_num, 0, fb);
  54 +
  55 +       /* TODO: Move more mappings here */
  56 +}
  57 +#undef ENTRY /* It's specififc, uses local variable, don't use it (again). */
  58 +
  59  static void bcm47xx_fill_sprom_r1234589(struct ssb_sprom *sprom,
  60                                         const char *prefix, bool fallback)
  61  {
  62 @@ -714,9 +741,6 @@ static void bcm47xx_fill_sprom_ethernet(
  63  static void bcm47xx_fill_board_data(struct ssb_sprom *sprom, const char *prefix,
  64                                     bool fallback)
  65  {
  66 -       nvram_read_u16(prefix, NULL, "boardrev", &sprom->board_rev, 0, true);
  67 -       nvram_read_u16(prefix, NULL, "boardnum", &sprom->board_num, 0,
  68 -                      fallback);
  69         nvram_read_u16(prefix, NULL, "boardtype", &sprom->board_type, 0, true);
  70         nvram_read_u32_2(prefix, "boardflags", &sprom->boardflags_lo,
  71                          &sprom->boardflags_hi, fallback);
  72 @@ -787,6 +811,8 @@ void bcm47xx_fill_sprom(struct ssb_sprom
  73                 bcm47xx_fill_sprom_r12389(sprom, prefix, fallback);
  74                 bcm47xx_fill_sprom_r1(sprom, prefix, fallback);
  75         }
  76 +
  77 +       bcm47xx_sprom_fill_auto(sprom, prefix, fallback);
  78  }
  79
  80  #ifdef CONFIG_BCM47XX_SSB