Re: [PATCH v6 0/6] Raspberry PI 4 V3D enablement

From: Melissa Wen <mwen@igalia.com>
To: Javier Martinez Canillas <javierm@redhat.com>
Cc: Florian Fainelli <f.fainelli@gmail.com>,
	Peter Robinson <pbrobinson@gmail.com>,
	Arnd Bergmann <arnd@arndb.de>,
	bcm-kernel-feedback-list@broadcom.com,
	Daniel Vetter <daniel@ffwll.ch>, David Airlie <airlied@linux.ie>,
	devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org,
	Emma Anholt <emma@anholt.net>,
	linux-arm-kernel@lists.infradead.org,
	linux-rpi-kernel@lists.infradead.org, maxime@cerno.tech,
	Nicolas Saenz Julienne <nsaenz@kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	Stefan Wahren <stefan.wahren@i2se.com>,
	Catalin Marinas <catalin.marinas@arm.com>,
	Will Deacon <will@kernel.org>
Subject: Re: [PATCH v6 0/6] Raspberry PI 4 V3D enablement
Date: Fri, 10 Jun 2022 10:05:56 -0100	[thread overview]
Message-ID: <20220610110556.63mddbye7vxq6mzb@mail.igalia.com> (raw)
In-Reply-To: <40f27e32-720a-2f51-2164-5e152f53443a@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 2073 bytes --]

On 06/08, Javier Martinez Canillas wrote:
> Hello Melissa,
> 
> On 6/8/22 17:51, Melissa Wen wrote:
> 
> [snip]
> 
> >>>
> >>> I can take the last 3 patches through the Broadcom ARM SoC pull request, 
> >>> but the first three should probably go via the DRM tree unless you want 
> >>> me to merge them all?
> >>
> >> I can merge the first 3 patches through the drm-misc tree. Can I get
> >> an ack from you for those ?
> >>
> >> The changes are independent so there's no need for an immutable branch
> >> or any kind of cross tree coordination.
> > 
> > Hi Javier,
> > 
> > I'm not sure if you're suggesting here to apply the entire series as it
> > is now.
> >
> 
> No. I suggested that could just apply the first 3 patches that were related
> to DRM, not the last 3 three since Florian will pick those.
>  
> > I'm not able to have a functional kernel from arm defconfig, only for
> > arm64. I'd like to have this issue clarified before merge this serie. I
> > tried multi_v7_defconfig on raspbian 32-bits and got a kernel panic.
> > Things work better when using downstream bcm2711_defconfig.
> > 
> > If you have an idea of what is going on, please, let me know. I can try
> 
> Can you please share for info? For example your boot log when it panics,
> maybe that could shed some light on what's going on.
> 
> > again and I'll be okay on merging it. Otherwise, let's wait for more
> > inputs to have a better picture of the situation.
> >
> 
> Of course I don't plan to push patches that are known to cause issues.
> 
> I mentioned that could help merging the DRM changes if needed before
> you sent your bug report.

Hi Javier,

Thanks for waiting a little.

Stefan guided me to the missing part and I'm okay on this serie.

If there's any r-b missing for drm/v3d, you can add mine:
Reviewed-by: Melissa Wen <mwen@igalia.com>

But if you prefer that I applied them, just let me know.

Best Regards,

Melissa

> 
> -- 
> Best regards,
> 
> Javier Martinez Canillas
> Linux Engineering
> Red Hat
> 

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