On Mon, Oct 07, 2013 at 03:16:11PM -0600, Stephen Warren wrote: > On 10/07/2013 05:48 AM, Thierry Reding wrote: > > Add a device node for the MIPI calibration block on Tegra114. There is > > no need to disable it by default because it only enables the clock while > > performing calibration and therefore shouldn't be consuming any power > > when unused. > > > include/dt-bindings/gpu/host1x/tegra114-mipi.h | 11 +++++++++++ > > Shouldn't that file be added in the same patch that defines the binding > that header contains constants for? What are those constants anyway? I'm not sure. The values themselves are not used within the driver. They are merely a convenience for DTS authors. In a nutshell, the values defined within that file are a bitmask of the pads that need calibration. There are 9 pads (5 for CSI, 4 for DSI) but the association to each input/output isn't immediately obvious. One DSI output uses DSIA and DSIB pads, while the other uses DSIC and DSID. Perhaps in this case it isn't useful to introduce the header at all and just using the numerical values would be fine? Thierry
Attachment:
pgp7qymOSfana.pgp
Description: PGP signature
