On 22/12/2022 14:01, Gatien CHEVALLIER wrote: > >> >> Anyway you duplicate work here: >> https://lore.kernel.org/all/c869d2751125181a55bc8a88c96e3a892b42f37a.1668070216.git.oleksii_moisieiev@xxxxxxxx/ >> and maybe you duplicate comments. >> >> I don't think there is point to review things twice, so NAK. > This is a result of me not knowing how to handle this particular case. > It is a patch that I need to have in my patch set in order to pass Rob's > bindings check. Otherwise, feature domains bindings defined here will > not be known in the STM32 System Bus binding file, where they are used. > > I wanted to illustrate the use of Oleksii's binding with a practical > use-case that we want to implement. Mention patch dependency in the path changelog (---) and that's it. No tests will be done. You are expected to run tests anyway on your side, before sending. > > What would be the correct way of managing this dependency? > -Specify something like "On top of ...." in the cover letter/patch and > reference the other thread? > -Use a "Depends-On" tag on the YAML binding commit? > -Something else? Best regards, Krzysztof
