Re: [PATCH] arch/powerpc: Remove unnecessary endian conversion code in XICS

2023-07-31 Thread Gautam Menghani
On Sat, Jul 29, 2023 at 08:54:26PM +1000, Michael Ellerman wrote: > Gautam Menghani writes: > > On Thu, Jul 06, 2023 at 05:50:57PM +1000, Jordan Niethe wrote: > >> On 30/6/23 3:56 pm, Gautam Menghani wrote: > >> > Remove an unnecessary piece of code that does an endianness conversion > >> > but >

Re: [PATCH] arch/powerpc: Remove unnecessary endian conversion code in XICS

2023-07-29 Thread Michael Ellerman
Gautam Menghani writes: > On Thu, Jul 06, 2023 at 05:50:57PM +1000, Jordan Niethe wrote: >> On 30/6/23 3:56 pm, Gautam Menghani wrote: >> > Remove an unnecessary piece of code that does an endianness conversion but >> > does not use the result. The following warning was reported by Clang's >> > st

Re: [PATCH] arch/powerpc: Remove unnecessary endian conversion code in XICS

2023-07-26 Thread Gautam Menghani
On Thu, Jul 06, 2023 at 05:50:57PM +1000, Jordan Niethe wrote: > > > On 30/6/23 3:56 pm, Gautam Menghani wrote: > > Remove an unnecessary piece of code that does an endianness conversion but > > does not use the result. The following warning was reported by Clang's > > static analyzer: > > > > a

Re: [PATCH] arch/powerpc: Remove unnecessary endian conversion code in XICS

2023-07-26 Thread Gautam Menghani
On Thu, Jul 06, 2023 at 05:50:57PM +1000, Jordan Niethe wrote: > > > On 30/6/23 3:56 pm, Gautam Menghani wrote: > > Remove an unnecessary piece of code that does an endianness conversion but > > does not use the result. The following warning was reported by Clang's > > static analyzer: > > > > a

Re: [PATCH] arch/powerpc: Remove unnecessary endian conversion code in XICS

2023-07-11 Thread Gautam Menghani
On Thu, Jul 06, 2023 at 05:50:57PM +1000, Jordan Niethe wrote: > > > On 30/6/23 3:56 pm, Gautam Menghani wrote: > > Remove an unnecessary piece of code that does an endianness conversion but > > does not use the result. The following warning was reported by Clang's > > static analyzer: > > > > a

Re: [PATCH] arch/powerpc: Remove unnecessary endian conversion code in XICS

2023-07-06 Thread Jordan Niethe
On 30/6/23 3:56 pm, Gautam Menghani wrote: Remove an unnecessary piece of code that does an endianness conversion but does not use the result. The following warning was reported by Clang's static analyzer: arch/powerpc/sysdev/xics/ics-opal.c:114:2: warning: Value stored to 'server' is never r

Re: [PATCH] arch/powerpc: Remove unnecessary endian conversion code in XICS

2023-07-02 Thread Michael Ellerman
Gautam Menghani writes: > Remove an unnecessary piece of code that does an endianness conversion but > does not use the result. The following warning was reported by Clang's > static analyzer: > > arch/powerpc/sysdev/xics/ics-opal.c:114:2: warning: Value stored to > 'server' is never read [deadcod

Re: [PATCH] arch/powerpc: Remove unnecessary endian conversion code in XICS

2023-06-29 Thread Gautam Menghani
On Fri, Jun 30, 2023 at 11:43:26AM +0530, Gautam Menghani wrote: > Remove an unnecessary piece of code that does an endianness conversion but > does not use the result. The following warning was reported by Clang's > static analyzer: > > arch/powerpc/sysdev/xics/ics-opal.c:114:2: warning: Value st

[PATCH] arch/powerpc: Remove unnecessary endian conversion code in XICS

2023-06-29 Thread Gautam Menghani
Remove an unnecessary piece of code that does an endianness conversion but does not use the result. The following warning was reported by Clang's static analyzer: arch/powerpc/sysdev/xics/ics-opal.c:114:2: warning: Value stored to 'server' is never read [deadcode.DeadStores] server = be16_

[PATCH] arch/powerpc: Remove unnecessary endian conversion code in XICS

2023-06-29 Thread Gautam Menghani
Remove an unnecessary piece of code that does an endianness conversion but does not use the result. The following warning was reported by Clang's static analyzer: arch/powerpc/sysdev/xics/ics-opal.c:114:2: warning: Value stored to 'server' is never read [deadcode.DeadStores] server = be16_