Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sat, 13 May 2023 11:53:26 +0200
From:   Henning Schild <henning.schild@siemens.com>
To:     Bartosz Golaszewski <brgl@bgdev.pl>
Cc:     "Wu, Xing Tong (DI FA CTR IPC CN PRC4)" <XingTong.Wu@siemens.com>,
        Simon Guinot <simon.guinot@sequanux.org>,
        Linus Walleij <linus.walleij@linaro.org>,
        "linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Hans de Goede <hdegoede@redhat.com>,
        Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Subject: Re: [PATCH v2 1/1] gpio-f7188x: fix chip name and pin count on
 Nuvoton chip
Message-ID: <20230513115326.0a9f669e@md1za8fc.ad001.siemens.net>
In-Reply-To: <CAMRc=Mdfptr0ZXV=fzBE0T+=vTxhL1tOKxRy+ccFLOqinb0w1A@mail.gmail.com>
References: <20230427152055.18380-1-henning.schild@siemens.com>
        <20230427152055.18380-2-henning.schild@siemens.com>
        <ZEuBMCxeWAx9OilV@76cbfcf04d45>
        <759b2df004e2445e850a01b33e748972@siemens.com>
        <20230428113332.3a7b9a18@md1za8fc.ad001.siemens.net>
        <CAMRc=Mdfptr0ZXV=fzBE0T+=vTxhL1tOKxRy+ccFLOqinb0w1A@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?UzcwT1BWYmZCV1RGSWJmUjJUSkNUYVdSWFZhTHdjenZ5YVJhczV3SDNlZFI2?=
 =?utf-8?B?VUJUNnFxK0dyakRPdW9jZVl0QWI2Q1VUREpjMzBzWk1LdGVCVXFwbzUyT3Vt?=
 =?utf-8?B?N1h3bjJhdHl4bDFJSHdyUzI2OFJpVVVHWlVFWHkvTmk0ZFZpeE50N29vUFFK?=
 =?utf-8?B?Uk9vd3NLaE9EVVV5RmtUWWdzVHE3c3dpNHlXay9rZ1g0NUNRNlNkVkdPZGh0?=
 =?utf-8?B?ODNmd051NnhmamwwY0dVT1psQmNVVE9MR0c4Q2t1MWxFZFg4S0pJbjBicGl2?=
 =?utf-8?B?ZVdhalRSc09yaWlEb3AyQy9VV0RFdG9Ha1ErWjdSYkNaY1RPVE1ZQnRFZXFl?=
 =?utf-8?B?RFUvWnFFSnRhZHlvZXRaczZqckE0UnhJQThtN1ljc3NRZEhZT2tIbWJmZ1g0?=
 =?utf-8?B?UG5KNkJLNkhaOUVPbi8zZjk4dWZOZWJSZ1AvUE9rMGRqMEdxZDVGSWpMRnp6?=
 =?utf-8?B?Q2lPK095ZWE5dTVRN25vKzZTRXNGTDhEOUpZdk1VWmh5aVJJQWpqRUVVMm00?=
 =?utf-8?B?ZkR5UTFiNmpQRFUyOHBQcW1mdG0xY2czSUdvOWpYeUdRclRUYnF4bFRXK1BV?=
 =?utf-8?B?bmNUTGNOclpOcVZ0dTFSek5jUnlHOGZRY0JCUHJBanJWYW51MzFtczI0dzVs?=
 =?utf-8?B?eEdmaldhaDZaUjFMb1FmUnNWSytHUDQ2R0dkeHBrWVQ4d1YzdWRvTjdTdkF5?=
 =?utf-8?B?cXV3eFFiUVRDMlN6QWxRdFlJTnFRUXpqMDUweWZtSWwvNUxxalF6T0VHS254?=
 =?utf-8?B?enNBbjBsSDhYWlY5TVVWb0R5RDFLaTZKVUZmLzJ2dUt1Tk1XT3VTejZhVXpo?=
 =?utf-8?B?YXVKUDlYMUY4V3lvZ1VHNy9jbWk1bStYT0J2K2NwaWRlWldhdk1DT2ptUGpp?=
 =?utf-8?B?VjZkVHI5TmZVTG05ZXpQcGJUZkE0NG9DMXB3UEw0c2RqaHUrTEdnNDJGYkNT?=
 =?utf-8?B?ZmdVczF0eHdDWFk4eUlsVUFseTk5Z1FhNGJmVnM5VXNZV05qNVFvaWNPWVVp?=
 =?utf-8?B?TjNSa1VNWm1zZ0ppaEw3ODFJZ1R5emY2WmtOL21KM2wxQU8zUGV2amw2a0RJ?=
 =?utf-8?B?c0xFUFo0YWdQeWpEZ1h0SlBZQU43VFFoYW1DY3VDUnhiaUUvdlc5aW82a2FT?=
 =?utf-8?B?ZklpY21vSnJGdUV0dkdPU1NEYS9TVTBxVXRUa0hoSWhPYW1zQlR4S0NhZDQr?=
 =?utf-8?B?a1lXdXc3VEhWK3g2bEFhdFFhSlJtdjJRY0RzR21uZ2ZocGo4Q1JwL0lySGFY?=
 =?utf-8?B?ZGdpMHdpT3o0L0JOQWJVNDRnZ20rTDFlOE1FRnEyTkZpcWVnSmh4SjhDQmg3?=
 =?utf-8?B?dzRPM3E0LzJXTWxMZzhtTHg4TzR2aVNIa2szMUhhSjQzQnovNU5oc0poQU51?=
 =?utf-8?B?TWc4Z213d2YrRy9zTVRTT2hETGpWMWVMZ3lXWFJmQklIbEQ5VEZJa08zc01r?=
 =?utf-8?B?RFVjdFNLVGNkaXhxbDlRTFpFcEhyNXgrWUo2NnZYdDQ5emxZRkgyZ3RMblRp?=
 =?utf-8?B?c29IcmF6T29qcW9HUSs4aWZ1YVN1eisxRzNrbWJOZFZjQXh6bmJhTXVEcUhT?=
 =?utf-8?B?TkI1L25tZFpDRFppYlFOdStiQzBXblhMdDFGMDhJc2l0WHdJMjBPZ2lTZzlm?=
 =?utf-8?B?Z1dtcTA5c05NdFQzZUlqZzJGNUhBdEZuRGVGTTFMaVJ1UnI2VW1GZnIrMWpl?=
 =?utf-8?B?ZmFsNnRqc1VHZFZHOWY0bmtCRG1Yb1lJRjFVdDB2c2R6Ui9PY2VWRVcvS29G?=
 =?utf-8?B?VFNWdzdRMTBWWnNYdWJ3ZEozUmUrRjdHUzFUcUcwaWJaZ0ZnUEJ0emF4SGpC?=
 =?utf-8?B?Y2QxdVM3QzNSdUJhTUJWK1hvazh6N2Q5TWFjTENtYXpDQXpZdURYa1FXWTIr?=
 =?utf-8?B?dGl4QkFrVk1DaDRtckZ3THo3NlVGZmh2LzBrWHpmbUc0ZisyaFU1NUtJSy9P?=
 =?utf-8?B?NXozMUo4RlhtcjZlTGc2ekVZb25wSHVDM2UzYTFVaHBWckd2S1VWeTBLYVRu?=
 =?utf-8?B?MmxEVjh3QlhuUWJsV1ZjZUU3bm55TmhsOTJ1SURQajExbHpkSkVHUHEzSEl1?=
 =?utf-8?B?dkNkR2pkanh1Y084UlFTS3gzenNuTGtIdzFoZ1RhOEtWVk55RWNJbGlteTNF?=
 =?utf-8?B?cUE1RWVqRHpoeC9sR3pOcDlucEhqcEFkQzNzN3NaR1A0VE9JZHhoWUVHWWxP?=
 =?utf-8?B?R0RKQUJaSW5BK2RqK1ZBN05HeGV0OWp0bDB5L1dtUjAwUkRvV2FEK21IZkdh?=
 =?utf-8?B?WElIdDltMnNDTGtmNXJWNEhUdUZ3PT0=?=
