On 05-09-23, 12:23, Lucas Stach wrote:
> > lots of magic numbers!
>
> Yes. Those are mostly tuning values for the PLL and I don't know if
> there is any computational way to come up with those numbers, so we're
> just using the values validated in downstream.
ok
>
> >
> > > +};
> > > +
> > > +struct reg_settings {
> > > + u8 reg;
> > > + u8 val;
> > > +};
> > > +
> > > +const struct reg_settings common_phy_cfg[] = {
> > > + { PHY_REG_00, 0x00 }, { PHY_REG_01, 0xD1 },
> > > + { PHY_REG_08, 0x4f }, { PHY_REG_09, 0x30 },
> > > + { PHY_REG_10, 0x33 }, { PHY_REG_11, 0x65 },
> > > + /* REG12 pixclk specific */
> > > + /* REG13 pixclk specific */
> > > + /* REG14 pixclk specific */
> > > + { PHY_REG_15, 0x80 }, { PHY_REG_16, 0x6C },
> > > + { PHY_REG_17, 0xF2 }, { PHY_REG_18, 0x67 },
> > > + { PHY_REG_19, 0x00 }, { PHY_REG_20, 0x10 },
> > > + /* REG21 pixclk specific */
> > > + { PHY_REG_22, 0x30 }, { PHY_REG_23, 0x32 },
> > > + { PHY_REG_24, 0x60 }, { PHY_REG_25, 0x8F },
> > > + { PHY_REG_26, 0x00 }, { PHY_REG_27, 0x00 },
> > > + { PHY_REG_28, 0x08 }, { PHY_REG_29, 0x00 },
> > > + { PHY_REG_30, 0x00 }, { PHY_REG_31, 0x00 },
> > > + { PHY_REG_32, 0x00 }, { PHY_REG_33, 0x80 },
> > > + { PHY_REG_34, 0x00 }, { PHY_REG_35, 0x00 },
> > > + { PHY_REG_36, 0x00 }, { PHY_REG_37, 0x00 },
> > > + { PHY_REG_38, 0x00 }, { PHY_REG_39, 0x00 },
> > > + { PHY_REG_40, 0x00 }, { PHY_REG_41, 0xE0 },
> > > + { PHY_REG_42, 0x83 }, { PHY_REG_43, 0x0F },
> > > + { PHY_REG_44, 0x3E }, { PHY_REG_45, 0xF8 },
> > > + { PHY_REG_46, 0x00 }, { PHY_REG_47, 0x00 }
> > > +};
> > > +
> > > +struct fsl_samsung_hdmi_phy {
> > > + struct device *dev;
> > > + void __iomem *regs;
> > > + struct clk *apbclk;
> > > + struct clk *refclk;
> > > +
> > > + /* clk provider */
> > > + struct clk_hw hw;
> > > + const struct phy_config *cur_cfg;
> > > +};
> > > +
> > > +static inline struct fsl_samsung_hdmi_phy *
> > > +to_fsl_samsung_hdmi_phy(struct clk_hw *hw)
> > > +{
> > > + return container_of(hw, struct fsl_samsung_hdmi_phy, hw);
> > > +}
> > > +
> > > +static void
> > > +fsl_samsung_hdmi_phy_configure_pixclk(struct fsl_samsung_hdmi_phy *phy,
> > > + const struct phy_config *cfg)
> > > +{
> > > + u8 div;
> > > +
> > > + switch (cfg->pixclk) {
> > > + case 22250000 ... 33750000: div = 0xf; break;
> > > + case 35000000 ... 40000000: div = 0xb; break;
> > > + case 43200000 ... 47500000: div = 0x9; break;
> > > + case 50349650 ... 63500000: div = 0x7; break;
> > > + case 67500000 ... 90000000: div = 0x5; break;
> > > + case 94000000 ... 148500000: div = 0x3; break;
> > > + case 154000000 ... 297000000: div = 0x1; break;
> >
> > lets do proper linux style please
>
> Do you mean moving the statements to separate lines?
Yes pls. Also check with scripts/checkpatch.pl --strict
--
~Vinod
