[PATCH 06/16] mach-aspeed: Add missing Firestone setup method and

Timothy Pearson tpearson at raptorengineering.com
Sat Sep 17 00:14:10 AEST 2016


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 09/15/2016 08:22 PM, Andrew Jeffery wrote:
> Can you please document exactly what this is (de)configuring and why?
> That way when we come to remove the hack we have some context.
> 
> Better yet, according to `bitfield`:
> 
> $ bitfield SCU88 0xD7000000
> decoding as SCU Multi-function Pin Control #3
> 0xd7000000 [3607101440]
>                                     Enable MAC#1 MDIO1: 0x1
>                                      Enable MAC#1 MDC1: 0x1
>                                Enable NOR flash ROMA25: 0x0
>                                Enable NOR flash ROMA24: 0x1
>                                   Enable flash ROMCS4#: 0x0
>                                   Enable flash ROMCS3#: 0x1
>                                   Enable flash ROMCS2#: 0x1
>                                   Enable flash ROMCS1#: 0x1
>                                   ...
> 
> And as such each of these can be configured through the devicetree via
> pinmux functions MDIO1, ROM16, ROMCS[321]. However, looking at the
> pinmux driver, whilst ROMCS[321] are defined for the relevant pins they
> don't look to have been added to the list of exported functions in dev-
> 4.7. I'll have to dig further into that, but it's probably an oversight
> on my part.
> 
> Out of interest, what are you doing with ROMA24? Is that actually
> required, and if so, is it meant to be independent of ROMA25? I may
> have to rethink my approach there.

No, it's not required, at least on Firestone.  In fact none of the
ROMAnn pins are required; the entire NOR interface is either used for
pin straps or is disconnected.

I'm going to say this was bad copypasta from the other _setup functions
and delete the line entirely.  I'll re-submit this patch shortly.

- -- 
Timothy Pearson
Raptor Engineering
+1 (415) 727-8645 (direct line)
+1 (512) 690-0200 (switchboard)
https://www.raptorengineering.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJX2/4yAAoJEK+E3vEXDOFbhPMIAJTNTlysdb60wt5igVSyZtZr
0xGwbzmO2hQZsEMqiyNIqCBejGZDKbu6je/GoONqmXPYbKfD5+8b4lB4RQBO8SDj
p7DGhXAX9x/ggbcHSjIF+I+PW0YJRep4/vYS1mKWp7M0w3LCku1DaWO8EoLenMZx
uj85mFkx8kixZFz2Iskm35PEqr8DlS6paTdIHcTA00+i8WohY9xM0SU1Lp5SfxKd
/2+J2Fhpzsg9myyCG5X/4MB0l6I/4cEtXL9litZ9I5XRiB+TuK86fNoAzCuQbno2
gSLGJTRGTYrDlvt7LEqk0s0EeX76z67MQmPexp5oPGMb5Wxa2Nqmnnyo1j8SMQU=
=cenu
-----END PGP SIGNATURE-----


More information about the openbmc mailing list