linux-kernel - RE: [PATCH v4 0/3] TypeC Connector Class driver improvements

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 21 Aug 2020 13:34:46 +0000
From:   "Shaikh, Azhar" <azhar.shaikh@...el.com>
To:     Enric Balletbo i Serra <enric.balletbo@...labora.com>,
        "bleung@...omium.org" <bleung@...omium.org>,
        "groeck@...omium.org" <groeck@...omium.org>,
        "heikki.krogerus@...ux.intel.com" <heikki.krogerus@...ux.intel.com>,
        "pmalani@...omium.org" <pmalani@...omium.org>
CC:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "Mani, Rajmohan" <rajmohan.mani@...el.com>,
        "Patel, Utkarsh H" <utkarsh.h.patel@...el.com>,
        "Bowman, Casey G" <casey.g.bowman@...el.com>
Subject: RE: [PATCH v4 0/3] TypeC Connector Class driver improvements

Hi Enric,

Sure, I will rebase, test and upload a new revision.

Regards,
Azhar Shaikh

> -----Original Message-----
> From: Enric Balletbo i Serra <enric.balletbo@...labora.com>
> Sent: Friday, August 21, 2020 12:13 AM
> To: Shaikh, Azhar <azhar.shaikh@...el.com>; bleung@...omium.org;
> groeck@...omium.org; heikki.krogerus@...ux.intel.com;
> pmalani@...omium.org
> Cc: linux-kernel@...r.kernel.org; Mani, Rajmohan
> <rajmohan.mani@...el.com>; Patel, Utkarsh H <utkarsh.h.patel@...el.com>;
> Bowman, Casey G <casey.g.bowman@...el.com>
> Subject: Re: [PATCH v4 0/3] TypeC Connector Class driver improvements
> 
> Hi Azhar,
> 
> I got a merge conflict because I applied [1] before these patches, fix the
> conflict is trivial but, as I can't test, can you rebase your patches on top of [1]
> and test and resend to make sure everything is working?
> 
> Thanks,
>  Enric
> 
> [1] https://lkml.org/lkml/2020/7/10/1063
> 
> 
> On 21/8/20 1:38, Azhar Shaikh wrote:
> > Changes in v4:
> > * Patch 1: "platform/chrome: cros_ec_typec: Send enum values to
> >             usb_role_switch_set_role()"
> >   - Remove extra line between Fixes and Signed-off tag.
> >   - Added Reviewed-by and Cc tags from v1 and v2.
> >
> > * Patch 2: "platform/chrome: cros_ec_typec: Avoid setting usb role twice
> >             during disconnect"
> >   - Added Suggested-by from v2.
> >
> > * Patch 3: "platform/chrome: cros_ec_typec: Re-order connector
> >             configuration steps"
> >   - No change
> >
> > Changes in v3:
> > * Patch 1: "platform/chrome: cros_ec_typec: Send enum values to
> >             usb_role_switch_set_role()"
> >  - No change
> >
> > * Patch 2: "platform/chrome: cros_ec_typec: Avoid setting usb role twice
> >             during disconnect"
> >  - Move the location of calling usb_role_switch_set_role() to
> >    end of function in cros_typec_configure_mux() to avoid any change
> >    in code flow.
> >
> > * Patch 3: "platform/chrome: cros_ec_typec: Re-order connector
> >             configuration steps"
> >  - New patch added
> >
> > Changes in v2:
> > * Patch 1: "platform/chrome: cros_ec_typec: Send enum values to
> >             usb_role_switch_set_role()"
> >   - Update the commit message to change 'USB_ROLE_HOST in case of
> >     UFP.'  to 'USB_ROLE_HOST in case of DFP.'
> >
> > * Patch 2: "platform/chrome: cros_ec_typec: Avoid setting usb role twice
> >             during disconnect"
> >   - New patch added.
> >
> > Azhar Shaikh (3):
> >   platform/chrome: cros_ec_typec: Send enum values to
> >     usb_role_switch_set_role()
> >   platform/chrome: cros_ec_typec: Avoid setting usb role twice during
> >     disconnect
> >   platform/chrome: cros_ec_typec: Re-order connector configuration
> > steps
> >
> >  drivers/platform/chrome/cros_ec_typec.c | 9 +++++++--
> >  1 file changed, 7 insertions(+), 2 deletions(-)
> >