On Fri, Apr 5, 2024 at 11:34 AM Kalle Valo <kvalo@xxxxxxxxxx> wrote: > > Bartosz Golaszewski <brgl@xxxxxxxx> writes: > > > On Mon, Mar 25, 2024 at 3:01 PM Kalle Valo <kvalo@xxxxxxxxxx> wrote: > >> > >> Bartosz Golaszewski <brgl@xxxxxxxx> writes: > >> > >> > From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> > >> > > >> > + > >> > +maintainers: > >> > + - Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx> > >> > >> IMHO it would be better to have just driver maintainers listed here. > >> > > > > Why? What's wrong with having the author of the bindings in the Cc list? > > If you want follow the ath12k development and review patches then you > can join the ath12k list. I'm not fond of having too many maintainers, > it's not really helping anything and just extra work to periodically > cleanup the silent maintainers. > > I would ask the opposite question: why add you as the maintainer? > There's not even a single ath12k patch from you, nor I haven't seen you > doing any patch review or otherwise helping others related to ath12k. > Don't get me wrong, I value the work you do with this important powerseq > feature and hopefully we get it into the tree soon. But I don't see > adding you as a maintainer at this point. > In addition to what Krzysztof already said about you seamingly confusing the maintenance of the driver vs maintenance of the device-tree bindings (IOW: structured hardware description) and in response to your question: I don't see any functional change to any dt-bindings neither from you nor from Jeff. Are you convinced you can maintain and properly review any changes? If so, I don't really care, I can drop myself and have less work. Bartosz > -- > https://patchwork.kernel.org/project/linux-wireless/list/ > > https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches