Re: [PATCH] dt-bindings: interrupt-controller: loongarch: Fix mismathed compatible

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2022/12/8 下午4:42, Krzysztof Kozlowski wrote:
On 08/12/2022 04:02, Liu Peibao wrote:
The "compatible" doesn't match what the kernel is using. Fix it as
kernel using.

Fixes: 855d4ca4bdb3 ("irqchip: loongarch-cpu: add DT support")

Really? You just added these patches...

1.  In the same time Loongson people keep pinging for accepting these:
https://lore.kernel.org/all/e3bf1f8a-e37e-46e3-f9b3-20a9031d49e2@xxxxxxxxxxx/
https://lore.kernel.org/all/f5d0f796-0147-8789-5bed-edf38b28229e@xxxxxxxxxxx/

All the time - several pings, every week.

2. In the last weeks, two of your engineers sent the same driver
(Loongson I2C) in the same time, with different names! For the same
hardware.

3. Several other patches from Loongson, when posted on mailing lists,
have multiple compile warnings. This suggests they were not really
compiled properly (with different test configs, with W=1, smatch,
sparse, coccinelle). There were so many compiler warnings, that I assume
your patches should wait on mailing list for two weeks before starting
any review. Otherwise it's a bit waste of reviewer's effort.

Patches are welcomed but all this points on poor quality of submissions
and sending anything, hoping it will get merged upstream, regardless of
quality.

I appreciate fixing the patch fast, it's a good sign, but maybe Loongson
could a bit improve internal processes of review before sending and
pinging on mailing list?

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>

Best regards,
Krzysztof


Hi, Krzysztof

As for me privately, I must say that I'm very appreciate patient helps and guides to us (some of us are new comers include me without more experience) from each maintainer and reviewer, thanks very much! From some patch series(include my patches), it literally seems that some low quality problem (e.g. since some reasons such as not familiar with rules of community patch, building warning by test robot checking and no internal review process) just as noise that bothered reviewers and consumed more time (back and forth), which is definitely due to our poor work in my opinion (though I'm not sure how others think about it). Have Loongson folks (especially new comers like me) put more efforts into patch work and learn about experience from reviewing process, the patch quality would be improved better.

Again, thanks for pointing out some problems you mentioned above(e.g. same patch with different name from different Loongson people, frequently pinging and patch quality), I think I can feed back them to company to pay more attention and take steps to improve community work.

Thanks!
Jianmin Lv




[Index of Archives]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux