Hi Saranya,

 thank you for the details.

  when i ran the lunch ,I see the conflict below.

Project kernel/project-celadon
        Already applied         kernel/project-celadon/0001-CELADON-Enabling-USB-ROLE-SWITCH-QUIRK-forINTEL_SUNR.patch
        Conflicts          kernel/project-celadon/0001-DO-NOT-MERGE-role-Toggle-to-host-mode.patch
        Already applied         kernel/project-celadon/0004-drm-i915-skl-ddb-allocation-algorithm-optimization.patch

===========================================================================
           ALERT : Conflicts Observed while patch application !!          
===========================================================================
kernel/project-celadon

I am using the below repo info. Is there a chance that , we need to have a different patch for this.

Thanks,
Sathish

On Mon, Aug 19, 2019 at 9:54 AM Gopal, Saranya <saranya.gopal@intel.com> wrote:

Hi Satish,

 

For non-upstreamed kernel changes, we keep them separately in google_diff.

So, when you do lunch celadon-userdebug, it will start applying patches from google_diff directory.

So, after doing source build/envsetup.sh and lunch celadon-userdebug, you can check the patches applied in kernel/project-celadon.

The patch does a reverse of what was done earlier.

It toggles to host mode instead of device mode. Check the code here:

In drivers/usb/roles/intel-xhci-usb-role-switch.c,

In function, intel_xhci_usb_probe:

intel_xhci_usb_set_role(dev, USB_ROLE_DEVICE);

msleep(10);

intel_xhci_usb_set_role(dev, USB_ROLE_HOST);

 

This is the change made. Previously, it was USB_ROLE_HOST and then USB_ROLE_DEVICE.

 

Thanks,

Saranya

 

From: Sathish [mailto:satish.chevuru@gmail.com]
Sent: Monday, August 19, 2019 9:34 AM
To: Gopal, Saranya <saranya.gopal@intel.com>
Subject: Re: [01.org Celadon] Unresponsive USB port on NUC7i7BNH - Celadon

 

hi Saranya,

 

  thank you , I tried  and it seems applied.

 

sathish@ubuntuIA:~/android-ia-celadon/vendor/intel/utils$ git apply ~/Desktop/0001-DO-NOT-MERGE-google_diff-Force-host-mode-on-Celadon.patch
sathish@ubuntuIA:~/android-ia-celadon/vendor/intel/utils$ 

 

But when i got to the specific file where changes are mentioned in the patches and do a diff , I don't see any difference.

 

sathish@ubuntuIA:~/android-ia-celadon/kernel/project-celadon/drivers/usb/roles$ git diff .
sathish@ubuntuIA:~/android-ia-celadon/kernel/project-celadon/drivers/usb/roles$ ls -lrt
total 32
-rw-rw-r-- 1 sathish sathish  132 Apr 22 11:12 Makefile
-rw-rw-r-- 1 sathish sathish 1014 Apr 22 11:12 Kconfig
-rw-rw-r-- 1 sathish sathish 7406 Apr 22 11:12 class.c
-rw-rw-r-- 1 sathish sathish 5811 Aug 18 18:23 intel-xhci-usb-role-switch.Modified
-rw-rw-r-- 1 sathish sathish 5772 Aug 18 18:27 intel-xhci-usb-role-switch.c

 

I am wondering what are the changes that went on with this patch and how do i see the differences ? 

not sure if i am missing anything.

 

Thanks,

Sathish

 

 

On Mon, Aug 19, 2019 at 9:24 AM Gopal, Saranya <saranya.gopal@intel.com> wrote:

Hi Sathish,

 

Please apply the patches from vendor/intel/utils directory.

 

Thanks,

Saranya

 

From: Sathish [mailto:satish.chevuru@gmail.com]
Sent: Sunday, August 18, 2019 6:57 PM
To: Gopal, Saranya <saranya.gopal@intel.com>
Subject: Re: [01.org Celadon] Unresponsive USB port on NUC7i7BNH - Celadon

 

Hi Saranya,

 

 I tried applying the patch and I see below errors , I am not sure what i am doing wrong.

 

sathish@ubuntuIA:~/android-ia-celadon/kernel/project-celadon$ git apply --stat ~/Desktop/0001-DO-NOT-MERGE-google_diff-Force-host-mode-on-Celadon.patch
 ...001-DO-NOT-MERGE-role-Toggle-to-host-mode.patch |   19 +++----
 ...s-Add-suspend-resume-call-backs-for-Intel.patch |   57 --------------------
 2 files changed, 9 insertions(+), 67 deletions(-)
