Re: [RFC, PATCH, v3.9] default exported asm symbols to zero

2016-12-02 Thread Geert Uytterhoeven
On Fri, Dec 2, 2016 at 1:40 PM, Arnd Bergmann <a...@arndb.de> wrote:
> With binutils-2.16 and before, a weak missing symbol was kept during the

2.26?

> final link, and a missing CRC for an export would lead to that CRC
> being treated as zero implicitly. With binutils-2.17, the crc

2.27?

> symbol gets dropped, and any module trying to use it will fail to
> load.

Gr{oetje,eeting}s,

    Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds



Bug#776211: Missing platform support in Debian's m68k kernel config

2015-01-25 Thread Geert Uytterhoeven
Source: linux
Version: 3.16.7-ckt2-1

Missing options in config-3.16.0-4-m68k.txt:

Generic:
  CONFIG_KEXEC=y
  CONFIG_BOOTINFO_PROC=y

Platform-specific:
  CONFIG_HP300=y
CONFIG_DIO=y
CONFIG_HPLANCE=m
CONFIG_SUN3LANCE=m
CONFIG_KEYBOARD_HIL_OLD=y
CONFIG_KEYBOARD_HIL=y
CONFIG_HP_SDC_RTC=m
CONFIG_HP_SDC=y
CONFIG_HIL_MLC=y
CONFIG_FB_HP300=y

  CONFIG_SUN3X=y
CONFIG_SUN3X_ESP=m
CONFIG_KEYBOARD_SUNKBD=y

  CONFIG_PARPORT_PC=m (Q40/Q60 parallel port)
  CONFIG_RTC_DRV_MSM6242=m (or y, cfr. CONFIG_RTC_DRV_RP5C01=y?) (Amiga RTC)

Thanks!

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/camuhmdu3e_d-n-nebuw_qmib8pqyykurrdkx37axu6-gf-z...@mail.gmail.com



Bug#775611: [sh]: FTBFS due to unknown compiler option '-m32'

2015-01-18 Thread Geert Uytterhoeven
On Sun, Jan 18, 2015 at 10:58 AM, Ben Hutchings b...@decadent.org.uk wrote:
 On Sun, 2015-01-18 at 00:35 +0100, John Paul Adrian Glaubitz wrote:
 Package: linux
 Version: 3.16.7-ckt4-1
 Severity: normal
 Tags: patch

 Hi!

 The kernel package currently fails to build from source on sh4 since
 the build scripts try to pass the '-m32' compiler option on gcc which
 is not available with gcc on sh4 (also according to the manpage).

 Selecting HAVE_C_RECORDMCOUNT in arch/sh/Kconfig (see attached patch
 by Ben Hutchings) fixes the issues. However, I also suggest removing
 the line $cc .=  -m32; for sh compiler options in scripts/
 recordmcount.pl.

 Steven and SH experts, what do you think of this?

It seems most compilers don't support -m32, unless they're biarch.
On an Ubuntu 14.04 LTS box with lots of cross-compilers from korg/crosstool:

/usr/bin/c89-gcc OK
/usr/bin/c99-gcc OK
/usr/bin/x86_64-linux-gnu-gcc OK
/opt/cross/gcc-4.2.4-nolibc/avr32-linux/bin/avr32-linux-gcc cc1:
error: unrecognized command line option -m32
/opt/cross/gcc-4.8.0-nolibc/microblaze-linux/bin/microblaze-linux-gcc
microblaze-linux-gcc: error: unrecognized command line option '-m32'
/opt/cross/gcc-4.8.0-nolibc/powerpc64-linux/bin/powerpc64-linux-gcc OK
/opt/cross/gcc-4.5.1-nolibc/or32-linux/bin/or32-linux-gcc cc1: error:
unrecognized command line option -m32
/opt/cross/gcc-4.6.3-nolibc/crisv32-linux/bin/crisv32-linux-gcc cc1:
error: unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/frv-linux/bin/frv-linux-gcc cc1: error:
unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/bfin-uclinux/bin/bfin-uclinux-gcc cc1:
error: unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/sparc64-linux/bin/sparc64-linux-gcc OK
/opt/cross/gcc-4.6.3-nolibc/m68k-linux/bin/m68k-linux-gcc cc1: error:
unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/alpha-linux/bin/alpha-linux-gcc cc1:
error: unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/sh4-linux/bin/sh4-linux-gcc cc1: error:
unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/m32r-linux/bin/m32r-linux-gcc cc1: error:
unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/ia64-linux/bin/ia64-linux-gcc cc1: error:
unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/xtensa-linux/bin/xtensa-linux-gcc cc1:
error: unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/arm-unknown-linux-gnueabi/bin/arm-unknown-linux-gnueabi-gcc
cc1: error: unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/am33_2.0-linux/bin/am33_2.0-linux-gcc cc1:
error: unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/mips64-linux/bin/mips64-linux-gcc cc1:
error: unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/s390x-linux/bin/s390x-linux-gcc cc1:
error: unrecognized command line option '-m32'
/opt/cross/gcc-4.6.3-nolibc/hppa64-linux/bin/hppa64-linux-gcc cc1:
error: unrecognized command line option '-m32'
/opt/cross/gcc-linaro-aarch64-none-elf-4.8-2013.08_linux/bin/aarch64-none-elf-gcc
aarch64-none-elf-gcc: error: unrecognized command line option ‘-m32’

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/CAMuHMdXe=hRZ_i=9-jpgmykfhnn4gtokheam+laa5qn0ywf...@mail.gmail.com



Re: [PATCH v2 5/5] m25p80,spi-nor: Share the list of supported chip type names again

2014-09-30 Thread Geert Uytterhoeven
Hi Ben,

On Tue, Sep 30, 2014 at 4:15 AM, Ben Hutchings b...@decadent.org.uk wrote:
 Move the list of chip type information to a macro in spi-nor.h, but
 leave the definitions of INFO and CAT25_INFO in spi-nor.

 In m25p80, define the INFO and CAT25_INFO macros to initialise a
 struct spi_device_id with the name, ignoring the remaining parameters.

 Signed-off-by: Ben Hutchings b...@decadent.org.uk

Thanks!

I didn't have an in-depth look, but this looks much better, and has less
opportunities for getting out-of-sync.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/CAMuHMdXRu0tMc1kbWfoVzh=efky31zqejdztrug_ehxdi-x...@mail.gmail.com



Re: [PATCH 5/5] m25p80,spi-nor: Share the list of supported chip type names again

2014-09-17 Thread Geert Uytterhoeven
Hi Ben,

