On Wed, 2010-03-10 at 10:55 +, i...@hellion.org.uk wrote:
arch_init_chip_data cannot be moved into struct irq_chip at this time
because irq_desc-chip is not known at the time the irq_desc is
setup. For now rename arch_init_chip_data to arch_init_irq_desc (for
PowerPC, the only other
On Wed, 2010-03-10 at 12:06 +, Yinghai Lu wrote:
On Wed, Mar 10, 2010 at 2:55 AM, i...@hellion.org.uk wrote:
From: Ian Campbell ian.campb...@citrix.com
Move arch_init_copy_chip_data and arch_free_chip_data into function
pointers in struct irq_chip since they operate on irq_desc
On Wed, 2010-03-10 at 17:18 +, Eric W. Biederman wrote:
Ian Campbell ian.campb...@citrix.com writes:
On Wed, 2010-03-10 at 10:55 +, i...@hellion.org.uk wrote:
arch_init_chip_data cannot be moved into struct irq_chip at this time
because irq_desc-chip is not known at the time
On Wed, 2010-03-10 at 17:42 +, Eric W. Biederman wrote:
Ian Xen in this sense is simply not x86. irq_cfg is not acpi or
ioapic or anything but x86 specific. It has everything to do with
having a per cpu vector table of 256 entries and architecturally
receiving a vector number when an
On Wed, 2010-03-10 at 18:15 +, Eric W. Biederman wrote:
Ian Campbell ian.campb...@citrix.com writes:
On Wed, 2010-03-10 at 17:42 +, Eric W. Biederman wrote:
Ian Xen in this sense is simply not x86. irq_cfg is not acpi or
ioapic or anything but x86 specific. It has
'
are available in the git repository at:
git://xenbits.xensource.com/people/ianc/linux-2.6.git for-x86/irq
Ian Campbell (2):
irq: move some interrupt arch_* functions into struct irq_chip.
x86: irq_desc-chip_data is always correct whether or not SPARSE_IRQ is
enabled.
arch/powerpc
need to take to actually exercise some of these code
paths.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Acked-by: Michael Ellerman mich...@ellerman.id.au [PowerPC rename portion]
Cc: Thomas Gleixner t...@linutronix.de
Cc: Ingo Molnar mi...@redhat.com
Cc: H. Peter Anvin h...@zytor.com
Cc: Eric
On Sat, 2010-03-13 at 00:29 +, Eric W. Biederman wrote:
[...]
after that xen could use
irq_to_desc_alloc_node_f(irq, node, xen_init_chip_data);
as need...
at last we don't need to call x86_init_chip_data everywhere.
This was one of the things I was considering. It seems like one
:
Ingo Molnar (1):
Merge branch 'perf/urgent'
are available in the git repository at:
git://xenbits.xensource.com/people/ianc/linux-2.6.git for-x86/irq
Ian Campbell (2):
irq: move some interrupt arch_* functions into struct irq_chip.
x86: irq_desc-chip_data is always correct
to actually exercise some of these code paths.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Acked-by: Michael Ellerman mich...@ellerman.id.au [PowerPC rename portion]
Cc: Thomas Gleixner t...@linutronix.de
Cc: Ingo Molnar mi...@redhat.com
Cc: H. Peter Anvin h...@zytor.com
Cc: Eric W. Biederman ebied
On Mon, 2010-03-22 at 10:19 +, Thomas Gleixner wrote:
On Sun, 21 Mar 2010, Yinghai Lu wrote:
From: Ian Campbell ian.campb...@citrix.com
Move arch_init_copy_chip_data and arch_free_chip_data into function
pointers in struct irq_chip since they operate on irq_desc-chip_data
On Tue, 2010-03-23 at 07:10 +, Paul Mundt wrote:
The function pointer thing itself is also a bit unorthodox to say the
least. You're introducing and passing around an opaque type just so you
can get to a 'return 0' in the xen case as far as I can tell,
The ultimate aim is to have Xen use
On Wed, 2010-03-24 at 17:44 +, Thomas Gleixner wrote:
On Wed, 24 Mar 2010, Ian Campbell wrote:
On Mon, 2010-03-22 at 10:19 +, Thomas Gleixner wrote:
On Sun, 21 Mar 2010, Yinghai Lu wrote:
From: Ian Campbell ian.campb...@citrix.com
Move arch_init_copy_chip_data
kw_i2c_irq and via_pmu_interrupt are not timer interrupts and
therefore should not use IRQF_TIMER. Use the recently introduced
IRQF_NO_SUSPEND instead since that is the actual desired behaviour.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Thomas Gleixner t...@linutronix.de
Cc
Continue to provide IRQF_TIMER as an alias to IRQF_NO_SUSPEND since I
think it is worth preserving the nice self-documenting name (where it
is used appropriately). It also avoid needing to patch all the many
users who are using the flag for an actual timer interrupt.
Signed-off-by: Ian Campbell
://git.kernel.org/.../ericvh/v9fs
are available in the git repository at:
git://xenbits.xensource.com/people/ianc/linux-2.6.git for-irq/irqf-no-suspend
Ian Campbell (4):
irq: rename IRQF_TIMER to IRQF_NO_SUSPEND
ixp4xx-beeper: Use IRQF_NO_SUSPEND not IRQF_TIMER for non-timer interrupt
On Thu, 2010-07-29 at 09:49 +0100, Thomas Gleixner wrote:
On Wed, 28 Jul 2010, Ian Campbell wrote:
Continue to provide IRQF_TIMER as an alias to IRQF_NO_SUSPEND since I
think it is worth preserving the nice self-documenting name (where it
is used appropriately). It also avoid needing
On Thu, 2010-07-29 at 09:49 +0100, Thomas Gleixner wrote:
On Wed, 28 Jul 2010, Ian Campbell wrote:
Continue to provide IRQF_TIMER as an alias to IRQF_NO_SUSPEND since I
think it is worth preserving the nice self-documenting name (where it
is used appropriately). It also avoid needing
A small number of users of IRQF_TIMER are using it for the implied no
suspend behaviour on interrupts which are not timer interrupts.
Therefore add a new IRQF_NO_SUSPEND flag, rename IRQF_TIMER to
__IRQF_TIMER and redefine IRQF_TIMER in terms of these new flags.
Signed-off-by: Ian Campbell
kw_i2c_irq and via_pmu_interrupt are not timer interrupts and
therefore should not use IRQF_TIMER. Use the recently introduced
IRQF_NO_SUSPEND instead since that is the actual desired behaviour.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Thomas Gleixner t...@linutronix.de
Cc
ping?
On Fri, 2013-05-31 at 11:14 +0100, Ian Campbell wrote:
This file is a common include for B4860 and B4420 but is not a valid DTS
itself:
DTC arch/powerpc/boot/b4qds.dtb
Error: arch/powerpc/boot/dts/b4qds.dts:35.1-2 syntax error
FATAL ERROR: Unable to parse input
On Mon, 2013-06-03 at 12:00 +0100, Ian Campbell wrote:
This fixes a warning:
DTC arch/powerpc/boot/virtex440-ml507.dtb
Warning (reg_format): reg property in
/plb@0/xps-ll-temac@81c0/ethernet@81c0/phy@7 has invalid length (4
bytes) (#address-cells == 2, #size-cells == 1
On Mon, 2013-06-03 at 09:36 -0400, Josh Boyer wrote:
On Mon, Jun 03, 2013 at 12:00:24PM +0100, Ian Campbell wrote:
Currently I see:
DTC arch/powerpc/boot/kilauea.dtb
Warning (reg_format): reg property in /plb/ppc4xx-msi@C1000 has
invalid length (12 bytes) (#address-cells == 1
Hi,
First off apologies for the large CC list -- I think this catches the
arch list for all the arches with device tree source in the tree.
Various folks have expressed an interest in eventually splitting the
device tree bindings out of the Linux git repository into a separate
tree. This should
On Mon, 2013-05-13 at 08:02 +0100, Michal Simek wrote:
Just small overview it is a Xilinx soft core cpu where you can even setup
some parameters for core itself - multiplier, divider, BS, fpu, cache sizes,
etc.
You have to also compose the whole system and every platform/configuration is
1
make: *** [b4qds.dtb] Error 2
I spotted in build tests of device-tree.git, announcement
https://lkml.org/lkml/2013/4/24/209, which builds *.dts. Probably no one would
do this this in real life on linux.git but it still seems worth fixing.
Signed-off-by: Ian Campbell ian.campb
This affects arch/powerpc/boot/dts/virtex440-ml510.dts but I think it is
actually a more general issue:
$ make ARCH=powerpc CROSS_COMPILE=powerpc-linux- virtex440-ml510.dtb
CC scripts/mod/devicetable-offsets.s
GEN scripts/mod/devicetable-offsets.h
On Fri, 2013-05-31 at 12:48 +0100, Grant Likely wrote:
On Fri, 31 May 2013 11:29:30 +0100, Ian Campbell ian.campb...@citrix.com
wrote:
This affects arch/powerpc/boot/dts/virtex440-ml510.dts but I think it is
actually a more general issue:
$ make ARCH=powerpc CROSS_COMPILE
On Fri, 2013-05-31 at 08:01 -0500, Jon Loeliger wrote:
Line 374 is the IDSEL 0x16... line here:
interrupt-map =
/* IRQ mapping for pci slots and ALI M1533
...
* management
On Fri, 2013-05-31 at 08:01 -0500, Jon Loeliger wrote:
Hrm. Is this a that's not in the kernel's copy yet problem?
BTW I'm using dtc.git 4e76ec796c90d44d417f82d9db2d67cfe575f8ed and not
the kernel copy.
dtc-lexer.l in my HEAD is identical to the current master
On Fri, 2013-05-31 at 17:31 +0100, Grant Likely wrote:
On Fri, May 31, 2013 at 5:04 PM, Stephen Warren swar...@wwwdotorg.org wrote:
On 05/31/2013 05:48 AM, Grant Likely wrote:
---
commit d01dccdcb3ea8233b09efb9c24db9f057fbd3b37
Author: Grant Likely grant.lik...@linaro.org
Date: Fri
-cells value for
/plb@0/xps-ll-temac@81c0/ethernet@81c0/phy@7
Warning (avoid_default_addr_size): Relying on default #size-cells value for
/plb@0/xps-ll-temac@81c0/ethernet@81c0/phy@7
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Benjamin Herrenschmidt b
this platform does not use
address-cells=2.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Rupjyoti Sarmah rsar...@apm.com
Cc: Tirumala R Marri tma...@apm.com
Cc: Josh Boyer jwbo...@linux.vnet.ibm.com
Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
Cc: Paul Mackerras pau...@samba.org
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
Cc: Paul Mackerras pau...@samba.org
Cc: linuxppc-dev@lists.ozlabs.org
Cc: linux-ker...@vger.kernel.org
---
arch/powerpc/boot/.gitignore |1 +
1 file changed, 1 insertion(+)
diff --git
On Mon, 2013-06-03 at 09:36 -0400, Josh Boyer wrote:
On Mon, Jun 03, 2013 at 12:00:24PM +0100, Ian Campbell wrote:
Currently I see:
DTC arch/powerpc/boot/kilauea.dtb
Warning (reg_format): reg property in /plb/ppc4xx-msi@C1000 has
invalid length (12 bytes) (#address-cells == 1
-by: Ian Campbell ian.campb...@citrix.com
to this change too.
Reported-by: Ian Campbell ian.campb...@citrix.com
Signed-off-by: Stephen Warren swar...@nvidia.com
---
This is a fix for 3.10.
scripts/dtc/dtc-lexer.l |2 +-
scripts/dtc/dtc-lexer.lex.c_shipped | 232
I noticed this with a defconfig build:
CC arch/powerpc/kernel/ptrace.o
arch/powerpc/kernel/ptrace.c: In function 'arch_ptrace':
arch/powerpc/kernel/ptrace.c:1502:5: error: array subscript is above array
bounds [-Werror=array-bounds]
arch/powerpc/kernel/ptrace.c:1530:5:
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Olof Johansson o...@lixom.net
Cc: Grant Likely grant.lik...@secretlab.ca
Cc: net...@vger.kernel.org
Cc: linuxppc-dev@lists.ozlabs.org
Cc: linux-ker...@vger.kernel.org
Cc: devicetree-disc...@lists.ozlabs.org
---
drivers/net/pasemi_mac.c
-cells value for
/plb@0/xps-ll-temac@81c0/ethernet@81c0/phy@7
Warning (avoid_default_addr_size): Relying on default #size-cells value for
/plb@0/xps-ll-temac@81c0/ethernet@81c0/phy@7
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Benjamin Herrenschmidt b
this platform does not use
address-cells=2.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Acked-by: Josh Boyer jwbo...@gmail.com
Cc: Rupjyoti Sarmah rsar...@apm.com
Cc: Tirumala R Marri tma...@apm.com
Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
Cc: Paul Mackerras pau...@samba.org
Cc
() and
syscore_resume() calls to arch/x86/kernel/apm_32.c, kernel/kexec.c
and drivers/xen/manage.c.
Xen bit looks ok to me:
Acked-by: Ian Campbell ian.campb...@citrix.com
Signed-off-by: Rafael J. Wysocki r...@sisk.pl
---
arch/x86/kernel/apm_32.c |5 +
drivers/xen/manage.c |9
On Thu, 2009-05-21 at 14:27 -0400, Becky Bruce wrote:
We have both in every case but one,
which is in swiotlb_map_page where we call address_needs_mapping()
without calling range_needs_mapping.
The reason it calls address_needs_mapping without range_needs_mapping is
that in the
On Thu, 2009-05-21 at 16:18 -0400, Jeremy Fitzhardinge wrote:
I guess the test is checking for a bad implementation of map_single().
More importantly the io_tlb_overflow_buffer is basically a second chance
if you exhaust the swiotlb pool. The check seems to be there to ensure
that the second
On Thu, 2009-05-21 at 14:27 -0400, Becky Bruce wrote:
I can work with that, but it's going to be a bit inefficient, as I
actually need the dma_addr_t, not the phys_addr_t, so I'll have to
convert. In every case, this is a conversion I've already done and
that I need in the calling code
On Fri, 2009-05-22 at 19:55 -0400, Jeremy Fitzhardinge wrote:
Ian Campbell wrote:
On Thu, 2009-05-21 at 14:27 -0400, Becky Bruce wrote:
I can work with that, but it's going to be a bit inefficient, as I
actually need the dma_addr_t, not the phys_addr_t, so I'll have to
convert
On Wed, 2009-05-27 at 15:05 -0400, Becky Bruce wrote:
On May 22, 2009, at 6:11 AM, Ian Campbell wrote:
BTW do you need swiotlb_bus_to_virt to be __weak or is the fact that
it
is implemented in terms of swiotlb_bus_to_phys sufficient?
The default one in swiotlb calls phys_to_virt
Subject should have been swiotlb: Introduce architecture-specific APIs
to replace __weak functions. Seems I never drive git send-email right
first time...
On Fri, 2009-05-29 at 04:43 -0400, Ian Campbell wrote:
This series does not contain any Xen or PowerPC specific changes, those
will follow
internals.
This series does not contain any Xen or PowerPC specific changes, those
will follow in separate postings. The complete patchset has been boot
tested under Xen and native-x86 and compiled for IA64 and PowerPC
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Becky Bruce bec
These new architecture-specific interfaces subsume the existing __weak
function hooks.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp
Cc: Jeremy Fitzhardinge jer...@goop.org
Cc: Olaf Kirch o...@suse.de
Cc: Greg KH gre...@suse.de
Cc: Tony Luck
the PowerPC swiotlb patchset by
Becky Bruce.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Becky Bruce bec...@kernel.crashing.org
Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
Cc: Kumar Gala ga...@kernel.crashing.org
Cc: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp
Cc: Ingo Molnar mi
This replaces usages of address_needs_mapping, range_needs_mapping and
is_buffer_dma_capable and the __weak architecture hooks to those
functions with a more flexible single function.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp
Cc: Jeremy
On Fri, 2009-05-29 at 08:58 -0700, Jeremy Fitzhardinge wrote:
Ian Campbell wrote:
-void * __weak swiotlb_bus_to_virt(struct device *hwdev, dma_addr_t address)
+#ifdef CONFIG_HIGHMEM
+static void * swiotlb_bus_to_virt(struct device *hwdev, dma_addr_t address)
I think it would
On Fri, 2009-05-29 at 14:09 -0700, Jeremy Fitzhardinge wrote:
Ian Campbell wrote:
void * __weak swiotlb_bus_to_virt(struct device *hwdev, dma_addr_t address)
Does this need to be weak too?
It's was wanted by PowerPC to support highmem, that's what patch 8/9
tries to address in an arch
and x86
pci-swiotlb.c and swiotlb_bus_to_phys/phys_to_bus implementations in
x86.
- additionally replaced __weak swiotlb_alloc* with
swiotlb_init_with_buffer.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Becky Bruce bec...@kernel.crashing.org
Cc: Benjamin Herrenschmidt b
These new architecture-specific interfaces subsume the existing __weak
function hooks.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp
Cc: Jeremy Fitzhardinge jer...@goop.org
Cc: Olaf Kirch o...@suse.de
Cc: Greg KH gre...@suse.de
Cc: Tony Luck
This replaces usages of address_needs_mapping, range_needs_mapping and
is_buffer_dma_capable and the __weak architecture hooks to those
functions with a more flexible single function.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp
Cc: Jeremy
the PowerPC swiotlb patchset by
Becky Bruce.
Signed-off-by: Ian Campbell ian.campb...@citrix.com
Cc: Becky Bruce bec...@kernel.crashing.org
Cc: Benjamin Herrenschmidt b...@kernel.crashing.org
Cc: Kumar Gala ga...@kernel.crashing.org
Cc: FUJITA Tomonori fujita.tomon...@lab.ntt.co.jp
Cc: Ingo Molnar mi
On Fri, 2009-07-10 at 15:55 +1000, Benjamin Herrenschmidt wrote:
On Mon, 2009-06-01 at 16:32 +0100, Ian Campbell wrote:
This series:[...]
Looks like I was only CCed on part of them... it's not very handy for me
as I end up having some of the patches in one folder and some
elsewhere
On Fri, 2009-07-10 at 14:35 +0900, FUJITA Tomonori wrote:
I don't think that we need to take account of dom0 support; we don't
have a clear idea about an acceptable dom0 design (it needs to use
swiotlb code? I don't know yet), we don't even know we will have dom0
support in mainline. That's
On Fri, 2009-07-10 at 06:12 +0100, Ingo Molnar wrote:
Hm, the functions and facilities you remove here were added as part
of preparatory patches for Xen guest support. You were aware of
them, you were involved in discussions about those aspects with Ian
and Jeremy but still you chose not to
On Tue, 2015-03-17 at 10:45 +0530, Manish Jaggi wrote:
On Monday 09 March 2015 08:04 AM, Yijing Wang wrote:
Now we could pass PCI domain combined with bus number
in u32 argu. Because in arm/arm64, PCI domain number
is assigned by pci_bus_assign_domain_nr(). So we leave
pci_scan_root_bus()
te:
I have no access to or information about this platform so this is purely a
guess as to the fix. An alternative would be to adjust #address-cells, but
whether that is correct or not depends on the platform.
[0]
https://git.kernel.org/pub/scm/linux/kernel/git/devicetree/devicetree-rebasing.git/
Signed-
On Fri, 2017-07-21 at 15:54 +0300, Ivan Mikhaylov wrote:
> Hi Ian,
> > Building the split device-tree tree[0] highlighted that upstream commit
> > 9eec6cb142bd ("powerpc/44x/fsp2: Add device tree for FSP2 board") introduced
> > this warning when building the device tree:
> >
> > $ make
63 matches
Mail list logo