Message ID | 1681468167-11689-1-git-send-email-quic_srichara@quicinc.com (mailing list archive) |
---|---|
Headers | show |
Series | Add minimal boot support for IPQ5018 | expand |
On 14.04.2023 12:29, Sricharan Ramabadhran wrote: > Add support for the global clock controller found on IPQ5018 > based devices. > > Co-developed-by: Varadarajan Narayanan <quic_varada@quicinc.com> > Signed-off-by: Varadarajan Narayanan <quic_varada@quicinc.com> > Co-developed-by: Gokul Sriram Palanisamy <quic_gokulsri@quicinc.com> > Signed-off-by: Gokul Sriram Palanisamy <quic_gokulsri@quicinc.com> > Signed-off-by: Sricharan Ramabadhran <quic_srichara@quicinc.com> > --- [...] > + .name = "adss_pwm_clk_src", > + .parent_data = gcc_xo_gpll0, > + .num_parents = 2, ARRAY_SIZE(), everywhere where you reference parent_data [...] > +MODULE_DESCRIPTION("Qualcomm Technologies, Inc. GCC IPQ5018 Driver"); > +MODULE_LICENSE("GPL v2"); "GPL" Konrad
On 4/14/2023 4:59 PM, Konrad Dybcio wrote: > > > On 14.04.2023 12:29, Sricharan Ramabadhran wrote: >> Add support for the global clock controller found on IPQ5018 >> based devices. >> >> Co-developed-by: Varadarajan Narayanan <quic_varada@quicinc.com> >> Signed-off-by: Varadarajan Narayanan <quic_varada@quicinc.com> >> Co-developed-by: Gokul Sriram Palanisamy <quic_gokulsri@quicinc.com> >> Signed-off-by: Gokul Sriram Palanisamy <quic_gokulsri@quicinc.com> >> Signed-off-by: Sricharan Ramabadhran <quic_srichara@quicinc.com> >> --- > [...] > >> + .name = "adss_pwm_clk_src", >> + .parent_data = gcc_xo_gpll0, >> + .num_parents = 2, > ARRAY_SIZE(), everywhere where you reference parent_data > ok. > [...] > >> +MODULE_DESCRIPTION("Qualcomm Technologies, Inc. GCC IPQ5018 Driver"); >> +MODULE_LICENSE("GPL v2"); > "GPL" ok, will fix. Regards, Sricharan
On 14/04/2023 12:29, Sricharan Ramabadhran wrote: > The IPQ5018 is Qualcomm's 802.11ax SoC for Routers, > Gateways and Access Points. > > This series adds minimal board boot support for ipq5018-mp03.1-c2 board. > > [v3] > Fixed all comments for clocks, schema fixes > Picked up Reviewed-by from Bjorn for pinctrl driver Didn't you just ignore all the tags you got? I was already acking and reviewing a lot of these... Best regards, Krzysztof
Hi Krysztof, On 4/16/2023 2:37 PM, Krzysztof Kozlowski wrote: > On 14/04/2023 12:29, Sricharan Ramabadhran wrote: >> The IPQ5018 is Qualcomm's 802.11ax SoC for Routers, >> Gateways and Access Points. >> >> This series adds minimal board boot support for ipq5018-mp03.1-c2 board. >> >> [v3] >> Fixed all comments for clocks, schema fixes >> Picked up Reviewed-by from Bjorn for pinctrl driver > > Didn't you just ignore all the tags you got? I was already acking and > reviewing a lot of these... > While i got review comments for V2, only one tag from Bjorn (reviewed-by) for the pinctrl driver. That one i picked. Regards, Sricharan