On Mon, Sep 15, 2014 at 5:07 PM, Ben Hutchings b...@decadent.org.uk wrote:
 
  +#define __SPI_NOR_ENUM_TYPES(c_id, str_and_c_id) \
  +   c_id(at25fs010) c_id(at25fs040) c_id(at25df041a)  \
  +   c_id(at25df321a)c_id(at25df641) c_id(at26f004)\

 Can't you just have the IDs in a header file only, and let the header file
 generate either a struct flash_info or a struct spi_device_id table, using
 a macro defined by the file that includes it?

 How would we match up the rest of the struct flash_info to the name?

Ah, you use the enums to match the names to the rest of the flash_info.
But you can do it in one-shot, can't you?

spi-nor-data.h:

FLASH_ENTRY(at25fs010, 0x1f6601, 0, 32 * 1024,   4, SECT_4K),
FLASH_ENTRY(at25fs040, 0x1f6604, 0, 64 * 1024,   8, SECT_4K),
...

m25p80.c:

#define FLASH_ENTRY(_name, _jedec_id, _ext_id, _sector_size,
_n_sectors, _flags) \
{ .name = _name },

static const struct spi_device_id m25p_ids[] = {
#include spi-nor-data.h
};

spi-nor.c:

#define FLASH_ENTRY(_name, _jedec_id, _ext_id, _sector_size,
_n_sectors, _flags) \
{ \
 .name = _name, \
.jedec_id = (_jedec_id), \
.ext_id = (_ext_id), \
.sector_size = (_sector_size), \
.n_sectors = (_n_sectors), \
.page_size = 256, \
.flags = (_flags), \
}

static const struct flash_info spi_nor_info[] = {
#include spi-nor-data.h
};

Then the whole table data in contained in one place (spi-nor-data.h).
Am I still missing something?

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/camuhmdvsyvuh3qvrhw15kmon0myocbjvqbr9tkhtebm19c8...@mail.gmail.com



Re: [PATCH 5/5] m25p80,spi-nor: Share the list of supported chip type names again

2014-09-15 Thread Geert Uytterhoeven
Hi Ben,

On Sun, Sep 14, 2014 at 7:11 PM, Ben Hutchings b...@decadent.org.uk wrote:
 Move the list of chip type names to spi-nor.h.  To avoid putting all
 the chip type information there, we define this list as a macro
 SPI_NOR_ENUM_TYPES() that invokes another macro SPI_NOR_ENUM_TYPE()
 that must be defined to expand each name to whatever form it's needed.

 In spi-nor.c, use it to define enumerators, then use the enumerators
 as indices when defining spi_nor_info so that we cannot accidentally
 use a name that's not on the list.

 This is somewhat complicated by the fact that some names include
 hyphens.  SPI_NOR_ENUM_TYPE() therefore takes separate string and C
 identifier parameters.

Thanks for doing this!

