On Thu, Jul 3, 2014 at 12:36 PM, <sviau@xxxxxxxxxxxxxx> wrote:
> Hi Rob,
>
>> Now that we (almost) have enough dependencies in place (MMCC, RPM, etc),
>> add necessary DT support so that we can use drm/msm on upstream kernel.
>>
>> Signed-off-by: Rob Clark <robdclark@xxxxxxxxx>
>> ---
>> Commence bikeshedding :-)
>>
> <snip>
>> diff --git a/Documentation/devicetree/bindings/drm/msm/hdmi.txt
>> b/Documentation/devicetree/bindings/drm/msm/hdmi.txt
>> new file mode 100644
>> index 0000000..051a49f
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/drm/msm/hdmi.txt
>> @@ -0,0 +1,43 @@
>> +Qualcomm adreno/snapdragon hdmi output
>> +
>> +Required properties:
>> +- compatible: "qcom,hdmi-tx-8x60", "qcom,hdmi-tx-8960",
>> "qcom,hdmi-tx-8x74"
>> +- reg: Physical base address and length of the controller's registers.
>
> Since you are adding "qcom,hdmi-tx-8x74" (separate address space for PHY
> registers) in the compatible entry, how about this for the register
> description:
> - reg: Physical base address and length of the controllers' registers.
> - reg-names: names corresponding to the defined register sets,
> - "core_physical": HDMI Core registers
> - (optional) "phy_physical": HDMI PHY registers
Oh, right.. I forgot 8x74 had phy in a separate region..
>> +- interrupts: The interrupt outputs from the controller.
>> +- clocks: device clocks
>> + See ../clocks/clock-bindings.txt for details.
>> +- qcom,hdmi-tx-ddc-clk: ddc clk pin
>> +- qcom,hdmi-tx-ddc-data: ddc data pin
>> +- qcom,hdmi-tx-hpd: hpd pin
>> +- core-vdda-supply: phandle to supply regulator
>> +- hdmi-mux-supply: phandle to mux regulator
>> +
>> +Optional properties:
>> +- qcom,hdmi-tx-mux-en: hdmi mux enable pin
>> +- qcom,hdmi-tx-mux-sel: hdmi mux select pin
>> +
>> +Example:
>> +
>> +/ {
>> + ...
>> +
>> + hdmi: qcom,hdmi-tx-8960@4a00000 {
>> + compatible = "qcom,hdmi-tx-8960";
>> + reg-names = "core_physical";
>> + reg = <0x04a00000 0x1000>;
>> + interrupts = <GIC_SPI 79 0>;
>> + clock-names =
>> + "core_clk",
>> + "master_iface_clk",
>> + "slave_iface_clk";
>> + clocks =
>> + <&mmcc HDMI_APP_CLK>,
>> + <&mmcc HDMI_M_AHB_CLK>,
>> + <&mmcc HDMI_S_AHB_CLK>;
>> + qcom,hdmi-tx-ddc-clk = <&msmgpio 70 GPIO_ACTIVE_HIGH>;
>> + qcom,hdmi-tx-ddc-data = <&msmgpio 71 GPIO_ACTIVE_HIGH>;
>> + qcom,hdmi-tx-hpd = <&msmgpio 72 GPIO_ACTIVE_HIGH>;
>> + core-vdda-supply = <&pm8921_hdmi_mvs>;
>> + hdmi-mux-supply = <&ext_3p3v>;
>> + };
>> +};
>> diff --git a/Documentation/devicetree/bindings/drm/msm/msm.txt
> b/Documentation/devicetree/bindings/drm/msm/msm.txt
>> new file mode 100644
>> index 0000000..484cc12
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/drm/msm/msm.txt
>> <at> <at> -0,0 +1,40 <at> <at>
>> +Qualcomm adreno/snapdragon
>> +
>> +Required properties:
>> +- compatible: "qcom,mdp" (mdp4) or "qcom,mdss_mdp" (mdp5)
>> +- reg: Physical base address and length of the controller's registers.
>
> As per the code (mdp5_kms.c), there are two sets of registers: "mdp_phys"
> and "vbif_phys". They should probably be added in the description here.
> "reg-names" entry might be needed as well.
hmm.. well, we aren't actually using vbif_phys *yet*.. although not
really sure if that will change.
I am wondering if maybe we should split out mdp4 vs mdp5 DT bindings.
Or at least remove the mdp5 compat string for now. So far I've only
got mdp4 working upstream, so probably a few things I've missed for
mdp5. Maybe just removing "qcom,mdss_mdp" for the time being is safer
until we decide.
I guess we should be really close to having all the needed
dependencies for 8074/dragonboard, so I suppose that is something I
should play with. Although seems unlikely that I'll have time until
after 3.17 merge window. Mostly I wanted to get mdp4 working first,
since I already have enough overlay support working there to be useful
for atomic modeset/pageflip ;-)
BR,
-R
>> +- interrupts: The interrupt outputs from the controller.
>> +- connectors: array of phandles for output device(s)
>> +- clocks: device clocks
>> + See ../clocks/clock-bindings.txt for details.
>> +
>> +Optional properties:
>> +- gpus: phandle for gpu device
>> +
>> +Example:
>> +
>> +/ {
>> + ...
>> +
>> + mdp: qcom,mdp <at> 5100000 {
>> + compatible = "qcom,mdp";
>> + reg = <0x05100000 0xf0000>;
>> + interrupts = <GIC_SPI 75 0>;
>> + connectors = <&hdmi>;
>> + gpus = <&gpu>;
>> + clock-names =
>> + "core_clk",
>> + "iface_clk",
>> + "lut_clk",
>> + "src_clk",
>> + "hdmi_clk",
>> + "mdp_clk";
>> + clocks =
>> + <&mmcc MDP_SRC>,
>> + <&mmcc MDP_AHB_CLK>,
>> + <&mmcc MDP_LUT_CLK>,
>> + <&mmcc TV_SRC>,
>> + <&mmcc HDMI_TV_CLK>,
>> + <&mmcc MDP_TV_CLK>;
>> + };
>> +};
> <snip>
>> diff --git a/drivers/gpu/drm/msm/hdmi/hdmi.c
> b/drivers/gpu/drm/msm/hdmi/hdmi.c
>> index 7f7aade..0ff8d46 100644
>> --- a/drivers/gpu/drm/msm/hdmi/hdmi.c
>> +++ b/drivers/gpu/drm/msm/hdmi/hdmi.c
> <snip>
>> <at> <at> -273,24 +275,37 <at> <at> static int hdmi_bind(struct
> device *dev, struct device *master, void *data)
>> return gpio;
>> }
>>
>> - /* TODO actually use DT.. */
>> - static const char *hpd_reg_names[] = {"hpd-gdsc", "hpd-5v"};
>> - static const char *pwr_reg_names[] = {"core-vdda", "core-vcc"};
>> - static const char *hpd_clk_names[] = {"iface_clk", "core_clk",
> "mdp_core_clk"};
>> - static unsigned long hpd_clk_freq[] = {0, 19200000, 0};
>> - static const char *pwr_clk_names[] = {"extp_clk", "alt_iface_clk"};
>> + if (of_device_is_compatible(of_node, "qcom,hdmi-tx-8x74")) {
>> + static const char *hpd_reg_names[] = {"hpd-gdsc", "hpd-5v"};
>> + static const char *pwr_reg_names[] = {"core-vdda", "core-vcc"};
>> + static const char *hpd_clk_names[] = {"iface_clk", "core_clk",
> "mdp_core_clk"};
>> + static unsigned long hpd_clk_freq[] = {0, 19200000, 0};
>> + static const char *pwr_clk_names[] = {"extp_clk", "alt_iface_clk"};
>> + config.phy_init = hdmi_phy_8x74_init;
>> + config.hpd_reg_names = hpd_reg_names;
>> + config.hpd_reg_cnt = ARRAY_SIZE(hpd_reg_names);
>> + config.pwr_reg_names = pwr_reg_names;
>> + config.pwr_reg_cnt = ARRAY_SIZE(pwr_reg_names);
>> + config.hpd_clk_names = hpd_clk_names;
>> + config.hpd_freq = hpd_clk_freq;
>> + config.hpd_clk_cnt = ARRAY_SIZE(hpd_clk_names);
>> + config.pwr_clk_names = pwr_clk_names;
>> + config.pwr_clk_cnt = ARRAY_SIZE(pwr_clk_names);
>> + } else if (of_device_is_compatible(of_node, "qcom,hdmi-tx-8960")) {
>> + static const char *hpd_clk_names[] = {"core_clk", "master_iface_clk",
> "slave_iface_clk"};
>> + static const char *hpd_reg_names[] = {"core-vdda", "hdmi-mux"};
>> + config.phy_init = hdmi_phy_8960_init;
>> + config.hpd_reg_names = hpd_reg_names;
>> + config.hpd_reg_cnt = ARRAY_SIZE(hpd_reg_names);
>> + config.hpd_clk_names = hpd_clk_names;
>> + config.hpd_clk_cnt = ARRAY_SIZE(hpd_clk_names);
>> + } else if (of_device_is_compatible(of_node, "qcom,hdmi-tx-8x60")) {
>> + config.phy_init = hdmi_phy_8x60_init;
>> + } else {
>> + dev_err(dev, "unknown phy: %s\n", of_node->name);
>> + }
>
> Stéphane.
>
--
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
