Re: [PATCH v10 1/4] MIPS: Loongson64: dts: update the display controller device node

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 2022/2/21 18:01, Krzysztof Kozlowski wrote:
On 21/02/2022 10:19, Sergei Shtylyov wrote:
On 2/20/22 5:55 PM, Sui Jingfeng wrote:

From: suijingfeng <suijingfeng@xxxxxxxxxxx>

The display controller is a pci device, its PCI vendor id is 0x0014
its PCI device id is 0x7a06.

1) In order to let the driver to know which chip the DC is contained
    in, the compatible string of the display controller is updated
    according to the chip's name.

2) Add display controller device node for ls2k1000 SoC

Reported-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
Signed-off-by: suijingfeng <suijingfeng@xxxxxxxxxxx>
Signed-off-by: Sui Jingfeng <15330273260@xxxxxx>
---
  arch/mips/boot/dts/loongson/loongson64-2k1000.dtsi | 8 ++++++++
  arch/mips/boot/dts/loongson/ls7a-pch.dtsi          | 7 ++-----
  2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/arch/mips/boot/dts/loongson/loongson64-2k1000.dtsi b/arch/mips/boot/dts/loongson/loongson64-2k1000.dtsi
index 768cf2abcea3..af9cda540f9e 100644
--- a/arch/mips/boot/dts/loongson/loongson64-2k1000.dtsi
+++ b/arch/mips/boot/dts/loongson/loongson64-2k1000.dtsi
@@ -209,6 +209,14 @@ gpu@5,0 {
  				interrupt-parent = <&liointc0>;
  			};
+ lsdc: display-controller@6,0 {
    Shouldn't the node name just be "display", according to the section 2.2.2
of the DT spec?
lcd-controller, led-controller. As I understood from the bindings, this
is not physical device displaying something (like a panel) but rather a
device controlling such panel. Therefore display-controller feels
appropriate.


Best regards,
Krzysztof
Extremely correct.



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux