All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Lunn <andrew@lunn.ch>
To: Lothar Wa??mann <LW@KARO-electronics.de>
Cc: Andrew Lunn <andrew@lunn.ch>,
	Russell King - ARM Linux <linux@arm.linux.org.uk>,
	sshtylyov@mvista.com, spear-devel@list.st.com,
	linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org,
	viresh kumar <viresh.linux@gmail.com>,
	mturquette@linaro.org, akpm@linux-foundation.org,
	jgarzik@redhat.com, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH V3 07/12] ata/sata_mv: Remove conditional compilation of clk code
Date: Wed, 25 Apr 2012 11:31:09 +0200	[thread overview]
Message-ID: <20120425093109.GC2116@lunn.ch> (raw)
In-Reply-To: <20375.46953.269684.680915@ipc1.ka-ro>

> If an arch has HAVE_CLK enabled it must provide valid clocks (be it
> dummy clocks) for all devices it supports.

So, lets take the theoretical exaple of a unicore32 PUV3

config ARCH_PUV3
        def_bool y
        select CPU_UCV2
        select GENERIC_CLOCKEVENTS
        select HAVE_CLK
        select ARCH_REQUIRE_GPIOLIB
        select ARCH_HAS_CPUFREQ

Seems like this somewhat unknown, to me at least, architecture, also
supports PCI. So i plug in an HP Adaptec AIC8120 SATA host bus adapter
into a spare PCI slot. This uses the Marvell 88SX6041, which the
SATA_MV driver supports.

Should i expect that the unicore32 PUV3 has created a dummy clk for
this case?

Thanks
	Andrew

WARNING: multiple messages have this Message-ID (diff)
From: andrew@lunn.ch (Andrew Lunn)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V3 07/12] ata/sata_mv: Remove conditional compilation of clk code
Date: Wed, 25 Apr 2012 11:31:09 +0200	[thread overview]
Message-ID: <20120425093109.GC2116@lunn.ch> (raw)
In-Reply-To: <20375.46953.269684.680915@ipc1.ka-ro>

> If an arch has HAVE_CLK enabled it must provide valid clocks (be it
> dummy clocks) for all devices it supports.

So, lets take the theoretical exaple of a unicore32 PUV3

config ARCH_PUV3
        def_bool y
        select CPU_UCV2
        select GENERIC_CLOCKEVENTS
        select HAVE_CLK
        select ARCH_REQUIRE_GPIOLIB
        select ARCH_HAS_CPUFREQ

Seems like this somewhat unknown, to me at least, architecture, also
supports PCI. So i plug in an HP Adaptec AIC8120 SATA host bus adapter
into a spare PCI slot. This uses the Marvell 88SX6041, which the
SATA_MV driver supports.

Should i expect that the unicore32 PUV3 has created a dummy clk for
this case?

Thanks
	Andrew

WARNING: multiple messages have this Message-ID (diff)
From: Andrew Lunn <andrew@lunn.ch>
To: Lothar Wa??mann <LW@KARO-electronics.de>
Cc: Andrew Lunn <andrew@lunn.ch>,
	viresh kumar <viresh.linux@gmail.com>,
	Russell King - ARM Linux <linux@arm.linux.org.uk>,
	sshtylyov@mvista.com, spear-devel@list.st.com,
	linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org,
	mturquette@linaro.org, akpm@linux-foundation.org,
	jgarzik@redhat.com, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH V3 07/12] ata/sata_mv: Remove conditional compilation of clk code
Date: Wed, 25 Apr 2012 11:31:09 +0200	[thread overview]
Message-ID: <20120425093109.GC2116@lunn.ch> (raw)
In-Reply-To: <20375.46953.269684.680915@ipc1.ka-ro>

> If an arch has HAVE_CLK enabled it must provide valid clocks (be it
> dummy clocks) for all devices it supports.

So, lets take the theoretical exaple of a unicore32 PUV3

config ARCH_PUV3
        def_bool y
        select CPU_UCV2
        select GENERIC_CLOCKEVENTS
        select HAVE_CLK
        select ARCH_REQUIRE_GPIOLIB
        select ARCH_HAS_CPUFREQ

Seems like this somewhat unknown, to me at least, architecture, also
supports PCI. So i plug in an HP Adaptec AIC8120 SATA host bus adapter
into a spare PCI slot. This uses the Marvell 88SX6041, which the
SATA_MV driver supports.

Should i expect that the unicore32 PUV3 has created a dummy clk for
this case?

Thanks
	Andrew

  reply	other threads:[~2012-04-25  9:31 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-24 11:21 [PATCH V3 00/12] clk: Add non CONFIG_HAVE_CLK routines Viresh Kumar
2012-04-24 11:21 ` Viresh Kumar
2012-04-24 11:21 ` [PATCH V3 01/12] " Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21 ` [PATCH V3 02/12] clk: Remove redundant depends on from drivers/Kconfig Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21 ` [PATCH V3 03/12] i2c/i2c-pxa: Remove conditional compilation of clk code Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:52   ` Wolfram Sang
2012-04-24 11:52     ` Wolfram Sang
2012-04-24 11:21 ` [PATCH V3 04/12] usb/marvell: " Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21 ` [PATCH V3 05/12] usb/musb: " Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21 ` [PATCH V3 06/12] ata/pata_arasan: " Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21 ` [PATCH V3 07/12] ata/sata_mv: " Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 12:00   ` Andrew Lunn
2012-04-24 12:00     ` Andrew Lunn
2012-04-24 12:51     ` Lothar Waßmann
2012-04-24 12:51       ` Lothar Waßmann
2012-04-24 13:42     ` viresh kumar
2012-04-24 13:42       ` viresh kumar
2012-04-24 14:29       ` Andrew Lunn
2012-04-24 14:29         ` Andrew Lunn
2012-04-24 17:02         ` viresh kumar
2012-04-24 17:02           ` viresh kumar
2012-04-25  5:42           ` Andrew Lunn
2012-04-25  5:42             ` Andrew Lunn
2012-04-24 17:05         ` viresh kumar
2012-04-24 17:05           ` viresh kumar
2012-04-24 20:18       ` Russell King - ARM Linux
2012-04-24 20:18         ` Russell King - ARM Linux
2012-04-25  3:02         ` viresh kumar
2012-04-25  3:02           ` viresh kumar
2012-04-25  5:28           ` Andrew Lunn
2012-04-25  5:28             ` Andrew Lunn
2012-04-25  6:43             ` Lothar Waßmann
2012-04-25  6:43               ` Lothar Waßmann
2012-04-25  7:14               ` Andrew Lunn
2012-04-25  7:14                 ` Andrew Lunn
2012-04-25  8:35                 ` Lothar Waßmann
2012-04-25  8:35                   ` Lothar Waßmann
2012-04-25  9:31                   ` Andrew Lunn [this message]
2012-04-25  9:31                     ` Andrew Lunn
2012-04-25  9:31                     ` Andrew Lunn
2012-04-25 10:37                     ` Russell King - ARM Linux
2012-04-25 10:37                       ` Russell King - ARM Linux
2012-04-25 11:24         ` Andrew Lunn
2012-04-25 11:24           ` Andrew Lunn
2012-04-24 11:21 ` [PATCH V3 08/12] net/c_can: " Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21 ` [PATCH V3 09/12] net/stmmac: " Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21 ` [PATCH V3 10/12] gadget/m66592: " Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21 ` [PATCH V3 11/12] gadget/r8a66597: " Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar
2012-04-24 11:21 ` [PATCH V3 12/12] usb/host/r8a66597: " Viresh Kumar
2012-04-24 11:21   ` Viresh Kumar

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120425093109.GC2116@lunn.ch \
    --to=andrew@lunn.ch \
    --cc=LW@KARO-electronics.de \
    --cc=akpm@linux-foundation.org \
    --cc=jgarzik@redhat.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mturquette@linaro.org \
    --cc=spear-devel@list.st.com \
    --cc=sshtylyov@mvista.com \
    --cc=viresh.linux@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.