On Sun, 1 Feb 2015, Russell King - ARM Linux wrote:
> On Sun, Feb 01, 2015 at 02:39:42PM +1100, Finn Thain wrote:
> > I find the ARM support in drivers/char/nvram to be surprising, not to
> > say questionable. The /proc/driver/nvram implementation, given
> > defined(__arm__), decodes the NVRAM
On Sun, 1 Feb 2015, Geert Uytterhoeven wrote:
> On Sun, Feb 1, 2015 at 4:39 AM, Finn Thain wrote:
> > On Sun, 4 Jan 2015, Geert Uytterhoeven wrote:
> > > On Sun, Jan 4, 2015 at 8:21 AM, Finn Thain wrote:
> > > > On Thu, 1 Jan 2015, Rickard Strandqvist wrote:
> > > > > Removes some functions that
On Sun, 1 Feb 2015, Geert Uytterhoeven wrote:
On Sun, Feb 1, 2015 at 4:39 AM, Finn Thain wrote:
On Sun, 4 Jan 2015, Geert Uytterhoeven wrote:
On Sun, Jan 4, 2015 at 8:21 AM, Finn Thain wrote:
On Thu, 1 Jan 2015, Rickard Strandqvist wrote:
Removes some functions that are not used
On Sun, 1 Feb 2015, Russell King - ARM Linux wrote:
On Sun, Feb 01, 2015 at 02:39:42PM +1100, Finn Thain wrote:
I find the ARM support in drivers/char/nvram to be surprising, not to
say questionable. The /proc/driver/nvram implementation, given
defined(__arm__), decodes the NVRAM
On Sun, Feb 1, 2015 at 4:39 AM, Finn Thain wrote:
> On Sun, 4 Jan 2015, Geert Uytterhoeven wrote:
>
>> On Sun, Jan 4, 2015 at 8:21 AM, Finn Thain
>> wrote:
>> > On Thu, 1 Jan 2015, Rickard Strandqvist wrote:
>> > > Removes some functions that are not used anywhere:
>> > > mac_pram_write()
On Sun, Feb 01, 2015 at 02:39:42PM +1100, Finn Thain wrote:
> I find the ARM support in drivers/char/nvram to be surprising, not to say
> questionable. The /proc/driver/nvram implementation, given
> defined(__arm__), decodes the NVRAM contents in exactly the same format as
> when
On Sun, Feb 01, 2015 at 02:39:42PM +1100, Finn Thain wrote:
I find the ARM support in drivers/char/nvram to be surprising, not to say
questionable. The /proc/driver/nvram implementation, given
defined(__arm__), decodes the NVRAM contents in exactly the same format as
when defined(__i386__)
On Sun, Feb 1, 2015 at 4:39 AM, Finn Thain fth...@telegraphics.com.au wrote:
On Sun, 4 Jan 2015, Geert Uytterhoeven wrote:
On Sun, Jan 4, 2015 at 8:21 AM, Finn Thain fth...@telegraphics.com.au
wrote:
On Thu, 1 Jan 2015, Rickard Strandqvist wrote:
Removes some functions that are not used
On Sun, 4 Jan 2015, Geert Uytterhoeven wrote:
> On Sun, Jan 4, 2015 at 8:21 AM, Finn Thain wrote:
> > On Thu, 1 Jan 2015, Rickard Strandqvist wrote:
> > > Removes some functions that are not used anywhere:
> > > mac_pram_write() mac_pram_read()
> >
> > ... I'd rather not remove all of this
On Sun, 4 Jan 2015, Geert Uytterhoeven wrote:
On Sun, Jan 4, 2015 at 8:21 AM, Finn Thain fth...@telegraphics.com.au wrote:
On Thu, 1 Jan 2015, Rickard Strandqvist wrote:
Removes some functions that are not used anywhere:
mac_pram_write() mac_pram_read()
... I'd rather not remove all
On Sun, Jan 4, 2015 at 8:21 AM, Finn Thain wrote:
> On Thu, 1 Jan 2015, Rickard Strandqvist wrote:
>> Removes some functions that are not used anywhere:
>> mac_pram_write() mac_pram_read()
>
> If you remove those functions, you'd then find that all of the called
> functions become unused:
>
>
On Sun, Jan 4, 2015 at 8:21 AM, Finn Thain fth...@telegraphics.com.au wrote:
On Thu, 1 Jan 2015, Rickard Strandqvist wrote:
Removes some functions that are not used anywhere:
mac_pram_write() mac_pram_read()
If you remove those functions, you'd then find that all of the called
functions
On Thu, 1 Jan 2015, Rickard Strandqvist wrote:
> Removes some functions that are not used anywhere:
> mac_pram_write() mac_pram_read()
>
If you remove those functions, you'd then find that all of the called
functions become unused:
maciisi_read_pram
maciisi_write_pram
pmu_read_pram
On Thu, 1 Jan 2015, Rickard Strandqvist wrote:
Removes some functions that are not used anywhere:
mac_pram_write() mac_pram_read()
If you remove those functions, you'd then find that all of the called
functions become unused:
maciisi_read_pram
maciisi_write_pram
pmu_read_pram
Removes some functions that are not used anywhere:
mac_pram_write() mac_pram_read()
This was partially found by using a static code analysis program called
cppcheck.
Signed-off-by: Rickard Strandqvist
---
arch/m68k/mac/misc.c | 46 --
1 file
Removes some functions that are not used anywhere:
mac_pram_write() mac_pram_read()
This was partially found by using a static code analysis program called
cppcheck.
Signed-off-by: Rickard Strandqvist rickard_strandqv...@spectrumdigital.se
---
arch/m68k/mac/misc.c | 46
16 matches
Mail list logo