Received-SPF: pass (google.com: domain of linux-kernel+bounces-120470-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
References: <20240322085606.993896-1-allencl_lin@hotmail.com>
 <TY0PR06MB56116F0902017225C78EDDDD9E312@TY0PR06MB5611.apcprd06.prod.outlook.com>
 <20240322-mammary-boil-f9a4c347fba1@spud> <20240322183009.GA1227164-robh@kernel.org>
 <20240322-rectified-udder-fef9102f58da@spud> <TY0PR06MB56110ADEA805B68BE2B887069E352@TY0PR06MB5611.apcprd06.prod.outlook.com>
 <20240326-whoever-spotter-1fe7ace35428@wendy> <TY0PR06MB561197578717990F4BEA93D29E352@TY0PR06MB5611.apcprd06.prod.outlook.com>
 <20240326-granite-snipping-7c8b04480b2e@spud>
In-Reply-To: <20240326-granite-snipping-7c8b04480b2e@spud>
From: Allen Lin <allencl_lin@hotmail.com>
Date: Wed, 27 Mar 2024 15:48:48 +0800
Message-ID:
 <TY0PR06MB5611AE812B72B349E85118D59E342@TY0PR06MB5611.apcprd06.prod.outlook.com>
Subject: Re: [PATCH v1 1/4] dt-bindings: input: Add Himax HX83102J touchscreen
To: Conor Dooley <conor@kernel.org>
Cc: Conor Dooley <conor.dooley@microchip.com>, Rob Herring <robh@kernel.org>, 
	dmitry.torokhov@gmail.com, krzysztof.kozlowski+dt@linaro.org, 
	conor+dt@kernel.org, jikos@kernel.org, benjamin.tissoires@redhat.com, 
	linux-input@vger.kernel.org, devicetree@vger.kernel.org, 
	linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: TY0PR06MB5611:EE_|PUZPR06MB6125:EE_
X-MS-Office365-Filtering-Correlation-Id: 44384e24-ea6f-4a71-22dd-08dc4e325f8e
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info:
	U0fBSDQdLZLUmC7RKr7sZanwMZkVDczBuVQO2LoZd4iecA+V3AoBsGHu6s0b5eCzlJ+SaFc/MNaPFNzdS0uwULCpS9PdbiWs/jpywmw4q+lBkUuvhzI5GdXLYcRs2uIyZ2PsP4ZGgJPNhnLsRq+ZvUiTPn3MNPLiDEYyoar52knBU8aE2uYAsJxNEhaZZaQyd1rFRqqKpAC0qzR1nwTqpqcIdoPUIvNj03PSgNpbi1ooAbP/k5BstP0gDJ09CYg29Pukoo2SULmQdOzNE5Rx9m2l0LfnHEgPdF4jKOaPHR0eX/OF7o1E5pVlvzRpvkN+l3xU7G7CZY20tCjV3h+oZIh3jFTSw7qchX024gSBV0wqWNvcyKPlXJvrtF1O+B10rWL8vcQC1q+pmvVmtS4X2nlHgT2P3X4lHSeRsBMaaoFHloMkf7FP2c5dS5w8ieBKeLOU5tJPcyhRTR9AkbxSHizPqbsgM4V0Y5zSWRlwxEf9eVY/W/+s4eo8Dl38Twk+nwu/yP6p5cDtJo7qErGcML+ynyR9SGi+I5rIU/flouGqK4funRRnQ0VtOHPR2KIVEDLyooQEk6zWEdpwsIV3+Pnb4QFtSYyx2k4HbtWWCi2owCyiuXy/lrzL/vzuJbUd8PpHJ36/dhMzYmTEbo7aqYJCwkiOIGat4GwdrM6VD7Q=
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?SFZac2tQTEZhTkV0N0JJRnlHMkpSN3NHaytrNXNidWtBaVUydmcwY1hPVjYz?=
 =?utf-8?B?S3BpOHA3UjFGN0FUWUNkWUE3d0NoT1ZobDloVkVFV3V6bkpuNUd2QTRnWlNm?=
 =?utf-8?B?Zk9INDgyMWVSWmovdHdpRFo4bDJLd3NkZUNYNDdYSGF0WGRNTzQwYmN5YTVm?=
 =?utf-8?B?SmhCTEVpck1qSWFqUEluVjRtOFNRdDNZVzA3NXp5dVVQN3ZYQVdpeVJFbUlV?=
 =?utf-8?B?K3ozQ1EwOTRIcjdYM0FuU3VXTkdpSWh5ODRuWVJabHZjVThDYXJ5QmJHU3lK?=
 =?utf-8?B?c3Q4cTYwSHF6elg5RXhCVFlLSExHMUlVeWNjSHErQzRvUjNhOVBnMzFKRzBs?=
 =?utf-8?B?KzJidFNSV25qUW94Rit0a1NwNU1UdWlJWG9zSzdIdWZMZXpxWGJpQ2VxNmk2?=
 =?utf-8?B?V1YwRXZZS0hJdTY3WnRaVythNHpLK0wxWUNrbERGVG92UFRpUnJPdVE4NnNk?=
 =?utf-8?B?dDVUcE5mTDlmd2NQVzJ3V2JucTdGWlN3VmJyTnVZMGJvZFA1L0swTFdUb3Rq?=
 =?utf-8?B?R0Q5bUtPaHhhYnpGZjJhSzFvZlhJYUlXRjE5c2M4Z0g3R01aclZwWkdkRm1W?=
 =?utf-8?B?SFpDRHlXZkRMMUlXODNtSUE3amNGdXljWGkzcytNQjlrU3R0emNLdFBtS0hl?=
 =?utf-8?B?YWt2VVFOb0N1T3hBWVlkQzAySnF1MFlsN2dGVkhHRHkxRk5USHBZenY5UWFB?=
 =?utf-8?B?SkIxSGxuSG5PREZ2RXdyUXcweit1ZHNYS1VndngvSzdVQitteGF4R3VNSW5m?=
 =?utf-8?B?SXQrYzZVOU9aQ2FQUndhYlJROGwzNFo1Y0Nkc0tkWXV1UHpqOEY4Z2RIWmFZ?=
 =?utf-8?B?ak9BODg2N3czdGVNUG1qTzN5eWI5SUNYZXkvckpjelN3cVVDTzJjWmt1cmxm?=
 =?utf-8?B?ZmJ4Z0R4clM1NHEydnVVOWoveUpZVnFRa1ZEVjF0ZnlrM1c0WW91ajc1M3Q2?=
 =?utf-8?B?WjlhNjh4MmUxT2VDYVEwYkVLL0xXVUp3c2NiOVJVQ0srajZWSkplMFdLbXNj?=
 =?utf-8?B?NDVwakpUWlZoRElleHMxVXFYaUVqK2NvdlZSUjhWVkd1WXQxb0xCYlN5VjhJ?=
 =?utf-8?B?TDdIR3JyS3lsUXpnUktzbWQ2NVh3VEVNcXF1eVlpMVlUQmh4VVIxVVBudDdZ?=
 =?utf-8?B?Tk1kVE91aUd2OVhiRENsVFQwd2dkTHRGTytvT1BkNkdOZFk1dThWbFNZSjlE?=
 =?utf-8?B?dUIza2dEUFhubm9oYkxyQ3lENlV6eUVTTzRrOU8xL1J1ZHVleUNzc3Bqb1BE?=
 =?utf-8?B?SWFOZ2dvSlRrbGhqNXg3V0x6OVpqem9IV0ROb2o5M09YT1FSTkNwOVZiU0VP?=
 =?utf-8?B?MWFDcXY1S2dGWGo1TDVyYThuOFRNMjZRdm5pcU1tekx6cE9qTWM5SjYwNU93?=
 =?utf-8?B?ZFVzRnZBN3diam5JVTRmV1BkbjNNbFJNU1Erb0V1amhnL3UzU1BHcUxpYmtD?=
 =?utf-8?B?VEdZVmNuU3JpMWJpaFdWK0dOMFo3d3huQ0dUdnlONGd6ZUE0ODJScVhuVTdl?=
 =?utf-8?B?R01iZVZFV1QvL1A0VTZUejFXNGJpUXlHSVplejR6bWlSSUNSU0hLenpkK2lW?=
 =?utf-8?B?cHovbGl6YW9wN2ZxTUpXTGtUNmRnOWVlbGRMOURCc1c1dUllTGtZaUdiUGlj?=
 =?utf-8?B?ZUlFdUtNNlFQZ29lNnFBNzNLU3FRMVZYcitpQ3lZYk5iWmV1blkxL0lKZWkw?=
 =?utf-8?Q?Q7TMSy0NZ2pMKKPBEfEg?=
X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-3208f.templateTenant
X-MS-Exchange-CrossTenant-Network-Message-Id: 44384e24-ea6f-4a71-22dd-08dc4e325f8e
X-MS-Exchange-CrossTenant-AuthSource: TY0PR06MB5611.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Mar 2024 07:49:04.0359
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg:
	00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PUZPR06MB6125

Conor Dooley <conor@kernel.org> =E6=96=BC 2024=E5=B9=B43=E6=9C=8827=E6=97=
=A5 =E9=80=B1=E4=B8=89 =E4=B8=8A=E5=8D=883:28=E5=AF=AB=E9=81=93=EF=BC=9A
>
> On Tue, Mar 26, 2024 at 06:40:28PM +0800, Allen Lin wrote:
> > Conor Dooley <conor.dooley@microchip.com> =E6=96=BC 2024=E5=B9=B43=E6=
=9C=8826=E6=97=A5 =E9=80=B1=E4=BA=8C =E4=B8=8B=E5=8D=884:48=E5=AF=AB=E9=81=
=93=EF=BC=9A
> > >
> > > On Tue, Mar 26, 2024 at 01:46:56PM +0800, Allen Lin wrote:
> > > > Conor Dooley <conor@kernel.org> =E6=96=BC 2024=E5=B9=B43=E6=9C=8823=
=E6=97=A5 =E9=80=B1=E5=85=AD =E4=B8=8A=E5=8D=882:34=E5=AF=AB=E9=81=93=EF=BC=
=9A
> > > > >
> > > > > On Fri, Mar 22, 2024 at 01:30:09PM -0500, Rob Herring wrote:
> > > > > > On Fri, Mar 22, 2024 at 05:54:08PM +0000, Conor Dooley wrote:
> > > > > > > On Fri, Mar 22, 2024 at 04:56:03PM +0800, Allen_Lin wrote:
> > > > > > > > Add the HX83102j touchscreen device tree bindings documents=
.
> > > > > > > > HX83102j is a Himax TDDI touchscreen controller.
> > > > > > > > It's power sequence should be bound with a lcm driver, thus=
 it
> > > > > > > > needs to be a panel follower. Others are the same as normal=
 SPI
> > > > > > > > touchscreen controller.
> > > > > > > >
> > > > > > > > Signed-off-by: Allen_Lin <allencl_lin@hotmail.com>
> > > > > > >
> > > > > > > note to self/Krzysztof/Rob:
> > > > > > > There was a previous attempt at this kind of device. This ver=
sion looks
> > > > > > > better but might be incomplete given there's a bunch more pro=
perties in
> > > > > > > that patchset:
> > > > > > > https://lore.kernel.org/all/20231017091900.801989-1-tylor_yan=
g@himax.corp-partner.google.com/
> > > > > >
> > > > > > Those don't look like properties we want coming back.
> > > > >
> > > > > Oh, I don't want most of them coming back either. There are some
> > > > > supplies in there though that I think we would like to come back,=
 no?
> > > > > Maybe this particular device doesn't have any supplies, but that =
doesn't
> > > > > really seem credible.
> > > >
> > > > We will use Firmware-name in Device Tree.
> > >
> > > > For power supply settings, because there may be other device using
> > > > same regulator.
> > >
> > > If there are other devices using the same regulator is it more
> > > important that you document it so that it doesn't get disabled by the
> > > other users.
> > >
> > > > We plan to define it as an optional property for user to control in
> > > > next release.
> > >
> > > I don't see how the regulator would not be required, the device doesn=
't
> > > function without power.
> > >
> > > Thanks,
> > > Conor.
> >
> > I will set power supply as required.
> > The description of power supply as below,
> >
> > properties:
> >   vccd-supply:
> >     description: A phandle for the regualtor supplying IO power. Should=
 be own
> >                  by TPIC only.
>
> What does "owned by TPIC" only mean? Why would the vccd supply not be
> allowed to be shared with another device?
>
> > This works for TP digital IO only, main power is
> >                  given by display part VSP/VSN power source which is co=
ntrolled
> >                  by lcm driver.
>
> What drivers control things doesn't really matter here, we're just
> describing the hardware. If there's another supply to the controller,
> then document it too please.
>

Below is IC power sequence introduction.
https://github.com/HimaxSoftware/Doc/tree/main/Himax_Chipset_Power_Sequence

TDDI IC, which means Touch and Display Driver is integrated in one IC,
So some power supplies will be controlled by Display driver.

In yaml Document, can we just list power supplies controlled by touch drive=
r?

Thanks,
Allen