Re: [PATCH] openrisc: Fix build warning in cache.c

2025-04-19 Thread Sahil Siddiq
Hi, On 4/19/25 10:42 AM, Stafford Horne wrote: On Fri, Apr 18, 2025 at 06:04:29PM +0530, Sahil Siddiq wrote: On 4/18/25 3:30 PM, Stafford Horne wrote: On Fri, Apr 18, 2025 at 02:42:20PM +0530, Sahil Siddiq wrote: [...] Maybe using another email in signed-off-by would be better. The current pa

Re: [PATCH] openrisc: Fix build warning in cache.c

2025-04-18 Thread Stafford Horne
On Fri, Apr 18, 2025 at 06:04:29PM +0530, Sahil Siddiq wrote: > Hi, > > On 4/18/25 3:30 PM, Stafford Horne wrote: > > On Fri, Apr 18, 2025 at 02:42:20PM +0530, Sahil Siddiq wrote: > > > On 4/18/25 1:17 PM, Stafford Horne wrote: > > > > On Fri, Apr 04, 2025 at 10:39:22AM +0530, Sahil Siddiq wrote:

Re: [PATCH] openrisc: Fix build warning in cache.c

2025-04-18 Thread Sahil Siddiq
Hi, On 4/18/25 3:30 PM, Stafford Horne wrote: On Fri, Apr 18, 2025 at 02:42:20PM +0530, Sahil Siddiq wrote: On 4/18/25 1:17 PM, Stafford Horne wrote: On Fri, Apr 04, 2025 at 10:39:22AM +0530, Sahil Siddiq wrote: [...] I was working on getting this patches ready for upstreaming and noticed one

Re: [PATCH] openrisc: Fix build warning in cache.c

2025-04-18 Thread Stafford Horne
On Fri, Apr 18, 2025 at 02:42:20PM +0530, Sahil Siddiq wrote: > Hi Stafford, > > On 4/18/25 1:17 PM, Stafford Horne wrote: > > On Fri, Apr 04, 2025 at 10:39:22AM +0530, Sahil Siddiq wrote: > > [...] > > Hi Sahil, > > > > Sorry for delay in getting back on this. > > No worries :) > > > I was wor

Re: [PATCH] openrisc: Fix build warning in cache.c

2025-04-18 Thread Sahil Siddiq
Hi Stafford, On 4/18/25 1:17 PM, Stafford Horne wrote: On Fri, Apr 04, 2025 at 10:39:22AM +0530, Sahil Siddiq wrote: [...] Hi Sahil, Sorry for delay in getting back on this. No worries :) I was working on getting this patches ready for upstreaming and noticed one thing: -

Re: [PATCH] openrisc: Fix build warning in cache.c

2025-04-18 Thread Stafford Horne
On Fri, Apr 04, 2025 at 10:39:22AM +0530, Sahil Siddiq wrote: > Hi Stafford, > > On 4/3/25 7:36 PM, Stafford Horne wrote: > > [...] > > I will just take this fix and apply it to the series (git fixup) rather than > > take this patch as is. Also, as registers should be unsigned short, I think > >

Re: [PATCH] openrisc: Fix build warning in cache.c

2025-04-03 Thread Sahil Siddiq
Hi Stafford, On 4/3/25 7:36 PM, Stafford Horne wrote: [...] I will just take this fix and apply it to the series (git fixup) rather than take this patch as is. Also, as registers should be unsigned short, I think we should change the type to that. I will fixup patches in place. Sure thing.

Re: [PATCH] openrisc: Fix build warning in cache.c

2025-04-03 Thread Stafford Horne
Hi Sahil, On Wed, Apr 02, 2025 at 01:31:29AM +0530, Sahil Siddiq wrote: > Commit c5c6fd8be51207f0abd3 ("openrisc: Introduce new utility functions > to flush and invalidate caches") introduced new functions to flush or > invalidate a range of addresses. These functions make use of the mtspr > macr

[PATCH] openrisc: Fix build warning in cache.c

2025-04-01 Thread Sahil Siddiq
Commit c5c6fd8be51207f0abd3 ("openrisc: Introduce new utility functions to flush and invalidate caches") introduced new functions to flush or invalidate a range of addresses. These functions make use of the mtspr macro. The kernel test robot reported an asm constraint-related warning and error rel