Re: [PATCH 02/11 v3] drm/i915/intel_i2c: assign HDMI port D to pin pair 6

[Daniel Vetter <daniel@ffwll.ch>]

On Tue, Mar 27, 2012 at 01:49:51AM +0800, Daniel Kurtz wrote:
> I'm not sure what a "Bspec" is, but here is the documents I used:
> 
> [DevSNB+]: http://intellinuxgraphics.org/documentation/SNB/IHD_OS_Vol3_Part3.pdf
> Section 2.2.2 lists the 6 gmbus ports (gpio pin pairs):
>   [ 5: HDMI/DPD, 4: HDMIB, 3: HDMI/DPC, 2: LVDS, 1: SSC, 0: VGA ]
> 2.2.2.1 lists the GPIO registers to control these 6 ports.
> 2.2.3.1 lists the mapping between 5 of these gmbus ports and the 3
> Pin_Pair_Select bits (of the GMBUS0 register).  This table is missing
> HDMIB (port 101).
> 
> [DevIBX]: http://intellinuxgraphics.org/IHD_OS_Vol3_Part3r2.pdf
> Section 2.2.2 lists the same 6 gmbus ports plus two 'reserved' gpio ports.
> 2.2.2.1 lists 8 GPIO registers... however, it says the size of the
> block is 6x32, which implies that those 2 reserved GPIO registers
> (GPIO_6 & GPIO_7) don't actually exist (or are irrelevant).
> 2.2.3.1 lists the mapping between the 6 named gmbus ports and the 3
> Pin_Pair_Select bits (of the GMBUS0 register).  This table has HDMIB.

Exactly this. Bspec is the internal name for what these public docs are
based on. When fixing up an inconsistency wrt the docs it's always good to
cite it like this, so that later on people can cross check. And we need
section numbers because they are the most stable - Internal Bspec is just
a collection of individual sections as html pages. And hw people also tend
to use the same sections for the same stuff in newer/older chips.

Cheers, Daniel
-- 
Daniel Vetter
Mail: daniel@ffwll.ch
Mobile: +41 (0)79 365 57 48