[PATCH 1/5] dt-bindings: arm: keystone: add ti,j7200-sci compatible

Conor Dooley conor at kernel.org
Thu Nov 30 07:42:48 PST 2023


On Wed, Nov 29, 2023 at 03:38:04PM +0000, Conor Dooley wrote:
> On Wed, Nov 29, 2023 at 03:34:20PM +0000, Conor Dooley wrote:
> > On Wed, Nov 29, 2023 at 04:31:17PM +0100, Thomas Richard wrote:
> > > On j7200, during suspend to ram the soc is powered-off.
> > > At resume requested irqs shall be restored which is a different behavior
> > > from other platforms.
> > > 
> > > Signed-off-by: Thomas Richard <thomas.richard at bootlin.com>
> > 
> > Acked-by: Conor Dooley <conor.dooley at microchip.com>
> 
> Un-Acked. Your dts patch contradicts this one.
> 
> Is the programming model compatible with the existing devices? To be
> compatible, the existing device only need to support a compatible subset
> of behaviours.
> If so, this patch is wrong. If not, then the dts one is.

Given Andrew's response, it looks like the dts patch is the correct one
of the two, and this patch should document the k2g as a fallback for the
jh7200.

Cheers,
Conor.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20231130/f1213474/attachment.sig>


More information about the linux-arm-kernel mailing list