Re: [linux][PATCH v2 00/20] Add support for Microchip PIC64GX Curiosity Kit

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1.1: Type: text/plain, Size: 1241 bytes --]

On Mon, Sep 30, 2024 at 10:54:29AM +0100, pierre-henry.moussay@microchip.com wrote:
> From: Pierre-Henry Moussay <pierre-henry.moussay@microchip.com>
> 
> Changes since v1:
> - Limit sdhci to HS (High Speed, 50MHz at 3.3V) mode max
> - Fix I2C dt-bindings, remove microchip,mpfs-i2c fallback
> - remove duplicated pic64gx-clock.h
> - Update Microchip dts Makefile
> - Update PIC64GX dts and dtsi (fix model, sort node)
> - Remove microchip,pic64gx-clock.h present in previous patchset
> - Reword commits messages
> 
> Hi all,
> 
> This V2 patchset is incomplete and doesn't address all the
> comments from the V1 patchset. More precisely the request on the following
> commit have not been addressed:
> [PATCH 08/17] dt-bindings: clock: mpfs-clkcfg: Add PIC64GX compatibility
> This will be fix in a V3 coming in due time.

It's not really clear from this comment that there are other problems
that were in the v1 that have remained in the v2 - the mailbox and
system controller feedback that I gave on v1. I've pointed all three out
again here against the relevant patches, and will hopefully have a
non-RFC implementation of what I want the pic64gx based on w.r.t
clock/mailbox sent this week.

Cheers,
Conor.

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 161 bytes --]

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv