Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752627AbdFLOZb (ORCPT ); Mon, 12 Jun 2017 10:25:31 -0400 Received: from mx08-00252a01.pphosted.com ([91.207.212.211]:58278 "EHLO mx08-00252a01.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752488AbdFLOZ2 (ORCPT ); Mon, 12 Jun 2017 10:25:28 -0400 From: Phil Elwell Subject: [PATCH v2 1/4] clk: bcm2835: More flexible IO register remapping To: Thomas Gleixner , Jason Cooper , Marc Zyngier , Rob Herring , Mark Rutland , Florian Fainelli , Stefan Wahren , Eric Anholt , Russell King , Michael Turquette , Stephen Boyd , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-rpi-kernel@lists.infradead.org, linux-clk@vger.kernel.org Message-ID: <28279803-78d2-4231-939b-89a8c895aba9@raspberrypi.org> Date: Mon, 12 Jun 2017 15:25:09 +0100 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 7bit X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-06-12_10:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_spam_notspam policy=outbound_spam score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1015 lowpriorityscore=0 impostorscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1703280000 definitions=main-1706120249 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1943 Lines: 55 The BCM2835 AUX block contains two registers - AUXIRQ and AUXENB. The addition of an irqchip driver using AUXIRQ is hampered by the current DT node reserving both registers with a compatible string claimed by this bcm2835-aux-clk driver. Ease the transition to separate DT nodes by detecting and handling the case where this driver's MEM resource has been reduced to include only the AUXENB register. Otherwise, use devm_ioremap to remap the region without reserving it. Signed-off-by: Phil Elwell --- drivers/clk/bcm/clk-bcm2835-aux.c | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/drivers/clk/bcm/clk-bcm2835-aux.c b/drivers/clk/bcm/clk-bcm2835-aux.c index bd750cf..70d389e 100644 --- a/drivers/clk/bcm/clk-bcm2835-aux.c +++ b/drivers/clk/bcm/clk-bcm2835-aux.c @@ -37,9 +37,22 @@ static int bcm2835_aux_clk_probe(struct platform_device *pdev) parent = __clk_get_name(parent_clk); res = platform_get_resource(pdev, IORESOURCE_MEM, 0); - reg = devm_ioremap_resource(dev, res); - if (IS_ERR(reg)) - return PTR_ERR(reg); + /* + * If the MEM resource is only 4 bytes long it covers just the + * AUXENB register, otherwise it is the entire AUX block. + */ + if (resource_size(res) == 4) { + reg = devm_ioremap_resource(dev, res); + if (IS_ERR(reg)) + return PTR_ERR(reg); + gate = reg; + } else { + reg = devm_ioremap(dev, res->start, resource_size(res)); + if (IS_ERR(reg)) + return PTR_ERR(reg); + gate = reg + BCM2835_AUXENB; + } + onecell = devm_kmalloc(dev, sizeof(*onecell) + sizeof(*onecell->hws) * BCM2835_AUX_CLOCK_COUNT, GFP_KERNEL); @@ -47,7 +60,6 @@ static int bcm2835_aux_clk_probe(struct platform_device *pdev) return -ENOMEM; onecell->num = BCM2835_AUX_CLOCK_COUNT; - gate = reg + BCM2835_AUXENB; onecell->hws[BCM2835_AUX_CLOCK_UART] = clk_hw_register_gate(dev, "aux_uart", parent, 0, gate, 0, 0, NULL); -- 1.9.1