sathish@ubuntuIA:~/android-ia-celadon/kernel/project-celadon$ 

 

sathish@ubuntuIA:~/android-ia-celadon/kernel/project-celadon$ git apply --check ~/Desktop/0001-DO-NOT-MERGE-google_diff-Force-host-mode-on-Celadon.patch
error: android_p/google_diff/celadon/kernel/project-celadon/0002-role-Toggle-role-in-probe-to-make-device-mode-work.patch: No such file or directory
error: android_p/google_diff/celadon/kernel/project-celadon/0003-usb-roles-Add-suspend-resume-call-backs-for-Intel.patch: No such file or directory


sathish@ubuntuIA:~/android-ia-celadon/kernel/project-celadon$ git apply ~/Desktop/0001-DO-NOT-MERGE-google_diff-Force-host-mode-on-Celadon.patch
error: android_p/google_diff/celadon/kernel/project-celadon/0002-role-Toggle-role-in-probe-to-make-device-mode-work.patch: No such file or directory
error: android_p/google_diff/celadon/kernel/project-celadon/0003-usb-roles-Add-suspend-resume-call-backs-for-Intel.patch: No such file or directory
sathish@ubuntuIA:~/android-ia-celadon/kernel/project-celadon$ 

 

Appreciate your help.

 

Thanks,

Sathish

 

On Tue, Aug 13, 2019 at 2:42 PM Sathish <satish.chevuru@gmail.com> wrote:

thank you

 

On Tue, 13 Aug, 2019, 2:37 PM Gopal, Saranya, <saranya.gopal@intel.com> wrote:

Hi Sathish,

 

PFA the patch for Celadon target.

 

Thanks,

Saranya

 

From: Sathish [mailto:satish.chevuru@gmail.com]
Sent: Tuesday, August 13, 2019 2:32 PM
To: Gopal, Saranya <saranya.gopal@intel.com>
Cc: Gaurav Gupta <gaurav@trainingdata.io>; Zhang, Yanmin <yanmin.zhang@intel.com>; celadon@lists.01.org; Prasad, RevatiX <revatix.prasad@intel.com>
Subject: Re: [01.org Celadon] Unresponsive USB port on NUC7i7BNH - Celadon

 

Hi Saranya,

 

  Could you please provide me the same patch for the celadon eng target.

 

Thanks,

Sathish

 

On Fri, Jul 5, 2019 at 9:34 AM Sathish <satish.chevuru@gmail.com> wrote:

Hi,

 

I tested the alternative and mouse is detected in the front port. 

Will use patches to build the OS.

 

Thank you.

 

Thanks,

Sathish

 

On Thu, Jul 4, 2019 at 3:56 PM Gopal, Saranya <saranya.gopal@intel.com> wrote:

Sorry.. corrected the typo below. It is ‘host’ to switch to host mode.

#echo host > /sys/class/usb_role/intel_xhci_usb_sw-role-switch/role

 

From: Celadon [mailto:celadon-bounces@lists.01.org] On Behalf Of Gopal, Saranya
Sent: Thursday, July 04, 2019 3:54 PM
To: Gaurav Gupta <gaurav@trainingdata.io>; Zhang, Yanmin <yanmin.zhang@intel.com>
Cc: celadon@lists.01.org; Prasad, RevatiX <revatix.prasad@intel.com>
Subject: Re: [01.org Celadon] Unresponsive USB port on NUC7i7BNH - Celadon

 

Hi Gaurav,

 

In Celadon platform, the front port is designed to work as device mode port.

For your case, you can apply the above two patches and build cel_kbl target to force host mode on your NUC (NUC7i7BNH).

Make sure that your build workspace is new. Otherwise, the applied google_diff patches may conflict with the current patches.

With the build with these two patches, the port is forced into host mode always.

 

Alternatively, in your existing build, you can still switch the port role to host through this command in adb shell:

#echo host > /sys/class/usb_role/intel_xhci_usb_sw-role-switch/role

 

Thanks,

Saranya

 

From: Celadon [mailto:celadon-bounces@lists.01.org] On Behalf Of Gaurav Gupta
Sent: Wednesday, July 03, 2019 4:15 AM
To: Zhang, Yanmin <yanmin.zhang@intel.com>
Cc: celadon@lists.01.org
Subject: Re: [01.org Celadon] Unresponsive USB port on NUC7i7BNH - Celadon

 

