Messages in this thread | | | Date | Wed, 3 Apr 2024 12:54:47 +0300 | Subject | Re: [PATCH v2 0/8] Move camss version related defs in to resources | From | "Gjorgji Rosikopulos (Consultant)" <> |
| |
Hi Bryan,
On 4/2/2024 1:35 AM, Bryan O'Donoghue wrote: > On 19/03/2024 17:39, Gjorgji Rosikopulos wrote: >> The different resources required for different camss soc versions are >> split in to two groups: > > General series comment => > > You need to add your Signed-off-by: Gjorgji Rosikopulos > <quic_grosikop@quicinc.com> > > for each of the patches you are submitting.
Agree that is missing i will do that for the next-patchset
> > Perhaps yourself or some of the other people in the series deserve to > have Co-developed-by for some of the patches too ?
I agree as well i will double check if some people are missing.
> > I really like the reduction of code and the definition in one-place of > things so definitely the right direction. > > I won't do a deeper dive into the code until V3 but for reference here > are the branches I verified db410c, rb3, rb5 and x13s with. > > x13s: > https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/sc8280xp-6.9-rc1-camss-resource-change-verification?ref_type=heads > > db410c/rb3/rb5: > https://git.codelinaro.org/bryan.odonoghue/kernel/-/tree/linux-stable-v6.7.9-camss-rb5-20240319173935.481-1-quic_grosikop@quicinc.com?ref_type=heads
Thanks, i will get your changes for x13's as commented in previous review and submit new patch-set.
~Gjorgji
> > --- > bod
| |