From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f47.google.com (mail-wm1-f47.google.com [209.85.128.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B664735CBA1 for ; Fri, 9 Jan 2026 12:51:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.47 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767963110; cv=none; b=tFVB/e9aWSIBi/XVOGO6S5KoY1RSncWGmTymJNlgDsvQlZL/kbQN6NuV5n4qB7DO01IiCd2EfNLiBwNKCVreAnqMMkjoxQRP3U6W61LgEktfQW0+HZ/uxokxTRWKARrFuUspAqun5bXfKXijLfGghziTO6Jvh94uBO6fN3ZNLT8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767963110; c=relaxed/simple; bh=b2pdfqBM8QcOuodTV2+EEKDjVt4RtMF2UOfs35NWCzc=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=d/jEb/Lc3OM7YnzdPe0j4VAL4041HW6fn3cS4yx5jGKI6ETM7G0mczqAh9MMvkaPNr4gWHxrVpOk/UTwP93epSJzUZ/jhsJDaHMk3O5tT5Qpa6p3N3fejVvy3CblvfTLEga3OIi2jHQlv/Zna5yY9sLkjy1zcEGjz60yFF35UJc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=FFYdg3WC; arc=none smtp.client-ip=209.85.128.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="FFYdg3WC" Received: by mail-wm1-f47.google.com with SMTP id 5b1f17b1804b1-477a219dbcaso34503265e9.3 for ; Fri, 09 Jan 2026 04:51:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1767963105; x=1768567905; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=wzhZ6qnhyB+hMkP+k7TywINSr9e1/VcrZMD9Hv7Wg9c=; b=FFYdg3WCgBIyCL5HmJm7P3u0kypDKbCo9XmFC/oaMQKVOl2Cdhw8ezCEbR5AcMGZmz JXMcQ/6qEvkafDc9hddjQXbi1hDW5NUd7pSk7MGgmFixPBph8LcVYEENMNoorp0gmfgG OBKYrvAYdmnt34X73AHaGvZ8GzzsFMkxY7GLkDSgxUzHJ7Va9jAVdd/i20JQKjQHYqsp CB7u/VeewJRUGGnvLz5FF1XRICVTIcGiu2dVz0GPk6fIAhiyfzvIUxd5K74DMKWLWMvv IDSdynLJU12jVUVNPklN8jwjWKe+4bzkNMvOOlhlmb6i5Ojqz7xi9We/YNfJCCCUnbaa ay/g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1767963105; x=1768567905; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=wzhZ6qnhyB+hMkP+k7TywINSr9e1/VcrZMD9Hv7Wg9c=; b=vB2E6+Pq6Wp9v8UvEVq9oYWFPQdWe6bAZMhwdgK3JsqzY1eO7NJ4iL5N0++kAg0Tqb ySVKMq+UPLe3lIUEfEqPROikQYzfxgMSIJQpS4IrMOdjdwyj1EYKqLyYJhCeekupiJdI ogrONMc8pELFtkjinTNAG1DF5vjPJnxGP7R5oSQOtEhUpEuerWUxCPpgJlQ7L5q8I/98 R8acHHtPvsmGRcVCkQUVveH7fVcQXFsrs74JdGiw/aIqcFX+Y3Go62eVjNj8xYLDDG09 yRDPCQTZh7Tl7R3YerqTUUgGSi1jz/D7mjYDzBfeve4IxMm6lVf6zfxiPLoGis8AGb7X VLbg== X-Forwarded-Encrypted: i=1; AJvYcCWEYkKKo8fBqV0NPfyPo2U95ODe4Y4E+xbqIPEvTpFqx2mh0n+tSpDNi1dWo9Ansee3uvgtdgAaIJMMXaQ=@vger.kernel.org X-Gm-Message-State: AOJu0YyZhiCf1HEnDTOiJ+hDZM2KzQ30q6ivFpGLwjkMWOUQhFgwh9L3 9Tfgo9VBY6CDd1/QyUnAzxWU0DIuWS9Hr4pb1bx0Ai39jTcJBSYxD7oh X-Gm-Gg: AY/fxX5FNn8+AMMKt+MeC4v4w2rkdaf0RLY5XW5lYPjMJmpLpcPpr2R/b+u7GOZez+g mwOraMwfla/MfVdeSVcGC05CN5Xyx341erK9m9PAsiq1z7pZbmdeZ+rG9kooXLa3L2o5BT7YuXd QJmPafp0flgf9WxuzpllBEzbAIIIPq4W6PX459Lv0z0vzHirZWbfTzpCda8/s7zM2MjfbmqRiKV JIUfqRtwgSl2WA8si0Q0ZfreTUGAEIgNXu+vRDuyYzqwX8qXUJ3CYnsBWB0Sn3fUzTB841/epiM qsKOL9JoxVV9HvM37YLGot8r/quB5PXqpujhDApiPKTkyWanCbhOJ3djWqmQdXGPdnp0osoTME/ gOn1M8nDFfflIjJE9CCwMMV57HrFQI6yv2d+/haOXU1RiVWETctbzbjbzMltsHEQRzIJ0xRAaHk uzvt6MgMq7sdr0lFun6M/PoSPEW/G0VdcGYv5HSuvCyvr8mf4N1HklhnETQe9IV5Qm X-Google-Smtp-Source: AGHT+IGcDcabrf0KiqabCe9igRWYL1ukIxgKapFHgqspyhXNU439sEDpU7Kdmj0l4SVed0DCTQbNxQ== X-Received: by 2002:a05:600c:1d14:b0:477:7b16:5fb1 with SMTP id 5b1f17b1804b1-47d84b0a96emr108677495e9.7.1767963105326; Fri, 09 Jan 2026 04:51:45 -0800 (PST) Received: from localhost (brnt-04-b2-v4wan-170138-cust2432.vm7.cable.virginm.net. [94.175.9.129]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-432bd5ee5e3sm22332156f8f.35.2026.01.09.04.51.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 09 Jan 2026 04:51:44 -0800 (PST) Date: Fri, 9 Jan 2026 12:51:43 +0000 From: Stafford Horne To: Linus Walleij Cc: Geert Uytterhoeven , LKML , Linux OpenRISC , Bartosz Golaszewski , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-gpio@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH v2 1/5] dt-bindings: Add compatible string opencores,gpio to gpio-mmio Message-ID: References: <20251217080843.70621-1-shorne@gmail.com> <20251217080843.70621-2-shorne@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Fri, Jan 09, 2026 at 11:07:17AM +0100, Linus Walleij wrote: > On Thu, Jan 8, 2026 at 9:41 AM Geert Uytterhoeven wrote: > > > > > What is the rationale behind using brcm,bcm6345-gpio? > > > > Given brcm,bcm6345-gpio has 32-bit registers, while opencores,gpio > > > > has 8-bit registers, I doubt the latter is compatible with the former... > > Yeah this needs to be fixed/reverted pronto :/ > > > > I switch the size from 32-bit to 8-bit using the reg = <* 0x1>, <* 0x1> setting. > > > Also the reg addresses of "dat" and "dirout" are different for the real > > > brcm,bcm6345-gpio. > > > > > > brcm,bcm6345-gpio. Example: > > > > > > /* GPIOs 192 .. 223 */ > > > gpio6: gpio@518 { > > > compatible = "brcm,bcm6345-gpio"; > > > reg = <0x518 0x04>, <0x538 0x04>; > > > reg-names = "dirout", "dat"; > > > gpio-controller; > > > #gpio-cells = <2>; > > > }; > > > > > > vs opencores,gpio Example: > > > > > > gpio0: gpio@91000000 { > > > compatible = "opencores,gpio", "brcm,bcm6345-gpio"; > > > reg = <0x91000000 0x1>, <0x91000001 0x1>; > > > reg-names = "dat", "dirout"; > > > gpio-controller; > > > #gpio-cells = <2>; > > > }; > > > > Exactly, the register space and register widths are different > > ...as proved here. > > Stafford can you send a fixup or revert patch? > (Only need to revert if you can't make a fix quick enough, which I > think you can.) Sure, I'll send a fixup to the devicetree binding and a update to the driver to just support opencores,gpio. Hopefully, that can be picked up in time by Bartosz who has this one staged in gpio/for-next. I'll send the 2 patches as part of my series for OpenRISC multicore fixups as the devicetree's I have added have a soft dependency the patches. After/if the patches are pulled to the gpio branch I can drop them from my queue and I'll just have to make sure Linux merged the GPIO changes binding updates before the OpenRISC updates during the merge window. Let me know if there are any issues. > > > The opencores,gpio setup does work. > > > > > > Now that I think about it, would it have been better to just add opencores,gpio > > > to gpio-mmio.c compatible list? > > > > I think that would be better. > > Yes this is better. > > I should have seen this, I guess I was sloppy :( I should have also thought more, but I don't do this often enough to remember all of the rules. Sorry for the head ache. -Stafford