Re: [PATCH v2 0/2] Add waveshare 7inch touchscreen panel support

[Stefan Wahren <wahrenst@gmx.net>]

Hi Neil,

Am 09.01.24 um 12:19 schrieb neil.armstrong@linaro.org:
> Hi,
>
> On 09/01/2024 08:09, Shengyang Chen wrote:
>> This patchset adds waveshare 7inch touchscreen panel support
>> for the StarFive JH7110 SoC.
>
> Could you precise which SKU you're referring to ? is it 19885 =>
> https://www.waveshare.com/7inch-dsi-lcd.htm ?
>
> Are you sure it requires different timings from the RPi one ? In the
> Waveshare
> wiki it explicitly uses the rpi setup (vc4-kms-dsi-7inch) to drive it:
> https://www.waveshare.com/wiki/7inch_DSI_LCD
i don't have an anser for your question, but the Raspberry Pi vendor
tree use different timings than the Mainline kernel:

https://github.com/raspberrypi/linux/commit/222b9baa97cc4c880d040a8c6a5da80d6a42c8e8

Additionally the
arm64/boot/dts/freescale/imx8mm-venice-gw72xx-0x-rpidsi.dtso suggests
that it uses the Raspberry Pi 7inch, but uses the timings of
powertip,ph800480t013-idf02 from panel-simple.

Maybe Shengyang could test these timings with the Waveshare touch. At
the end this rely on a proper implementation on the underlying drivers.

Sorry, for adding more confusion.

Regards
>
> Neil
>
>>
>>
>> changes since v1:
>> - Rebased on tag v6.7.
>>
>> patch 1:
>> - Gave up original changing.
>> - Changed the commit message.
>> - Add compatible in panel-simple.yaml
>>
>> patch 2:
>> - Gave up original changing.
>> - Changed the commit message.
>> - Add new mode for the panel in panel-simple.c
>>
>> v1:
>> https://patchwork.kernel.org/project/dri-devel/cover/20231124104451.44271-1-shengyang.chen@starfivetech.com/
>>
>> Shengyang Chen (2):
>>    dt-bindings: display: panel: panel-simple: Add compatible property
>> for
>>      waveshare 7inch touchscreen panel
>>    gpu: drm: panel: panel-simple: add new display mode for waveshare
>>      7inch touchscreen panel
>>
>>   .../bindings/display/panel/panel-simple.yaml  |  2 ++
>>   drivers/gpu/drm/panel/panel-simple.c          | 28 +++++++++++++++++++
>>   2 files changed, 30 insertions(+)
>>
>