From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 46042C001B0 for ; Mon, 7 Aug 2023 13:10:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Cc:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=rVTO/8dD6TQpWiX2tIm/WvpMaroAHKKWKKnoBrh0tTY=; b=NFheWjudA9P8Es X9z8/vnjnBQ8JHtPu2CU7RGRT58ZgZF9qiirPLRVjbSjb9rsY0Fs40jQ8jL4PLSTWsPyd/aacVcZZ s2tLlwbbIPMhGtfX7+bt0Fy/UQxwPMUM8/giA7NVdQvQKDlUV4vcQAm2n1wLMdptamussJ0CzGDBQ ajMLGRJsd9Ii5HgMztTWxcXu+YxEm52Cwo90r6/7XLzwOQrzFAotx+cx142BPEi+Bp6FHspR4oDEp Oz1i4mv5uaquQX9p8BJd00ACJEljm4v5rmTNvRIB6j2ys3Xx1njkvoybRIZHU0csil6w6g+9TMLgM BPBUVFpNYKH+tz/wKHMQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qSzzY-00HJHE-2G; Mon, 07 Aug 2023 13:09:48 +0000 Received: from mail-ej1-x62f.google.com ([2a00:1450:4864:20::62f]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qSzzV-00HJGQ-33 for linux-arm-kernel@lists.infradead.org; Mon, 07 Aug 2023 13:09:47 +0000 Received: by mail-ej1-x62f.google.com with SMTP id a640c23a62f3a-991c786369cso612086666b.1 for ; Mon, 07 Aug 2023 06:09:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1691413783; x=1692018583; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id:from :to:cc:subject:date:message-id:reply-to; bh=OUBX0M73wFHT9eoECwDsnUeuQ/GOufxE2TDJPi/hBi4=; b=CwY2YCXlMDtzQMebTwv7Ckjxds6YTVDHwGNS5YtrmZA4PmSJ/ycHgSkYcVgoWDliT4 UVBTr3vEfsIYoOlvXhyz0ir/u+DA6IiXumxTV5Cl+OTarcmOWyunKGwZlIUxlO0x3F41 hzVz2vEg4/uLUgoZHp8FTWjhUhw7dfJfNYivPid+KP5TWrr9O5eyq0cvuENoKjBiG7aJ 1Cq+v+c6HF8VkPWnYVlGpZAHnGA0POZKB2pezEhj7wP+kfnsOV4uZlxFxoRBCe91iYZD ZmwCtYXU+lBdFXpbn/JDa9rome/yfaB270MCMFxRSlGKlttEn/5V0nj3V+3+3mAZ048i 1YEQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691413783; x=1692018583; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=OUBX0M73wFHT9eoECwDsnUeuQ/GOufxE2TDJPi/hBi4=; b=AX14jI65JN1Q9YeR3rm7LaguP+urGpm7D1tOsKGEKHEt5czcB4711Q2avg0vu7Izll DstU5O+T6qo4Xv2yl00Oe2u5el+EUkijRjxkwzc+u+1APj6i8gZz0Owvg3xuMydN6rdq MClUhktslhtCRenZBZUF0EZUWaExHe4oqr0c2YmSSx+KJjWr3FYNlxYj4r8hkkP1FQC+ LPgyKqeRU8MLdyABPprztqi4uhCrWRE3QIClJjAyYM7jsylZDn6AZ3UxVKhs/bGLgG/o DmWzYXy2Cpsn3c0shCBFxg0uMGg4XgjtYIY8tC6C43jOaZ4200/vcME8Nkpat2IeiUnA gGog== X-Gm-Message-State: AOJu0Ywcyq5QLOQmzIk1rSfsPwXi0+l2/Eu7FjTqKj0ezhrmp9yK+o1Z R0Fit2rcHcvRUclxCmOs+HByrg== X-Google-Smtp-Source: AGHT+IE2cGa7ZQhye3Rv8ehiG2rTumLO8Og+QEsxDBnUJkU/mU8QmIulq6RaJ48j6i1orRq+apPqgw== X-Received: by 2002:a17:907:7757:b0:99b:499d:4635 with SMTP id kx23-20020a170907775700b0099b499d4635mr7903641ejc.75.1691413783095; Mon, 07 Aug 2023 06:09:43 -0700 (PDT) Received: from [192.168.1.20] ([178.197.222.113]) by smtp.gmail.com with ESMTPSA id o17-20020a17090611d100b0099bd0b5a2bcsm5231021eja.101.2023.08.07.06.09.41 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 07 Aug 2023 06:09:42 -0700 (PDT) Message-ID: <9c878317-18d6-c483-d182-4c25200ee305@linaro.org> Date: Mon, 7 Aug 2023 15:09:40 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.14.0 Subject: Re: [PATCH 3/3] media: exynos4-is: fimc-is: replace duplicate pmu node with phandle To: Hans Verkuil , Sylwester Nawrocki , Mauro Carvalho Chehab , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Alim Akhtar , linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org References: <20230722115441.139628-1-krzysztof.kozlowski@linaro.org> <20230722115441.139628-3-krzysztof.kozlowski@linaro.org> <0b361e6c-d141-4758-edc2-c75b6f0efbe3@xs4all.nl> Content-Language: en-US From: Krzysztof Kozlowski In-Reply-To: <0b361e6c-d141-4758-edc2-c75b6f0efbe3@xs4all.nl> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230807_060946_002401_A5102308 X-CRM114-Status: GOOD ( 18.26 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 07/08/2023 15:06, Hans Verkuil wrote: > Hi Krzysztof, > > On 22/07/2023 13:54, Krzysztof Kozlowski wrote: >> Devicetree for the FIMC IS camera included duplicated PMU node as its >> child like: >> >> soc@0 { >> system-controller@10020000 { ... }; // Real PMU >> >> camera@11800000 { >> fimc-is@12000000 { >> // FIMC IS camera node >> pmu@10020000 { >> reg = <0x10020000 0x3000>; // Fake PMU node >> }; >> }; >> }; >> }; >> >> This is not a correct representation of the hardware. Mapping the PMU >> (Power Management Unit) IO memory should be via syscon-like phandle >> (samsung,pmu-syscon, already used for other drivers), not by duplicating >> "pmu" Devicetree node inside the FIMC IS. Backward compatibility is >> preserved. >> >> Signed-off-by: Krzysztof Kozlowski >> --- >> .../platform/samsung/exynos4-is/fimc-is.c | 33 ++++++++++++++----- >> 1 file changed, 24 insertions(+), 9 deletions(-) >> >> diff --git a/drivers/media/platform/samsung/exynos4-is/fimc-is.c b/drivers/media/platform/samsung/exynos4-is/fimc-is.c >> index 530a148fe4d3..c4c191771093 100644 >> --- a/drivers/media/platform/samsung/exynos4-is/fimc-is.c >> +++ b/drivers/media/platform/samsung/exynos4-is/fimc-is.c >> @@ -767,12 +767,32 @@ static void fimc_is_debugfs_create(struct fimc_is *is) >> static int fimc_is_runtime_resume(struct device *dev); >> static int fimc_is_runtime_suspend(struct device *dev); >> >> +static void __iomem *fimc_is_get_pmu_regs(struct device *dev) >> +{ >> + struct device_node *node; >> + void __iomem *regs; >> + >> + node = of_parse_phandle(dev->of_node, "samsung,pmu-syscon", 0); >> + if (!node) { >> + dev_warn(dev, "Finding PMU node via deprecated method, update your DTB\n"); >> + node = of_get_child_by_name(dev->of_node, "pmu"); >> + if (!node) >> + return ERR_PTR(-ENODEV); >> + } >> + >> + regs = of_iomap(node, 0); >> + of_node_put(node); >> + if (!regs) >> + return ERR_PTR(-ENOMEM); > > sparse gives me these warnings for these ERR_PTR returns: > > drivers/media/platform/samsung/exynos4-is/fimc-is.c:780:39: warning: incorrect type in return expression (different address spaces) > drivers/media/platform/samsung/exynos4-is/fimc-is.c:780:39: expected void [noderef] __iomem * > drivers/media/platform/samsung/exynos4-is/fimc-is.c:780:39: got void * Thanks, indeed. I will use IOMEM_ERR_PTR() Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel