On Fri, 22 Jan 2021 22:19:51 PST (-0800), Damien Le Moal wrote:
On 2021/01/23 13:15, Palmer Dabbelt wrote:
On Thu, 21 Jan 2021 00:32:38 PST (-0800), Damien Le Moal wrote:
On 2021/01/21 17:25, Linus Walleij wrote:
On Thu, Jan 21, 2021 at 1:15 AM Damien Le Moal <Damien.LeMoal@xxxxxxx> wrote:
On 2021/01/21 3:21, Palmer Dabbelt wrote:
On Mon, 18 Jan 2021 05:33:05 PST (-0800), linus.walleij@xxxxxxxxxx wrote:
Hi Damien,
this looks all right to me.
Reviewed-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
Shall I apply just this one patch to the pinctrl tree?
That's fine with me. The DT bindings are in riscv/for-next, maybe it's best to
take those as well and I'll drop them? I don't generally like to drop stuff
from for-next, but that's probably better than having everything all mixed up.
https://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux.git/commit/?h=for-next&id=ed3137edb31b86702511e7ad12b4abe8686b6805
I think the line that touches arch/*/Kconfig should be dropped
then, that better go to the SoC tree.
I'm OK with you taking them along with my Ack, but if you don't want to that's
fine. Just LMK
Acked-by: Palmer Dabbelt <palmerdabbelt@xxxxxxxxxx>
Linus,
Please let me know which way you want to proceed. If you want to take the patch,
I will resend without the Kconfig change and move that change to another patch.
It's fine to proceed as is, since Palmer already applied the bindings to
his tree, just apply the pinctrl driver there as well. I don't see any
risk of collisions in the tree.
Sounds good. Thanks !
I don't see an actual Ack here, but I'm OK taking it. IIRC there were some
issues with the DT stuff as well, I don't remember if they got resolved or if I
should be expecting a v14.
Palmer,
Linus sent a reviewed-by tag (see above).
For the dt-bindings problems, the v13 that I sent already addresses all make
dtbs_check warning and also addresses Rob's comments. The v13 dt-bindings
patches need review, as well as the clk driver patch. Will ping Stephen about
that one.
I will send a v14 only if there are more comments that need to be addressed.
Ah, thanks. This is on for-next. I'll look at the rest, but probably not
tonight as it's a bit late here.