X-OriginatorOrg: siemens.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 65bad70a-fc61-4c36-fe4c-08db5397ebbe
X-MS-Exchange-CrossTenant-AuthSource: PA4PR10MB5780.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 May 2023 09:53:36.6507
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 38ae3bcd-9579-4fd4-adda-b42e1495d55a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: wpLir44XFXU+lV1Fj0I76D6FFT8aOzMq475ewpgo7vMfc4ZFYXcJ/3Z/k5cBNzCkxmIdFbRKR97ZS7c1dsgrjbEMMUOpPDSQovM3uDYvYXc=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAVPR10MB6813
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIMWL_WL_MED,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=no autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Am Thu, 11 May 2023 14:52:17 +0200
schrieb Bartosz Golaszewski <brgl@bgdev.pl>:

> On Fri, Apr 28, 2023 at 11:33=E2=80=AFAM Henning Schild
> <henning.schild@siemens.com> wrote:
> >
> > Am Fri, 28 Apr 2023 10:26:18 +0200
> > schrieb "Wu, Xing Tong (DI FA CTR IPC CN PRC4)"
> > <XingTong.Wu@siemens.com>:
> > =20
> > > Hi all
> > >
> > > The chip id of NCT6116D is 0XD281, you can refer to
> > > NCT6116D_Datasheet_V1_0.pdf, Page 291 =20
> >
> > Thanks Xing Tong. I think we have come to agree that for now
> > NCT6116D 0xD281 will not be supported in the kernel. Maybe until
> > someone has access to that very chip and a use-case.
> >
> > But you managed to somehow get these datasheets, which are still not
> > publicly available. Maybe you can use your contacts at Nuvoton to
> > kindly ask them to publish those specs on their website for future
> > reference. Some specs are there, but not all. That would help
> > people to add more chips and avoid mistakes like they happened to
> > me.
> >
> > Henning
> > =20
>=20
> Henning, do you plan to respin this with the ID corrected?

Bart, no this one fixes the name of the chip i have at hand, and fixes
the size of its last bank. So it is valid on its own and should IMHO be
merged as is.

Since i do not have a NCT6116D (0xD281) i could basically just guess
(not too hard) but not test. And i do not really feel like contributing
untested code for which there is no known user/tester.

Henning

> Bart