However, the table generation still looks overly complicated to me, with
too much duplication which needs to be kept in sync.

 --- a/drivers/mtd/devices/m25p80.c
 +++ b/drivers/mtd/devices/m25p80.c
 @@ -245,51 +245,10 @@ static int m25p_remove(struct spi_device *spi)
  }


 -/*
 - * XXX This needs to be kept in sync with spi_nor_info.  We can't share
 - * it with spi-nor, because if this is built as a module then modpost
 - * won't be able to read it and add appropriate aliases.
 - */
  static const struct spi_device_id m25p_ids[] = {

[...]

 +#define SPI_NOR_ENUM_TYPE(s, c) { s },
 +   SPI_NOR_ENUM_TYPES()
 +#undef SPI_NOR_ENUM_TYPE
 { },

 --- a/drivers/mtd/spi-nor/spi-nor.c
 +++ b/drivers/mtd/spi-nor/spi-nor.c

  /* NOTE: double check command sets and memory organization when you add
   * more nor chips.  This current list focusses on newer chips, which
   * have been converging on command sets which including JEDEC ID.
 + *
 + * All chip type names must be added to __SPI_NOR_ENUM_TYPES in

Oops... These will get out of sync.

 + * spi-nor.h before they can be used here (enforced by using the
 + * names to index this array).
   */
  static const struct flash_info spi_nor_info[] = {
 /* Atmel -- some are (confusingly) marketed as DataFlash */
 -   INFO(at25fs010,  0x1f6601, 0, 32 * 1024,   4, SECT_4K),
 -   INFO(at25fs040,  0x1f6604, 0, 64 * 1024,   8, SECT_4K),
 +   INFO(at25fs010,  0x1f6601, 0, 32 * 1024,   4, SECT_4K),
 +   INFO(at25fs040,  0x1f6604, 0, 64 * 1024,   8, SECT_4K),

[...]


 --- a/include/linux/mtd/spi-nor.h
 +++ b/include/linux/mtd/spi-nor.h
 @@ -198,4 +198,76 @@ struct spi_nor {
   */
  int spi_nor_scan(struct spi_nor *nor, const char *name, enum read_mode mode);

 +#define __SPI_NOR_ENUM_TYPES(c_id, str_and_c_id) \
 +   c_id(at25fs010) c_id(at25fs040) c_id(at25df041a)  \
 +   c_id(at25df321a)c_id(at25df641) c_id(at26f004)\

Can't you just have the IDs in a header file only, and let the header file
generate either a struct flash_info or a struct spi_device_id table, using
a macro defined by the file that includes it?
Cfr. include/uapi/asm-generic/unistd.h and its use of __SYSCALL()?

Or am I missing something (e.g. this is impossible due to the hyphens in the
names?).

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/camuhmdus+7uidcd6tivnq6+tbd4_piixlvqwusa0z1u3sf7...@mail.gmail.com



Re: Autoloading of SPI/nor driver on kirkwood (qnap devices)

2014-09-10 Thread Geert Uytterhoeven
+ devicetree, as the names are used in DTS file.

On Wed, Sep 10, 2014 at 8:11 AM, Brian Norris
computersforpe...@gmail.com wrote:
 + Huang (his Freescale address is bouncing), Rafal

 On Tue, Sep 9, 2014 at 11:00 PM, Brian Norris
 computersforpe...@gmail.com wrote:
 On Thu, Sep 04, 2014 at 09:02:04AM +0200, Geert Uytterhoeven wrote:
 On Thu, Sep 4, 2014 at 4:11 AM, Ben Hutchings b...@decadent.org.uk wrote:
  I noticed that many platforms declare the flash chip as compatible =
  st,m25pXXX whereas the ts219.dtsi just said m25p128 but I tried
  changing that and it didn't help. In any case without spi-nor.ko being
  autoloaded I don't support m25p80.ko ever would be.
 
  m25p80.c has:
 
  static struct spi_driver m25p80_driver = {
  ...
  .id_table   = spi_nor_ids,
  ...
  };
 
  while spi_nor_ids is defined in spi-nor.c.  Since they end up as two
  separate modules, modpost can't read the ID table and add the device ID
  aliases to m25p80.ko.

 Woops. This indeed doesn't work.
 Note that the MODULE_DEVICE_TABLE() is also gone from m25p80.c

 So m25p80.c needs to contain the IDs, while spi-nor.c also needs the IDs
 because it's a framework/library.

 Actually, m25p80.c only needs the strings (i.e., the named aliases --
 character data), and for the most part, spi-nor.c only needs the IDs (i.e.,
 the identification bytes).

 What's more, I don't think that any modern SPI NOR user really needs to
 be specifying exactly which SPI device it is via a specific string. Our
 driver code pretty much always re-detects the device by reading its
 device ID. All the SPI NOR code needs to know is how to read its ID.

But the proper name should be in the compatible property.

 A quick solution would be to move the ID table to a header file, and include
 that by both, and re-add the lost MODULE_DEVICE_TABLE() to m25p80.c.
 That duplicates the data, though.

 Hmm, for the built-in case, we can avoid the duplication by letting m25p80.c
 refer to the table in spi-nor.c if !MODULE.
 Does anyone see a better solution?

 A little bit of a shot in the dark, as I haven't fleshed this one out:

 Would it work to just copy the SPI ID strings into m25p80.c, keep the
 full table in spi-nor.c, stop adding SPI ID string (and DT) bindings to
 the m25p80 table (force platforms to use m25p80-compatible probing, or
 maybe something generic like spi-nor-rdid, spi-nor-sfdp, etc.) and
 eventually kill the strings from spi-nor.c entirely? I really don't want
 to propagate string-binding much further into the SPI NOR library, since
 everything should be auto-detectable--partly just by an ID table as we
 have now, but eventually we can use CFI or SFDP parameters provided by
 relatively new SPI NOR chips.

How does the SPI core know to use m25p80? From the ids, and the compatible
value. If you won't add new names, all devices should declare compatiblity
with m25p80 (that's spansion,m25p80, right?) as a fallback.
However, are all devices really compatible with m25p80?
I.e. does m25p80.c work will all devices, in the absense of support for the
real device? If not, the fallback must not be there, and new device names
must be added.

 I'm not sure if this is great for the short-term problem of fixing
 module-autoloading. Perhaps we should do a short-term hack to duplicate
 the SPI ID strings to m25p80.c by including them in a header (and
 backport for 3.16.y stable?), but I'd like to disentangle this.

 Brian

Autoloading is based on spi:name. If new names aren't added, m25p80
won't be loaded for new SPI FLASHes, unless I'm missing something?

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/CAMuHMdXO8+9UWgYTv-fcostAipoqjZ=wqe56l4l-sr09l33...@mail.gmail.com



Re: Autoloading of SPI/nor driver on kirkwood (qnap devices)

2014-09-04 Thread Geert Uytterhoeven
Hi Ben,

On Thu, Sep 4, 2014 at 4:11 AM, Ben Hutchings b...@decadent.org.uk wrote:
 I noticed that many platforms declare the flash chip as compatible =
 st,m25pXXX whereas the ts219.dtsi just said m25p128 but I tried
 changing that and it didn't help. In any case without spi-nor.ko being
 autoloaded I don't support m25p80.ko ever would be.

 m25p80.c has:

 static struct spi_driver m25p80_driver = {
 ...
 .id_table   = spi_nor_ids,
 ...
 };

 while spi_nor_ids is defined in spi-nor.c.  Since they end up as two
 separate modules, modpost can't read the ID table and add the device ID
 aliases to m25p80.ko.

Woops. This indeed doesn't work.
Note that the MODULE_DEVICE_TABLE() is also gone from m25p80.c

So m25p80.c needs to contain the IDs, while spi-nor.c also needs the IDs
because it's a framework/library.

A quick solution would be to move the ID table to a header file, and include
that by both, and re-add the lost MODULE_DEVICE_TABLE() to m25p80.c.
That duplicates the data, though.

Hmm, for the built-in case, we can avoid the duplication by letting m25p80.c
refer to the table in spi-nor.c if !MODULE.
Does anyone see a better solution?

 It looks like m25p80 has been split up in order that the new fsl-qspi
 driver can share its code (bypassing the SPI core and its device
 model!).  Why doesn't fsl-qspi implement an SPI controller device?  I
 think it is totally unacceptable to regress m25p80 in this way for the
 sake of a new driver.

Fsl-qspi is not a generic SPI driver, but a dedicated SPI NOR driver.
It's programmed using high-level SPI NOR commands, instead of low-level
SPI commands.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/CAMuHMdWBr=b0_xivti9k5reryhheunqj2p26fjhmzho-l9t...@mail.gmail.com



Re: linux 3.10.1 with initrd (was Re: linux 3.8.12-1 (atari flavour) does not boot)

2013-07-30 Thread Geert Uytterhoeven
On Sat, Jul 20, 2013 at 7:27 PM, Thorsten Glaser t...@mirbsd.de wrote:
 I’d like the m68k people to read the whole eMail and test
 https://www.freewrt.org/~tg/dp/dists/hacks/dirty/Crossbuilt/linux/linux-image-3.10-0+m68k.1-m68k_3.10.1-1+m68k.1_m68k.deb

404

 on all their boxen, at least Amiga/Atari/Macintosh
 though the image also has BVME and MVME support
 (no Sun3 or Q40 but that can easily be added ofc).

 Also, I’d like the m68k people *and* the Debian Linux
 kernel people to review the configuration, to see whether
 there are things either can absolutely not live with:
 https://www.freewrt.org/~tg/dp/dists/hacks/dirty/Crossbuilt/linux/config-3.10-0+m68k.1-m68k

404

 So good, so far. An ARAnyM log of the build (kernel
 messages) is attached. There’s one thing sticking out:

 [0.16] WARNING: at /root/linux-3.10.1/init/main.c:698 
 do_one_initcall+0x12e/0x13a()
 [0.16] initcall param_sysfs_init+0x0/0x1a4 returned with disabled 
 interrupts

 This seems to happen for several initcalls. Geert et al,
 can you have a look at them, they’re scary ☺

At first sight, this doesn't look like an m68k-specific problem.
Do you see it with mainline 3.10 (or any other recent version) too? I don't.

I wanted to try your binary, but 404.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/camuhmdu136v5n23ujk+qxboqcxtxovjn2geusxnxxjhmtx_...@mail.gmail.com



Re: Bug#717689: linux: please review and merge m68k patch

2013-07-24 Thread Geert Uytterhoeven
On Wed, Jul 24, 2013 at 9:34 AM, Thorsten Glaser t...@mirbsd.de wrote:
   +## choice: Preemption Model
 […]
 Now why do you want to change that?

 My naïve reasoning is like that: less preemption = less
 interruption / context switches = more work done in total
 at the cost of a bit interactivity (probably not all that
 much because it’s already slow).

 If that’s wrong I’ll remove this part too, just say so.

It doesn't matter changing these, as arch/m68k/Kconfig has:

if COLDFIRE
source kernel/Kconfig.preempt
endif

so none of these symbols should be in your .config file?

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/camuhmdx7nnhy7c56zdq5h623p9qepzsmep3dgcan0ox53tk...@mail.gmail.com



Re: linux 3.8.12-1 (atari flavour) does not boot

2013-05-14 Thread Geert Uytterhoeven
On Mon, May 13, 2013 at 11:29 PM, Thorsten Glaser t...@mirbsd.de wrote:
 OK, thanks. What about the others (amiga, bvme*, mac, two mvme*)?

arch/m68k/kernel/head.S has code to set up a 4 MiB mapping (1 page (4 KiB)
of 1024 PTEs) for the kernel, so it applies to all subplatforms:

/*
 *  mmu_init
 *
 *  This block of code does what's necessary to map in the various kinds
 *  of machines for execution of Linux.
 *  First map the first 4 MB of kernel code  data
 */

mmu_map #PAGE_OFFSET,%pc@(L(phys_kernel_start)),#4*1024*1024,\
%pc@(m68k_supervisor_cachemode)

I guess it can be extended, if really needed. Does it work if you
change the #4
above to #8?

Note that  4 MiB kernels won't work on platforms where the first memory block
is smaller than the kernel size.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/camuhmdukta4yap_jmod+8fxqy40iuh9szojmnz2+dxmzmwp...@mail.gmail.com



Re: linux 3.8.12-1 (atari flavour) does not boot

2013-05-13 Thread Geert Uytterhoeven
Hi Thorsten,

On Mon, May 13, 2013 at 9:30 PM, Thorsten Glaser t...@mirbsd.de wrote:
 ARAnyM cannot boot the latest kernel images (still compiling,
 but I copied out vmlinux.gz):

 tglase@tglase:~/stuff/aranym/vm2 $ ./run
 Running Ara2 on X11: :2
 ARAnyM 0.9.15
 Using config file: 'aranym.config.x11'
 Could not open joystick 0
 ARAnyM RTC Timer: /dev/rtc: Permission denied
 ARAnyM LILO: Error loading ramdisk 'root.bin'
 CPU: Double bus fault detected !
 CPU: Halting

 Unfortunately, the SDL window stops there. “Headless” does
 not show anything either.

 I notice its uncompressed size is 4361000 as opposed to the
 latest previous kernel image of 3790400 bytes (3.2.35-2).
 Is there a known size limit (architectural, like sparc, or
 otherwise)?

Kernel images fail to boot on Atari if they're bigger than 4 MiB.
I also noticed this a while ago when bisecting an allmodconfig boot failure.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/CAMuHMdX+t0=J+8a1JoF6Jf=go_roul_kgavjf0dja-cgpgp...@mail.gmail.com



Re: [RFC] Simplifying kernel configuration for distro issues

2012-07-19 Thread Geert Uytterhoeven
On Thu, Jul 19, 2012 at 8:22 PM, Josh Boyer jwbo...@redhat.com wrote:
 On Thu, Jul 19, 2012 at 08:20:36PM +0200, Paul Bolle wrote:
 On Thu, 2012-07-19 at 13:19 -0400, Josh Boyer wrote:
  kconfig already spits out warnings for symbols being selected that
  don't exist.

 Does it? Since when does it do that? Or do you mean select in a more
 general way (not just meaning Kconfig's select statement)?

 I believe Alan was more correct than me when he said it was 'make
 oldconfig' that produced the warnings.

Kconfig does spit out warnings for selecting things with unmet dependencies.
But does anyone care?

[...checking logs...]

Oh, only 12 warnings in the v3.5-rc7 builds. Not that bad as my gut feeling
said...

warning: (ETRAX_USB_HOST  MOUSE_APPLETOUCH  MOUSE_BCM5974 
MOUSE_SYNAPTICS_USB  JOYSTICK_XPAD  TABLET_USB_ACECAD 
TABLET_USB_AIPTEK  TABLET_USB_HANWANG  TABLET_USB_KBTAB 
TABLET_USB_WACOM  TOUCHSCREEN_USB_COMPOSITE  INPUT_ATI_REMOTE2 
INPUT_KEYSPAN_REMOTE  INPUT_POWERMATE  INPUT_YEALINK 
INPUT_CM109) selects USB which has unmet direct dependencies
(USB_SUPPORT  USB_ARCH_HAS_HCD): 10 warnings in 2 logs
warning: (IWLWIFI  IWLEGACY  ATH5K  ATH9K  ATH9K_HTC 
CARL9170_LEDS) selects MAC80211_LEDS which has unmet direct
dependencies (NET  WIRELESS  MAC80211  LEDS_CLASS): 10 warnings
in 2 logs
warning: (LOCKDEP  FAULT_INJECTION_STACKTRACE_FILTER  LATENCYTOP
 KMEMCHECK) selects FRAME_POINTER which has unmet direct
dependencies (DEBUG_KERNEL  (CRIS || M68K || FRV || UML || AVR32 ||
SUPERH || BLACKFIN || MN10300) || ARCH_WANT_FRAME_POINTERS): 10
warnings in 2 logs
warning: (MPC836x_RDK  MTD_NAND_FSL_ELBC  MTD_NAND_FSL_UPM)
selects FSL_LBC which has unmet direct dependencies (FSL_SOC): 9
warnings in 2 logs
warning: (SINGLE_MEMORY_CHUNK) selects NEED_MULTIPLE_NODES which has
unmet direct dependencies (DISCONTIGMEM || NUMA): 9 warnings in 2 logs
warning: (COMPAT) selects COMPAT_BINFMT_ELF which has unmet direct
dependencies (COMPAT  BINFMT_ELF): 3 warnings in 1 logs
warning: (DRM) selects DMA_SHARED_BUFFER which has unmet direct
dependencies (EXPERIMENTAL): 3 warnings in 1 logs
warning: (DRM_RADEON_KMS  DRM_I915  STUB_POULSBO  FB_BACKLIGHT
 USB_APPLEDISPLAY  FB_OLPC_DCON  ASUS_LAPTOP  SONY_LAPTOP 
THINKPAD_ACPI  EEEPC_LAPTOP  ACPI_CMPC  SAMSUNG_Q10 
APPLE_GMUX) selects BACKLIGHT_CLASS_DEVICE which has unmet direct
dependencies (HAS_IOMEM  BACKLIGHT_LCD_SUPPORT): 3 warnings in 1
logs
warning: (IA64) selects PM which has unmet direct dependencies
(PM_SLEEP || PM_RUNTIME): 4 warnings in 1 logs
warning: (MOUSE_APPLETOUCH  MOUSE_BCM5974  MOUSE_SYNAPTICS_USB 
JOYSTICK_XPAD  TABLET_USB_ACECAD  TABLET_USB_AIPTEK 
TABLET_USB_HANWANG  TABLET_USB_KBTAB  TABLET_USB_WACOM 
TOUCHSCREEN_USB_COMPOSITE  INPUT_ATI_REMOTE2  INPUT_KEYSPAN_REMOTE
 INPUT_POWERMATE  INPUT_YEALINK  INPUT_CM109  RC_ATI_REMOTE 
IR_IMON  IR_MCEUSB  IR_REDRAT3  IR_STREAMZAP  DRM_USB) selects
USB which has unmet direct dependencies (USB_SUPPORT 
USB_ARCH_HAS_HCD): 5 warnings in 1 logs
warning: (PLATFORM_AT32AP) selects HAVE_NET_MACB which has unmet
direct dependencies (NETDEVICES  ETHERNET): 3 warnings in 1 logs
warning: (PREEMPT  DEBUG_ATOMIC_SLEEP) selects PREEMPT_COUNT which
has unmet direct dependencies (COLDFIRE): 5 warnings in 1 logs

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/camuhmdwunmatvf3sefwbtrfczwy01rdzvcx_htwe2q62rpc...@mail.gmail.com



Re: [opensuse-kernel] Re: [RFC] Simplifying kernel configuration for distro issues

2012-07-17 Thread Geert Uytterhoeven
On Mon, Jul 16, 2012 at 10:56 PM, Linus Torvalds
torva...@linux-foundation.org wrote:
 On Mon, Jul 16, 2012 at 12:26 PM,  da...@lang.hm wrote:
 Some of the proposed ways to implement the minimum distro kernel would not
 allow you to override the distro defaults because they would be implemented
 by setting dependancies, not by selecting options that you as the user could
 then unselect.

 The sanest thing to do is just a list of select statements. And in
 any case it would have to depend on the distro config entry, so EVEN
 THEN you could just create the Kconfig file, then edit out the distro
 config thing, and then do whatever you want.

Except that select is one of the ugliest things in Kconfig, as it
blindly sets a symbol
without checking if its dependencies are fulfilled.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/CAMuHMdWopF1MjHjNT_CAnu=ptu8x8oxyw4m+y2-xhehff2x...@mail.gmail.com



Re: [RFC] Simplifying kernel configuration for distro issues

2012-07-13 Thread Geert Uytterhoeven
On Fri, Jul 13, 2012 at 11:02 PM, Dave Jones da...@redhat.com wrote:
 I wish defconfig was actually something useful like this, instead of..
 what the hell is it exactly ? No-one even seems to agree, other than
 random selection of options, many of which were removed n years ago

It's just to difficult to update them in a sane way.

I mean, I have my own set of defconfigs for all supported m68k-platforms,
but getting them in sync and ready for submitting an update seems to be a
multi-year project, and there are always more important (and more fun) things
to do. So that's why I haven't gotten to updating them since the defconfig
reduction.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
-- Linus Torvalds


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/camuhmduaksrexgxwdzmzs3ji7fcf8p-hhqbz-zuznkwfz2+...@mail.gmail.com



Re: [PATCH 1/2] topology: Check for missing CPU devices

2012-01-09 Thread Geert Uytterhoeven
On Sun, Jan 8, 2012 at 21:48, Ben Hutchings b...@decadent.org.uk wrote:
 Commit ccbc60d3e19a1b6ae66ca0d89b3da02dde62088b ('topology: Provide
 CPU topology in sysfs in !SMP configurations') causes a crash at boot
 on a several architectures.  The topology sysfs code assumes that
 there is a CPU device for each online CPU whereas some architectures
 that do not support SMP or cpufreq do not register any CPU devices.
 Check for this before trying to use a device.

 Signed-off-by: Ben Hutchings b...@decadent.org.uk

Thanks!
Works on its own or combined with [PATCH 2/2] cpu: Register a generic
CPU device on architectures that currently do not.

Tested-by: Geert Uytterhoeven ge...@linux-m68k.org

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
                                -- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/camuhmdv614hb9dfykfnbkwf-6hqx36qg_wftnqsexehq8...@mail.gmail.com



Re: [PATCH 1/2] topology: Check for missing CPU devices

2012-01-09 Thread Geert Uytterhoeven
On Mon, Jan 9, 2012 at 02:06, richard -rw- weinberger
richard.weinber...@gmail.com wrote:
 Instead of testing kernels I really should read more LKML. ;-)

As an architecture maintainer, you want to read at least linux-arch.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
                                -- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/camuhmdwrj5qgvnwaxl62btsusjf1od3gjiyrd1kvc2bwwgd...@mail.gmail.com



Bug#649216: [PATCH] topology: Provide CPU topology in sysfs in !SMP configurations

2012-01-03 Thread Geert Uytterhoeven
On Thu, Nov 24, 2011 at 08:04, Ben Hutchings b...@decadent.org.uk wrote:
 We should provide topology information to userland even if it's not
 very interesting.  The current code appears to work properly for !SMP
 (tested on i386).

 Reference: http://bugs.debian.org/649216
 Reported-by: Marcus Osdoba marcus.osd...@googlemail.com
 Signed-off-by: Ben Hutchings b...@decadent.org.uk
 ---
  drivers/base/Makefile |    4 ++--
  1 files changed, 2 insertions(+), 2 deletions(-)

 diff --git a/drivers/base/Makefile b/drivers/base/Makefile
 index 99a375a..1334d89 100644
 --- a/drivers/base/Makefile
 +++ b/drivers/base/Makefile
 @@ -3,7 +3,8 @@
  obj-y                  := core.o sys.o bus.o dd.o syscore.o \
                           driver.o class.o platform.o \
                           cpu.o firmware.o init.o map.o devres.o \
 -                          attribute_container.o transport_class.o
 +                          attribute_container.o transport_class.o \
 +                          topology.o
  obj-$(CONFIG_DEVTMPFS) += devtmpfs.o
  obj-y                  += power/
  obj-$(CONFIG_HAS_DMA)  += dma-mapping.o
 @@ -12,7 +13,6 @@ obj-$(CONFIG_ISA)     += isa.o
  obj-$(CONFIG_FW_LOADER)        += firmware_class.o
  obj-$(CONFIG_NUMA)     += node.o
  obj-$(CONFIG_MEMORY_HOTPLUG_SPARSE) += memory.o
 -obj-$(CONFIG_SMP)      += topology.o
  ifeq ($(CONFIG_SYSFS),y)
  obj-$(CONFIG_MODULES)  += module.o
  endif

Taking this to linux-arch with a follow up from Ben due to crashes
reported on the
debian-m68k mailing list:

  My guess is that on m68k get_cpu_sysdev(0) returns NULL and thus
  topology_add_dev() passes an invalid pointer into sysfs_create_group().
  So either m68k (and maybe some other architectures with no SMP support)
  need to be fixed or the CPU topology code needs to allow for this.

 None of these architectures appears to call register_cpu():

c6x frv h8300 m68k microblaze openrisc score um xtensa

 and therefore they will all panic at boot following this change (commit
 ccbc60d3e19a1b6ae66ca0d89b3da02dde62088b).  So either I can try to fix
 them or else it must be reverted for now.

The commit mentioned above is in linux-next (and in Debian's 3.2-rc7).

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
                                -- Linus Torvalds



--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/CAMuHMdX6rcoyts=o5EEZTDb4CWHY-sYU=p4_f6u4o1lnk...@mail.gmail.com



Re: [PATCH] aufs: Do not refer to AUFS_NAME in pr_fmt

2012-01-01 Thread Geert Uytterhoeven
Hi Thorsten,

On Sat, Dec 31, 2011 at 23:55, Thorsten Glaser t...@mirbsd.de wrote:
 Dixi quod…

This one does work:
http://lists.debian.org/debian-kernel/2011/12/msg00474.html

 Geert, would you please apply the attached patch, independent of
 whether we are able to find a solution in aufs, as it avoids an
 entire problem class?

Sorry, static inline functions are preferred over macros, unless there's
a really good reason. Out-of-tree kernel code doing stupid things doesn't
count.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
                                -- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/CAMuHMdWoJYkRJ8NnS8xLnmBF9+tyNYew+O9+0O=v6td0mzz...@mail.gmail.com



Re: [PATCH] aufs: Do not refer to AUFS_NAME in pr_fmt

2011-12-31 Thread Geert Uytterhoeven
On Sat, Dec 31, 2011 at 05:46,  sf...@users.sourceforge.net wrote:
 +/* fs/aufs/Makefile refers this macro */
 +#define AUFS_NAME              aufs

If the Makefile refers to the macro, perhaps the Makefile should
define it, and pass it with -D?

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
                                -- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/camuhmdxfrfqxjebr5ig4s7nabrom6fzm6wpgi1f9hyvsv_c...@mail.gmail.com



Re: [PATCH] m68k/irq: don't use pr_crit in an header

2011-12-18 Thread Geert Uytterhoeven
2011/12/17 Uwe Kleine-König u.kleine-koe...@pengutronix.de:
 Using pr_crit in an header results in funny messages. Consider

        #define pr_fmt(fmt) mydriver:  fmt
        #include linux/hardirq.h

 which makes the message from ack_bad_irq

        mydriver: unexpected IRQ trap...

 so better use plain printk with KERN_CRIT directly.

Yep, that's expected behavior, as defining pr_fmt() modifies all kernel messages
generated from that module.

 This fixes a build problem on m68k with aufs3 en passant because the
 latter builds with

        ccflags-y += 
 -D'pr_fmt(fmt)=AUFS_NAME\040%s:%d:%s[%d]:\040fmt,__func__,__LINE__,current-comm,current-pid'

 without providing AUFS_NAME early enough for ack_bad_irq (which is the
 problem of aufs).

Isn't this a problem with (out of tree) aufs?
Why does it put a define that relies on an (apparently sometimes still
undefined)
variable on the build command line?
Any header may contain calls to pr_*().

 Cc: Thorsten Glaser t...@debian.org
 Signed-off-by: Uwe Kleine-König u.kleine-koe...@pengutronix.de
 ---
  arch/m68k/include/asm/hardirq.h |    2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

 diff --git a/arch/m68k/include/asm/hardirq.h b/arch/m68k/include/asm/hardirq.h
 index db30ed2..1f652e0 100644
 --- a/arch/m68k/include/asm/hardirq.h
 +++ b/arch/m68k/include/asm/hardirq.h
 @@ -20,7 +20,7 @@

  static inline void ack_bad_irq(unsigned int irq)
  {
 -       pr_crit(unexpected IRQ trap at vector %02x\n, irq);
 +       printk(KERN_CRIT unexpected IRQ trap at vector %02x\n, irq);

Nack. Nowadays pr_crit(...) is recommended over printk(KERN_CRIT ...).

Besides, there are (albeit not that many yet) other callers of pr_*() in
header files. Do you plan to revert them to printk(), too?

Please fix aufs instead. Thanks!

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
                                -- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/camuhmdxyf_ofm+y6ykrzftyktoyrskzw7fuvajyy6xtv_fv...@mail.gmail.com



Bug#648996: linux-2.6: FTBFS [m68k] in modpost: strlen [iscsi_target_mod.ko] undefined! (and others)

2011-11-30 Thread Geert Uytterhoeven
On Wed, Nov 30, 2011 at 18:53, Thorsten Glaser t...@mirbsd.de wrote:
 ERROR: strlen [drivers/target/iscsi/iscsi_target_mod.ko] undefined!
 ERROR: strcpy [drivers/target/iscsi/iscsi_target_mod.ko] undefined!

Haven't seen that issue here.

Can you please send me:
  - your .config
  - The output file(s) created by make drivers/target/iscsi/file.s
for the object
files that contain references to strlen and strcpy.

Thanks!

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
                                -- Linus Torvalds



--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/camuhmdw8t8q7oqxjm602ilg4asv1kmty-fhigs7hyy8xbre...@mail.gmail.com



Re: linux-2.6_2.6.38-5_m68k.changes ACCEPTED

2011-05-12 Thread Geert Uytterhoeven
On Thu, May 12, 2011 at 09:31, Thorsten Glaser t...@mirbsd.de wrote:
 WARNING: modpost: Found 1 section mismatch(es).
 To see full details build your kernel with:
 'make CONFIG_DEBUG_SECTION_MISMATCH=y'

Don't know; do what it says if you care.

 OK, will do that. Will they show up when cross-compiling?

Yes they do. I just haven't got to fixing them.
Patches are accepted :-)

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
                                -- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/BANLkTi=+hvdfcs4juh_pv6ym5xqa7jn...@mail.gmail.com



Re: linux-2.6_2.6.38-5_m68k.changes ACCEPTED

2011-05-12 Thread Geert Uytterhoeven
On Thu, May 12, 2011 at 09:24, Finn Thain fth...@telegraphics.com.au wrote:

 On Thu, 12 May 2011, Geert Uytterhoeven wrote:

 Do we really want to compile wireless on m68k? AFAIK we don't have any.

 http://marc.info/?l=linux-m68km=128628785718187w=2

 I suspect this list is also applicable to atari and amiga (?)

Yeah, I should look into updating the defconfigs again.

Haven't done that anymore since the concept of minimal defconfigs was
introduced, as I'm still a bit worried if I can trust them or not (options not
mentioned in the minimal defconfig may silently change their value
depending on Kconfig logic; with the full ones I would immeditaly notice).

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
                                -- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/BANLkTi=fz_wt3j_ceqwjdqbprxlkasq...@mail.gmail.com



Re: linux-2.6_2.6.38-5_m68k.changes ACCEPTED

2011-05-12 Thread Geert Uytterhoeven
On Fri, May 13, 2011 at 06:13, Finn Thain fth...@telegraphics.com.au wrote:
 On Thu, 12 May 2011, Geert Uytterhoeven wrote:
 On Thu, May 12, 2011 at 09:24, Finn Thain fth...@telegraphics.com.au
 wrote:
 
  On Thu, 12 May 2011, Geert Uytterhoeven wrote:
 
  Do we really want to compile wireless on m68k? AFAIK we don't have
  any.
 
  http://marc.info/?l=linux-m68km=128628785718187w=2
 
  I suspect this list is also applicable to atari and amiga (?)

 Excepting the PS2 mouse I should add. (And is there PCMCIA on
 atari/amiga?)

Yes, so far used only by the apne network driver.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
                                -- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/banlktinjt3njr3cpmzbkvhrduw6zczo...@mail.gmail.com



Re: linux-2.6_2.6.38-5_m68k.changes ACCEPTED

2011-05-11 Thread Geert Uytterhoeven
On Thu, May 12, 2011 at 04:36, Ben Hutchings b...@decadent.org.uk wrote:
 stdin:1522:2: warning: #warning syscall recvmmsg not implemented

 Missing feature on m68k.

False positive, as this is implemented via the sys_socketcall() demultiplexer.

 drivers/net/wireless/p54/main.c: In function 'p54_register_common':
 drivers/net/wireless/p54/main.c:614: warning: unused variable 'priv'
 drivers/net/wireless/p54/main.c: In function 'p54_unregister_common':
 drivers/net/wireless/p54/main.c:656: warning: unused variable 'priv'

 Fixed in 2.6.39.

Do we really want to compile wireless on m68k? AFAIK we don't have any.

 CC [M]  drivers/net/zorro8390.o
 drivers/net/lib8390.c:256: warning: '__ei_tx_timeout' defined but not used
 drivers/net/lib8390.c:302: warning: '__ei_start_xmit' defined but not used
 drivers/net/lib8390.c:526: warning: '__ei_poll' defined but not used
 drivers/net/lib8390.c:879: warning: '__ei_get_stats' defined but not used
 drivers/net/lib8390.c:981: warning: '__ei_set_multicast_list' defined but 
 not used
 drivers/net/lib8390.c:1016: warning: 'alloc_ei_netdev' defined but not 
 used

 It looks like zorro8390.c should be using these functions instead of the
 corresponding functions without the double-underscore prefixes.  At the
 moment it's using some functions built from the #included lib8390.c
 using the special definition of EI_SHIFT (from zorro8390.c), and some
 functions exported from the 8390 module, which are built with the
 default definition of EI_SHIFT (from 8390.h).

 CC [M]  drivers/net/hydra.o
 drivers/net/lib8390.c:256: warning: '__ei_tx_timeout' defined but not used
 drivers/net/lib8390.c:302: warning: '__ei_start_xmit' defined but not used
 drivers/net/lib8390.c:526: warning: '__ei_poll' defined but not used
 drivers/net/lib8390.c:879: warning: '__ei_get_stats' defined but not used
 drivers/net/lib8390.c:981: warning: '__ei_set_multicast_list' defined but 
 not used
 drivers/net/lib8390.c:1016: warning: 'alloc_ei_netdev' defined but not 
 used

 Similar issue in hydra.c.

Will be fixed. Turned out to be a real bug recently.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say programmer or something like that.
                                -- Linus Torvalds


--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/BANLkTikT29u7CYv+2htP08bfi=u3q_b...@mail.gmail.com



Bug#541463: linux-libc-dev and LTP compile failures on ppc

2009-08-14 Thread Geert Uytterhoeven
Package: linux-libc-dev
Version: 2.6.26-17lenny1

When configuring upstream LTP (http://ltp.sourceforge.net/) on ppc, I get:

| checking linux/cgroupstats.h usability... no
| checking linux/cgroupstats.h presence... yes
| configure: WARNING: linux/cgroupstats.h: present but cannot be compiled
| configure: WARNING: linux/cgroupstats.h: check for missing prerequisite 
headers?
| configure: WARNING: linux/cgroupstats.h: see the Autoconf documentation
| configure: WARNING: linux/cgroupstats.h: section Present But Cannot Be 
Compiled
| configure: WARNING: linux/cgroupstats.h: proceeding with the preprocessor's 
result
| configure: WARNING: linux/cgroupstats.h: in the future, the compiler will 
take precedence
| configure: WARNING: ##  ##
| configure: WARNING: ## Report this to ltp-resu...@lists.sourceforge.net ##
| configure: WARNING: ##  ##
| checking for linux/cgroupstats.h... yes

This seems to be caused by linux/cgroupstats.h not being self-contained
(it needs but doesn't include linux/types.h):

| In file included from /usr/include/linux/cgroupstats.h:18,
|  from /tmp/a.c:1:
| /usr/include/linux/taskstats.h:44: error: expected specifier-qualifier-list 
before ‘__u16’
| In file included from /tmp/a.c:1:
| /usr/include/linux/cgroupstats.h:31: error: expected specifier-qualifier-list 
before ‘__u64’

| # dpkg -S cgroupstats.h
| linux-libc-dev: /usr/include/linux/cgroupstats.h
| # apt-cache policy linux-libc-dev
| linux-libc-dev:
|   Installed: 2.6.26-17lenny1
|   Candidate: 2.6.26-17lenny1
|   Version table:
|  2.6.30-5 0
| 500 http://ftp.de.debian.org sid/main Packages
|  *** 2.6.26-17lenny1 0
| 990 http://security.debian.org lenny/updates/main Packages
| 100 /var/lib/dpkg/status
|  2.6.26-17 0
| 990 http://http.us.debian.org lenny/main Packages
| 500 http://ftp.de.debian.org squeeze/main Packages
| # 

LTP does compile fine, though.


After upgrading linux-libc-dev to version 2.6.30-5 from sid, configure works
fine again.  However, the build of LTP fails with:

| gcc -g -O2 -I../../include -g -Wall -I../../../../include -Wall
signalfd01.c  -L../../../../lib -lltp -o signalfd01
| In file included from /usr/include/asm/fcntl.h:9,
|  from /usr/include/linux/fcntl.h:4,
|  from /usr/include/linux/signalfd.h:13,
|  from signalfd01.c:62:
| /usr/include/asm-generic/fcntl.h:117: error: redefinition of ‘struct flock’
| /usr/include/asm-generic/fcntl.h:140: error: redefinition of ‘struct flock64’
| make[4]: *** [signalfd01] Error 1
| make[4]: Leaving directory 
`/root/ltp-full-20090731/testcases/kernel/syscalls/signalfd'

The original definitions of `struct flock' and `struct flock64' are in
/usr/include/bits/fcntl.h.

Presumably /usr/include/bits/fcntl.h should use the definition pulled in from
/usr/include/asm/fcntl.h, instead of defining its own?

With kind regards,

Geert Uytterhoeven
Software Architect
Techsoft Centre

Technology and Software Centre Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium

Phone:+32 (0)2 700 8453
Fax:  +32 (0)2 700 8622
E-mail:   geert.uytterhoe...@sonycom.com
Internet: http://www.sony-europe.com/

A division of Sony Europe (Belgium) N.V.
VAT BE 0413.825.160 · RPR Brussels
Fortis · BIC GEBABEBB · IBAN BE41293037680010



--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#508173: Acknowledgement (fbdev/logo: Corrupted SPE penguins on PS3)

2009-01-07 Thread Geert Uytterhoeven
Hi Moritz,

On Sun, 4 Jan 2009, Moritz Muehlenhoff wrote:
 Geert Uytterhoeven wrote:
  Patch available in http://lkml.org/lkml/2008/12/9/78
 
 The bugzilla entry has been closed, but the patch doesn't appear
 in the current Linus git tree so far. What's the merge status?
 Patches need to be merged into mainline prior to inclusion in
 Debian.

The patch has been in Andrew's mm-tree for a while, and got into Linus' tree
last night:
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=4fb6de2561d5c47f3539a7b412108164d79fbb96

With kind regards,

Geert Uytterhoeven
Software Architect

Sony Techsoft Centre Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium

Phone:+32 (0)2 700 8453
Fax:  +32 (0)2 700 8622
E-mail:   geert.uytterhoe...@sonycom.com
Internet: http://www.sony-europe.com/

A division of Sony Europe (Belgium) N.V.
VAT BE 0413.825.160 · RPR Brussels
Fortis · BIC GEBABEBB · IBAN BE41293037680010



--
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#508173: Acknowledgement (fbdev/logo: Corrupted SPE penguins on PS3)

2008-12-09 Thread Geert Uytterhoeven

Patch available in http://lkml.org/lkml/2008/12/9/78

With kind regards,

Geert Uytterhoeven
Software Architect

Sony Techsoft Centre Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium

Phone:+32 (0)2 700 8453
Fax:  +32 (0)2 700 8622
E-mail:   [EMAIL PROTECTED]
Internet: http://www.sony-europe.com/

A division of Sony Europe (Belgium) N.V.
VAT BE 0413.825.160 · RPR Brussels
Fortis · BIC GEBABEBB · IBAN BE41293037680010



--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#462529: linux-2.6: Add config file support for efika and PS3 (preliminary)

2008-05-31 Thread Geert Uytterhoeven
On Fri, 30 May 2008, Geoff Levand wrote:
 You don't have CONFIG_LOGO_LINUX_CLUT224 set, so the SPE logos are not
 rendered correctly.  It looks like the lines of pixel are not aligned.
 Geert, can you say why?  Is it a bug in the logo code?

If there's no main logo, it also skips (using `return') drawing the extra
SPE logos.

I looked into it a few days ago, and came up with the patch below. But this
is not sufficient to fix the issue, as fb_show_logo_line() needs fb_logo.logo.
As this looks very low-priority to me, I didn't go further.
---
 drivers/video/fbmem.c |   15 ---
 1 file changed, 8 insertions(+), 7 deletions(-)

--- a/drivers/video/fbmem.c
+++ b/drivers/video/fbmem.c
@@ -553,6 +553,7 @@ static inline int fb_show_extra_logos(st
 int fb_prepare_logo(struct fb_info *info, int rotate)
 {
int depth = fb_get_color_depth(info-var, info-fix);
+   int height = 0;
unsigned int yres;
 
memset(fb_logo, 0, sizeof(struct logo_data));
@@ -574,12 +575,10 @@ int fb_prepare_logo(struct fb_info *info
depth = 4;
}
 
-   /* Return if no suitable logo was found */
+   /* Skip if no suitable logo was found */
fb_logo.logo = fb_find_logo(depth);
-
-   if (!fb_logo.logo) {
-   return 0;
-   }
+   if (!fb_logo.logo)
+   goto out;
 
if (rotate == FB_ROTATE_UR || rotate == FB_ROTATE_UD)
yres = info-var.yres;
@@ -588,8 +587,9 @@ int fb_prepare_logo(struct fb_info *info
 
if (fb_logo.logo-height  yres) {
fb_logo.logo = NULL;
-   return 0;
+   goto out;
}
+   height = fb_logo.logo-height;
 
/* What depth we asked for might be different from what we get */
if (fb_logo.logo-type == LINUX_LOGO_CLUT224)
@@ -615,7 +615,8 @@ int fb_prepare_logo(struct fb_info *info
}
}
 
-   return fb_prepare_extra_logos(info, fb_logo.logo-height, yres);
+out:
+   return fb_prepare_extra_logos(info, height, yres);
 }
 
 int fb_show_logo(struct fb_info *info, int rotate)

With kind regards,

Geert Uytterhoeven
Software Architect

Sony Network and Software Technology Center Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium

Phone:+32 (0)2 700 8453
Fax:  +32 (0)2 700 8622
E-mail:   [EMAIL PROTECTED]
Internet: http://www.sony-europe.com/

Sony Network and Software Technology Center Europe
A division of Sony Service Centre (Europe) N.V.
Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium
VAT BE 0413.825.160 · RPR Brussels
Fortis Bank Zaventem · BIC GEBABEBB08A · IBAN BE39001382358619

Bug#462529: linux-2.6: Add config file support for efika and PS3 (preliminary)

2008-02-22 Thread Geert Uytterhoeven
While Efika support is included in 2.6.24-1, PS3 support is still not enabled
in 2.6.24-4.

As upstream 2.6.24 has full support for PS3 (except for wireless), it would be
very easy to add support for it in the Debian kernel package by enabling it in
config.powerpc64, or by adding a new config.ps3.

With kind regards,

Geert Uytterhoeven
Software Architect

Sony Network and Software Technology Center Europe
The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium

Phone:+32 (0)2 700 8453
Fax:  +32 (0)2 700 8622
E-mail:   [EMAIL PROTECTED]
Internet: http://www.sony-europe.com/

Sony Network and Software Technology Center Europe
A division of Sony Service Centre (Europe) N.V.
Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium
VAT BE 0413.825.160 · RPR Brussels
Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619