Messages in this thread | | | Date | Tue, 23 Aug 2022 13:23:32 +0300 | Subject | Re: [PATCH RESEND v10 6/7] clk: baikal-t1: Add DDR/PCIe directly controlled resets support | From | Krzysztof Kozlowski <> |
| |
On 22/08/2022 21:29, Serge Semin wrote: > Aside with a set of the trigger-like resets Baikal-T1 CCU provides two > additional blocks with directly controlled reset signals. In particular it > concerns DDR full and initial resets and various PCIe sub-domains resets. > Let's add the direct reset assertion/de-assertion of the corresponding > flags support into the Baikal-T1 CCU driver then. It will be required at > least for the PCIe platform driver. Obviously the DDR controller isn't > supposed to be fully reset in the kernel, so the corresponding controls > are added just for the sake of the interface implementation completeness. > > Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> > Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de> > > --- > > Changelog v6: > - Refactor the code to support the linear reset IDs only. (@Philipp) > > Changelog v7: > - Drop empty line from the sys_rst_info structure initialization block. > (@Philipp) > --- > drivers/clk/baikal-t1/ccu-rst.c | 66 +++++++++++++++++++++++++++++ > drivers/clk/baikal-t1/ccu-rst.h | 10 +++++ > include/dt-bindings/reset/bt1-ccu.h | 9 ++++
Do not mix drivers and bindings. Bindings always go separately, so this has to be split.
Best regards, Krzysztof
| |