Perhaps the port is forced to USB Device Mode. Can you use ‘adb shell’ to connect to Android form that port?

how to connect to 'adb shell' using USB port?  USB-USB cable?

I though adb connect was only enabled over network for NUC, am I right?

Can adb be connected over USB for NUC? 

Check BIOS menu to disable USB XDCI, then check if that port can be used as Host Mode.

Is it possible to disable XDCI and force Host-Mode in Android OS or BSP? 

 

 

From: Celadon [mailto:celadon-bounces@lists.01.org] On Behalf Of Sathish
Sent: Tuesday, July 02, 2019 1:31 PM
To: Tan, Ming <ming.tan@intel.com>
Cc: Gaurav Gupta <gaurav@trainingdata.io>; celadon@lists.01.org
Subject: Re: [01.org Celadon] Unresponsive USB port on NUC7i7BNH - Celadon

 

Hi,

 

 BIOS Version is  : BNKBL357.86A.0047.2017.0518.1824

 

 I used the same port to plugin the USB Storage Disk to Install the Celadon OS and installation is successful.

But once the OS boots up and I connect camera or mouse , to it , It won't work.

 

Thanks,

Sathish

 

On Tue, Jul 2, 2019 at 10:49 AM Tan, Ming <ming.tan@intel.com> wrote:

Sathish:

  What is your BIOS version?

  If you enter the BIOS setup program, does the key board or mouse do work if connect to this port?

 

  BR/Tan Ming.

 

From: Sathish [mailto:satish.chevuru@gmail.com]
Sent: Tuesday, July 2, 2019 1:13 PM
To: Tan, Ming <ming.tan@intel.com>
Cc: Gaurav Gupta <gaurav@trainingdata.io>; celadon@lists.01.org
Subject: Re: [01.org Celadon] Unresponsive USB port on NUC7i7BNH - Celadon

 

Out of four , three works fine , including back one , I am seeing the issue on front right side port .

 

Thanks,

Sathish

 

On Tue, 2 Jul, 2019, 10:39 AM Tan, Ming, <ming.tan@intel.com> wrote:

Sathish:

  How about the USB ports in the back side?

 

  BR/Tan Ming.

 

From: Celadon [mailto:celadon-bounces@lists.01.org] On Behalf Of Gaurav Gupta
Sent: Tuesday, July 2, 2019 1:03 PM
To: Sathish <satish.chevuru@gmail.com>
Cc: celadon@lists.01.org
Subject: Re: [01.org Celadon] Unresponsive USB port on NUC7i7BNH - Celadon

 

Hi All,

Has anyone observed USB ports not functioning on officially supported hardware?

 

On Mon, Jul 1, 2019 at 9:48 PM Sathish <satish.chevuru@gmail.com> wrote:

Hi,

 

 I installed celadon-eng build on NUC7i7BNH. I am using this for a video conferencing application.

 

PLATFORM_VERSION_CODENAME=REL
PLATFORM_VERSION=9
TARGET_PRODUCT=celadon
TARGET_BUILD_VARIANT=eng
TARGET_BUILD_TYPE=release
TARGET_ARCH=x86_64
TARGET_ARCH_VARIANT=x86_64

 

The front USB is non responsive for either Mouse, Keyboard, Camera or to Jabra Speaker.

 

I am seeing this on the latest build , this was fine with omr1 version.

 

I am seeing the same issue on cel_apl build as well.

 

[ro.vendor.build.fingerprint]: [cel_apl/cel_apl/cel_apl:9/PPR2.181005.003.A1/sathish06162135:eng/test-keys]

 

I am wondering , whether any of you have seen this issue during your installs. Do i need to use celadon certified NUC hardware ?

 

Appreciate your inputs.

 

Thanks,

Sathish

--
Celadon mailing list
Celadon@lists.01.org
https://lists.01.org/mailman/listinfo/celadon


 

--

Gaurav,

Co-founder, TrainingData.io

+1-917-386-3396

228 Webster Street,           
Palo Alto,                             

CA, 94301                          

 

--
Celadon mailing list
Celadon@lists.01.org
https://lists.01.org/mailman/listinfo/celadon


 

--

Gaurav,

Co-founder, TrainingData.io

+1-917-386-3396

228 Webster Street,           
Palo Alto,                             

CA, 94301                          

 

--
Celadon mailing list
Celadon@lists.01.org
https://lists.01.org/mailman/listinfo/celadon