On 2016-11-02 11:36 AM, Stephen Boyd wrote:
On 11/01, Michael Scott wrote:On 11/01/2016 04:53 PM, Stephen Boyd wrote:On 10/31, Michael Scott wrote:+ +static const struct msm_pingroup msm8994_groups[] = { + PINGROUP(0, blsp_spi1, blsp_uart1, blsp_uim1, NA, NA, NA, NA, NA, NA, + NA, NA),I see an hdmi_rcv group here after blsp_uim1. Please add it for this gpio.Hi Stephen, Thank you for reviewing the data-- this is really tedious stuff. I'd like to make sure that we're both using tech data for msm8994. I don't see the hdmi_rcv function at all in my docs.+ PINGROUP(1, blsp_spi1, blsp_uart1, blsp_uim1, NA, NA, NA, NA, NA, NA, + NA, NA), + PINGROUP(2, blsp_spi1, blsp_uart1, blsp_i2c1, NA, NA, NA, NA, NA, NA, + NA, NA), + PINGROUP(3, blsp_spi1, blsp_uart1, blsp_i2c1, NA, NA, NA, NA, NA, NA, + NA, NA), + PINGROUP(4, blsp_spi2, blsp_uart2, blsp_uim2, qdss_cti_trig_out_b, + NA, NA, NA, NA, NA, NA, NA), + PINGROUP(5, blsp_spi2, blsp_uart2, blsp_uim2, qdss_cti_trig_in_b, NA,The qdss_cti_* is in function 5 for both of these, not function 4.These are indeed wrong, but my docs both show as function 7.Ok. I think your document is for 8994v1, which never got commercialized. I certainly see that this function assignment changed between v1 and v2 from 7 to 5.
This is a case where not having the docs and only downstream code as a reference is probably a good thing (tm), although the downstream code may end up hurting more than helping. -jeremy -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html